staging: iio: tsl2x7x: remove unnecessary parentheses
authorBrian Masney <masneyb@onstation.org>
Thu, 19 Oct 2017 20:06:26 +0000 (16:06 -0400)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Sat, 21 Oct 2017 17:37:28 +0000 (18:37 +0100)
This patch fixes the error 'Unnecessary parentheses around 'XXX' from
checkpatch.pl. It also fixes several other places with unnecessary
parentheses that checkpatch.pl did not detect.

Signed-off-by: Brian Masney <masneyb@onstation.org>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/staging/iio/light/tsl2x7x.c

index 80968dd..6cc89cd 100644 (file)
@@ -348,9 +348,9 @@ static int tsl2x7x_get_lux(struct iio_dev *indio_dev)
 
        /* clear any existing interrupt status */
        ret = i2c_smbus_write_byte(chip->client,
-                                  (TSL2X7X_CMD_REG |
+                                  TSL2X7X_CMD_REG |
                                   TSL2X7X_CMD_SPL_FN |
-                                  TSL2X7X_CMD_ALS_INT_CLR));
+                                  TSL2X7X_CMD_ALS_INT_CLR);
        if (ret < 0) {
                dev_err(&chip->client->dev,
                        "i2c_write_command failed - err = %d\n", ret);
@@ -364,7 +364,7 @@ static int tsl2x7x_get_lux(struct iio_dev *indio_dev)
        chip->als_cur_info.als_ch0 = ch0;
        chip->als_cur_info.als_ch1 = ch1;
 
-       if ((ch0 >= chip->als_saturation) || (ch1 >= chip->als_saturation)) {
+       if (ch0 >= chip->als_saturation || ch1 >= chip->als_saturation) {
                lux = TSL2X7X_LUX_CALC_OVER_FLOW;
                goto return_max;
        }
@@ -697,14 +697,14 @@ static int tsl2x7x_chip_on(struct iio_dev *indio_dev)
                dev_info(&chip->client->dev, "Setting Up Interrupt(s)\n");
 
                reg_val = TSL2X7X_CNTL_PWR_ON | TSL2X7X_CNTL_ADC_ENBL;
-               if ((chip->settings.interrupts_en == 0x20) ||
-                   (chip->settings.interrupts_en == 0x30))
+               if (chip->settings.interrupts_en == 0x20 ||
+                   chip->settings.interrupts_en == 0x30)
                        reg_val |= TSL2X7X_CNTL_PROX_DET_ENBL;
 
                reg_val |= chip->settings.interrupts_en;
                ret = i2c_smbus_write_byte_data(chip->client,
-                                               (TSL2X7X_CMD_REG |
-                                               TSL2X7X_CNTRL), reg_val);
+                                               TSL2X7X_CMD_REG | TSL2X7X_CNTRL,
+                                               reg_val);
                if (ret < 0)
                        dev_err(&chip->client->dev,
                                "%s: failed in tsl2x7x_IOCTL_INT_SET.\n",
@@ -1721,7 +1721,7 @@ static int tsl2x7x_probe(struct i2c_client *clientp,
                return -EINVAL;
        }
 
-       ret = i2c_smbus_write_byte(clientp, (TSL2X7X_CMD_REG | TSL2X7X_CNTRL));
+       ret = i2c_smbus_write_byte(clientp, TSL2X7X_CMD_REG | TSL2X7X_CNTRL);
        if (ret < 0) {
                dev_err(&clientp->dev, "write to cmd reg failed. err = %d\n",
                        ret);