From mboxrd@z Thu Jan 1 00:00:00 1970 From: linux@prisktech.co.nz (Tony Prisk) Date: Fri, 15 Mar 2013 06:48:27 +1300 Subject: [PATCH] drivers/clk: fixup commit 72480014b86c8b51fb51c5c6a0525876055c37c7 In-Reply-To: <514191F3.7030203@asianux.com> References: <513C0280.7040004@asianux.com> <514191F3.7030203@asianux.com> Message-ID: <1363283307.12019.2.camel@gitbox> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, 2013-03-14 at 17:01 +0800, Chen Gang wrote: > is there any member to help checking and applying it ? > > this issue is repeated again by randconfig make, today. > > > ? 2013?03?10? 11:48, Chen Gang ??: > > > > prate is not defined, will cause compiling issue. > > need use parent_rate instead of prate. > > > > Signed-off-by: Chen Gang > > --- > > drivers/clk/clk-vt8500.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/drivers/clk/clk-vt8500.c b/drivers/clk/clk-vt8500.c > > index b5538bb..09c6331 100644 > > --- a/drivers/clk/clk-vt8500.c > > +++ b/drivers/clk/clk-vt8500.c > > @@ -157,7 +157,7 @@ static int vt8500_dclk_set_rate(struct clk_hw *hw, unsigned long rate, > > divisor = parent_rate / rate; > > > > /* If prate / rate would be decimal, incr the divisor */ > > - if (rate * divisor < *prate) > > + if (rate * divisor < parent_rate) > > divisor++; > > > > if (divisor == cdev->div_mask + 1) > > > > The patch for this is in Mike's clk-fixes branch. Regards Tony Prisk