twl4030: make twl4030_i2c_write_u8 prototype consistent

u-boot standard i2c register write prototype is
i2c_reg_write(u8 addr, u8 reg, u8 val)

twl4030_i2c_write_u8(u8 addr, u8 val, u8 reg)
does not provide consistency, so switch the prototype to be
consistent with rest of u-boot i2c operations:
twl4030_i2c_write_u8(u8 addr, u8 reg, u8 val)

Signed-off-by: Nishanth Menon <nm@ti.com>

diff --git a/board/cm_t35/cm_t35.c b/board/cm_t35/cm_t35.c
index 84c36bafb414d56004fa8de98682bd2995ab5c1d..87d73546a2fd96f414c7fdc5ae918723c622ae82 100644 (file)
--- a/board/cm_t35/cm_t35.c
+++ b/board/cm_t35/cm_t35.c
@@ -493,17 +493,17 @@ static void setup_net_chip_gmpc(void)
 static void reset_net_chip(void)
 {
        /* Set GPIO1 of TPS65930 as output */
-       twl4030_i2c_write_u8(TWL4030_CHIP_GPIO, 0x02,
-                               TWL4030_BASEADD_GPIO + 0x03);
+       twl4030_i2c_write_u8(TWL4030_CHIP_GPIO, TWL4030_BASEADD_GPIO + 0x03,
+                            0x02);
        /* Send a pulse on the GPIO pin */
-       twl4030_i2c_write_u8(TWL4030_CHIP_GPIO, 0x02,
-                               TWL4030_BASEADD_GPIO + 0x0C);
+       twl4030_i2c_write_u8(TWL4030_CHIP_GPIO, TWL4030_BASEADD_GPIO + 0x0C,
+                            0x02);
        udelay(1);
-       twl4030_i2c_write_u8(TWL4030_CHIP_GPIO, 0x02,
-                               TWL4030_BASEADD_GPIO + 0x09);
+       twl4030_i2c_write_u8(TWL4030_CHIP_GPIO, TWL4030_BASEADD_GPIO + 0x09,
+                            0x02);
        mdelay(40);
-       twl4030_i2c_write_u8(TWL4030_CHIP_GPIO, 0x02,
-                               TWL4030_BASEADD_GPIO + 0x0C);
+       twl4030_i2c_write_u8(TWL4030_CHIP_GPIO, TWL4030_BASEADD_GPIO + 0x0C,
+                            0x02);
        mdelay(1);
 }
 #else
@@ -600,10 +600,10 @@ int ehci_hcd_init(int index, struct ehci_hccr **hccr, struct ehci_hcor **hcor)
        twl4030_i2c_read_u8(TWL4030_CHIP_GPIO, &val, offset);
        /* Set GPIO6 and GPIO7 of TPS65930 as output */
        val |= 0xC0;
-       twl4030_i2c_write_u8(TWL4030_CHIP_GPIO, val, offset);
+       twl4030_i2c_write_u8(TWL4030_CHIP_GPIO, offset, val);
        offset = TWL4030_BASEADD_GPIO + TWL4030_GPIO_SETGPIODATAOUT1;
        /* Take both PHYs out of reset */
-       twl4030_i2c_write_u8(TWL4030_CHIP_GPIO, 0xC0, offset);
+       twl4030_i2c_write_u8(TWL4030_CHIP_GPIO, offset, 0xC0);
        udelay(1);
 
        return omap_ehci_hcd_init(&usbhs_bdata, hccr, hcor);

diff --git a/board/nokia/rx51/rx51.c b/board/nokia/rx51/rx51.c
index 48eb65f8968365a918adf893cd1780f3e8f16d74..860d52da3f141aa9b7e7555098c957182200633c 100644 (file)
--- a/board/nokia/rx51/rx51.c
+++ b/board/nokia/rx51/rx51.c
@@ -332,10 +332,10 @@ void *video_hw_init(void)
 static void twl4030_regulator_set_mode(u8 id, u8 mode)
 {
        u16 msg = MSG_SINGULAR(DEV_GRP_P1, id, mode);
-       twl4030_i2c_write_u8(TWL4030_CHIP_PM_MASTER, msg >> 8,
-                       TWL4030_PM_MASTER_PB_WORD_MSB);
-       twl4030_i2c_write_u8(TWL4030_CHIP_PM_MASTER, msg & 0xff,
-                       TWL4030_PM_MASTER_PB_WORD_LSB);
+       twl4030_i2c_write_u8(TWL4030_CHIP_PM_MASTER,
+                            TWL4030_PM_MASTER_PB_WORD_MSB, msg >> 8);
+       twl4030_i2c_write_u8(TWL4030_CHIP_PM_MASTER,
+                            TWL4030_PM_MASTER_PB_WORD_LSB, msg & 0xff);
 }
 
 static void omap3_emu_romcode_call(u32 service_id, u32 *parameters)
