ppc4xx: Remove mtsdram0() marcos and use common mtsdram() instead
authorStefan Roese <sr@denx.de>
Thu, 24 Sep 2009 12:10:30 +0000 (14:10 +0200)
committerStefan Roese <sr@denx.de>
Mon, 28 Sep 2009 08:46:05 +0000 (10:46 +0200)
Additionally some whitespace coding style fixes.

Signed-off-by: Stefan Roese <sr@denx.de>
board/g2000/g2000.c
cpu/ppc4xx/40x_spd_sdram.c

index 092a3d8..03cd6b8 100644 (file)
@@ -114,18 +114,17 @@ int checkboard (void)
 
 long int init_sdram_static_settings(void)
 {
-#define mtsdram0(reg, data)  mtdcr(SDRAM0_CFGADDR,reg);mtdcr(SDRAM0_CFGDATA,data)
        /* disable memcontroller so updates work */
-       mtsdram0( SDRAM0_CFG, MEM_MCOPT1_INIT_VAL );
-       mtsdram0( SDRAM0_RTR   , MEM_RTR_INIT_VAL   );
-       mtsdram0( SDRAM0_PMIT  , MEM_PMIT_INIT_VAL  );
-       mtsdram0( SDRAM0_B0CR , MEM_MB0CF_INIT_VAL );
-       mtsdram0( SDRAM0_B1CR , MEM_MB1CF_INIT_VAL );
-       mtsdram0( SDRAM0_TR , MEM_SDTR1_INIT_VAL );
+       mtsdram(SDRAM0_CFG, MEM_MCOPT1_INIT_VAL);
+       mtsdram(SDRAM0_RTR, MEM_RTR_INIT_VAL);
+       mtsdram(SDRAM0_PMIT, MEM_PMIT_INIT_VAL);
+       mtsdram(SDRAM0_B0CR, MEM_MB0CF_INIT_VAL);
+       mtsdram(SDRAM0_B1CR, MEM_MB1CF_INIT_VAL);
+       mtsdram(SDRAM0_TR, MEM_SDTR1_INIT_VAL);
 
        /* SDRAM have a power on delay,  500 micro should do */
        udelay(500);
-       mtsdram0( SDRAM0_CFG, MEM_MCOPT1_INIT_VAL|SDRAM0_CFG_ENABLE );
+       mtsdram(SDRAM0_CFG, MEM_MCOPT1_INIT_VAL|SDRAM0_CFG_ENABLE);
 
        return (CONFIG_SYS_SDRAM_SIZE); /* CONFIG_SYS_SDRAM_SIZE is in G2000.h */
  }
index c50f673..5e6dbe3 100644 (file)
@@ -422,32 +422,31 @@ long int spd_sdram(int(read_spd)(uint addr))
         * program all the registers.
         * -------------------------------------------------------------------*/
 
-#define mtsdram0(reg, data)  mtdcr(SDRAM0_CFGADDR,reg);mtdcr(SDRAM0_CFGDATA,data)
        /* disable memcontroller so updates work */
-       mtsdram0( SDRAM0_CFG, 0 );
+       mtsdram(SDRAM0_CFG, 0);
 
 #ifndef CONFIG_405EP /* not on PPC405EP */
-       mtsdram0( SDRAM0_BESR0 , sdram0_besr0 );
-       mtsdram0( SDRAM0_BESR1 , sdram0_besr1 );
-       mtsdram0( SDRAM0_ECCCFG , sdram0_ecccfg );
-       mtsdram0( SDRAM0_ECCESR, sdram0_eccesr );
+       mtsdram(SDRAM0_BESR0, sdram0_besr0);
+       mtsdram(SDRAM0_BESR1, sdram0_besr1);
+       mtsdram(SDRAM0_ECCCFG, sdram0_ecccfg);
+       mtsdram(SDRAM0_ECCESR, sdram0_eccesr);
 #endif
-       mtsdram0( SDRAM0_RTR   , sdram0_rtr );
-       mtsdram0( SDRAM0_PMIT  , sdram0_pmit );
-       mtsdram0( SDRAM0_B0CR , sdram0_b0cr );
-       mtsdram0( SDRAM0_B1CR , sdram0_b1cr );
+       mtsdram(SDRAM0_RTR, sdram0_rtr);
+       mtsdram(SDRAM0_PMIT, sdram0_pmit);
+       mtsdram(SDRAM0_B0CR, sdram0_b0cr);
+       mtsdram(SDRAM0_B1CR, sdram0_b1cr);
 #ifndef CONFIG_405EP /* not on PPC405EP */
-       mtsdram0( SDRAM0_B2CR , sdram0_b2cr );
-       mtsdram0( SDRAM0_B3CR , sdram0_b3cr );
+       mtsdram(SDRAM0_B2CR, sdram0_b2cr);
+       mtsdram(SDRAM0_B3CR, sdram0_b3cr);
 #endif
-       mtsdram0( SDRAM0_TR , sdram0_tr );
+       mtsdram(SDRAM0_TR, sdram0_tr);
 
        /* SDRAM have a power on delay,  500 micro should do */
        udelay(500);
        sdram0_cfg = SDRAM0_CFG_DCE | SDRAM0_CFG_BRPF(1) | SDRAM0_CFG_ECCDD | SDRAM0_CFG_EMDULR;
        if (ecc_on)
                sdram0_cfg |= SDRAM0_CFG_MEMCHK;
-       mtsdram0(SDRAM0_CFG, sdram0_cfg);
+       mtsdram(SDRAM0_CFG, sdram0_cfg);
 
        return (total_size);
 }