ARM: ks8695: use [readl|writel]_relaxed()
authorLinus Walleij <linus.walleij@linaro.org>
Wed, 29 Aug 2012 18:27:02 +0000 (20:27 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Wed, 5 Sep 2012 06:49:00 +0000 (08:49 +0200)
I have no clue why __raw* macros are used here, but I strongly
suspect there is no good reason at all for this, so removing
another bad example.

Tested-by: Greg Ungerer <gerg@snapgear.com>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
arch/arm/mach-ks8695/time.c

index 6427b7c2c9faccc785a2ef8e09bd9b0f870c3ee8..ae9daccca4c169d755ce292779c08a97a9bfa0c2 100644 (file)
@@ -67,11 +67,11 @@ static unsigned long ks8695_gettimeoffset (void)
         * interrupt.  We solve this by ensuring that the counter has not
         * reloaded between our two reads.
         */
-       elapsed = __raw_readl(KS8695_TMR_VA + KS8695_T1TC) + __raw_readl(KS8695_TMR_VA + KS8695_T1PD);
+       elapsed = readl_relaxed(KS8695_TMR_VA + KS8695_T1TC) + readl_relaxed(KS8695_TMR_VA + KS8695_T1PD);
        do {
                tick2 = elapsed;
-               intpending = __raw_readl(KS8695_IRQ_VA + KS8695_INTST) & (1 << KS8695_IRQ_TIMER1);
-               elapsed = __raw_readl(KS8695_TMR_VA + KS8695_T1TC) + __raw_readl(KS8695_TMR_VA + KS8695_T1PD);
+               intpending = readl_relaxed(KS8695_IRQ_VA + KS8695_INTST) & (1 << KS8695_IRQ_TIMER1);
+               elapsed = readl_relaxed(KS8695_TMR_VA + KS8695_T1TC) + readl_relaxed(KS8695_TMR_VA + KS8695_T1PD);
        } while (elapsed > tick2);
 
        /* Convert to number of ticks expired (not remaining) */
@@ -106,14 +106,14 @@ static void ks8695_timer_setup(void)
        unsigned long tmcon;
 
        /* disable timer1 */
-       tmcon = __raw_readl(KS8695_TMR_VA + KS8695_TMCON);
-       __raw_writel(tmcon & ~TMCON_T1EN, KS8695_TMR_VA + KS8695_TMCON);
+       tmcon = readl_relaxed(KS8695_TMR_VA + KS8695_TMCON);
+       writel_relaxed(tmcon & ~TMCON_T1EN, KS8695_TMR_VA + KS8695_TMCON);
 
-       __raw_writel(tmout / 2, KS8695_TMR_VA + KS8695_T1TC);
-       __raw_writel(tmout / 2, KS8695_TMR_VA + KS8695_T1PD);
+       writel_relaxed(tmout / 2, KS8695_TMR_VA + KS8695_T1TC);
+       writel_relaxed(tmout / 2, KS8695_TMR_VA + KS8695_T1PD);
 
        /* re-enable timer1 */
-       __raw_writel(tmcon | TMCON_T1EN, KS8695_TMR_VA + KS8695_TMCON);
+       writel_relaxed(tmcon | TMCON_T1EN, KS8695_TMR_VA + KS8695_TMCON);
 }
 
 static void __init ks8695_timer_init (void)
@@ -138,12 +138,12 @@ void ks8695_restart(char mode, const char *cmd)
                soft_restart(0);
 
        /* disable timer0 */
-       reg = __raw_readl(KS8695_TMR_VA + KS8695_TMCON);
-       __raw_writel(reg & ~TMCON_T0EN, KS8695_TMR_VA + KS8695_TMCON);
+       reg = readl_relaxed(KS8695_TMR_VA + KS8695_TMCON);
+       writel_relaxed(reg & ~TMCON_T0EN, KS8695_TMR_VA + KS8695_TMCON);
 
        /* enable watchdog mode */
-       __raw_writel((10 << 8) | T0TC_WATCHDOG, KS8695_TMR_VA + KS8695_T0TC);
+       writel_relaxed((10 << 8) | T0TC_WATCHDOG, KS8695_TMR_VA + KS8695_T0TC);
 
        /* re-enable timer0 */
-       __raw_writel(reg | TMCON_T0EN, KS8695_TMR_VA + KS8695_TMCON);
+       writel_relaxed(reg | TMCON_T0EN, KS8695_TMR_VA + KS8695_TMCON);
 }