[PATCH] i2c: cadence: Avoid fifo clear after start

Boddu, Sai Pavan sai.pavan.boddu at amd.com
Fri May 3 02:05:37 PDT 2024


Hi Andi,

Sorry, I did not close on this one.
Anyway I will re-spin fixing the commit message issues. More comments inline below.

>-----Original Message-----
>From: Andi Shyti <andi.shyti at kernel.org>
>Sent: Thursday, January 18, 2024 2:36 AM
>To: Boddu, Sai Pavan <sai.pavan.boddu at amd.com>
>Cc: linux-kernel at vger.kernel.org; linux-i2c at vger.kernel.org; linux-arm-
>kernel at lists.infradead.org; Simek, Michal <michal.simek at amd.com>; Lars-
>Peter Clausen <lars at metafoo.de>; Wolfram Sang <wsa at kernel.org>
>Subject: Re: [PATCH] i2c: cadence: Avoid fifo clear after start
>
>Hi,
>
>> >> b/drivers/i2c/busses/i2c-cadence.c
>> >> index de3f58b60dce..6f7d753a8197 100644
>> >> --- a/drivers/i2c/busses/i2c-cadence.c
>> >> +++ b/drivers/i2c/busses/i2c-cadence.c
>> >> @@ -633,6 +633,7 @@ static void cdns_i2c_mrecv(struct cdns_i2c *id)
>> >>
>> >>  	if (hold_clear) {
>> >>  		ctrl_reg &= ~CDNS_I2C_CR_HOLD;
>> >> +		ctrl_reg &= ~CDNS_I2C_CR_CLR_FIFO;
>> >
>> >I'm wondering whether the whole ctrl_reg should be reset after the first
>write.
>
>> [Boddu, Sai Pavan] previous implementation of read-modify-write was good
>then ?
>
>I don't know, I'm just asking... because rather than read-modify-write, this is
>read-modify-write-modify-write :-)
>
>I'm just wondering if after the first write ctrl_reg is still holding a valid value.
[Boddu, Sai Pavan] Yes, all bits in ctrl_reg stay as configured except CLR_FIFO which is self-clearing.
None of the other bits would change state.

CLR_FIFO post start of transactions should not be allowed; this patch address the same.

Regards,
Sai Pavan 
>
>Andi



More information about the linux-arm-kernel mailing list