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

[Len Brown <lenb@kernel.org>]

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@mail.ru>
> >
> > 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@mail.ru>
> >
> >   
> Acked-by: Alexey Starikovskiy <astarikovskiy@suse.de>
> > ---
> >
> >  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@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>