On 2012?07?16? 21:10, Hiremath, Vaibhav wrote:

On Sun, Jul 15, 2012 at 13:14:03, Zumeng Chen wrote:

quoted

Hello,

The following patches based on the 3.5-rc6 from Wim, which
focus on:

1. bootstatus fix for omap3,

2. omap-wdt framework update cater for the current framework
    as Shubhrajyoti comments mentioned.

V3 changes:

1. New comments updated as Kevin mentioned in the third patch;
2. 3530evm works well,
    AM33xx seems work well with the following changes:
	+   if (cpu_is_am335x())
	+       return omap2_prm_read_mod_reg(AM33XX_PRM_DEVICE_MOD,
	+           AM33XX_PRM_RSTST_OFFSET)&  0x7f;
    But since some definitions not ready for am33xx, so I don't
    give the patch, if they have been updated, feel free to take these.

Did you test it on any of AM33xx platform?

Yes. I ever did in v2 as follows shown:

The same case works well on am335xevm but with a new patch in
mach-omap2/prcm.c

I'll send it later.

root at ti-omap3:~# uname -a
Linux ti-omap3 3.4.3-00635-g82d1d26-dirty #32 Wed Jul 11 16:02:12 CST
2012 armv7l GNU/Linux
root at ti-omap3:~# dmesg|grep WDT
[ 1.921173] omap_wdt: OMAP WDTimer Rev 0x01: Initial timeout 0sec status= 0x1
root at ti-omap3:~# ./a.out -i 20; for i in `seq 1 20`; do echo $i ; sleep
1;done
Set watchdog interval to 20
Current watchdog interval is 20
Last boot is caused by : Watchdog
Use:
<w>  to kick through writing over device file
<i>  to kick through IOCTL
<x>  to exit the program
x

1
2
[snip]
U-Boot SPL 2011.09 (Feb 09 2012 - 15:38:59)
Texas Instruments Revision detection unimplemented


U-Boot 2011.09 (Feb 09 2012 - 15:11:31)

I2C: ready
DRAM: 256 MiB
WARNING: Caches not enabled
Found a daughter card connected
NAND: HW ECC Hamming Code selected
256 MiB
MMC: OMAP SD/MMC: 0
Net: cpsw
Hit any key to stop autoboot: 0
U-Boot#

  If you use linux-omap/master pr
linux-next branch as a baseline you should have basic things (except hwmod
data) available.

I will test it and send a patch for this on your behalf (if you are ok with
it).

I'm OK for this, thanks Vaibhav.

Note: Change in convention, cpu_is_am335x() =>  soc_is_am335x()

Yes, agreed.

Regards,
Zumeng

Thanks,
Vaibhav