Re: [PATCH] i2c: core: Improve i2c_new_scanned_device

[Wolfram Sang <wsa+renesas@sang-engineering.com>]

[-- Attachment #1: Type: text/plain, Size: 495 bytes --]


> No functional change is intended. I have hw using i801, which calls
> i2c_register_spd() -> i2c_new_scanned_device(). With this patch
> tools like decode-dimms still work.

Thanks. Still, thinking more about it, I am not comfortable with this
patch. Kernel code where the intended exit of a function is not at the
end but somewhere in the middle often enough confused me in the past.

So, although it is not as concise as your suggestion, I would prefer to
leave the code as-is for clarity.


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]