[PATCH] ARM: ixp4xx: Add "ask" handler for timer interrupts
Arnd Bergmann
arnd at arndb.de
Sat Dec 1 16:25:51 EST 2012
On Saturday 01 December 2012, Alexander Shiyan wrote:
> + switch (d->irq) {
> + case IRQ_IXP4XX_TIMER1:
> + *IXP4XX_OSST = IXP4XX_OSST_TIMER_1_PEND;
> + break;
> + case IRQ_IXP4XX_TIMER2:
> + *IXP4XX_OSST = IXP4XX_OSST_TIMER_2_PEND;
> + break;
> + case IRQ_IXP4XX_TIMESTAMP:
> + *IXP4XX_OSST = IXP4XX_OSST_TIMER_TS_PEND;
> + break;
> + case IRQ_IXP4XX_WDOG:
> + *IXP4XX_OSST = IXP4XX_OSST_TIMER_WDOG_PEND;
> + break;
Since you are touching these lines, it probably makes sense to convert them
to use writel_relaxed() in the process. Dereferencing a volatile pointer
in order to do MMIO is strongly discouraged, see
Documentation/volatile-considered-harmful.txt
Arnd
More information about the linux-arm-kernel
mailing list