Re: [PATCH 1/3] i2c: qcom-geni: Use the correct return value

Johan Hovold
2022-07-24T07:30:52Z

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

On Sun, Jul 24, 2022 at 07:04:12AM +0200, Wolfram Sang wrote:
>
> > - return num;
> > + return ret;
>
> Not quite. Kdoc describes the retval in i2c_transfer.
>
> 2136 * i2c_transfer - execute a single or combined I2C message
> 2137 * @adap: Handle to I2C bus
> 2138 * @msgs: One or more messages to execute before STOP is issued to
> 2139 * terminate the operation; each message begins with a START.
> 2140 * @num: Number of messages to be executed.
> 2141 *
> 2142 * Returns negative errno, else the number of messages executed.

And this is exactly what the driver is returning after this fix.

Note that ret above is assigned based on the return value from two
helper functions that return either a negative errno or the number
messages executed.

So I believe this patch is still needed to fix the error handling.

Reviewed-by: Johan Hovold <johan+linaro@kernel.org>

Johan

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