From: Hsin-Yu.Chen <harry021633@gmail.com>
Date: Wed, 6 Mar 2024 04:19:00 +0000 (+0800)
Subject: i2c: remove redundant condition
X-Git-Tag: v6.9-rc1~98^2~2
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=f311507c5336ad6b9ca7687c35c3bd7a4fe2868c;p=thirdparty%2Fkernel%2Flinux.git

i2c: remove redundant condition

I2C_M_RD is defined as and guaranteed to be 1 and 'flag & I2C_M_RD' is
one or zero. No need for an additional condition to obtain the value.

Signed-off-by: Hsin-Yu.Chen <harry021633@gmail.com>
Reviewed-by: Andi Shyti <andi.shyti@kernel.org>
[wsa: slightly updated commit message]
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---

diff --git a/include/linux/i2c.h b/include/linux/i2c.h
index ff93ff8b257ce..5e6cd43a6dbdd 100644
--- a/include/linux/i2c.h
+++ b/include/linux/i2c.h
@@ -931,7 +931,7 @@ static inline int i2c_adapter_id(struct i2c_adapter *adap)
 
 static inline u8 i2c_8bit_addr_from_msg(const struct i2c_msg *msg)
 {
-	return (msg->addr << 1) | (msg->flags & I2C_M_RD ? 1 : 0);
+	return (msg->addr << 1) | (msg->flags & I2C_M_RD);
 }
 
 u8 *i2c_get_dma_safe_msg_buf(struct i2c_msg *msg, unsigned int threshold);