Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Sun, 24 Jul 2022 09:30:53 +0200
From:   Johan Hovold <johan@kernel.org>
To:     Wolfram Sang <wsa@kernel.org>
Cc:     Bjorn Andersson <bjorn.andersson@linaro.org>,
        Konrad Dybcio <konrad.dybcio@somainline.org>,
        Vinod Koul <vkoul@kernel.org>, linux-arm-msm@vger.kernel.org,
        linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/3] i2c: qcom-geni: Use the correct return value
Message-ID: <Ytz1LUSwyZ/8LiN6@hovoldconsulting.com>
References: <20220717035027.2135106-1-bjorn.andersson@linaro.org>
 <20220717035027.2135106-2-bjorn.andersson@linaro.org>
 <YtzSzKmakEWe1usg@shikoro>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="SCcni7T232IeqbHy"
Content-Disposition: inline
In-Reply-To: <YtzSzKmakEWe1usg@shikoro>
Precedence: bulk


--SCcni7T232IeqbHy
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Sun, Jul 24, 2022 at 07:04:12AM +0200, Wolfram Sang wrote:
>=20
> > -	return num;
> > +	return ret;
>=20
> Not quite. Kdoc describes the retval in i2c_transfer.
>=20
> 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

--SCcni7T232IeqbHy
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYIAB0WIQQHbPq+cpGvN/peuzMLxc3C7H1lCAUCYtz1KAAKCRALxc3C7H1l
CBHVAP0UEiURyG3ZhQ8PSe/GPPDFlqH+xvSr8mdxxD4xF88HgAEA8ZIUri/6FFa+
4us9bAiitlXLdlyS7lHfWeAQhBmemQc=
=wjOq
-----END PGP SIGNATURE-----

--SCcni7T232IeqbHy--