RE: [PATCH 0/6] Fix R-Car I2C data byte sent twice issue

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Wolfram,

Thank you for your feedback!

> -----Original Message-----
> From: Wolfram Sang [mailto:wsa@xxxxxxxxxxxxx]
> Sent: 29 May 2018 18:20
> To: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx>
> Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>; linux-i2c@xxxxxxxxxxxxxxx; Chris Paterson <Chris.Paterson2@xxxxxxxxxxx>;
> Biju Das <biju.das@xxxxxxxxxxxxxx>; stable@xxxxxxxxxxxxxxx; Ben Hutchings <ben.hutchings@xxxxxxxxxxxxxxx>
> Subject: Re: [PATCH 0/6] Fix R-Car I2C data byte sent twice issue
>
>
> > this series fixes an issue with the I2C driver of the Renesas R-Car and
> > RZ/G1 family of chips. The issue is clearly visible with the CIP kernel
> > (4.4) running on a iwg20d board from iWave due to the way the bq32000
> > driver/device is interacting with the I2C driver/controller.
> > In the stable kernel (4.4) there is no support for the iwg20d, I tried
> > to replicate the same problem on a Koelsch board with no success, but
> > the problem is there.
>
> For the record, this patchset was developed on a Lager board (R-Car H2)
> but the issue was known to be present on other Gen2 SoCs, too. This was
> a nasty race condition. IIRC I was able to reproduce the issue only with
> the first transfer after boot (for whatever reason). And fixing it,
> together with another issue (double address byte), needed all this
> refactoring.

Thank you for the information, I may give it a try.

>
> I might try to recap how this double data byte problem was triggered.
> This is somewhere on my todo-list anyhow. Just to check if the same
> problem is still present with Gen3 SoCs. No promises, but I'll see what
> I can do.

Thanks,
Fab




Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.





[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux