On Thursday 16 August 2012, Chao Xie wrote: > +static int clk_apbc_prepare(struct clk_hw *hw) > +{ > + struct clk_apbc *apbc = to_clk_apbc(hw); > + unsigned int data; > + unsigned long flags = 0; > + > + /* > + * It may share same register as MUX clock, > + * and it will impact FNCLK enable. Spinlock is needed > + */ > + if (apbc->lock) > + spin_lock_irqsave(apbc->lock, flags); > + > + data = __raw_readl(apbc->base); > + if (apbc->flags & APBC_POWER_CTRL) > + data |= APBC_POWER; > + data |= APBC_FNCLK; > + __raw_writel(data, apbc->base);
It seems you have missed a few __raw_readl/__raw_writel that still want to get changed to readl_relaxed/writel_relaxed. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/