From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Subject: Re: [PATCHv3 05/16] hwmon: lm75: expose to thermal fw via DT nodes Date: Sat, 21 Sep 2013 19:24:55 -0700 Message-ID: <523E54F7.9010403@roeck-us.net> References: <1379457325-18192-1-git-send-email-eduardo.valentin@ti.com> <1379521314-16959-1-git-send-email-eduardo.valentin@ti.com> <523DE01F.4020607@roeck-us.net> <523E2C1B.9010307@ti.com> <523E321F.8050502@roeck-us.net> <523E388F.80707@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <523E388F.80707-l0cyMroinI0@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Eduardo Valentin Cc: swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org, pawel.moll-5wv7dgnIgG8@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, ian.campbell-Sxgqhf6Nn4DQT0dZR+AlfA@public.gmane.org, rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org, rui.zhang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, wni-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org, grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org, durgadoss.r-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, lm-sensors-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Jean Delvare List-Id: linux-pm@vger.kernel.org On 09/21/2013 05:23 PM, Eduardo Valentin wrote: > On 21-09-2013 19:56, Guenter Roeck wrote: >> On 09/21/2013 04:30 PM, Eduardo Valentin wrote: >>> On 21-09-2013 14:06, Guenter Roeck wrote: >>>> On 09/18/2013 09:21 AM, Eduardo Valentin wrote: >>>>> This patch adds to lm75 temperature sensor the possibility >>>>> to expose itself as thermal zone device, registered on the >>>>> thermal framework. >>>>> >>>>> The thermal zone is built only if a device tree node >>>>> describing a thermal zone for this sensor is present >>>>> inside the lm75 DT node. Otherwise, the driver behavior >>>>> will be the same. >>>>> >>>>> Cc: Jean Delvare >>>>> Cc: Guenter Roeck >>>>> Cc: lm-sensors-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org >>>>> Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org >>>>> Signed-off-by: Eduardo Valentin >>>> >>>> Acked-by: Guenter Roeck >>>> >>> >>> Cool! thanks again for taking the time to review this. >>> >>>> I assume this will be sent upstream through the thermal tree ? >>> >>> Yeah, that is the idea, at least for the core part. The hwmon changes I >>> believe goes via hwmon tree, right? >>> >> >> Not necessarily, as the hwmon code depends on the core code. >> In such cases it is quite common that both are sent through the same tree. > > If this works for you I can queue these changes via the thermal tree. > Ok with me. Guenter -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Date: Sun, 22 Sep 2013 02:24:55 +0000 Subject: Re: [lm-sensors] [PATCHv3 05/16] hwmon: lm75: expose to thermal fw via DT nodes Message-Id: <523E54F7.9010403@roeck-us.net> List-Id: References: <1379457325-18192-1-git-send-email-eduardo.valentin@ti.com> <1379521314-16959-1-git-send-email-eduardo.valentin@ti.com> <523DE01F.4020607@roeck-us.net> <523E2C1B.9010307@ti.com> <523E321F.8050502@roeck-us.net> <523E388F.80707@ti.com> In-Reply-To: <523E388F.80707@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Eduardo Valentin Cc: swarren@wwwdotorg.org, pawel.moll@arm.com, mark.rutland@arm.com, ian.campbell@citrix.com, rob.herring@calxeda.com, rui.zhang@intel.com, wni@nvidia.com, grant.likely@linaro.org, durgadoss.r@intel.com, linux-pm@vger.kernel.org, devicetree@vger.kernel.org, lm-sensors@lm-sensors.org, linux-kernel@vger.kernel.org, Jean Delvare On 09/21/2013 05:23 PM, Eduardo Valentin wrote: > On 21-09-2013 19:56, Guenter Roeck wrote: >> On 09/21/2013 04:30 PM, Eduardo Valentin wrote: >>> On 21-09-2013 14:06, Guenter Roeck wrote: >>>> On 09/18/2013 09:21 AM, Eduardo Valentin wrote: >>>>> This patch adds to lm75 temperature sensor the possibility >>>>> to expose itself as thermal zone device, registered on the >>>>> thermal framework. >>>>> >>>>> The thermal zone is built only if a device tree node >>>>> describing a thermal zone for this sensor is present >>>>> inside the lm75 DT node. Otherwise, the driver behavior >>>>> will be the same. >>>>> >>>>> Cc: Jean Delvare >>>>> Cc: Guenter Roeck >>>>> Cc: lm-sensors@lm-sensors.org >>>>> Cc: linux-kernel@vger.kernel.org >>>>> Signed-off-by: Eduardo Valentin >>>> >>>> Acked-by: Guenter Roeck >>>> >>> >>> Cool! thanks again for taking the time to review this. >>> >>>> I assume this will be sent upstream through the thermal tree ? >>> >>> Yeah, that is the idea, at least for the core part. The hwmon changes I >>> believe goes via hwmon tree, right? >>> >> >> Not necessarily, as the hwmon code depends on the core code. >> In such cases it is quite common that both are sent through the same tree. > > If this works for you I can queue these changes via the thermal tree. > Ok with me. Guenter _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752708Ab3IVCZH (ORCPT ); Sat, 21 Sep 2013 22:25:07 -0400 Received: from mail.active-venture.com ([67.228.131.205]:61111 "EHLO mail.active-venture.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752577Ab3IVCZE (ORCPT ); Sat, 21 Sep 2013 22:25:04 -0400 X-Originating-IP: 108.223.40.66 Message-ID: <523E54F7.9010403@roeck-us.net> Date: Sat, 21 Sep 2013 19:24:55 -0700 From: Guenter Roeck User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130803 Thunderbird/17.0.8 MIME-Version: 1.0 To: Eduardo Valentin CC: swarren@wwwdotorg.org, pawel.moll@arm.com, mark.rutland@arm.com, ian.campbell@citrix.com, rob.herring@calxeda.com, rui.zhang@intel.com, wni@nvidia.com, grant.likely@linaro.org, durgadoss.r@intel.com, linux-pm@vger.kernel.org, devicetree@vger.kernel.org, lm-sensors@lm-sensors.org, linux-kernel@vger.kernel.org, Jean Delvare Subject: Re: [PATCHv3 05/16] hwmon: lm75: expose to thermal fw via DT nodes References: <1379457325-18192-1-git-send-email-eduardo.valentin@ti.com> <1379521314-16959-1-git-send-email-eduardo.valentin@ti.com> <523DE01F.4020607@roeck-us.net> <523E2C1B.9010307@ti.com> <523E321F.8050502@roeck-us.net> <523E388F.80707@ti.com> In-Reply-To: <523E388F.80707@ti.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 09/21/2013 05:23 PM, Eduardo Valentin wrote: > On 21-09-2013 19:56, Guenter Roeck wrote: >> On 09/21/2013 04:30 PM, Eduardo Valentin wrote: >>> On 21-09-2013 14:06, Guenter Roeck wrote: >>>> On 09/18/2013 09:21 AM, Eduardo Valentin wrote: >>>>> This patch adds to lm75 temperature sensor the possibility >>>>> to expose itself as thermal zone device, registered on the >>>>> thermal framework. >>>>> >>>>> The thermal zone is built only if a device tree node >>>>> describing a thermal zone for this sensor is present >>>>> inside the lm75 DT node. Otherwise, the driver behavior >>>>> will be the same. >>>>> >>>>> Cc: Jean Delvare >>>>> Cc: Guenter Roeck >>>>> Cc: lm-sensors@lm-sensors.org >>>>> Cc: linux-kernel@vger.kernel.org >>>>> Signed-off-by: Eduardo Valentin >>>> >>>> Acked-by: Guenter Roeck >>>> >>> >>> Cool! thanks again for taking the time to review this. >>> >>>> I assume this will be sent upstream through the thermal tree ? >>> >>> Yeah, that is the idea, at least for the core part. The hwmon changes I >>> believe goes via hwmon tree, right? >>> >> >> Not necessarily, as the hwmon code depends on the core code. >> In such cases it is quite common that both are sent through the same tree. > > If this works for you I can queue these changes via the thermal tree. > Ok with me. Guenter