Re: [PATCH] gpio: make of_get_named_gpiod_flags() private
From: Alexandre Courbot <hidden>
Date: 2014-05-13 13:24:07
Also in:
lkml
From: Alexandre Courbot <hidden>
Date: 2014-05-13 13:24:07
Also in:
lkml
On Tue, May 13, 2014 at 10:17 PM, Linus Walleij [off-list ref] wrote:
On Sun, May 4, 2014 at 5:45 AM, Alexandre Courbot [off-list ref] wrote:quoted
of_get_named_gpiod_flags() is visible and directly usable by GPIO consumers, but it really should not as the gpiod interface relies on the simpler gpiod_get() to provide properly-configured GPIOs. of_get_named_gpiod_flags() is just used internally by gpiolib to implement gpiod_get(), and by the old of_get_named_gpio_flags() function, therefore it makes sense to make it gpiolib-private. As a side-effect, the unused (and unneeded) of_get_gpiod_flags() inline function is also removed, and of_get_named_gpio_flags() is moved from a static inline function to a regular one in gpiolib-of.c This results in all references to gpiod_* functions in of_gpio.h being gone, which is the way it should be since this file is part of the old integer GPIO interface. Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>This patch broke a lot of drivers so I had to take it out again...
That's unexpected - what happens with it?