On Mon, Jul 14, 2014 at 08:23:55PM +0200, Arnd Bergmann wrote:
On Monday 14 July 2014 18:18:12 Lars-Peter Clausen wrote:
Yes. But now that you say it the gpiod_direction_output() call is missing from this patch.
I'm lost now. The GPIO_ACTIVE_HIGH I added comes from Documentation/gpio/board.txt and as Linus Walleij explained to me the other day, the lookup is supposed to replace devm_gpio_request_one(), which in turn replaced both the gpio_request and the gpio_direction_output(). Do I need to put the gpiod_direction_output() back or is there another interface for that when registering the board gpios?
Indeed. If you *do* need an explicit _output() then that sounds to me like we either need a gpiod_get_one() or an extension to the table, looking at the code it seems like this is indeed the case. We can set if the GPIO is active high/low, or open source/drain but there's no flag for the initial state.