Re: [PATCH] 2.6.24-rc: fix ACPI battery technology reporting

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



applied.

though kmail found it to be impossible to save
an un-corrupted plain text version of andrezy's message.

should the other string compares be updated like this one?

-len

On Sunday 11 November 2007 17:38, Alexey Starikovskiy wrote:
> Andrey Borzenkov wrote:
> > Subject: [PATCH] 2.6.24-rc: fix ACPI battery technology reporting
> > From: Andrey Borzenkov <arvidjaar@xxxxxxx>
> >
> > At least some systems report technology information with trailing spaces:
> >
> > {pts/1}% cat -E /var/tmp/bat/2.6.23 | grep type
> > battery type:            Li-ION  $
> >
> > Use strncasecmp to compare model string to skip trailing part
> >
> > Signed-off-by: Andrey Borzenkov <arvidjaar@xxxxxxx>
> >
> >   
> Acked-by: Alexey Starikovskiy <astarikovskiy@xxxxxxx>
> > ---
> >
> >  drivers/acpi/battery.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/drivers/acpi/battery.c b/drivers/acpi/battery.c
> > index c2ce0ad..1905b88 100644
> > --- a/drivers/acpi/battery.c
> > +++ b/drivers/acpi/battery.c
> > @@ -125,7 +125,7 @@ static int acpi_battery_technology(struct acpi_battery 
> > *battery)
> >  		return POWER_SUPPLY_TECHNOLOGY_NiMH;
> >  	if (!strcasecmp("LION", battery->type))
> >  		return POWER_SUPPLY_TECHNOLOGY_LION;
> > -	if (!strcasecmp("LI-ION", battery->type))
> > +	if (!strncasecmp("LI-ION", battery->type, 6))
> >  		return POWER_SUPPLY_TECHNOLOGY_LION;
> >  	if (!strcasecmp("LiP", battery->type))
> >  		return POWER_SUPPLY_TECHNOLOGY_LIPO;
> >   
> 
> -
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
-
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux