From mboxrd@z Thu Jan  1 00:00:00 1970
From: peter.chen@freescale.com (Peter Chen)
Date: Fri, 7 Jun 2013 16:31:35 +0800
Subject: [PATCH v2 1/1] ARM: imx: clk-pllv3: change wait method for PLL
 lock
In-Reply-To: <20130607082457.GF12361@pengutronix.de>
References: <1370593146-14025-1-git-send-email-peter.chen@freescale.com>
 <20130607082457.GF12361@pengutronix.de>
Message-ID: <20130607083134.GE21641@nchen-desktop>
To: linux-arm-kernel@lists.infradead.org
List-Id: linux-arm-kernel.lists.infradead.org

On Fri, Jun 07, 2013 at 10:24:57AM +0200, Uwe Kleine-K?nig wrote:
> On Fri, Jun 07, 2013 at 04:19:06PM +0800, Peter Chen wrote:
> > For some unknown reasons, the jiffies will be updated more
> > than one tick at every short time. Eg, at this code:
> > After timeout = jiffies + msecs_to_jiffies(10),
> > the interrupt occurs, and the softirq updates jiffies (eg,  + 2 jiffies),
> > then return back from interrupt, the time between above operations
> > are tiny, the PLL is still not locked, but the timeout condition is satisfied.
> > 
> > Signed-off-by: Peter Chen <peter.chen@freescale.com>
> Does this mean my patch doesn't solve the issue for you?

Not try, but your patch just delay timeout assignment, if there
is jiffies update after that but before timeout check, it still
has problem.

> 
> Best regards
> Uwe
> 
> 
> -- 
> Pengutronix e.K.                           | Uwe Kleine-K?nig            |
> Industrial Linux Solutions                 | http://www.pengutronix.de/  |
> 

-- 

Best Regards,
Peter Chen