Messages in this thread | | | Date | Fri, 1 Aug 2008 12:28:53 +0400 | From | Anton Vorontsov <> | Subject | Re: [PATCH] power: support for Texas Instruments BQ27x00 battery managers. |
| |
On Thu, Jul 31, 2008 at 11:03:54PM +0200, Rodolfo Giometti wrote: > Sorry for the long delay... I was very busy in these days! :'(
No problem. ;-)
> On Mon, Jun 23, 2008 at 10:38:17AM +0400, Anton Vorontsov wrote: > > > Here is the patch to show the idea. I only tested it to build, so no > > guaranties that it will work on the actual hardware. :-) > > > > This patch should be separated: one patch is for I2C subsystem > > maintainer (drivers/i2c/* changes) and another (drivers/power/* + > > include/linux/bq27x00.h) is for me. > > On my system I need to specify 2 battery chips into the platform > specific file as follow (file is arch/arm/mach-pxa/wr1100.c): > > static struct i2c_board_info __initdata wr1100_i2c_mux_devices[] = { > { > I2C_BOARD_INFO("bq27200", 0x55), > }, > }; > > i2c_register_board_info(2, wr1100_i2c_mux_devices, > ARRAY_SIZE(wr1100_i2c_mux_devices)); > i2c_register_board_info(3, wr1100_i2c_mux_devices, > ARRAY_SIZE(wr1100_i2c_mux_devices)); > > By using your solution I cannot use this code, is that right?
Why not? I think there should be no problem.
[...] > if you agree, I prefere resend my > patch without the #ifdef and the w1 support (since I cannot test it at > all) but living the possibility to add that support later.
Yes, that would be also fine.
Thanks,
-- Anton Vorontsov email: cbouatmailru@gmail.com irc://irc.freenode.net/bd2
| |