From: "Tc, Jenny" <jenny.tc@intel.com>
To: Anton Vorontsov <cbouatmailru@gmail.com>
CC: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "R, Durgadoss" <durgadoss.r@intel.com>
Subject: RE: [PATCHv2] power_supply: Register battery as a thermal zone
Thread-Topic: [PATCHv2] power_supply: Register battery as a thermal zone
Thread-Index: AQHNTQs71I137SCSiUGywBoF1Z1h2Jb/PkgAgABc6RA=
Date: Mon, 18 Jun 2012 06:23:47 +0000
Message-ID: <20ADAB092842284E95860F279283C5642825FA@BGSMSX101.gar.corp.intel.com>
References: <1336576007-8772-1-git-send-email-jenny.tc@intel.com>
 <20120618042927.GA25372@lizard> <20120618061916.GA27786@lizard>
In-Reply-To: <20120618061916.GA27786@lizard>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Sender: linux-kernel-owner@vger.kernel.org
Content-Transfer-Encoding: 8bit
Content-Length: 1122
Lines: 32

> -----Original Message-----
> From: Anton Vorontsov [mailto:cbouatmailru@gmail.com]
> Sent: Monday, June 18, 2012 11:49 AM
> To: Tc, Jenny
> Cc: linux-kernel@vger.kernel.org; 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@gmail.com
????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m????????????I?