@@ -410,8 +410,8 @@ int misc_init_r(void)
                        TWL4030_PM_MASTER_PB_CFG);
 
        /* enable I2C access to powerbus (needed for twl4030 regulator) */
-       twl4030_i2c_write_u8(TWL4030_CHIP_PM_MASTER, 0x02,
-                       TWL4030_PM_MASTER_PB_CFG);
+       twl4030_i2c_write_u8(TWL4030_CHIP_PM_MASTER, TWL4030_PM_MASTER_PB_CFG,
+                            0x02);
 
        /* set VAUX3, VSIM and VMMC1 state to active - enable eMMC memory */
        twl4030_regulator_set_mode(RES_VAUX3, RES_STATE_ACTIVE);
@@ -419,8 +419,8 @@ int misc_init_r(void)
        twl4030_regulator_set_mode(RES_VMMC1, RES_STATE_ACTIVE);
 
        /* restore I2C access state */
-       twl4030_i2c_write_u8(TWL4030_CHIP_PM_MASTER, state,
-                       TWL4030_PM_MASTER_PB_CFG);
+       twl4030_i2c_write_u8(TWL4030_CHIP_PM_MASTER, TWL4030_PM_MASTER_PB_CFG,
+                            state);
 
        /* set env variable attkernaddr for relocated kernel */
        sprintf(buf, "%#x", KERNEL_ADDRESS);
@@ -481,8 +481,8 @@ void hw_watchdog_reset(void)
        /* timeout 0 means watchdog is disabled */
        /* reset watchdog timeout to 31s (maximum) */
        if (timeout != 0)
-               twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER, 31,
-                               TWL4030_PM_RECEIVER_WATCHDOG_CFG);
+               twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER,
+                                    TWL4030_PM_RECEIVER_WATCHDOG_CFG, 31);
 
        /* store last watchdog reset time */
        twl_wd_time = get_timer(0);
@@ -541,18 +541,18 @@ int rx51_kp_init(void)
        ctrl |= TWL4030_KEYPAD_CTRL_KBD_ON;
        ctrl |= TWL4030_KEYPAD_CTRL_SOFT_NRST;
        ctrl |= TWL4030_KEYPAD_CTRL_SOFTMODEN;
-       ret |= twl4030_i2c_write_u8(TWL4030_CHIP_KEYPAD, ctrl,
-                               TWL4030_KEYPAD_KEYP_CTRL_REG);
+       ret |= twl4030_i2c_write_u8(TWL4030_CHIP_KEYPAD,
+                                   TWL4030_KEYPAD_KEYP_CTRL_REG, ctrl);
        /* enable key event status */
-       ret |= twl4030_i2c_write_u8(TWL4030_CHIP_KEYPAD, 0xfe,
-                               TWL4030_KEYPAD_KEYP_IMR1);
+       ret |= twl4030_i2c_write_u8(TWL4030_CHIP_KEYPAD,
+                                   TWL4030_KEYPAD_KEYP_IMR1, 0xfe);
        /* enable interrupt generation on rising and falling */
        /* this is a workaround for qemu twl4030 emulation */
-       ret |= twl4030_i2c_write_u8(TWL4030_CHIP_KEYPAD, 0x57,
-                               TWL4030_KEYPAD_KEYP_EDR);
+       ret |= twl4030_i2c_write_u8(TWL4030_CHIP_KEYPAD,
+                                   TWL4030_KEYPAD_KEYP_EDR, 0x57);
        /* enable ISR clear on read */
-       ret |= twl4030_i2c_write_u8(TWL4030_CHIP_KEYPAD, 0x05,
-                               TWL4030_KEYPAD_KEYP_SIH_CTRL);
+       ret |= twl4030_i2c_write_u8(TWL4030_CHIP_KEYPAD,
+                                   TWL4030_KEYPAD_KEYP_SIH_CTRL, 0x05);
        return 0;
 }
 

