lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<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

Powered by Openwall GNU/*/Linux Powered by OpenVZ