From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dawei Chien Subject: Re: [PATCH v2 3/3] arm: dts: thermal: add thermal/auxadc node. Date: Wed, 7 Sep 2016 17:47:11 +0800 Message-ID: <1473241631.8994.2.camel@mtksdaap41> References: <1471492253-7185-1-git-send-email-dawei.chien@mediatek.com> <1471492253-7185-4-git-send-email-dawei.chien@mediatek.com> <1473121447.2201.20.camel@intel.com> <1473121807.2201.23.camel@intel.com> <1473132510.15922.13.camel@mtksdaap41> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: <1473132510.15922.13.camel@mtksdaap41> Sender: linux-pm-owner@vger.kernel.org To: Zhang Rui Cc: Eduardo Valentin , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , Matthias Brugger , linux-pm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, srv_heupstream@mediatek.com, Sascha Hauer , Fan Chen , Eddie Huang , Yingjoe Chen , Erin Lo List-Id: devicetree@vger.kernel.org On Tue, 2016-09-06 at 11:28 +0800, Dawei Chien wrote: > Dear Rui, > > On Tue, 2016-09-06 at 08:30 +0800, Zhang Rui wrote: > > On 二, 2016-09-06 at 08:24 +0800, Zhang Rui wrote: > > > On 四, 2016-08-18 at 11:50 +0800, Dawei Chien wrote: > > > > > > > > This adds the thermal controller and auxadc nodes > > > > to the Mediatek MT2701 dtsi file. > > > > > > > > Signed-off-by: Dawei Chien > > > > --- > > > > This patch depned on: > > > > https://patchwork.kernel.org/patch/9249589/ > > > > https://patchwork.kernel.org/patch/9249589/ > > > Hmm, does this mean that I should take all these three patches and > > > hold > > > the patches in next merge window until the above patch has been > > > merged? > > > > Yes, this patch is base on auxadc patchset. > > > Patch 3/3 can not be applied on top of upstream kernel, so I guess it > > still has some other dependencies. > > I think I can apply patch 1, 2 first. And patch 3/3 should be handled > > in the dt tree. > > Sorry, this patch is also need Mediatek's clock patchset what I miss to > add in dependency list, I would resend this dt tree once clock patch > merge, or resend this for right dependency on next version, thank you. Actually, since patch1/patch2 would be OK so far, I would just only resend patch3 after MTK clock pathcset ready, I prefer to merge patch1/patch2 if possible, thank you. > > thanks, > > rui > > > thanks, > > > rui > > > > > > > > --- > > > > arch/arm/boot/dts/mt2701.dtsi | 44 > > > > +++++++++++++++++++++++++++++++++++++++++ > > > > 1 file changed, 44 insertions(+) > > > > > > > > diff --git a/arch/arm/boot/dts/mt2701.dtsi > > > > b/arch/arm/boot/dts/mt2701.dtsi > > > > index e9150a4..cee4724 100644 > > > > --- a/arch/arm/boot/dts/mt2701.dtsi > > > > +++ b/arch/arm/boot/dts/mt2701.dtsi > > > > @@ -87,6 +87,36 @@ > > > > clock-output-names = "rtc32k"; > > > > }; > > > > > > > > + thermal-zones { > > > > + cpu_thermal: cpu_thermal { > > > > + polling-delay-passive = <1000>; /* > > > > milliseconds */ > > > > + polling-delay = <1000>; /* milliseconds */ > > > > + > > > > + thermal-sensors = <&thermal 0>; > > > > + sustainable-power = <1000>; > > > > + > > > > + trips { > > > > + threshold: trip-point@0 { > > > > + temperature = <68000>; > > > > + hysteresis = <2000>; > > > > + type = "passive"; > > > > + }; > > > > + > > > > + target: trip-point@1 { > > > > + temperature = <85000>; > > > > + hysteresis = <2000>; > > > > + type = "passive"; > > > > + }; > > > > + > > > > + cpu_crit: cpu_crit@0 { > > > > + temperature = <115000>; > > > > + hysteresis = <2000>; > > > > + type = "critical"; > > > > + }; > > > > + }; > > > > + }; > > > > + }; > > > > + > > > > timer { > > > > compatible = "arm,armv7-timer"; > > > > interrupt-parent = <&gic>; > > > > @@ -222,4 +252,18 @@ > > > > clock-names = "baud", "bus"; > > > > status = "disabled"; > > > > }; > > > > + > > > > + thermal: thermal@1100b000 { > > > > + #thermal-sensor-cells = <0>; > > > > + compatible = "mediatek,mt2701-thermal"; > > > > + reg = <0 0x1100b000 0 0x1000>; > > > > + interrupts = ; > > > > + clocks = <&pericfg CLK_PERI_THERM>, <&pericfg > > > > CLK_PERI_AUXADC>; > > > > + clock-names = "therm", "auxadc"; > > > > + resets = <&pericfg 0x10>; > > > > + reset-names = "therm"; > > > > + mediatek,auxadc = <&auxadc>; > > > > + mediatek,apmixedsys = <&apmixedsys>; > > > > + }; > > > > + > > > > }; > > > -- > > > To unsubscribe from this list: send the line "unsubscribe linux-pm" > > > in > > > the body of a message to majordomo@vger.kernel.org > > > More majordomo info at http://vger.kernel.org/majordomo-info.html >