Hi Rob, On 09/07/19 4:06 AM, Rob Herring wrote: > On Fri, Jun 28, 2019 at 3:31 AM Harish Jenny K N > <harish_kandiga@xxxxxxxxxx> wrote: >> Document the device tree binding for the inverter gpio >> controller to configure the polarity of the gpio pins >> used by the consumers. >> >> Signed-off-by: Harish Jenny K N <harish_kandiga@xxxxxxxxxx> >> --- >> .../devicetree/bindings/gpio/gpio-inverter.txt | 29 ++++++++++++++++++++++ >> 1 file changed, 29 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/gpio/gpio-inverter.txt >> >> diff --git a/Documentation/devicetree/bindings/gpio/gpio-inverter.txt b/Documentation/devicetree/bindings/gpio/gpio-inverter.txt >> new file mode 100644 >> index 0000000..8bb6b2e >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/gpio/gpio-inverter.txt >> @@ -0,0 +1,29 @@ >> +GPIO-INVERTER >> +====== >> +This binding defines the gpio-inverter. The gpio-inverter is a driver that >> +allows to properly describe the gpio polarities on the hardware. > I don't understand. Please explain this in terms of the hardware, not a driver. gpio inverters can be used on different hardware to alter the polarity of gpio chips. The polarity of pins can change from hardware to hardware with the use of inverters. This device tree binding models gpio inverters in the device tree to properly describe the hardware. Please let me know if this is enough and needs to be updated in the documentation patch. I am sorry I did not include device tree list in the original discussion ( i.e first version of the patch https://www.spinics.net/lists/linux-gpio/msg39681.html). Thanks. Best Regards, Harish Jenny K N
