1. Patchew
  2. linux
  3. View series

[PATCH] iio: adc: axp288: Add TS bias override for Haier HV103H

Vladyslav Ivashchenko posted 1 patch 3 weeks ago 
drivers/iio/adc/axp288_adc.c | 8 ++++++++
1 file changed, 8 insertions(+)
[PATCH] iio: adc: axp288: Add TS bias override for Haier HV103H
Posted by Vladyslav Ivashchenko 3 weeks ago 
The Haier HV103H firmware configures the AXP288 TS pin bias
current to 60 uA. This causes the battery temperature reading to
cross the charging temperature threshold under load, incorrectly
stopping battery charging.

Setting the TS bias current to 80 uA fixes the temperature
measurement and prevents charging from being incorrectly disabled.

Add a DMI quirk to use the 80 uA TS bias current on the Haier
HV103H.

Signed-off-by: Vladyslav Ivashchenko <cactucatu4eckuu@gmail.com>
---
 drivers/iio/adc/axp288_adc.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/drivers/iio/adc/axp288_adc.c b/drivers/iio/adc/axp288_adc.c
index c8283279c47..c98863d612a 100644
--- a/drivers/iio/adc/axp288_adc.c
+++ b/drivers/iio/adc/axp288_adc.c
@@ -207,6 +207,14 @@ static const struct dmi_system_id axp288_adc_ts_bias_override[] = {
 		},
 		.driver_data = (void *)(uintptr_t)AXP288_ADC_TS_BIAS_80UA,
 	},
+	{
+		/* Haier HV103H */
+		.matches = {
+		  DMI_MATCH(DMI_SYS_VENDOR, "Haier"),
+		  DMI_MATCH(DMI_PRODUCT_NAME, "HV103H"),
+		},
+		.driver_data = (void *)(uintptr_t)AXP288_ADC_TS_BIAS_80UA,
+	},
 	{ }
 };
 
-- 
2.55.0
Re: [PATCH] iio: adc: axp288: Add TS bias override for Haier HV103H
Posted by Hans de Goede 3 weeks ago 
Hi,

On 4-Sep-26 3:31 PM, Vladyslav Ivashchenko wrote:
> The Haier HV103H firmware configures the AXP288 TS pin bias
> current to 60 uA. This causes the battery temperature reading to
> cross the charging temperature threshold under load, incorrectly
> stopping battery charging.
> 
> Setting the TS bias current to 80 uA fixes the temperature
> measurement and prevents charging from being incorrectly disabled.
> 
> Add a DMI quirk to use the 80 uA TS bias current on the Haier
> HV103H.
> 
> Signed-off-by: Vladyslav Ivashchenko <cactucatu4eckuu@gmail.com>

Thanks, patch looks good to me:

Reviewed-by: Hans de Goede <johannes.goede@oss.qualcomm.com>

Regards,

Hans


> ---
>  drivers/iio/adc/axp288_adc.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/drivers/iio/adc/axp288_adc.c b/drivers/iio/adc/axp288_adc.c
> index c8283279c47..c98863d612a 100644
> --- a/drivers/iio/adc/axp288_adc.c
> +++ b/drivers/iio/adc/axp288_adc.c
> @@ -207,6 +207,14 @@ static const struct dmi_system_id axp288_adc_ts_bias_override[] = {
>  		},
>  		.driver_data = (void *)(uintptr_t)AXP288_ADC_TS_BIAS_80UA,
>  	},
> +	{
> +		/* Haier HV103H */
> +		.matches = {
> +		  DMI_MATCH(DMI_SYS_VENDOR, "Haier"),
> +		  DMI_MATCH(DMI_PRODUCT_NAME, "HV103H"),
> +		},
> +		.driver_data = (void *)(uintptr_t)AXP288_ADC_TS_BIAS_80UA,
> +	},
>  	{ }
>  };
>
Re: [PATCH] iio: adc: axp288: Add TS bias override for Haier HV103H
Posted by Jonathan Cameron 3 weeks ago 
On Fri, 4 Sep 2026 21:48:21 +0200
Hans de Goede <hansg@kernel.org> wrote:

> Hi,
> 
> On 4-Sep-26 3:31 PM, Vladyslav Ivashchenko wrote:
> > The Haier HV103H firmware configures the AXP288 TS pin bias
> > current to 60 uA. This causes the battery temperature reading to
> > cross the charging temperature threshold under load, incorrectly
> > stopping battery charging.
> > 
> > Setting the TS bias current to 80 uA fixes the temperature
> > measurement and prevents charging from being incorrectly disabled.
> > 
> > Add a DMI quirk to use the 80 uA TS bias current on the Haier
> > HV103H.
> > 
> > Signed-off-by: Vladyslav Ivashchenko <cactucatu4eckuu@gmail.com>  
> 
> Thanks, patch looks good to me:
> 
> Reviewed-by: Hans de Goede <johannes.goede@oss.qualcomm.com>
> 
> Regards,
> 
> Hans
> 

