RE: [PATCHv2] power_supply: Register battery as a thermal zone

From: Tc, Jenny
Date: Mon Jun 18 2012 - 02:23:49 EST


> -----Original Message-----
> From: Anton Vorontsov [mailto:cbouatmailru@xxxxxxxxx]
> Sent: Monday, June 18, 2012 11:49 AM
> To: Tc, Jenny
> Cc: linux-kernel@xxxxxxxxxxxxxxx; R, Durgadoss
> Subject: Re: [PATCHv2] power_supply: Register battery as a thermal zone
>
> On Sun, Jun 17, 2012 at 09:29:28PM -0700, Anton Vorontsov wrote:
> [...]
> > +static int psy_register_thermal(struct power_supply *psy) {
> > + int i;
> > +
> > + /* Register battery zone device psy reports temperature */
> > + for (i = 0; i < psy->num_properties; i++) {
> > + if (psy->properties[i] == POWER_SUPPLY_PROP_TEMP) {
> > + psy->tzd = thermal_zone_device_register(psy-
> >name, 0,
>
> Oh, and just FYI, there was a (char *) cast for psy->name, but I removed it.
> The cast should be not needed w/ this patch:
>
> http://lkml.org/lkml/2012/6/18/28

Thanks. I applied it locally when I noticed the warning.

>
> Thanks,
>
> --
> Anton Vorontsov
> Email: cbouatmailru@xxxxxxxxx
N‹§²æìr¸›yúèšØb²X¬¶ÇvØ^–)Þ{.nÇ+‰·¥Š{±‘êçzX§¶›¡Ü}©ž²ÆzÚ&j:+v‰¨¾«‘êçzZ+€Ê+zf£¢·hšˆ§~†­†Ûiÿûàz¹®w¥¢¸?™¨è­Ú&¢)ßf”ù^jÇy§m…á@A«a¶Úÿ 0¶ìh®å’i