All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] CONFIG_WATCHDOG on MPC85XX and QorIQ P1/P2
@ 2011-03-03  0:10 Felix Radensky
  2011-03-03  4:18 ` Aggrwal Poonam-B10812
  0 siblings, 1 reply; 10+ messages in thread
From: Felix Radensky @ 2011-03-03  0:10 UTC (permalink / raw)
  To: u-boot

Hi,

I'm trying to enable watchdog in u-boot on P1011 based platform.
I've noticed that all 85XX platforms have CONFIG_WATCHDOG
disabled. There's a definition of reset_85xx_watchdog() in
cpu/mpc85xx/cpu.c but no code that actually enables the watchdog.

What should be done to make this feature working on these
platforms ?

Thanks a lot.

Felix.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [U-Boot] CONFIG_WATCHDOG on MPC85XX and QorIQ P1/P2
  2011-03-03  0:10 [U-Boot] CONFIG_WATCHDOG on MPC85XX and QorIQ P1/P2 Felix Radensky
@ 2011-03-03  4:18 ` Aggrwal Poonam-B10812
  2011-03-03  4:20   ` Tabi Timur-B04825
  0 siblings, 1 reply; 10+ messages in thread
From: Aggrwal Poonam-B10812 @ 2011-03-03  4:18 UTC (permalink / raw)
  To: u-boot

Hello Felix

We have not used watchdog feature in u-boot as such.

Timur, do you have any idea about this? Can you give some pointers here.

I will also try to explore it more.

Regards
Poonam 

> -----Original Message-----
> From: Felix Radensky [mailto:felix at embedded-sol.com]
> Sent: Thursday, March 03, 2011 5:41 AM
> To: U-Boot-Denx; Kumar Gala; Aggrwal Poonam-B10812
> Subject: CONFIG_WATCHDOG on MPC85XX and QorIQ P1/P2
> 
> Hi,
> 
> I'm trying to enable watchdog in u-boot on P1011 based platform.
> I've noticed that all 85XX platforms have CONFIG_WATCHDOG disabled.
> There's a definition of reset_85xx_watchdog() in cpu/mpc85xx/cpu.c but no
> code that actually enables the watchdog.
> 
> What should be done to make this feature working on these platforms ?
> 
> Thanks a lot.
> 
> Felix.
> 
> 
> 
> 

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [U-Boot] CONFIG_WATCHDOG on MPC85XX and QorIQ P1/P2
  2011-03-03  4:18 ` Aggrwal Poonam-B10812
@ 2011-03-03  4:20   ` Tabi Timur-B04825
  2011-03-03  4:27     ` Dudhat Dipen-B09055
  0 siblings, 1 reply; 10+ messages in thread
From: Tabi Timur-B04825 @ 2011-03-03  4:20 UTC (permalink / raw)
  To: u-boot

I don't think anyone here has ever turned on watchdog support on an 85xx board.  I think it works on 86xx boards, though.

Aggrwal Poonam-B10812 wrote:
> Hello Felix
>
> We have not used watchdog feature in u-boot as such.
>
> Timur, do you have any idea about this? Can you give some pointers here.
>
> I will also try to explore it more.
>
> Regards
> Poonam
>
>> -----Original Message-----
>> From: Felix Radensky [mailto:felix at embedded-sol.com]
>> Sent: Thursday, March 03, 2011 5:41 AM
>> To: U-Boot-Denx; Kumar Gala; Aggrwal Poonam-B10812
>> Subject: CONFIG_WATCHDOG on MPC85XX and QorIQ P1/P2
>>
>> Hi,
>>
>> I'm trying to enable watchdog in u-boot on P1011 based platform.
>> I've noticed that all 85XX platforms have CONFIG_WATCHDOG disabled.
>> There's a definition of reset_85xx_watchdog() in cpu/mpc85xx/cpu.c but no
>> code that actually enables the watchdog.
>>
>> What should be done to make this feature working on these platforms ?
>>
>> Thanks a lot.
>>
>> Felix.
>>
>>
>>
>>
>


