Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1289931

Re: [PATCH 1/2] dt-bindings: GPIO: Add generic serializer binding

Show all headers | View raw

On Fri, Dec 11, 2015 at 8:46 PM, Andrew F. Davis <afd@ti.com> wrote:

> Add binding for generic parallel-in/serial-out shift register devices
> used as GPIO.
>
> Signed-off-by: Andrew F. Davis <afd@ti.com>

> +Generic Parallel-in/Serial-out Shift Register GPIO Driver
> +
> +This binding describes generic parallel-in/serial-out shift register
> +devices that can be used for GPI (General Purpose Input). This includes
> +SN74165 serial-out shift registers and the SN65HVS88x series of
> +industrial serializers.
> +
> +Required properties:
> + - compatible          : Should be "pisosr-gpio".

I think it should also define compatible strings on the "vendor,device"
format apart from the generic compatible. Sooner or later we may need
to differentiate them and then that comes in handy.

> + - gpio-controller     : Marks the device node as a GPIO controller.
> + - #gpio-cells         : Should be two. For consumer use see gpio.txt.
> +
> +Optional properties:
> + - ngpios              : Number of GPIO lines, default is 8.

If you didn't do "pisosr-gpio" but instead "foo,sn74165", maybe you
don't need to have this in the device tree but instead it can be
determined from the compatible string?

In that case do that.

> + - load-gpios          : GPIO pin specifier attached to load enable, this
> +                         pin is pulsed before reading from the device to
> +                         load input pin values into the the device.

OK seems necessary.

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

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 0/2] gpio: Add driver for SPI serializers "Andrew F. Davis" <afd@ti.com> - 2015-12-11 20:50 +0100
  [PATCH 1/2] dt-bindings: GPIO: Add generic serializer binding "Andrew F. Davis" <afd@ti.com> - 2015-12-11 20:50 +0100
    Re: [PATCH 1/2] dt-bindings: GPIO: Add generic serializer binding Linus Walleij <linus.walleij@linaro.org> - 2015-12-11 22:50 +0100
      Re: [PATCH 1/2] dt-bindings: GPIO: Add generic serializer binding "Andrew F. Davis" <afd@ti.com> - 2015-12-14 17:50 +0100
        Re: [PATCH 1/2] dt-bindings: GPIO: Add generic serializer binding Rob Herring <robh+dt@kernel.org> - 2015-12-14 23:40 +0100
          Re: [PATCH 1/2] dt-bindings: GPIO: Add generic serializer binding "Andrew F. Davis" <afd@ti.com> - 2015-12-15 00:30 +0100
            Re: [PATCH 1/2] dt-bindings: GPIO: Add generic serializer binding Rob Herring <robh+dt@kernel.org> - 2015-12-16 17:40 +0100
              Re: [PATCH 1/2] dt-bindings: GPIO: Add generic serializer binding "Andrew F. Davis" <afd@ti.com> - 2015-12-30 18:10 +0100
            Re: [PATCH 1/2] dt-bindings: GPIO: Add generic serializer binding Linus Walleij <linus.walleij@linaro.org> - 2015-12-22 11:00 +0100
              Re: [PATCH 1/2] dt-bindings: GPIO: Add generic serializer binding "Andrew F. Davis" <afd@ti.com> - 2015-12-30 18:10 +0100
        Re: [PATCH 1/2] dt-bindings: GPIO: Add generic serializer binding Geert Uytterhoeven <geert@linux-m68k.org> - 2015-12-17 09:30 +0100
        Re: [PATCH 1/2] dt-bindings: GPIO: Add generic serializer binding Geert Uytterhoeven <geert@linux-m68k.org> - 2015-12-17 09:30 +0100
  Re: [PATCH 0/2] gpio: Add driver for SPI serializers Linus Walleij <linus.walleij@linaro.org> - 2015-12-11 22:50 +0100

csiph-web