Thread (8 messages) 8 messages, 3 authors, 2014-05-16

Re: [PATCH] gpio: make of_get_named_gpiod_flags() private

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?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help