Re: [RFC] qnap-tsx51: add new driver for leds and button support on QNAP TS-x51 series

[Linus Walleij <linus.walleij@xxxxxxxxxx>]

On Wed, Jun 15, 2016 at 2:45 AM, Vincent Pelletier
<plr.vincent@xxxxxxxxx> wrote:

> So I wonder if it makes sense at all for a driver to both define a gpio
> lookup table and then look for each gpio it just defined.
>
> Maybe I am doing it wrong and/or at the wrong level ?

Well ask yourself what you are doing I guess, GPIO is
General Purpose Input Output, a trap some people fall into
is confusing that with their special purpose input/outputs,
and sometimes they are input-only or output-only on top of
that. That is not GPIO.

So what is the use case for these electronics? If it is not
supposed to be used by misc input/output, it is not GPIO,
and it is better for the driver to manage it at will.

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html