The existing API has function `i2c_xfer` that transfers one or more messages, and it only returns a single error code if the transfer was failed. It doesn't allow to know how many of the messages were transferred successfully, neither how many bytes were transferred in the message that caused the fault, and also it drops all data received from target device before the fault. There is a comment about this in drivers/i2c/i2c-core-base.c: "REVISIT the fault reporting model here is weak".
This patch series implements new API function `i2c_xfer_v2` that does the same as `i2c_xfer` but also returns detailed transfer report, including number of messages and bytes transferred before the fault. This also allows client to get the bytes read from the target before the fault occurred. For user space clients, new ioctl `I2C_RDWR_V2` is introduced. Patch for `i2c-tools` user space utility that introduces support of this feature: https://lore.kernel.org/linux-i2c/[email protected]/ A usage example (the first message changes the target address, so the second message is not ACKed): # i2ctransfer -r -y 0 w2@0x20 0 0x55 r1 Error: No such device or address at message 1 1 out of 2 messages sent # In this patchset, the introduced functionality is implemented in `i2c-nomadik` and `i2c-designware` drivers. Several other improvements in these drivers related to fault handling are also included in this patchset. `i2c-nomadik` driver has been tested on EyeQ6H. `i2c-designware` driver has been tested on EyeQ6L+. The implementation is split up into patches: Patch #1 Introduce callback `xfer_v2` in struct `i2c_algorithm`, function `i2c_xfer_v2`, ioctl `I2C_RDWR_V2`, structures for I2C transfer reporting and implement all driver-independent functionality. Patch #2 Optimize struct layout in `i2c-nomadik`. Patch #3 Remove automatic retransfer in `i2c-nomadik`. Patch #4 Fix error codes returned by `xfer` callback in `i2c-nomadik`. Patch #5 Replace `dev_err` with `dev_dbg` on I2C faults in `i2c-nomadik`. Patch #6 Add quirks that describe some limitations of `i2c-nomadik`. Patch #7 Add support for `xfer_v2` in `i2c-nomadik`. Patch #8 Fix error codes returned by `xfer` callback in `i2c-designware`. Patch #9 Reset SMBUS_READ_BLOCK_DATA bit in functionality mask for `designware` variants without EMPTYFIFO_HOLD_MASTER feature. Patch #10 Terminate transfer if spurious STOP is detected in `i2c-designware` driver. Patch #11 Separate `i2c_algorithm` structure for AMD_NAVI_GPU variant of designware controller. Patch #12 Add support for `xfer_v2` in `i2c-designware` driver. Signed-off-by: Dmitry Guzman <[email protected]> --- Changes in v2: - Add support for detailed transfer reporting in designware driver (patches #8..#12). - Change `unsigned char` into `u8` in patch #2. - Added code-review trailers for patches #3..#7. - Rebase on new kernel version (no conflicting code changes). - Link to v1: https://lore.kernel.org/r/[email protected] --- Dmitry Guzman (12): i2c: core: add I2C_XFER_V2 - support for detailed transfer reporting i2c: nomadik: optimize struct layout, change unsigned char into u8 i2c: nomadik: do not try to retransmit I2C message series on errors i2c: nomadik: return proper fault codes i2c: nomadik: change print level for fault messages to debug i2c: nomadik: add quirks max_len=2047 and no_zero_len_read i2c: nomadik: add support for I2C_XFER_V2 - detailed fault reporting i2c: designware: return proper fault codes i2c: designware: no SMBUS_READ_BLOCK_DATA without EMPTYFIFO_HOLD_MASTER i2c: designware: stop transfer if spurious STOP is detected i2c: designware: use separate `i2c_algorithm` for AMD_NAVI_GPU i2c: designware: add support for I2C_XFER_V2 - detailed fault reporting Documentation/i2c/dev-interface.rst | 46 +++++++++ drivers/i2c/busses/i2c-designware-common.c | 23 ++++- drivers/i2c/busses/i2c-designware-core.h | 14 ++- drivers/i2c/busses/i2c-designware-master.c | 160 ++++++++++++++++++++++++++--- drivers/i2c/busses/i2c-nomadik.c | 111 ++++++++++++++------ drivers/i2c/i2c-core-base.c | 107 +++++++++++++------ drivers/i2c/i2c-dev.c | 79 +++++++++++--- include/linux/i2c.h | 12 +++ include/trace/events/i2c.h | 6 +- include/uapi/linux/i2c-dev.h | 9 ++ include/uapi/linux/i2c.h | 21 ++++ 11 files changed, 489 insertions(+), 99 deletions(-) --- base-commit: 940de590b839f71d6dc846160534bf202401b8b7 change-id: 20260623-i2c-fault-reporting-9236c9affc2d Best regards, -- Dmitry Guzman <[email protected]>