-- 
Timur Tabi
Linux kernel developer

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [U-Boot] CONFIG_WATCHDOG on MPC85XX and QorIQ P1/P2
  2011-03-03  4:20   ` Tabi Timur-B04825
@ 2011-03-03  4:27     ` Dudhat Dipen-B09055
  2011-03-03  7:41       ` Felix Radensky
  0 siblings, 1 reply; 10+ messages in thread
From: Dudhat Dipen-B09055 @ 2011-03-03  4:27 UTC (permalink / raw)
  To: u-boot

Hi All,

Just clarification, felix is asking for WatchDog support on u-boot not on Linux.
If he is asking on Linux then it's there and we have tested it.

Regards,
Dipen

-----Original Message-----
From: u-boot-bounces@lists.denx.de [mailto:u-boot-bounces at lists.denx.de] On Behalf Of Tabi Timur-B04825
Sent: Thursday, March 03, 2011 9:51 AM
To: Aggrwal Poonam-B10812
Cc: U-Boot-Denx; Felix Radensky; Kumar Gala
Subject: Re: [U-Boot] CONFIG_WATCHDOG on MPC85XX and QorIQ P1/P2

I don't think anyone here has ever turned on watchdog support on an 85xx board.  I think it works on 86xx boards, though.

Aggrwal Poonam-B10812 wrote:
> Hello Felix
>
> We have not used watchdog feature in u-boot as such.
>
> Timur, do you have any idea about this? Can you give some pointers here.
>
> I will also try to explore it more.
>
> Regards
> Poonam
>
>> -----Original Message-----
>> From: Felix Radensky [mailto:felix at embedded-sol.com]
>> Sent: Thursday, March 03, 2011 5:41 AM
>> To: U-Boot-Denx; Kumar Gala; Aggrwal Poonam-B10812
>> Subject: CONFIG_WATCHDOG on MPC85XX and QorIQ P1/P2
>>
>> Hi,
>>
>> I'm trying to enable watchdog in u-boot on P1011 based platform.
>> I've noticed that all 85XX platforms have CONFIG_WATCHDOG disabled.
>> There's a definition of reset_85xx_watchdog() in cpu/mpc85xx/cpu.c 
>> but no code that actually enables the watchdog.
>>
>> What should be done to make this feature working on these platforms ?
>>
>> Thanks a lot.
>>
>> Felix.
>>
>>
>>
>>
>


--
Timur Tabi
Linux kernel developer
_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [U-Boot] CONFIG_WATCHDOG on MPC85XX and QorIQ P1/P2
  2011-03-03  4:27     ` Dudhat Dipen-B09055
