Hi Aryan, > +/* Construct and send i2c transaction core cmd for read ops */ > +static int octeon_i2c_hlc_read_cmd(struct octeon_i2c *i2c, struct i2c_msg msg, u64 cmd) > +{ > + u64 ext = 0; > + > + if (octeon_i2c_hlc_ext(i2c, msg, &cmd, &ext)) > + octeon_i2c_writeq_flush(ext, i2c->twsi_base + SW_TWSI_EXT(i2c)); I think this check here is the only logical change I see. Right? If so, can you please describe in the log why you made this change? Thanks, Andi > + return octeon_i2c_hlc_cmd_send(i2c, cmd); > +}