rockchip/rk3399: convert to for-loops to save code space
authorDerek Basehore <dbasehore@chromium.org>
Sat, 13 May 2017 04:29:13 +0000 (21:29 -0700)
committerCaesar Wang <wxt@rock-chips.com>
Thu, 8 Jun 2017 01:59:53 +0000 (09:59 +0800)
This converts two functions to use for-loops. This saves a bit of
space to help moving DRAM resume code to PMUSRAM.

Change-Id: Ie6ca490cf50c2ec83335cf1845b337c3e8a47496
Signed-off-by: Derek Basehore <dbasehore@chromium.org>
plat/rockchip/rk3399/drivers/dram/suspend.c

index e892844ea0db0cd5c8d6c8f60c7903add24c34b2..608482ab740ae5b79f28f0c0b8f602ad24229fec 100644 (file)
@@ -121,11 +121,12 @@ static __sramfunc void phy_pctrl_reset(uint32_t ch)
 
 static __sramfunc void set_cs_training_index(uint32_t ch, uint32_t rank)
 {
+       uint32_t byte;
+
        /* PHY_8/136/264/392 phy_per_cs_training_index_X 1bit offset_24 */
-       mmio_clrsetbits_32(PHY_REG(ch, 8), 0x1 << 24, rank << 24);
-       mmio_clrsetbits_32(PHY_REG(ch, 136), 0x1 << 24, rank << 24);
-       mmio_clrsetbits_32(PHY_REG(ch, 264), 0x1 << 24, rank << 24);
-       mmio_clrsetbits_32(PHY_REG(ch, 392), 0x1 << 24, rank << 24);
+       for (byte = 0; byte < 4; byte++)
+               mmio_clrsetbits_32(PHY_REG(ch, 8 + (128 * byte)), 0x1 << 24,
+                                  rank << 24);
 }
 
 static __sramfunc void select_per_cs_training_index(uint32_t ch, uint32_t rank)
@@ -139,19 +140,17 @@ static void override_write_leveling_value(uint32_t ch)
 {
        uint32_t byte;
 
-       /*
-        * PHY_8/136/264/392
-        * phy_per_cs_training_multicast_en_X 1bit offset_16
-        */
-       mmio_clrsetbits_32(PHY_REG(ch, 8), 0x1 << 16, 1 << 16);
-       mmio_clrsetbits_32(PHY_REG(ch, 136), 0x1 << 16, 1 << 16);
-       mmio_clrsetbits_32(PHY_REG(ch, 264), 0x1 << 16, 1 << 16);
-       mmio_clrsetbits_32(PHY_REG(ch, 392), 0x1 << 16, 1 << 16);
-
-       for (byte = 0; byte < 4; byte++)
+       for (byte = 0; byte < 4; byte++) {
+               /*
+                * PHY_8/136/264/392
+                * phy_per_cs_training_multicast_en_X 1bit offset_16
+                */
+               mmio_clrsetbits_32(PHY_REG(ch, 8 + (128 * byte)), 0x1 << 16,
+                                  1 << 16);
                mmio_clrsetbits_32(PHY_REG(ch, 63 + (128 * byte)),
                                   0xffff << 16,
                                   0x200 << 16);
+       }
 
        /* CTL_200 ctrlupd_req 1bit offset_8 */
        mmio_clrsetbits_32(CTL_REG(ch, 200), 0x1 << 8, 0x1 << 8);