i3c: master: svc: fix ibi may not return mandatory data byte
authorFrank Li <Frank.Li@nxp.com>
Mon, 23 Oct 2023 16:16:55 +0000 (12:16 -0400)
committerAlexandre Belloni <alexandre.belloni@bootlin.com>
Fri, 3 Nov 2023 18:48:17 +0000 (19:48 +0100)
commitc85e209b799f12d18a90ae6353b997b1bb1274a5
treebc6fb5a10b3e67aa30d9bc727036618d0ab588c7
parent5e5e3c92e748a6d859190e123b9193cf4911fcca
i3c: master: svc: fix ibi may not return mandatory data byte

MSTATUS[RXPEND] is only updated after the data transfer cycle started. This
creates an issue when the I3C clock is slow, and the CPU is running fast
enough that MSTATUS[RXPEND] may not be updated when the code reaches
checking point. As a result, mandatory data can be missed.

Add a wait for MSTATUS[COMPLETE] to ensure that all mandatory data is
already in FIFO. It also works without mandatory data.

Fixes: dd3c52846d59 ("i3c: master: svc: Add Silvaco I3C master driver")
Cc: <stable@vger.kernel.org>
Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
Signed-off-by: Frank Li <Frank.Li@nxp.com>
Link: https://lore.kernel.org/r/20231023161658.3890811-4-Frank.Li@nxp.com
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
drivers/i3c/master/svc-i3c-master.c