[PATCH v6 0/3] add support for Socionext SynQuacer I2C controller

Wolfram Sang wsa at the-dreams.de
Tue Apr 3 08:01:39 PDT 2018


> - switch to generic recovery using minimal helpers to drive the SDA/SCL lines
>   directly

If you had added this incrementally, reviewing would have been easier (=
faster). The callbacks look okay...

> - don't recover on every error

... but you are now never recovering. You don't call i2c_recover_bus().

> - add patch to sanity check i2c_transfer() arguments in core code (#3)

See small comment there.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20180403/817d23e9/attachment.sig>


More information about the linux-arm-kernel mailing list