Messages in this thread | | | Date | Mon, 6 Aug 2012 21:41:43 -0700 | From | Anton Vorontsov <> | Subject | Re: [PATCH v2] power_supply: Added support for power supply attribute sources |
| |
On Fri, Jul 27, 2012 at 06:28:31PM +0000, Pallala, Ramakrishna wrote: [...] > > > Signed-off-by: Ramakrishna Pallala <ramakrishna.pallala@intel.com> > > > --- > > [...] > > > +extern int power_supply_attributes_register(struct device *parent, > > > + struct power_supply_attr_source *psy_attr); > > > > Can you please show some user of the new calls? If I understand correctly, > > you're going to call these from sensing (ADC, or some > > other) drivers, which would be very very wrong thing to do. > > I have submitted two patches, one on smb347_charger driver and one on max17042_battery driver > to demonstrate the use of these API's. > > [PATCH] smb347_charger: Add support for battery power supply attributes registration > [PATCH] max17042_battery: add support for battery STATUS and CHARGE_TYPE
Yeah, I noticed them, I'll review the whole approach tomorrow.
Thanks!
-- Anton Vorontsov Email: cbouatmailru@gmail.com
| |