On Wed 2019-10-09 17:12:23, Tony Lindgren wrote:
> Commit f74297dd9354 ("ARM: OMAP2+: Make sure LOGICRETSTATE bits are not
> cleared") disabled oswr (open switch retention) for per and core domains
> as various GPIO related issues were noticed if the bootloader had
> configured the bits for LOGICRETSTATE for per and core domains.
>
> With the recent gpio-omap fixes, mostly related to commit e6818d29ea15
> ("gpio: gpio-omap: configure edge detection for level IRQs for idle
> wakeup"), things now behave for enabling core oswr for omap4.
>
> Cc: Merlijn Wajer <merlijn@xxxxxxxxxx>
> Cc: Pavel Machek <pavel@xxxxxx>
> Cc: Sebastian Reichel <sre@xxxxxxxxxx>
> Signed-off-by: Tony Lindgren <tony@xxxxxxxxxxx>
2,7,8 basically modify same lines of code? Should that be done in one
patch?
Best regards,
Pavel
>
> - /*
> - * Bootloader or kexec boot may have LOGICRETSTATE cleared
> - * for some domains. This is the case when kexec booting from
> - * Android kernels that support off mode for example.
> - * Make sure it's set at least for core and per, otherwise
> - * we currently will see lost GPIO interrupts for wlcore and
> - * smsc911x at least if per hits retention during idle.
> - */
> - if (!strncmp(pwrdm->name, "core", 4)
> - pwrdm_set_logic_retst(pwrdm, PWRDM_POWER_RET);
> -
> - if (!strncmp(pwrdm->name, "l4per", 5)
> + if (!strncmp(pwrdm->name, "core", 4) ||
> + !strncmp(pwrdm->name, "l4per", 5))
> pwrdm_set_logic_retst(pwrdm, PWRDM_POWER_OFF);
>
> pwrst = kmalloc(sizeof(struct power_state), GFP_ATOMIC);
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
Attachment:
signature.asc
Description: Digital signature
