Re: [PATCH 1/2] gpio: add pin biasing and drive mode to gpiolib

From: Alan Cox
Date: Sun Apr 17 2011 - 18:02:36 EST


On Sun, 17 Apr 2011 23:58:16 +0200
Linus Walleij <linus.walleij@xxxxxxxxxx> wrote:

> 2011/4/17 Alan Cox <alan@xxxxxxxxxxxxxxxxxxx>:
>
> >> + * enum gpio_drive - drive modes for GPIOs (output)
> >> + * @GPIO_DRIVE_PUSH_PULL: the GPIO will be driven actively high and low, this
> >> + *   is the most typical case and is typically achieved with two active
> >> + *   transistors on the output
> >> + * @GPIO_DRIVE_OPEN_DRAIN: the GPIO will be driven with open drain (open
> >> + *   collector) which means it is usually wired with other output ports
> >> + *   which are then pulled up with an external resistor
> >> + * @GPIO_DRIVE_OPEN_SOURCE: the GPIO will be driven with open drain
> >> + *   (open emitter) which is the same as open drain mutatis mutandis but
> >> + *   pulled to ground
> >
> > How about "off". As I've pointed out at various times some devices have
> > GPIO pins that may or may not be connected depending upon what other chip
> > logic is doing.
>
> Sure, why not.
>
> > As I've also noted with various other previous suggestions for API
> > additions you also need an "unknown" state so that a device can implement
> > the methods but indicate for some pins the value is unknown due to
> > hardware or firmware limitations.
>
> OK do you want an unknown biasing mode too, or is "floating" enough
> of a default to cover that?

I think I'd rather also have unknown for both the drive and the biasing.
It's a way of ensuring we get some kind of complete coverage even if not
perfect.

In the Intel case we also have 'alt status' stuff where the line has been
borrowed by firmware for example so we really genuinly haven't got a clue
what the state is right now.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/