Heiko Stübner wrote: > > The changed statement should set the old armdiv bits to 0 > and not everything else, before setting the new value. > > Signed-off-by: Heiko Stuebner <heiko@xxxxxxxxx> > --- > arch/arm/mach-s3c2443/clock.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/mach-s3c2443/clock.c b/arch/arm/mach-s3c2443/clock.c > index 966bde5..27549c4 100644 > --- a/arch/arm/mach-s3c2443/clock.c > +++ b/arch/arm/mach-s3c2443/clock.c > @@ -124,7 +124,7 @@ static int s3c2443_armclk_setrate(struct clk *clk, unsigned > long rate) > unsigned long clkcon0; > > clkcon0 = __raw_readl(S3C2443_CLKDIV0); > - clkcon0 &= S3C2443_CLKDIV0_ARMDIV_MASK; > + clkcon0 &= ~S3C2443_CLKDIV0_ARMDIV_MASK; > clkcon0 |= val << S3C2443_CLKDIV0_ARMDIV_SHIFT; > __raw_writel(clkcon0, S3C2443_CLKDIV0); > } > -- > 1.7.2.3 Hi Heiko, Yes, you're right. Applied, thanks. Best regards, Kgene. -- Kukjin Kim <kgene.kim@xxxxxxxxxxx>, Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd. -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html