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:   Thu, 30 Mar 2023 01:24:35 +0200
From:   Sebastian Reichel <sre@kernel.org>
To:     Linus Walleij <linus.walleij@linaro.org>,
        Matti Vaittinen <mazziesaccount@gmail.com>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org,
        devicetree@vger.kernel.org
Subject: Re: [PATCHv3 02/14] power: supply: core: auto-exposure of
 simple-battery data
Message-ID: <20230329232435.rjdzoluxagwy4tvu@mercury.elektranox.org>
References: <20230317225707.1552512-1-sre@kernel.org>
 <20230317225707.1552512-3-sre@kernel.org>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="uv7vcc4lwuwob7hz"
Content-Disposition: inline
In-Reply-To: <20230317225707.1552512-3-sre@kernel.org>
Precedence: bulk


--uv7vcc4lwuwob7hz
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

Hi,

On Fri, Mar 17, 2023 at 11:56:55PM +0100, Sebastian Reichel wrote:
> [...]
> +	/*
> +	 * Expose constant battery info, if it is available. While there are
> +	 * some chargers accessing constant battery data, we only want to
> +	 * expose battery data to userspace for battery devices.
> +	 */
> +	if (desc->type == POWER_SUPPLY_TYPE_BATTERY) {
> +		rc = power_supply_get_battery_info(psy, &psy->battery_info);
> +		if (rc && rc != -ENODEV)
> +			goto check_supplies_failed;

I merged this a few days ago and got a bug report from KernelCI,
that -ENOENT also needs to be handled. I fixed this in place.

-- Sebastian

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

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

iQIzBAABCgAdFiEE72YNB0Y/i3JqeVQT2O7X88g7+poFAmQkyKgACgkQ2O7X88g7
+pouzBAAjZh4e+txg6+30IosEQnaNrcWMB1IY/DYmXJA6cMCfSAyuUEY6j/bwcUM
3qXIHRtm4qrWN3IX2CWLu/28F4BiXUjSmTLlzWqzptLhCRRbJijzqrgzGkLo1jCl
ZBG4gYYdYo+sDVsIPSDLyosqVghCrclnRLjsEYF6lTtDmPNmwB1z/euqLC/lgndy
nvVusMVW3DB2G3tsxtXyfEY68GGZOpZgTnuZrle6zjnRYAD5ZQVJhsFoEsIT/dgJ
oRY1aX7jZy7Ha/vzdzypKiZM8V1UAWPnu+45LcTjy2NgQRp5mj6ERFJ9m3/Myv/r
leVbdGiWi5HIQiPZvF7h1QQDN7fZ4wU3abVis6iBnnHuOzdOFVF8BvaGGgPvRFn/
HcgdtK2notxCbONYo5moKDAjCGoxqJMY4J5ZLt5Caweg+oZ5u7IKtLd6f975DrSt
4949HmPXaDFdZMdWn8/fF6AcVsVNx5pM70Va52sIkVTF0WyFlOk3BEAypRllc3Dq
dWSqZwSLkF0T6GAC1eScuaTp6xhMxmct3qhG7893+AzXy2Lu3LW10tOhoiBXsQPQ
dalfxb217mhM50ld6VEkm6Bnge5X4vBz1aGjxGeT0FsDRbI718+tJV2cUZRMdgSE
vy9rimB4KUdMDT5LYCXd4vdN/wTHIkQ0Tkbg9ObU6VccVnXTw2I=
=kyq1
-----END PGP SIGNATURE-----

--uv7vcc4lwuwob7hz--