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

Groups > linux.kernel > #1618669

Re: [PATCH 1/1] gpio: core: Decouple open drain/source flag with active low/high

Show all headers | View raw

On Friday 07 April 2017 03:55 PM, Linus Walleij wrote:
> On Thu, Apr 6, 2017 at 3:35 PM, Laxman Dewangan <ldewangan@nvidia.com> wrote:
>
>> Currently, the GPIO interface is said to Open Drain if it is Single
>> Ended and active LOW. Similarly, it is said as Open Source if it is
>> Single Ended and active HIGH.
>>
>> The active HIGH/LOW is used in the interface for setting the pin
>> state to HIGH or LOW when enabling/disabling the interface.
>>
>> In Open Drain interface, pin is set to HIGH by putting pin in
>> high impedance and LOW by driving to the LOW.
>>
>> In Open Source interface, pin is set to HIGH by driving pin to
>> HIGH and set to LOW by putting pin in high impedance.
>>
>> With above, the Open Drain/Source is unrelated to the active LOW/HIGH
>> in interface. There is interface where the enable/disable of interface
>> is ether active LOW or HIGH but it is Open Drain type.
>>
>> Hence decouple the Open Drain with Single Ended + Active LOW and
>> Open Source with Single Ended + Active HIGH.
>>
>> Adding different flag for the Open Drain/Open Source which is valid
>> only when Single ended flag is enabled.
>>
>> Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com>
> Patch applied.
>
> Good that you found this and fixed it before someone git hurt.

Thanks you very much.


Now we are going to use the descriptor based GPIO APIs as this is very 
powerful and passing this info from DT is very useful.

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

Thread

[PATCH 1/1] gpio: core: Decouple open drain/source flag with active low/high Laxman Dewangan <ldewangan@nvidia.com> - 2017-04-06 16:00 +0200
  Re: [PATCH 1/1] gpio: core: Decouple open drain/source flag with  active low/high Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-04-06 18:20 +0200
    Re: [PATCH 1/1] gpio: core: Decouple open drain/source flag with  active low/high Laxman Dewangan <ldewangan@nvidia.com> - 2017-04-06 19:30 +0200
      Re: [PATCH 1/1] gpio: core: Decouple open drain/source flag with  active low/high Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-04-06 19:40 +0200
  Re: [PATCH 1/1] gpio: core: Decouple open drain/source flag with  active low/high Linus Walleij <linus.walleij@linaro.org> - 2017-04-07 12:30 +0200
    Re: [PATCH 1/1] gpio: core: Decouple open drain/source flag with  active low/high Laxman Dewangan <ldewangan@nvidia.com> - 2017-04-07 12:40 +0200

csiph-web