linux-kernel - Re: [PATCH] power: supply: qcom_battmgr: Add lithium-polymer entry

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <bje4frydn6bw4qwu4s3di4uxppx3tvqidsl2gmyb7tlqihi63o@5v24ta3wpl54>
Date: Sun, 22 Jun 2025 02:58:12 +0200
From: Sebastian Reichel <sebastian.reichel@...labora.com>
To: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
Cc: Abel Vesa <abel.vesa@...aro.org>, linux-arm-msm@...r.kernel.org, 
	linux-pm@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] power: supply: qcom_battmgr: Add lithium-polymer entry

Hi,

On Fri, May 23, 2025 at 01:44:09PM +0300, Dmitry Baryshkov wrote:
> On Fri, May 23, 2025 at 01:14:22PM +0300, Abel Vesa wrote:
> > On some Dell XPS 13 (9345) variants, the battery used is lithium-polymer
> > based. Currently, this is reported as unknown technology due to the entry
> > missing.
> > 
> > [ 4083.135325] Unknown battery technology 'LIP'
> > 
> > Add another check for lithium-polymer in the technology parsing callback
> > and return that instead of unknown.
> > 
> > Signed-off-by: Abel Vesa <abel.vesa@...aro.org>
> > ---
> >  drivers/power/supply/qcom_battmgr.c | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/drivers/power/supply/qcom_battmgr.c b/drivers/power/supply/qcom_battmgr.c
> > index fe27676fbc7cd12292caa6fb3b5b46a18c426e6d..32c85939b8973422ee417c3f1552f2355658cf06 100644
> > --- a/drivers/power/supply/qcom_battmgr.c
> > +++ b/drivers/power/supply/qcom_battmgr.c
> > @@ -981,6 +981,8 @@ static unsigned int qcom_battmgr_sc8280xp_parse_technology(const char *chemistry
> >  {
> >  	if (!strncmp(chemistry, "LIO", BATTMGR_CHEMISTRY_LEN))
> >  		return POWER_SUPPLY_TECHNOLOGY_LION;
> > +	else if (!strncmp(chemistry, "LIP", BATTMGR_CHEMISTRY_LEN))
> 
> no need for 'else'.

I fixed that up while applying.

Greetings,

-- Sebastian

> 
> With that fixed:
> 
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
> 
> 
> > +		return POWER_SUPPLY_TECHNOLOGY_LIPO;
> >  
> >  	pr_err("Unknown battery technology '%s'\n", chemistry);
> >  	return POWER_SUPPLY_TECHNOLOGY_UNKNOWN;
> > 
> > ---
> > base-commit: 176e917e010cb7dcc605f11d2bc33f304292482b
> > change-id: 20250523-psy-qcom-battmgr-add-lipo-entry-e75b3be303b6
> > 
> > Best regards,
> > -- 
> > Abel Vesa <abel.vesa@...aro.org>
> > 
> 
> -- 
> With best wishes
> Dmitry

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)