i2c: smbus: don't abuse loop variable

Don't use 'i' to carry data, make a specific variable for it. After the
move to memcpy recently, we can even remove 'i'.

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Signed-off-by: Wolfram Sang <wsa@kernel.org>

diff --git a/drivers/i2c/i2c-core-smbus.c b/drivers/i2c/i2c-core-smbus.c
index bf6bfe7c5cfe139f252f768b1756f6e240679b74..6cda46913d89923aa76172c120dd0e2f1d4c331f 100644 (file)
--- a/drivers/i2c/i2c-core-smbus.c
+++ b/drivers/i2c/i2c-core-smbus.c
@@ -324,7 +324,6 @@ static s32 i2c_smbus_xfer_emulated(struct i2c_adapter *adapter, u16 addr,
        unsigned char msgbuf0[I2C_SMBUS_BLOCK_MAX+3];
        unsigned char msgbuf1[I2C_SMBUS_BLOCK_MAX+2];
        int num = read_write == I2C_SMBUS_READ ? 2 : 1;
-       int i;
        u8 partial_pec = 0;
        int status;
        struct i2c_msg msg[2] = {
@@ -340,6 +339,8 @@ static s32 i2c_smbus_xfer_emulated(struct i2c_adapter *adapter, u16 addr,
                        .buf = msgbuf1,
                },
        };
+       bool wants_pec = ((flags & I2C_CLIENT_PEC) && size != I2C_SMBUS_QUICK
+                         && size != I2C_SMBUS_I2C_BLOCK_DATA);
 
        msgbuf0[0] = command;
        switch (size) {
@@ -443,9 +444,7 @@ static s32 i2c_smbus_xfer_emulated(struct i2c_adapter *adapter, u16 addr,
                return -EOPNOTSUPP;
        }
 
-       i = ((flags & I2C_CLIENT_PEC) && size != I2C_SMBUS_QUICK
-                                     && size != I2C_SMBUS_I2C_BLOCK_DATA);
-       if (i) {
+       if (wants_pec) {
                /* Compute PEC if first message is a write */
                if (!(msg[0].flags & I2C_M_RD)) {
                        if (num == 1) /* Write only */
@@ -468,7 +467,7 @@ static s32 i2c_smbus_xfer_emulated(struct i2c_adapter *adapter, u16 addr,
        status = 0;
 
        /* Check PEC if last message is a read */
-       if (i && (msg[num-1].flags & I2C_M_RD)) {
+       if (wants_pec && (msg[num-1].flags & I2C_M_RD)) {
                status = i2c_smbus_check_pec(partial_pec, &msg[num-1]);
                if (status < 0)
                        goto cleanup;