Re: [PATCH v4 2/7] OMAP3: PM: Erratum i581 support: dll kick strategy

2010-12-20 Thread Jean Pihet
On Sat, Dec 18, 2010 at 11:53 PM, Nishanth Menon n...@ti.com wrote: From: Peter 'p2' De Schrijver peter.de-schrij...@nokia.com Erratum i581 impacts OMAP3 platforms. PRCM DPLL control FSM removes SDRC_IDLEREQ before DPLL3 locks causing the DPLL not to be locked at times. IMPORTANT: *) This

Re: [PATCH v4 2/7] OMAP3: PM: Erratum i581 support: dll kick strategy

2010-12-20 Thread Peter 'p2' De Schrijver
On Mon, Dec 20, 2010 at 11:23:27AM +0100, ext Jean Pihet wrote: On Sat, Dec 18, 2010 at 11:53 PM, Nishanth Menon n...@ti.com wrote: From: Peter 'p2' De Schrijver peter.de-schrij...@nokia.com Erratum i581 impacts OMAP3 platforms. PRCM DPLL control FSM removes SDRC_IDLEREQ before DPLL3

Re: [PATCH v4 2/7] OMAP3: PM: Erratum i581 support: dll kick strategy

2010-12-20 Thread Nishanth Menon
Santosh Shilimkar had written, on 12/20/2010 12:47 AM, the following: [..] diff --git a/arch/arm/mach-omap2/sleep34xx.S b/arch/arm/mach- omap2/sleep34xx.S index 2c20fcf..3fbd1e5 100644 --- a/arch/arm/mach-omap2/sleep34xx.S +++ b/arch/arm/mach-omap2/sleep34xx.S @@ -42,6 +42,7 @@

Re: [PATCH v4 2/7] OMAP3: PM: Erratum i581 support: dll kick strategy

2010-12-20 Thread Nishanth Menon
Peter 'p2' De Schrijver had written, on 12/20/2010 05:33 AM, the following: [..] + /* these 2 words need to be at the end !!! */ +kick_counter: + .word 0 +wait_dll_lock_counter: + .word 0 Why do they need to be at the end? Also, at the end of what do they need to be? At

RE: [PATCH v4 2/7] OMAP3: PM: Erratum i581 support: dll kick strategy

2010-12-19 Thread Santosh Shilimkar
-Original Message- From: linux-omap-ow...@vger.kernel.org [mailto:linux-omap- ow...@vger.kernel.org] On Behalf Of Nishanth Menon Sent: Sunday, December 19, 2010 4:24 AM To: linux-omap; linux-arm Cc: Jean Pihet; Kevin; Tony Subject: [PATCH v4 2/7] OMAP3: PM: Erratum i581 support: dll

[PATCH v4 2/7] OMAP3: PM: Erratum i581 support: dll kick strategy

2010-12-18 Thread Nishanth Menon
From: Peter 'p2' De Schrijver peter.de-schrij...@nokia.com Erratum i581 impacts OMAP3 platforms. PRCM DPLL control FSM removes SDRC_IDLEREQ before DPLL3 locks causing the DPLL not to be locked at times. IMPORTANT: *) This is not a complete workaround implementation as recommended by the silicon