On Monday 02 November 2015 13:56:31 Daniel Lezcano wrote: > static inline void rk_timer_disable(struct clock_event_device *ce) > { > writel_relaxed(TIMER_DISABLE, rk_base(ce) + TIMER_CONTROL_REG); > - dsb(); > + dsb(sy); > } > > static inline void rk_timer_enable(struct clock_event_device *ce, u32 flags) > { > writel_relaxed(TIMER_ENABLE | TIMER_INT_UNMASK | flags, > rk_base(ce) + TIMER_CONTROL_REG); > - dsb(); > + dsb(sy); > } > > This will fail the compile test, because dsb() is not available on non-ARM architectures. Would it be enough to just use the normal writel() accessor here? Arnd