Hi Daniel, On Thu, May 25, 2017 at 10:54:55PM +0200, Daniel Lezcano wrote: > On Sat, May 13, 2017 at 03:29:35PM +0800, Dong Aisheng wrote: > > IMX Timer/PWM Module (TPM) supports both timer and pwm function while > > this patch only adds the timer support. PWM would be added later. > > > > The TPM counter, compare and capture registers are clocked by an > > asynchronous clock that can remain enabled in low power modes. > > > > Cc: Daniel Lezcano <[email protected]> > > Cc: Thomas Gleixner <[email protected]> > > Cc: Shawn Guo <[email protected]> > > Signed-off-by: Anson Huang <[email protected]> > > Signed-off-by: Bai Ping <[email protected]> > > Signed-off-by: Dong Aisheng <[email protected]> > > --- > > [ ... ] > > > + val = __raw_readl(timer_base + TPM_C0SC); > > + val &= ~(TPM_C0SC_MODE_MASK | TPM_C0SC_CHIE); > > + __raw_writel(val, timer_base + TPM_C0SC); > > Hi Dong, > > why are you using __raw_write/__raw_read instead of regular readl/writel? >
Sorry for missing the second throught of it. Will change to readl/writel as Arnd pointed out in another mail. Regards Dong Aisheng

