Re: [U-Boot] [PATCH v2][repost] arm: Kirkwood: add SYSRSTn Duration Counter Support

2009-08-24 Thread Prafulla Wadaskar
> -Original Message- > From: Wolfgang Denk [mailto:w...@denx.de] > Sent: Saturday, August 22, 2009 3:37 AM > To: Prafulla Wadaskar > Cc: u-boot@lists.denx.de; Ashish Karkare; Prabhanjan Sarnaik; > Ronen Shitrit > Subject: Re: [U-Boot] [PATCH v2][repost] arm: Kir

Re: [U-Boot] [PATCH v2][repost] arm: Kirkwood: add SYSRSTn Duration Counter Support

2009-08-21 Thread Wolfgang Denk
Dear Prafulla Wadaskar, In message <73173d32e9439e4abb5151606c3e19e202e3915...@sc-vexch1.marvell.com> you wrote: > > > > > > + sysrst_cnt = (0x1fff & readl(KW_REG_SYSRST_CNT)); > > > > > + printf("H/w Rst hold time: %d.%d secs\n", > > > > > + sysrst_cnt / SYSRST_CNT_1SEC_

Re: [U-Boot] [PATCH v2][repost] arm: Kirkwood: add SYSRSTn Duration Counter Support

2009-08-20 Thread Jean-Christophe PLAGNIOL-VILLARD
> > Okay I got it > > WE can do this but, > > There are just two APIs, overall size impact is not much > > (<100 bytes max). > The actual u-boot.bin size diff for newly posted patch is (169464- 169344= > 120 bytes) ok fine Best Regards, J. ___ U-Boot m

Re: [U-Boot] [PATCH v2][repost] arm: Kirkwood: add SYSRSTn Duration Counter Support

2009-08-20 Thread Prafulla Wadaskar
Sarnaik; > Ronen Shitrit > Subject: Re: [U-Boot] [PATCH v2][repost] arm: Kirkwood: add > SYSRSTn Duration Counter Support > > > > > -Original Message- > > From: Jean-Christophe PLAGNIOL-VILLARD > [mailto:plagn...@jcrosoft.com] > > Sent: Thursday,

Re: [U-Boot] [PATCH v2][repost] arm: Kirkwood: add SYSRSTn Duration Counter Support

2009-08-20 Thread Prafulla Wadaskar
> -Original Message- > From: Wolfgang Denk [mailto:w...@denx.de] > Sent: Thursday, August 20, 2009 3:08 PM > To: Prafulla Wadaskar > Cc: u-boot@lists.denx.de; Ashish Karkare; Prabhanjan Sarnaik; > Ronen Shitrit > Subject: Re: [U-Boot] [PATCH v2][repost] arm: Kir

Re: [U-Boot] [PATCH v2][repost] arm: Kirkwood: add SYSRSTn Duration Counter Support

2009-08-20 Thread Prafulla Wadaskar
ot] [PATCH v2][repost] arm: Kirkwood: add > SYSRSTn Duration Counter Support > > > > > Signed-off-by: Prafulla Wadaskar > > > > --- > > > > Change log: > > > > v2: updated as per review feedback for v1 > > > > bug fix in the previous

Re: [U-Boot] [PATCH v2][repost] arm: Kirkwood: add SYSRSTn Duration Counter Support

2009-08-20 Thread Wolfgang Denk
Dear Prafulla Wadaskar, In message <73173d32e9439e4abb5151606c3e19e202e3915...@sc-vexch1.marvell.com> you wrote: > > > > + if (!s) { > > > + printf("Error.. %s failed, check sysrstcmd\n", > > > + __FUNCTION__); > > > + return; > > > > Why is this considered an er

Re: [U-Boot] [PATCH v2][repost] arm: Kirkwood: add SYSRSTn Duration Counter Support

2009-08-20 Thread Prafulla Wadaskar
> -Original Message- > From: Wolfgang Denk [mailto:w...@denx.de] > Sent: Wednesday, August 19, 2009 12:50 PM > To: Prafulla Wadaskar > Cc: u-boot@lists.denx.de; Ashish Karkare; Prabhanjan Sarnaik; > Ronen Shitrit > Subject: Re: [U-Boot] [PATCH v2][repost] arm: Kir

Re: [U-Boot] [PATCH v2][repost] arm: Kirkwood: add SYSRSTn Duration Counter Support

2009-08-19 Thread Jean-Christophe PLAGNIOL-VILLARD
> > > Signed-off-by: Prafulla Wadaskar > > > --- > > > Change log: > > > v2: updated as per review feedback for v1 > > > bug fix in the previous post (V2) fixed > > ok > > > > but I think make optionnal will be better > Hi Jean > Thanks.. > I didn't understod what you want to say here, can you pl

Re: [U-Boot] [PATCH v2][repost] arm: Kirkwood: add SYSRSTn Duration Counter Support

2009-08-19 Thread Prafulla Wadaskar
> -Original Message- > From: Jean-Christophe PLAGNIOL-VILLARD [mailto:plagn...@jcrosoft.com] > Sent: Thursday, August 20, 2009 4:00 AM > To: Prafulla Wadaskar > Cc: u-boot@lists.denx.de; Ashish Karkare; Prabhanjan Sarnaik; > Ronen Shitrit > Subject: Re: [U-Boot] [

Re: [U-Boot] [PATCH v2][repost] arm: Kirkwood: add SYSRSTn Duration Counter Support

2009-08-19 Thread Jean-Christophe PLAGNIOL-VILLARD
On 16:24 Wed 19 Aug , Prafulla Wadaskar wrote: > I am sorry for previous post v2, pls ignore it, this is the right patch for > the same > > This feature can be used to trigger special command "sysrstcmd" using > reset key long press event and environment variable "sysrstdelay" is set > (usefu

Re: [U-Boot] [PATCH v2][repost] arm: Kirkwood: add SYSRSTn Duration Counter Support

2009-08-19 Thread Wolfgang Denk
Dear Prafulla Wadaskar, In message <1250679240-17557-1-git-send-email-prafu...@marvell.com> you wrote: > I am sorry for previous post v2, pls ignore it, this is the right patch for > the same This comment does not belong to the commit message. Please move below the "---" line. > This feature ca

[U-Boot] [PATCH v2][repost] arm: Kirkwood: add SYSRSTn Duration Counter Support

2009-08-18 Thread Prafulla Wadaskar
I am sorry for previous post v2, pls ignore it, this is the right patch for the same This feature can be used to trigger special command "sysrstcmd" using reset key long press event and environment variable "sysrstdelay" is set (useful for reset to factory or manufacturing mode execution) Kirkwo