Thread (11 messages) flat view 11 messages, 4 authors, 2011-03-10
STALE5667d

[PATCH] omap: wd_timer: Fix crash frm wdt_probe when!CONFIG_RUNTIME_PM

From: Santosh Shilimkar <hidden>
Date: 2011-01-06 03:57:32
Also in: linux-omap

-----Original Message-----
From: Russell King - ARM Linux [mailto:linux at arm.linux.org.uk]
Sent: Thursday, January 06, 2011 3:49 AM
To: Santosh Shilimkar
Cc: linux-omap at vger.kernel.org; Paul Walmsley; linux-arm-
kernel at lists.infradead.org
Subject: Re: [PATCH] omap: wd_timer: Fix crash frm wdt_probe
when!CONFIG_RUNTIME_PM

On Wed, Jan 05, 2011 at 07:10:55PM +0530, Santosh Shilimkar wrote:
quoted
Commit ff2516fb 'wd_timer: disable on boot via hwmod postsetup
mechanism'
quoted
introduced watchdog timer state state management using
postsetup_state.
quoted
This was done to allow some board files to support watchdog
coverage
quoted
throughout kernel initialization and it work as intended when
RUNTIME_PM
quoted
is enabled.

With !CONFIG_RUNTIME_PM and no board is specifically requests
watchdog
quoted
to remain enabled the omap_wdt_probe crashesh. This is because
hwmod
quoted
in absense of runtime PM unable to turn watchdog clocks because
it's
quoted
state is set to be disabled. For rest of the device, the state is
set as enabled in absense of RUNTIME_PM
Err... wasn't this provoked by an attempt to fix the LDP issue, that
is
(I believe) because the boot loader enables the watchdog and pre-
hwmod
kernels used to disable it.  Post-hwmod kernels stopped disabling
the
watchdog, resulting in a few seconds booting userspace before the
system
resets itself.
Yes. That's managed through the shutdown part. Apart from that there
is another enhancement done in case some one wants to have WDT running
throughout the kernel boot.

Regards,
Santosh
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help