Commit dca2d0eb authored by Kevin Hilman's avatar Kevin Hilman

OMAP3: PM: fix UART handling when using no_console_suspend

During the idle/suspend path, we expect the console lock to be held so
that no console output is done during/after the UARTs are idled.

However, when using the no_console_suspend argument on the
command-line, the console driver does not take the console lock.  This
allows the possibility of console activity after UARTs have been
disabled.

To fix, update the current is_suspending() to also check the
console_suspend_enabled flag.
Reported-by: default avatarAbhilash Koyamangalath <abhilash.kv@ti.com>
Tested-by: default avatarAbhilash Koyamangalath <abhilash.kv@ti.com>
Signed-off-by: default avatarKevin Hilman <khilman@ti.com>
parent e9a5190a
...@@ -55,7 +55,7 @@ ...@@ -55,7 +55,7 @@
static suspend_state_t suspend_state = PM_SUSPEND_ON; static suspend_state_t suspend_state = PM_SUSPEND_ON;
static inline bool is_suspending(void) static inline bool is_suspending(void)
{ {
return (suspend_state != PM_SUSPEND_ON); return (suspend_state != PM_SUSPEND_ON) && console_suspend_enabled;
} }
#else #else
static inline bool is_suspending(void) static inline bool is_suspending(void)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment