Re: [PATCH V5 0/4] gpio: tegra: Cleanups and support for debounce

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 05/02/2016 11:58 AM, Laxman Dewangan wrote:

On Monday 02 May 2016 09:42 PM, Stephen Warren wrote:
On 04/30/2016 05:07 AM, Linus Walleij wrote:
On Fri, Apr 29, 2016 at 11:20 AM, Laxman Dewangan
<ldewangan@xxxxxxxxxx> wrote:
On Friday 29 April 2016 02:37 PM, Linus Walleij wrote:
On Mon, Apr 25, 2016 at 12:38 PM, Laxman Dewangan
<ldewangan@xxxxxxxxxx>
wrote:

Add support for the debounce as Tegra210 support debounce in HW.
Also do the clenaups to remove all global variables.

OK this v5 is applied.

Laxman does this GPIO also have open drain and/or open source
handling?


Some of the pins support the open drain and these are part of pinmux
register set.
For that we have property for setting open drain.

IIRC, Tegra has open-drain control in both the GPIO controller for all
pins (OE bit) and in the pinmux controller for a small subset of pins.
For GPIOs, why wouldn't we just use the control bit in the GPIO
controller for all GPIOs. This would avoid any special-cases, and
minimize coupling between the GPIO and pinctrl drivers.


Toggling OE bit is something emulating the open drain here.

From the perspective of the external HW that's attached to the GPIO, I believe there's no difference.

I think idea is that when we configure the pin in open drain then it
should be automatically handled by HW  when we want to set pin state
high or low. When we set low, the pin should be driven and when high
then it should be tristated input. We should not need any direction bit
setting.

I don't imagine anything in the kernel cares, so long as the correct logic level is present on the pin based on whatever GPIO API was last called.

I'd be very surprised if there wasn't hardware that could only implement open-drain by this "emulation" method, so I'd be very surprised if something prohibited that implementation style.

Otherwise, if pin is configured as open drain then:

Set out = 0

and when it need to set pin to high then oe = 0 else oe =1. Do not
toggle any other bits.

On this case, we need to store that pin is configured as open drain so
that set_value should toggle OE instead of OUT.

That sounds like a reasonable implementation.

Or do you want to have different implementation?

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



[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux