On Fri, Jul 27, 2012 at 06:28:31PM +0000, Pallala, Ramakrishna wrote:
[...]
> > > Signed-off-by: Ramakrishna Pallala <[email protected]>
> > > ---
> > [...]
> > > +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: [email protected]