From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933781AbaHZCV1 (ORCPT ); Mon, 25 Aug 2014 22:21:27 -0400 Received: from hqemgate15.nvidia.com ([216.228.121.64]:6053 "EHLO hqemgate15.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933682AbaHZCVZ (ORCPT ); Mon, 25 Aug 2014 22:21:25 -0400 X-PGP-Universal: processed; by hqnvupgp07.nvidia.com on Mon, 25 Aug 2014 19:07:15 -0700 Message-ID: <53FBEF37.2040008@nvidia.com> Date: Tue, 26 Aug 2014 10:21:43 +0800 From: Wei Ni User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0 MIME-Version: 1.0 To: Eduardo Valentin CC: , , , , , Subject: Re: [PATCH v3 4/4] ARM: tegra: dalmore: add thermal zones for nct1008 References: <1408948188-4181-1-git-send-email-wni@nvidia.com> <1408948188-4181-5-git-send-email-wni@nvidia.com> <20140825110844.GB12349@developer> In-Reply-To: <20140825110844.GB12349@developer> X-Originating-IP: [10.19.224.131] X-ClientProxiedBy: HKMAIL101.nvidia.com (10.18.16.10) To HKMAIL101.nvidia.com (10.18.16.10) Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 08/25/2014 07:08 PM, Eduardo Valentin wrote: > On Mon, Aug 25, 2014 at 02:29:48PM +0800, Wei Ni wrote: >> From: lightning314 >> >> Add dt node to describe the thermal zone for the nct1008. >> >> Signed-off-by: Wei Ni >> --- >> arch/arm/boot/dts/tegra114-dalmore.dts | 20 +++++++++++++++++++- >> 1 file changed, 19 insertions(+), 1 deletion(-) >> >> diff --git a/arch/arm/boot/dts/tegra114-dalmore.dts b/arch/arm/boot/dts/tegra114-dalmore.dts >> index 5c21d21..94a1b5d 100644 >> --- a/arch/arm/boot/dts/tegra114-dalmore.dts >> +++ b/arch/arm/boot/dts/tegra114-dalmore.dts >> @@ -779,12 +779,14 @@ >> <&gpio TEGRA_GPIO(V, 3) GPIO_ACTIVE_HIGH>; >> }; >> >> - temperature-sensor@4c { >> + nct1008: temperature-sensor@4c { >> compatible = "onnn,nct1008"; >> reg = <0x4c>; >> vcc-supply = <&palmas_ldo6_reg>; >> interrupt-parent = <&gpio>; >> interrupts = ; >> + >> + #thermal-sensor-cells = <1>; >> }; >> }; >> >> @@ -1283,4 +1285,20 @@ >> <&tegra_car TEGRA114_CLK_EXTERN1>; >> clock-names = "pll_a", "pll_a_out0", "mclk"; >> }; >> + >> + thermal-zones { >> + nct1008-local { >> + polling-delay-passive = <2000>; /* milliseconds */ >> + polling-delay = <0>; /* milliseconds */ >> + >> + thermal-sensors = <&nct1008 0>; >> + }; >> + >> + nct1008-remote { >> + polling-delay-passive = <1000>; /* milliseconds */ >> + polling-delay = <0>; /* milliseconds */ >> + >> + thermal-sensors = <&nct1008 1>; >> + }; >> + }; > > The above zones misses the required properties, as per the thermal.txt > binding description. Could you please have a look on those that are > required and improve the zones above? I changed the thermal.txt in the patch 3/4, to move these two properties to optional property. On the Dalmore, we just need to register these two sensors as thermal zone devices. Indeed, we have a skin-temperature driver, which used these two thermal zone deives' temperature to estimator the skin temperature, so we doesn't need to set trips and bind with any cooling devices on them. Thanks. Wei. > >> }; >> -- >> 1.8.1.5 >> > -- > To unsubscribe from this list: send the line "unsubscribe linux-tegra" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >