lkml.org 
[lkml]   [2008]   [Nov]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/2] power_supply: Add cold to the POWER_SUPPLY_HEALTH report values
On Tue, Nov 25, 2008 at 02:25:12PM +0300, Anton Vorontsov wrote:
> On Thu, Nov 20, 2008 at 03:23:40PM +0000, Mark Brown wrote:
> > Some systems are able to report problems with batteries being under
> > temperature.
> >
> > Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
>
> Looks good to me.
>
> Acked-by: Anton Vorontsov <cbouatmailru@gmail.com>
Thanks guys, both patches applied.

Cheers,
Samuel.

> > ---
> > drivers/power/power_supply_sysfs.c | 2 +-
> > include/linux/power_supply.h | 1 +
> > 2 files changed, 2 insertions(+), 1 deletions(-)
> >
> > diff --git a/drivers/power/power_supply_sysfs.c b/drivers/power/power_supply_sysfs.c
> > index 23ae846..ac01e06 100644
> > --- a/drivers/power/power_supply_sysfs.c
> > +++ b/drivers/power/power_supply_sysfs.c
> > @@ -45,7 +45,7 @@ static ssize_t power_supply_show_property(struct device *dev,
> > };
> > static char *health_text[] = {
> > "Unknown", "Good", "Overheat", "Dead", "Over voltage",
> > - "Unspecified failure"
> > + "Unspecified failure", "Cold",
> > };
> > static char *technology_text[] = {
> > "Unknown", "NiMH", "Li-ion", "Li-poly", "LiFe", "NiCd",
> > diff --git a/include/linux/power_supply.h b/include/linux/power_supply.h
> > index f9348cb..8ff25e0 100644
> > --- a/include/linux/power_supply.h
> > +++ b/include/linux/power_supply.h
> > @@ -45,6 +45,7 @@ enum {
> > POWER_SUPPLY_HEALTH_DEAD,
> > POWER_SUPPLY_HEALTH_OVERVOLTAGE,
> > POWER_SUPPLY_HEALTH_UNSPEC_FAILURE,
> > + POWER_SUPPLY_HEALTH_COLD,
> > };
> >
> > enum {
> > --
> > 1.5.6.5
> >
>
> --
> Anton Vorontsov
> email: cbouatmailru@gmail.com
> irc://irc.freenode.net/bd2

--
Intel Open Source Technology Centre
http://oss.intel.com/


\
 
 \ /
  Last update: 2008-12-01 01:11    [W:0.041 / U:1.044 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site