On Fri, Nov 21, 2014 at 04:16:51AM +0400, Alexander Kochetkov wrote: > commit 6d9939f651419a63e091105663821f9c7d3fec37 (i2c: omap: split out [XR]DR > and [XR]RDY) changed the way how errata i207 (I2C: RDR Flag May Be Incorrectly > Set) get handled. 6d9939f6514 code doesn't correspond to workaround provided by > errata. > > According to errata ISR must filter out spurious RDR before data read not after. > ISR must read RXSTAT to get number of bytes available to read. Because RDR > could be set while there could no data in the receive FIFO. > > Restored pre 6d9939f6514 way of handling errata. > > Found by code review. Real impact haven't seen. > Tested on Beagleboard XM C. > > Signed-off-by: Alexander Kochetkov <al.kochet@xxxxxxxxx> > Fixes: 6d9939f651419a63e09110 i2c: omap: split out [XR]DR and [XR]RDY Applied to for-current, thanks!
Attachment:
signature.asc
Description: Digital signature
