Tero Kristo <[email protected]> writes:

> This prevents system hang while attempting to access suspended console.

Please add more detail.  Who is accessing console?   This sounds more
like it's masking a UART/console bug.

> Signed-off-by: Tero Kristo <[email protected]>
> ---
>  arch/arm/mach-omap2/pm34xx.c |    6 ++++++
>  1 files changed, 6 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/pm34xx.c b/arch/arm/mach-omap2/pm34xx.c
> index e626a3d..dae0a8a 100644
> --- a/arch/arm/mach-omap2/pm34xx.c
> +++ b/arch/arm/mach-omap2/pm34xx.c
> @@ -418,6 +418,10 @@ void omap_sram_idle(void)
>       pwrdm_post_transition();
>  
>       clkdm_allow_idle(mpu_pwrdm->pwrdm_clkdms[0]);
> +     if (suspend_state != PM_SUSPEND_ON) {
> +             disable_irq(prcm_io_irq);
> +             disable_irq(prcm_wkup_irq);
> +     }

>  }
>  
>  int omap3_can_sleep(void)
> @@ -524,6 +528,8 @@ static void omap3_pm_end(void)
>       suspend_state = PM_SUSPEND_ON;
>       omap_uart_enable_irqs(1);
>       enable_hlt();
> +     enable_irq(prcm_io_irq);
> +     enable_irq(prcm_wkup_irq);
>       return;
>  }

The re-enable is in ->end(), shouldn't the disable be in ->begin() ?

Kevin


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

Reply via email to