From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wei Ni Subject: Re: [RFC PATCH 1/9] ARM: dt: t30 cardhu: add dt entry for lm90 Date: Tue, 19 Feb 2013 17:52:35 +0800 Message-ID: <51234B63.7080104@nvidia.com> References: <1361187031-3679-1-git-send-email-wni@nvidia.com> <1361187031-3679-2-git-send-email-wni@nvidia.com> <5122F162.9030103@nvidia.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Return-path: Received: from hqemgate04.nvidia.com ([216.228.121.35]:8897 "EHLO hqemgate04.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750791Ab3BSJwv (ORCPT ); Tue, 19 Feb 2013 04:52:51 -0500 In-Reply-To: <5122F162.9030103@nvidia.com> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: Alex Courbot Cc: "durgadoss.r@intel.com" , "rui.zhang@intel.com" , Matthew Longnecker , "khali@linux-fr.org" , "devicetree-discuss@lists.ozlabs.org" , "linux-tegra@vger.kernel.org" , "lm-sensors@lm-sensors.org" , "linux-pm@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" On 02/19/2013 11:28 AM, Alex Courbot wrote: > On 02/18/2013 08:30 PM, Wei Ni wrote: >> Enable thermal sensor lm90 in the dts file. > > Acked-by: Alexandre Courbot > > Btw, shouldn't this patch come last, after all the changes you did to > lm90? If you keep the current order, you will need to ensure that the > device performs as expected on Cardhu for each of the remaining patches > of the series. It is probably safer to just move this one at the end to > avoid trouble. I will try split this patch set into 3 separate sets in my next version, like Matthew's suggestion, so the patches will be clear. Wei. > > Alex. > From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wei Ni Date: Tue, 19 Feb 2013 09:52:35 +0000 Subject: Re: [lm-sensors] [RFC PATCH 1/9] ARM: dt: t30 cardhu: add dt entry for lm90 Message-Id: <51234B63.7080104@nvidia.com> List-Id: References: <1361187031-3679-1-git-send-email-wni@nvidia.com> <1361187031-3679-2-git-send-email-wni@nvidia.com> <5122F162.9030103@nvidia.com> In-Reply-To: <5122F162.9030103@nvidia.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Alex Courbot Cc: "durgadoss.r@intel.com" , "rui.zhang@intel.com" , Matthew Longnecker , "khali@linux-fr.org" , "devicetree-discuss@lists.ozlabs.org" , "linux-tegra@vger.kernel.org" , "lm-sensors@lm-sensors.org" , "linux-pm@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" On 02/19/2013 11:28 AM, Alex Courbot wrote: > On 02/18/2013 08:30 PM, Wei Ni wrote: >> Enable thermal sensor lm90 in the dts file. > > Acked-by: Alexandre Courbot > > Btw, shouldn't this patch come last, after all the changes you did to > lm90? If you keep the current order, you will need to ensure that the > device performs as expected on Cardhu for each of the remaining patches > of the series. It is probably safer to just move this one at the end to > avoid trouble. I will try split this patch set into 3 separate sets in my next version, like Matthew's suggestion, so the patches will be clear. Wei. > > Alex. > _______________________________________________ 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 From: wni@nvidia.com (Wei Ni) Date: Tue, 19 Feb 2013 17:52:35 +0800 Subject: [RFC PATCH 1/9] ARM: dt: t30 cardhu: add dt entry for lm90 In-Reply-To: <5122F162.9030103@nvidia.com> References: <1361187031-3679-1-git-send-email-wni@nvidia.com> <1361187031-3679-2-git-send-email-wni@nvidia.com> <5122F162.9030103@nvidia.com> Message-ID: <51234B63.7080104@nvidia.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 02/19/2013 11:28 AM, Alex Courbot wrote: > On 02/18/2013 08:30 PM, Wei Ni wrote: >> Enable thermal sensor lm90 in the dts file. > > Acked-by: Alexandre Courbot > > Btw, shouldn't this patch come last, after all the changes you did to > lm90? If you keep the current order, you will need to ensure that the > device performs as expected on Cardhu for each of the remaining patches > of the series. It is probably safer to just move this one at the end to > avoid trouble. I will try split this patch set into 3 separate sets in my next version, like Matthew's suggestion, so the patches will be clear. Wei. > > Alex. >