On Sat, Jan 07, 2023 at 01:18:12PM -0800, Lars-Peter Clausen wrote: > In the mrecv() function the Cadence I2C driver has the following expression > in an if clause. > > ((id->p_msg->flags & I2C_M_RECV_LEN) != I2C_M_RECV_LEN) && > (id->recv_count <= CDNS_I2C_FIFO_DEPTH)) > > Earlier in the same function when I2C_M_RECV_LEN is set the recv_count is > initialized to a value that is larger than CDNS_I2C_FIFO_DEPTH. This means > if the first expression is false the second expression is also false. > Checking the first expression is thus redundant and can be removed. > > This slightly simplifies the logic. > > Signed-off-by: Lars-Peter Clausen <lars@xxxxxxxxxx> Applied to for-next, thanks!
Attachment:
signature.asc
Description: PGP signature