diff --git a/board/pandora/pandora.c b/board/pandora/pandora.c
index 9ff5dd76641b15fd1a1f127f39846001673a5d59..5f0c58d8d2ce1ab47fef6b54bd9a68bd7bd55ef6 100644 (file)
--- a/board/pandora/pandora.c
+++ b/board/pandora/pandora.c
@@ -114,8 +114,9 @@ int misc_init_r(void)
 
        /* Enable battery backup capacitor (3.2V, 0.5mA charge current) */
        twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER,
+               TWL4030_PM_RECEIVER_BB_CFG,
                TWL4030_BB_CFG_BBCHEN | TWL4030_BB_CFG_BBSEL_3200MV |
-               TWL4030_BB_CFG_BBISEL_500UA, TWL4030_PM_RECEIVER_BB_CFG);
+               TWL4030_BB_CFG_BBISEL_500UA);
 
        dieid_num_r();
 

diff --git a/drivers/misc/twl4030_led.c b/drivers/misc/twl4030_led.c
index 33cea116d2920792d7e0cce5687603ae9fc323aa..e150d8f0372b44bc133483501dd34d8a5f70ebf3 100644 (file)
--- a/drivers/misc/twl4030_led.c
+++ b/drivers/misc/twl4030_led.c
@@ -42,7 +42,7 @@ void twl4030_led_init(unsigned char ledon_mask)
        if (ledon_mask & TWL4030_LED_LEDEN_LEDBON)
                ledon_mask |= TWL4030_LED_LEDEN_LEDBPWM;
 
-       twl4030_i2c_write_u8(TWL4030_CHIP_LED, ledon_mask,
-                            TWL4030_LED_LEDEN);
+       twl4030_i2c_write_u8(TWL4030_CHIP_LED, TWL4030_LED_LEDEN,
+                            ledon_mask);
 
 }

diff --git a/drivers/power/twl4030.c b/drivers/power/twl4030.c
index e7d5f132b2bccb037de256ec005260ddc026c2e5..2bf94b134c64e363757045076313aa7648d1573b 100644 (file)
--- a/drivers/power/twl4030.c
+++ b/drivers/power/twl4030.c
@@ -51,8 +51,8 @@ void twl4030_power_reset_init(void)
                printf("Could not initialize hardware reset\n");
        } else {
                val |= TWL4030_PM_MASTER_SW_EVENTS_STOPON_PWRON;
-               if (twl4030_i2c_write_u8(TWL4030_CHIP_PM_MASTER, val,
-                                        TWL4030_PM_MASTER_P1_SW_EVENTS)) {
+               if (twl4030_i2c_write_u8(TWL4030_CHIP_PM_MASTER,
+                                        TWL4030_PM_MASTER_P1_SW_EVENTS, val)) {
                        printf("Error:TWL4030: failed to write the power register\n");
                        printf("Could not initialize hardware reset\n");
                }
@@ -68,8 +68,8 @@ void twl4030_pmrecv_vsel_cfg(u8 vsel_reg, u8 vsel_val,
        int ret;
 
        /* Select the Voltage */
-       ret = twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER, vsel_val,
-                               vsel_reg);
+       ret = twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER, vsel_reg,
+                                  vsel_val);
        if (ret != 0) {
                printf("Could not write vsel to reg %02x (%d)\n",
                        vsel_reg, ret);
@@ -77,8 +77,8 @@ void twl4030_pmrecv_vsel_cfg(u8 vsel_reg, u8 vsel_val,
        }
 
        /* Select the Device Group (enable the supply if dev_grp_sel != 0) */
-       ret = twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER, dev_grp_sel,
-                               dev_grp);
+       ret = twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER, dev_grp,
+                                  dev_grp_sel);
        if (ret != 0)
                printf("Could not write grp_sel to reg %02x (%d)\n",
                        dev_grp, ret);

