Commit acf7146e authored by Daniel Baluta's avatar Daniel Baluta Committed by Jonathan Cameron

iio: imu: inv_mpu6050: Fix Yoda conditions

This fixes the following checkpatch warning:
	* WARNING: Comparisons should place the constant
	  on the right side of the test
Signed-off-by: default avatarDaniel Baluta <daniel.baluta@intel.com>
Acked-by: default avatarCrt Mori <cmo@melexis.com>
Signed-off-by: default avatarJonathan Cameron <jic23@kernel.org>
parent 23c5edcc
...@@ -84,7 +84,7 @@ int inv_mpu6050_switch_engine(struct inv_mpu6050_state *st, bool en, u32 mask) ...@@ -84,7 +84,7 @@ int inv_mpu6050_switch_engine(struct inv_mpu6050_state *st, bool en, u32 mask)
* clock source be switched to gyro. Otherwise, it must be set to * clock source be switched to gyro. Otherwise, it must be set to
* internal clock * internal clock
*/ */
if (INV_MPU6050_BIT_PWR_GYRO_STBY == mask) { if (mask == INV_MPU6050_BIT_PWR_GYRO_STBY) {
result = regmap_read(st->map, st->reg->pwr_mgmt_1, &mgmt_1); result = regmap_read(st->map, st->reg->pwr_mgmt_1, &mgmt_1);
if (result) if (result)
return result; return result;
...@@ -92,7 +92,7 @@ int inv_mpu6050_switch_engine(struct inv_mpu6050_state *st, bool en, u32 mask) ...@@ -92,7 +92,7 @@ int inv_mpu6050_switch_engine(struct inv_mpu6050_state *st, bool en, u32 mask)
mgmt_1 &= ~INV_MPU6050_BIT_CLK_MASK; mgmt_1 &= ~INV_MPU6050_BIT_CLK_MASK;
} }
if ((INV_MPU6050_BIT_PWR_GYRO_STBY == mask) && (!en)) { if ((mask == INV_MPU6050_BIT_PWR_GYRO_STBY) && (!en)) {
/* /*
* turning off gyro requires switch to internal clock first. * turning off gyro requires switch to internal clock first.
* Then turn off gyro engine * Then turn off gyro engine
...@@ -117,7 +117,7 @@ int inv_mpu6050_switch_engine(struct inv_mpu6050_state *st, bool en, u32 mask) ...@@ -117,7 +117,7 @@ int inv_mpu6050_switch_engine(struct inv_mpu6050_state *st, bool en, u32 mask)
if (en) { if (en) {
/* Wait for output stabilize */ /* Wait for output stabilize */
msleep(INV_MPU6050_TEMP_UP_TIME); msleep(INV_MPU6050_TEMP_UP_TIME);
if (INV_MPU6050_BIT_PWR_GYRO_STBY == mask) { if (mask == INV_MPU6050_BIT_PWR_GYRO_STBY) {
/* switch internal clock to PLL */ /* switch internal clock to PLL */
mgmt_1 |= INV_CLK_PLL; mgmt_1 |= INV_CLK_PLL;
result = regmap_write(st->map, result = regmap_write(st->map,
......
...@@ -168,7 +168,7 @@ irqreturn_t inv_mpu6050_read_fifo(int irq, void *p) ...@@ -168,7 +168,7 @@ irqreturn_t inv_mpu6050_read_fifo(int irq, void *p)
result = kfifo_out(&st->timestamps, &timestamp, 1); result = kfifo_out(&st->timestamps, &timestamp, 1);
/* when there is no timestamp, put timestamp as 0 */ /* when there is no timestamp, put timestamp as 0 */
if (0 == result) if (result == 0)
timestamp = 0; timestamp = 0;
result = iio_push_to_buffers_with_timestamp(indio_dev, data, result = iio_push_to_buffers_with_timestamp(indio_dev, data,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment