Linux kernel mirror (for testing)
git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel
os
linux
drivers/i2c/busses: use min() instead of min_t()
min_t(u8, a, b) casts both its arguments to u8 potentially discarding
signifinact bits.
Use min(a, b) instead as it cannot discard significant bits.
Detected by an extra check added to min_t().
Signed-off-by: David Laight <david.laight.linux@gmail.com>
Signed-off-by: Andi Shyti <andi.shyti@kernel.org>
Link: https://lore.kernel.org/r/20251119224140.8616-21-david.laight.linux@gmail.com
authored by
David Laight
and committed by
Andi Shyti
fae88e03
949f647e
+1
-1
+1
-1
drivers/i2c/busses/i2c-designware-master.c
+1
-1
drivers/i2c/busses/i2c-designware-master.c
···
575
575
* after receiving the first byte.
576
576
*/
577
577
len += (flags & I2C_CLIENT_PEC) ? 2 : 1;
578
-
dev->tx_buf_len = len - min_t(u8, len, dev->rx_outstanding);
578
+
dev->tx_buf_len = len - min(len, dev->rx_outstanding);
579
579
msgs[dev->msg_read_idx].len = len;
580
580
msgs[dev->msg_read_idx].flags &= ~I2C_M_RECV_LEN;
581
581