diff --git a/drivers/usb/phy/twl4030.c b/drivers/usb/phy/twl4030.c
index 54d2e615c8076030a4a6c452edad274c0474fd24..f41cc07fe709459e1c45a3f7b4132ae3c594cedf 100644 (file)
--- a/drivers/usb/phy/twl4030.c
+++ b/drivers/usb/phy/twl4030.c
@@ -54,7 +54,7 @@ static int twl4030_usb_write(u8 address, u8 data)
 {
        int ret;
 
-       ret = twl4030_i2c_write_u8(TWL4030_CHIP_USB, data, address);
+       ret = twl4030_i2c_write_u8(TWL4030_CHIP_USB, address, data);
        if (ret != 0)
                printf("TWL4030:USB:Write[0x%x] Error %d\n", address, ret);
 
@@ -78,40 +78,40 @@ static int twl4030_usb_read(u8 address)
 static void twl4030_usb_ldo_init(void)
 {
        /* Enable writing to power configuration registers */
-       twl4030_i2c_write_u8(TWL4030_CHIP_PM_MASTER, 0xC0,
-                            TWL4030_PM_MASTER_PROTECT_KEY);
-       twl4030_i2c_write_u8(TWL4030_CHIP_PM_MASTER, 0x0C,
-                            TWL4030_PM_MASTER_PROTECT_KEY);
+       twl4030_i2c_write_u8(TWL4030_CHIP_PM_MASTER,
+                            TWL4030_PM_MASTER_PROTECT_KEY, 0xC0);
+       twl4030_i2c_write_u8(TWL4030_CHIP_PM_MASTER,
+                            TWL4030_PM_MASTER_PROTECT_KEY, 0x0C);
 
        /* put VUSB3V1 LDO in active state */
-       twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER, 0x00,
-                            TWL4030_PM_RECEIVER_VUSB_DEDICATED2);
+       twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER,
+                            TWL4030_PM_RECEIVER_VUSB_DEDICATED2, 0x00);
 
        /* input to VUSB3V1 LDO is from VBAT, not VBUS */
-       twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER, 0x14,
-                            TWL4030_PM_RECEIVER_VUSB_DEDICATED1);
+       twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER,
+                            TWL4030_PM_RECEIVER_VUSB_DEDICATED1, 0x14);
 
        /* turn on 3.1V regulator */
-       twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER, 0x20,
-                            TWL4030_PM_RECEIVER_VUSB3V1_DEV_GRP);
-       twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER, 0x00,
-                            TWL4030_PM_RECEIVER_VUSB3V1_TYPE);
+       twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER,
+                            TWL4030_PM_RECEIVER_VUSB3V1_DEV_GRP, 0x20);
+       twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER,
+                            TWL4030_PM_RECEIVER_VUSB3V1_TYPE, 0x00);
 
        /* turn on 1.5V regulator */
-       twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER, 0x20,
-                            TWL4030_PM_RECEIVER_VUSB1V5_DEV_GRP);
-       twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER, 0x00,
-                            TWL4030_PM_RECEIVER_VUSB1V5_TYPE);
+       twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER,
+                            TWL4030_PM_RECEIVER_VUSB1V5_DEV_GRP, 0x20);
+       twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER,
+                            TWL4030_PM_RECEIVER_VUSB1V5_TYPE, 0x00);
 
        /* turn on 1.8V regulator */
-       twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER, 0x20,
-                            TWL4030_PM_RECEIVER_VUSB1V8_DEV_GRP);
-       twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER, 0x00,
-                            TWL4030_PM_RECEIVER_VUSB1V8_TYPE);
+       twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER,
+                            TWL4030_PM_RECEIVER_VUSB1V8_DEV_GRP, 0x20);
+       twl4030_i2c_write_u8(TWL4030_CHIP_PM_RECEIVER,
+                            TWL4030_PM_RECEIVER_VUSB1V8_TYPE, 0x00);
 
        /* disable access to power configuration registers */
-       twl4030_i2c_write_u8(TWL4030_CHIP_PM_MASTER, 0x00,
-                            TWL4030_PM_MASTER_PROTECT_KEY);
+       twl4030_i2c_write_u8(TWL4030_CHIP_PM_MASTER,
+                            TWL4030_PM_MASTER_PROTECT_KEY, 0x00);
 }
 
 static void twl4030_phy_power(void)

diff --git a/include/twl4030.h b/include/twl4030.h
index 5aa184183e7cffef60942464290c73c9ecd7d41b..aca695d5bb5cc2790de1ab4b858c67c6ab631f50 100644 (file)
--- a/include/twl4030.h
+++ b/include/twl4030.h
@@ -638,7 +638,7 @@
  *   examples are TWL4030_PM_RECEIVER_VMMC1_DEV_GRP and
  *   TWL4030_LED_LEDEN.
  */
-static inline int twl4030_i2c_write_u8(u8 chip_no, u8 val, u8 reg)
+static inline int twl4030_i2c_write_u8(u8 chip_no, u8 reg, u8 val)
 {
        return i2c_write(chip_no, reg, 1, &val, 1);
 }