Re: [PATCH v3 2/3] gpio/omap: modify wake-up register with interrupt enable.

[Santosh Shilimkar <santosh.shilimkar@ti.com>]

On Wednesday 15 May 2013 02:24 AM, Andreas Fenkart wrote:
> OMAP4430 TRM chap. 25.4.5.2
> To reduce dynamic consumption, an efficient idle scheme is based on the
> following:
> • An efficient local autoclock gating for each module
> • The implementation of control sideband signals between the PRCM module
>   and each module
> This enhanced idle control allows clocks to be activated and deactivated
> safely without requiring complex software management. The idle mode
> request, idle acknowledge, and wake-up request are sideband signals
> between the PRCM module and the general-purpose interface
> 
> OMAP4430 TRM chap. 25.4.6.2
> There must be a correlation between the wake-up enable and interrupt
> enable register. If a GPIO pin has a wake-up configured on it, it must
> also have the corresponding interrupt enabled. Otherwise, it is possible
> there is a wake-up event, but after exiting the IDLE state, no interrupt
> is generated; the corresponding bit from the interrupt status register is
> not cleared, and the module does not acknowledge a future idle request.
> 
> Up to now _set_gpio_triggering() is also handling the wake-up enable
> register. According the TRM this should be in sync with the interrupt
> enable. Wakeup is still enabled by default, since the module would not
> wake from idle otherwise.
> The enabled_wakeup_gpios was introduced to remember an explicit
> _set_gpio_wakeup beyond a mask/unmask cycle. Calling the flag flag
> disabled_wakeup_gpios would spare the problem of initializing it, but
> feels very unnatural to read.
> 
> Wakeup functionality is completely untested, since the AM335x
> lacks a IRQWAKEN register.
> 
> Signed-off-by: Andreas Fenkart <andreas.fenkart@streamunlimited.com>
> ---
I am ok with the subject patch and other two patches.
Would be good to get a tested by at least for though for OMAP4
and OMAP3. I have no access to boards for few weeks so
can't test it myself.

Other than that, for all three patches,
Acked-by: Santosh Shilimkar <santosh.shilimkar@ti.com>

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