[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20070507141238.GA10858@zarina>
Date: Mon, 7 May 2007 18:12:38 +0400
From: Anton Vorontsov <cbou@...l.ru>
To: Pavel Machek <pavel@....cz>
Cc: Greg KH <gregkh@...e.de>, linux-kernel@...r.kernel.org,
kernel-discuss@...dhelds.org, David Woodhouse <dwmw2@...radead.org>
Subject: Re: [PATCH 8/8] One Laptop Per Child power/battery driver
On Mon, May 07, 2007 at 02:04:54PM +0000, Pavel Machek wrote:
> Hi!
>
> > This probably should be marked as BROKEN because, according
> > to David Woodhouse, EC-acccess method will change. Plus currently
> > it lacks mutexes. So this driver is just for reference. Converted
> > from the battery-2.6 repository, 1:1.
> >
> > But nevertheless it should work.
> >
> > Signed-off-by: Anton Vorontsov <cbou@...l.ru>
> > ---
> > drivers/power/Kconfig | 6 +
> > drivers/power/Makefile | 1 +
> > drivers/power/olpc_battery.c | 300 ++++++++++++++++++++++++++++++++++++++++++
> > 3 files changed, 307 insertions(+), 0 deletions(-)
> > create mode 100644 drivers/power/olpc_battery.c
> >
> > diff --git a/drivers/power/Kconfig b/drivers/power/Kconfig
> > index 051724f..3ac79c3 100644
> > --- a/drivers/power/Kconfig
> > +++ b/drivers/power/Kconfig
> > @@ -42,4 +42,10 @@ config BATTERY_PMU
> > Say Y here to expose battery information on Apple machines
> > through the generic battery class.
> >
> > +config BATTERY_OLPC
> > + tristate "One Laptop Per Child battery"
> > + depends on X86_32
> > + help
> > + Say Y to enable support for the battery on the $100 laptop.
> > +
> > endif # POWER_SUPPLY
> > diff --git a/drivers/power/Makefile b/drivers/power/Makefile
> > index 0ebdc6d..62b58ca 100644
> > --- a/drivers/power/Makefile
> > +++ b/drivers/power/Makefile
> > @@ -19,3 +19,4 @@ obj-$(CONFIG_APM_POWER) += apm_power.o
> >
> > obj-$(CONFIG_BATTERY_DS2760) += ds2760_battery.o
> > obj-$(CONFIG_BATTERY_PMU) += pmu_battery.o
> > +obj-$(CONFIG_BATTERY_OLPC) += olpc_battery.o
> > diff --git a/drivers/power/olpc_battery.c b/drivers/power/olpc_battery.c
> > new file mode 100644
> > index 0000000..40f76bb
> > --- /dev/null
> > +++ b/drivers/power/olpc_battery.c
> > @@ -0,0 +1,300 @@
> > +/*
> > + * Battery driver for One Laptop Per Child ($100 laptop) board.
> > + *
> > + * Copyright ?? 2006 David Woodhouse <dwmw2@...radead.org>
>
> Can we stick to ascii in sources?
>
> > +#define wBAT_VOLTAGE 0xf900 /* *9.76/32, mV */
> > +#define wBAT_CURRENT 0xf902 /* *15.625/120, mA */
> > +#define wBAT_TEMP 0xf906 /* *256/1000, ??C */
> > +#define wAMB_TEMP 0xf908 /* *256/1000, ??C */
>
> Ascii? And those defines seem affected by hungarian convention.
That's up to David Woodhouse. In my code I'm using (c), though David
insisting on true "(c)" symbol. So, this is copyright thing, I can't
resist to the author.
> > +#define sMBAT_STATUS 0xfaa4
> > +#define sBAT_PRESENT 1
> > +#define sBAT_FULL 2
> > +#define sBAT_DESTROY 4 /* what is this exactly? */
> > +#define sBAT_LOW 32
> > +#define sBAT_DISCHG 64
> > +#define sMCHARGE_STATUS 0xfaa5
> > +#define sBAT_CHARGE 1
> > +#define sBAT_OVERTEMP 4
> > +#define sBAT_NiMH 8
> > +#define sPOWER_FLAG 0xfa40
>
>
> Pavel
> --
> (english) http://www.livejournal.com/~pavelmachek
> (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
>
--
Anton Vorontsov
email: cbou@...l.ru
backup email: ya-cbou@...dex.ru
irc://irc.freenode.org/bd2
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists