diff mbox

[U-Boot,1/2] mx31pdk: Disable watchdog

Message ID 1316128703-1330-1-git-send-email-festevam@gmail.com
State Changes Requested
Delegated to: Stefano Babic
Headers show

Commit Message

Fabio Estevam Sept. 15, 2011, 11:18 p.m. UTC
When booting a mainline kernel on a mx31pdk the system gets getting resets from the watchdog.

As the kernel has watchdog support, disable it from U-boot.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 include/configs/mx31pdk.h |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

Comments

Stefano Babic Sept. 16, 2011, 7:46 a.m. UTC | #1
On 09/16/2011 01:18 AM, Fabio Estevam wrote:

Hi Fabio,

> When booting a mainline kernel on a mx31pdk the system gets getting resets from the watchdog.
> 
> As the kernel has watchdog support, disable it from U-boot.

But if the kernel has support for watchdog, why does the system reset ?
I checked this board and the watchdog in u-boot is set to 64 seconds,
that is the maximum value. It is surely enough for kernel to boot and to
start triggering the watchdog. If the system still reboots, it seems to
me that this is an issue to be fixed in kernel, not in u-boot. I am sure
I have tested in the past (not recently) the QONG board with enabled
watchdog, using also the imx2_wdt driver in kernel.

The watchdog in u-boot helps if u-boot hangs somewhere, for example
transfering the image from media. It is a different feature.

Best regards,
Stefano Babic
Sergei Shtylyov Sept. 16, 2011, 11:10 a.m. UTC | #2
Hello.

On 16-09-2011 3:18, Fabio Estevam wrote:

> When booting a mainline kernel on a mx31pdk the system gets getting resets from the watchdog.

    That's tautological. Maybe "is getting reset"?

> As the kernel has watchdog support, disable it from U-boot.

> Signed-off-by: Fabio Estevam<fabio.estevam@freescale.com>

WBR, Sergei
Fabio Estevam Sept. 19, 2011, 3:04 p.m. UTC | #3
Hi Stefano,

On Fri, Sep 16, 2011 at 4:46 AM, Stefano Babic <sbabic@denx.de> wrote:
> On 09/16/2011 01:18 AM, Fabio Estevam wrote:
>
> Hi Fabio,
>
>> When booting a mainline kernel on a mx31pdk the system gets getting resets from the watchdog.
>>
>> As the kernel has watchdog support, disable it from U-boot.
>
> But if the kernel has support for watchdog, why does the system reset ?
> I checked this board and the watchdog in u-boot is set to 64 seconds,
> that is the maximum value. It is surely enough for kernel to boot and to
> start triggering the watchdog. If the system still reboots, it seems to
> me that this is an issue to be fixed in kernel, not in u-boot. I am sure
> I have tested in the past (not recently) the QONG board with enabled
> watchdog, using also the imx2_wdt driver in kernel.

I have sent a patch to the ARM kernel list to add watchdog support for
qong board:
http://lists.infradead.org/pipermail/linux-arm-kernel/2011-September/065698.html

Can you please try it and let me know if you get resets when watchdog
is enabled in U-boot
and kernel?

Thanks,

Fabio Estevam
Stefano Babic Sept. 19, 2011, 4:24 p.m. UTC | #4
On 09/19/2011 05:04 PM, Fabio Estevam wrote:
> Hi Stefano,
> 
> On Fri, Sep 16, 2011 at 4:46 AM, Stefano Babic <sbabic@denx.de> wrote:
>> On 09/16/2011 01:18 AM, Fabio Estevam wrote:
>>
>> Hi Fabio,
>>
>>> When booting a mainline kernel on a mx31pdk the system gets getting resets from the watchdog.
>>>
>>> As the kernel has watchdog support, disable it from U-boot.
>>
>> But if the kernel has support for watchdog, why does the system reset ?
>> I checked this board and the watchdog in u-boot is set to 64 seconds,
>> that is the maximum value. It is surely enough for kernel to boot and to
>> start triggering the watchdog. If the system still reboots, it seems to
>> me that this is an issue to be fixed in kernel, not in u-boot. I am sure
>> I have tested in the past (not recently) the QONG board with enabled
>> watchdog, using also the imx2_wdt driver in kernel.
> 
> I have sent a patch to the ARM kernel list to add watchdog support for
> qong board:
> http://lists.infradead.org/pipermail/linux-arm-kernel/2011-September/065698.html

Ok, thanks.

> 
> Can you please try it and let me know if you get resets when watchdog
> is enabled in U-boot
> and kernel?

It works, the board does not get reset (of course, it is required to
trigger at least once the watchdog under the kernel). I send my
Tested-by to linux-arm.

Best regards,
Stefano Babic
Fabio Estevam Sept. 19, 2011, 4:57 p.m. UTC | #5
On Mon, Sep 19, 2011 at 1:24 PM, Stefano Babic <sbabic@denx.de> wrote:
...
> It works, the board does not get reset (of course, it is required to
> trigger at least once the watchdog under the kernel). I send my
> Tested-by to linux-arm.

Yes, looks like on my mx31pdk the kernel never services the watchdog.

Will try to debug the kernel then.

Thanks for testing it on qong board.

Regards,

Fabio Estevam
Fabio Estevam Sept. 19, 2011, 5:47 p.m. UTC | #6
Hi Stefano,

On Mon, Sep 19, 2011 at 1:57 PM, Fabio Estevam <festevam@gmail.com> wrote:
> On Mon, Sep 19, 2011 at 1:24 PM, Stefano Babic <sbabic@denx.de> wrote:
> ...
>> It works, the board does not get reset (of course, it is required to
>> trigger at least once the watchdog under the kernel). I send my
>> Tested-by to linux-arm.

I managed to fix this and will submit a patch for U-boot shortly.

Regards,

Fabio Estevam
diff mbox

Patch

diff --git a/include/configs/mx31pdk.h b/include/configs/mx31pdk.h
index 1a5bdd1..e77805c 100644
--- a/include/configs/mx31pdk.h
+++ b/include/configs/mx31pdk.h
@@ -60,7 +60,6 @@ 
 
 #define CONFIG_MXC_UART
 #define CONFIG_SYS_MX31_UART1
-#define CONFIG_HW_WATCHDOG
 #define CONFIG_MXC_GPIO
 
 #define CONFIG_HARD_SPI