Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Sun, 8 Mar 2020 04:24:15 +0100
From:   Sebastian Reichel <sre@kernel.org>
To:     Baolin Wang <baolin.wang7@gmail.com>
Cc:     linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org,
        zhang.lyra@gmail.com, orsonzhai@gmail.com
Subject: Re: [PATCH] power: supply: sc27xx: Add POWER_SUPPLY_PROP_CHARGE_NOW
 attribute
Message-ID: <20200308032415.pirjewmu4wjut2xo@earth.universe>
References: <33dba0929575461e38c2e17ac0cc4a8a4e46fa2f.1581658633.git.baolin.wang7@gmail.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="rk2x2c6jwavldn4i"
Content-Disposition: inline
In-Reply-To: <33dba0929575461e38c2e17ac0cc4a8a4e46fa2f.1581658633.git.baolin.wang7@gmail.com>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


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

Hi,

On Fri, Feb 14, 2020 at 01:38:58PM +0800, Baolin Wang wrote:
> Add the POWER_SUPPLY_PROP_CHARGE_NOW attribute to allow user to get
> current battery capacity (uAh) to do measurement.
>=20
> Signed-off-by: Baolin Wang <baolin.wang7@gmail.com>
> ---

Thanks, queued to power-supply's for-next branch.

-- Sebastian

>  drivers/power/supply/sc27xx_fuel_gauge.c |   12 ++++++++++++
>  1 file changed, 12 insertions(+)
>=20
> diff --git a/drivers/power/supply/sc27xx_fuel_gauge.c b/drivers/power/sup=
ply/sc27xx_fuel_gauge.c
> index 469c83f..a7c8a84 100644
> --- a/drivers/power/supply/sc27xx_fuel_gauge.c
> +++ b/drivers/power/supply/sc27xx_fuel_gauge.c
> @@ -614,6 +614,17 @@ static int sc27xx_fgu_get_property(struct power_supp=
ly *psy,
>  		val->intval =3D data->total_cap * 1000;
>  		break;
> =20
> +	case POWER_SUPPLY_PROP_CHARGE_NOW:
> +		ret =3D sc27xx_fgu_get_clbcnt(data, &value);
> +		if (ret)
> +			goto error;
> +
> +		value =3D DIV_ROUND_CLOSEST(value * 10,
> +					  36 * SC27XX_FGU_SAMPLE_HZ);
> +		val->intval =3D sc27xx_fgu_adc_to_current(data, value);
> +
> +		break;
> +
>  	default:
>  		ret =3D -EINVAL;
>  		break;
> @@ -682,6 +693,7 @@ static int sc27xx_fgu_property_is_writeable(struct po=
wer_supply *psy,
>  	POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE,
>  	POWER_SUPPLY_PROP_ENERGY_FULL_DESIGN,
>  	POWER_SUPPLY_PROP_CALIBRATE,
> +	POWER_SUPPLY_PROP_CHARGE_NOW
>  };
> =20
>  static const struct power_supply_desc sc27xx_fgu_desc =3D {
> --=20
> 1.7.9.5
>=20

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

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

iQIzBAEBCgAdFiEE72YNB0Y/i3JqeVQT2O7X88g7+poFAl5kZV8ACgkQ2O7X88g7
+pqhgRAAqmnqhBKya7Xa3yAztFO1pmXITppHrlVqecqljHx3sUH+8OvMyUSY64IJ
dxP1Kra/zcUyw44HCwJMeI/pmE4WdXddlcCWtmhcJks98WCJG20OUv9E4qMee96h
2bNsf79hMi05Zhs7N3d835sfLLCEIRfFP+rN/EJVLZXXsUtX2XCNbmr+D5ZQmxyZ
Dxg82PryGUbmPucF7Y+/t8BqjEOG6ct5FMqtqOckrtdKDHUXgqTFbICpH2JXjli5
UaaHv3gUPuiYvTe9U5/t6EvgmQ0J66fQ628G41CzTvsRAKvIO4o0ENd17gwAbHno
83b/bxN6E5hx+8+BfSSlUjtF5wpA/WatvFw63R13wO8cl3sBH6DGJ51f3maeDcGC
Ett0msiALhoS3AKO1IzcK2DJSwUfP8xutV0igOpouE/cmlSb2ZPN+hT7v6UAsTAA
38PSXj5edRcFlDF3ygjr1jao0QZyYZ9myL32bU52HEDmei1axXnN3vvEOWtT8cY6
BIoj+EerytLInoc03fbornNFc+dHXjPbGm236YUd7Yhw0z7ayJoHo4osjzlyqVuX
JxqbGGYOtTdAgvc0Au2fdNYuE/5TVfNlXuy+beXrRukChYijiBfNYe/zmCBoEQOx
s6ftb2jrCdfaXw/WmpMQNQQzO/ggfY4RgDiRg3sKUY9moMAVGS0=
=2A4B
-----END PGP SIGNATURE-----

--rk2x2c6jwavldn4i--