Fixes tag appropriate?  There was the original dance of keeping the firmware
version, then this table was added to fix that up.  Anyhow, I'm thinking yes
and tag should be:


Fixes: 9bcf15f75cac ("iio: adc: axp288: Fix TS-pin handling")

But bit awkward as anyone trying to just back port to that better have
been paying attention and picked up the following, so lets have that as well with comment
Fixes: 048058399f19 ("iio: adc: axp288: Override TS pin bias current for some models") # Add necessary infrastructure

With those two, applied to the fixes-togreg branch of iio.git

Shout if you think this should be done differently.
> 
> > ---
> >  drivers/iio/adc/axp288_adc.c | 8 ++++++++
> >  1 file changed, 8 insertions(+)
> > 
> > diff --git a/drivers/iio/adc/axp288_adc.c b/drivers/iio/adc/axp288_adc.c
> > index c8283279c47..c98863d612a 100644
> > --- a/drivers/iio/adc/axp288_adc.c
> > +++ b/drivers/iio/adc/axp288_adc.c
> > @@ -207,6 +207,14 @@ static const struct dmi_system_id axp288_adc_ts_bias_override[] = {
> >  		},
> >  		.driver_data = (void *)(uintptr_t)AXP288_ADC_TS_BIAS_80UA,
> >  	},
> > +	{
> > +		/* Haier HV103H */
> > +		.matches = {
> > +		  DMI_MATCH(DMI_SYS_VENDOR, "Haier"),
> > +		  DMI_MATCH(DMI_PRODUCT_NAME, "HV103H"),
> > +		},
> > +		.driver_data = (void *)(uintptr_t)AXP288_ADC_TS_BIAS_80UA,
> > +	},
> >  	{ }
> >  };
> >    
>
Re: [PATCH] iio: adc: axp288: Add TS bias override for Haier HV103H
Posted by Hans de Goede 2 weeks, 6 days ago 
Hi Jonathan,

On 5-Sep-26 03:42, Jonathan Cameron wrote:
> On Fri, 4 Sep 2026 21:48:21 +0200
> Hans de Goede <hansg@kernel.org> wrote:
> 
>> Hi,
>>
>> On 4-Sep-26 3:31 PM, Vladyslav Ivashchenko wrote:
>>> The Haier HV103H firmware configures the AXP288 TS pin bias
>>> current to 60 uA. This causes the battery temperature reading to
>>> cross the charging temperature threshold under load, incorrectly
>>> stopping battery charging.
>>>
>>> Setting the TS bias current to 80 uA fixes the temperature
>>> measurement and prevents charging from being incorrectly disabled.
>>>
>>> Add a DMI quirk to use the 80 uA TS bias current on the Haier
>>> HV103H.
>>>
>>> Signed-off-by: Vladyslav Ivashchenko <cactucatu4eckuu@gmail.com>  
>>
>> Thanks, patch looks good to me:
>>
>> Reviewed-by: Hans de Goede <johannes.goede@oss.qualcomm.com>
>>
>> Regards,
>>
>> Hans
>>
> 
> Fixes tag appropriate?  There was the original dance of keeping the firmware
> version, then this table was added to fix that up.  Anyhow, I'm thinking yes
> and tag should be:
> 
> 
> Fixes: 9bcf15f75cac ("iio: adc: axp288: Fix TS-pin handling")
> 
> But bit awkward as anyone trying to just back port to that better have
> been paying attention and picked up the following, so lets have that as well with comment
> Fixes: 048058399f19 ("iio: adc: axp288: Override TS pin bias current for some models") # Add necessary infrastructure
> 
> With those two, applied to the fixes-togreg branch of iio.git
> 
> Shout if you think this should be done differently.

This sounds good to me, thank you for taking care of this.

Regards,

Hans


>>
>>> ---
>>>  drivers/iio/adc/axp288_adc.c | 8 ++++++++
>>>  1 file changed, 8 insertions(+)
>>>
>>> diff --git a/drivers/iio/adc/axp288_adc.c b/drivers/iio/adc/axp288_adc.c
>>> index c8283279c47..c98863d612a 100644
>>> --- a/drivers/iio/adc/axp288_adc.c
>>> +++ b/drivers/iio/adc/axp288_adc.c
>>> @@ -207,6 +207,14 @@ static const struct dmi_system_id axp288_adc_ts_bias_override[] = {
>>>  		},
>>>  		.driver_data = (void *)(uintptr_t)AXP288_ADC_TS_BIAS_80UA,
>>>  	},
>>> +	{
>>> +		/* Haier HV103H */
>>> +		.matches = {
>>> +		  DMI_MATCH(DMI_SYS_VENDOR, "Haier"),
>>> +		  DMI_MATCH(DMI_PRODUCT_NAME, "HV103H"),
>>> +		},
>>> +		.driver_data = (void *)(uintptr_t)AXP288_ADC_TS_BIAS_80UA,
>>> +	},
>>>  	{ }
>>>  };
>>>    
>>
>

Patchew 2.1-devel-ea86937

© 2016 - 2026 Red Hat, Inc.