Re: [PATCH] i2c: mxs: Rework the PIO mode operation

[Marek Vasut <marex-ynQEQJNshbs@public.gmane.org>]

Dear to-fleischer-zqRNUXuvxA0b1SvskN2V4Q@public.gmane.org,

> Hi,
> 
> > > Analyze and rework the PIO mode operation. The PIO mode operation
> > > was unreliable on MX28, by analyzing the bus with LA, the checks for
> > > when data were available or were to be sent were wrong.
> > > 
> > > The PIO WRITE has to be completely reworked as it multiple problems.
> > > The MX23 datasheet helped here, see comments in the code for
> > > details.
> > > The problems boil down to:
> > > - RUN bit in CTRL0 must be set after DATA register was written - The
> > > PIO transfer must be 4 bytes long tops, otherwise use
> > > clock stretching.
> > > Both of these fixes are implemented.
> > > 
> > > The PIO READ operation can only be done for up to four bytes as we
> > > are unable to read out the data from the DATA register fast enough.
> > > 
> > > This patch also tries to document the investigation within the code.
> > > 
> > > Signed-off-by: Marek Vasut <marex-ynQEQJNshbs@public.gmane.org>
> > > Cc: Alexandre Belloni <alexandre.belloni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
> > > Cc: Fabio Estevam <r49496-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
> > > Cc: Lucas Stach <l.stach-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
> > > Cc: Shawn Guo <shawn.guo-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
> > > Cc: Wolfram Sang <wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org>
> > > Cc: <to-fleischer-zqRNUXuvxA0b1SvskN2V4Q@public.gmane.org>
> > 
> > I so far got confirmation from ALexandre this patch works. Can someone
> > else test it please? I'd like to roll out a final version to close
> > this issue.
> 
> sorry, I couldn't  test the patch until today on my iMX283 board.
> It works when reading the registers from the temperature sensor TCN75A.
> But it fails when writing less than 7 bytes to the EEPROM (24LC32). Then
> I always get a 'Connection timed out' error in the user space.

Thinking about this, did you correctly configure the EEPROM write sector size? I 
think it's 32 bytes for this device.

Best regards,
Marek Vasut