@ 2011-03-03  7:41       ` Felix Radensky
  2011-03-03  7:49         ` Albert ARIBAUD
  2011-03-03 12:33         ` Tabi Timur-B04825
  0 siblings, 2 replies; 10+ messages in thread
From: Felix Radensky @ 2011-03-03  7:41 UTC (permalink / raw)
  To: u-boot

Hi Dipen,

On 03/03/2011 06:27 AM, Dudhat Dipen-B09055 wrote:
> Hi All,
>
> Just clarification, felix is asking for WatchDog support on u-boot not on Linux.
> If he is asking on Linux then it's there and we have tested it.

Correct. It works fine in linux. I've even tried to port linux code that
enables watchdog (booke_wdt.c/__booke_wdt_enable()) to u-boot,
but was not successful. Watchdog never triggered, although I did
not refresh it.

I'd appreciate some advice on what may be different in u-boot
vs. linux.

Thanks a lot.

Felix.


> Regards,
> Dipen
>
> -----Original Message-----
> From: u-boot-bounces at lists.denx.de [mailto:u-boot-bounces at lists.denx.de] On Behalf Of Tabi Timur-B04825
> Sent: Thursday, March 03, 2011 9:51 AM
> To: Aggrwal Poonam-B10812
> Cc: U-Boot-Denx; Felix Radensky; Kumar Gala
> Subject: Re: [U-Boot] CONFIG_WATCHDOG on MPC85XX and QorIQ P1/P2
>
> I don't think anyone here has ever turned on watchdog support on an 85xx board.  I think it works on 86xx boards, though.
>
> Aggrwal Poonam-B10812 wrote:
>> Hello Felix
>>
>> We have not used watchdog feature in u-boot as such.
>>
>> Timur, do you have any idea about this? Can you give some pointers here.
>>
>> I will also try to explore it more.
>>
>> Regards
>> Poonam
>>
>>> -----Original Message-----
>>> From: Felix Radensky [mailto:felix at embedded-sol.com]
>>> Sent: Thursday, March 03, 2011 5:41 AM
>>> To: U-Boot-Denx; Kumar Gala; Aggrwal Poonam-B10812
>>> Subject: CONFIG_WATCHDOG on MPC85XX and QorIQ P1/P2
>>>
>>> Hi,
>>>
>>> I'm trying to enable watchdog in u-boot on P1011 based platform.
>>> I've noticed that all 85XX platforms have CONFIG_WATCHDOG disabled.
>>> There's a definition of reset_85xx_watchdog() in cpu/mpc85xx/cpu.c
>>> but no code that actually enables the watchdog.
>>>
>>> What should be done to make this feature working on these platforms ?
>>>
>>> Thanks a lot.
>>>
>>> Felix.
>>>
>>>
>>>
>>>
>
> --
> Timur Tabi
> Linux kernel developer
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>
>

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [U-Boot] CONFIG_WATCHDOG on MPC85XX and QorIQ P1/P2
  2011-03-03  7:41       ` Felix Radensky
@ 2011-03-03  7:49         ` Albert ARIBAUD
  2011-03-03 12:33         ` Tabi Timur-B04825
  1 sibling, 0 replies; 10+ messages in thread
From: Albert ARIBAUD @ 2011-03-03  7:49 UTC (permalink / raw)
  To: u-boot

Hi Felix,

Le 03/03/2011 08:41, Felix Radensky a ?crit :

>> Just clarification, felix is asking for WatchDog support on u-boot not on Linux.
>> If he is asking on Linux then it's there and we have tested it.
>
> Correct. It works fine in linux. I've even tried to port linux code that
> enables watchdog (booke_wdt.c/__booke_wdt_enable()) to u-boot,
> but was not successful. Watchdog never triggered, although I did
> not refresh it.
>
> I'd appreciate some advice on what may be different in u-boot
> vs. linux.
>
> Thanks a lot.

Asking the obvious just in case: are you sure the watchdog is *armed*? 
Maybe you could add debug code to test if it is, e.g. regularly 
displaying the remaining count?

> Felix.

Amicalement,
-- 
Albert.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [U-Boot] CONFIG_WATCHDOG on MPC85XX and QorIQ P1/P2
  2011-03-03  7:41       ` Felix Radensky
  2011-03-03  7:49         ` Albert ARIBAUD
@ 2011-03-03 12:33         ` Tabi Timur-B04825
  2011-03-03 13:08           ` Felix Radensky
  1 sibling, 1 reply; 10+ messages in thread
From: Tabi Timur-B04825 @ 2011-03-03 12:33 UTC (permalink / raw)
  To: u-boot

Felix Radensky wrote:
> Correct. It works fine in linux. I've even tried to port linux code that
> enables watchdog (booke_wdt.c/__booke_wdt_enable()) to u-boot,
> but was not successful. Watchdog never triggered, although I did
> not refresh it.
>
> I'd appreciate some advice on what may be different in u-boot
> vs. linux.

I don't know what to tell you.  I would need to try to make it myself in order to see what's going on.  Maybe you really didn't program TCR correctly.  Or maybe it is working, but the default behavior of U-Boot is to handle everything silently.

-- 
Timur Tabi
Linux kernel developer

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [U-Boot] CONFIG_WATCHDOG on MPC85XX and QorIQ P1/P2
  2011-03-03 12:33         ` Tabi Timur-B04825
@ 2011-03-03 13:08           ` Felix Radensky
  2011-03-03 13:11             ` Tabi Timur-B04825
  0 siblings, 1 reply; 10+ messages in thread
From: Felix Radensky @ 2011-03-03 13:08 UTC (permalink / raw)
  To: u-boot

Hi Timur,

On 03/03/2011 02:33 PM, Tabi Timur-B04825 wrote:
> Felix Radensky wrote:
>> Correct. It works fine in linux. I've even tried to port linux code that
>> enables watchdog (booke_wdt.c/__booke_wdt_enable()) to u-boot,
>> but was not successful. Watchdog never triggered, although I did
>> not refresh it.
>>
>> I'd appreciate some advice on what may be different in u-boot
>> vs. linux.
> I don't know what to tell you.  I would need to try to make it myself in order to see what's going on.  Maybe you really didn't program TCR correctly.  Or maybe it is working, but the default behavior of U-Boot is to handle everything silently.

Thanks. I suspect u-boot WDT behaviour should be different from Linux.
Linux sets TCR_WIE bit to enable watchdog interrupt,  u-boot should not
do this and just reset the CPU on first time-out. The TCR values are 
identical
in u-boot and linux, except for TCR_WIE and TCR_DIE bits that are set 
in linux.

Except not setting TCI_WIE bit and setting WRC to 10b, is there 
anything else
that should be done to cause reset of first time-out ?

Felix.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [U-Boot] CONFIG_WATCHDOG on MPC85XX and QorIQ P1/P2
  2011-03-03 13:08           ` Felix Radensky
@ 2011-03-03 13:11             ` Tabi Timur-B04825
  2011-03-07  9:38               ` Felix Radensky
  0 siblings, 1 reply; 10+ messages in thread
From: Tabi Timur-B04825 @ 2011-03-03 13:11 UTC (permalink / raw)
  To: u-boot

Felix Radensky wrote:
>
> Except not setting TCI_WIE bit and setting WRC to 10b, is there anything
> else
> that should be done to cause reset of first time-out ?

Like I said, I would need to debug it myself to know more, and I don't have the time for that.

-- 
Timur Tabi
Linux kernel developer

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [U-Boot] CONFIG_WATCHDOG on MPC85XX and QorIQ P1/P2
  2011-03-03 13:11             ` Tabi Timur-B04825
@ 2011-03-07  9:38               ` Felix Radensky
  0 siblings, 0 replies; 10+ messages in thread
From: Felix Radensky @ 2011-03-07  9:38 UTC (permalink / raw)
  To: u-boot

Hi Timur,

On 03/03/2011 03:11 PM, Tabi Timur-B04825 wrote:
> Felix Radensky wrote:
>> Except not setting TCI_WIE bit and setting WRC to 10b, is there anything
>> else
>> that should be done to cause reset of first time-out ?
> Like I said, I would need to debug it myself to know more, and I don't have the time for that.
>
Thanks anyway. I've found the problem and watchdog works fine now.

Felix.

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2011-03-07  9:38 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-03-03  0:10 [U-Boot] CONFIG_WATCHDOG on MPC85XX and QorIQ P1/P2 Felix Radensky
2011-03-03  4:18 ` Aggrwal Poonam-B10812
2011-03-03  4:20   ` Tabi Timur-B04825
2011-03-03  4:27     ` Dudhat Dipen-B09055
2011-03-03  7:41       ` Felix Radensky
2011-03-03  7:49         ` Albert ARIBAUD
2011-03-03 12:33         ` Tabi Timur-B04825
2011-03-03 13:08           ` Felix Radensky
2011-03-03 13:11             ` Tabi Timur-B04825
2011-03-07  9:38               ` Felix Radensky

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.