From: Zhang Rui <rui.zhang@intel.com> To: Dawei Chien <dawei.chien@mediatek.com>, Eduardo Valentin <edubezval@gmail.com> Cc: Rob Herring <robh+dt@kernel.org>, Pawel Moll <pawel.moll@arm.com>, Mark Rutland <mark.rutland@arm.com>, Ian Campbell <ijc+devicetree@hellion.org.uk>, Kumar Gala <galak@codeaurora.org>, Russell King <linux@armlinux.org.uk>, Matthias Brugger <matthias.bgg@gmail.com>, 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 <kernel@pengutronix.de>, Fan Chen <fan.chen@mediatek.com>, Eddie Huang <eddie.huang@mediatek.com>, Yingjoe Chen <yingjoe.chen@mediatek.com>, Erin Lo <erin.lo@mediatek.com> Subject: Re: [PATCH v2 3/3] arm: dts: thermal: add thermal/auxadc node. Date: Tue, 06 Sep 2016 08:30:07 +0800 [thread overview] Message-ID: <1473121807.2201.23.camel@intel.com> (raw) In-Reply-To: <1473121447.2201.20.camel@intel.com> 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 <dawei.chien@mediatek.com> > > --- > > 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? > 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. 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 = <GIC_SPI 70 IRQ_TYPE_LEVEL_LOW>; > > + 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
WARNING: multiple messages have this Message-ID (diff)
From: rui.zhang@intel.com (Zhang Rui) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v2 3/3] arm: dts: thermal: add thermal/auxadc node. Date: Tue, 06 Sep 2016 08:30:07 +0800 [thread overview] Message-ID: <1473121807.2201.23.camel@intel.com> (raw) In-Reply-To: <1473121447.2201.20.camel@intel.com> 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 <dawei.chien@mediatek.com> > > --- > > 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? > 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. 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 at 0 { > > + temperature = <68000>; > > + hysteresis = <2000>; > > + type = "passive"; > > + }; > > + > > + target: trip-point at 1 { > > + temperature = <85000>; > > + hysteresis = <2000>; > > + type = "passive"; > > + }; > > + > > + cpu_crit: cpu_crit at 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 at 1100b000 { > > + #thermal-sensor-cells = <0>; > > + compatible = "mediatek,mt2701-thermal"; > > + reg = <0 0x1100b000 0 0x1000>; > > + interrupts = <GIC_SPI 70 IRQ_TYPE_LEVEL_LOW>; > > + 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 at vger.kernel.org > More majordomo info at??http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2016-09-06 0:30 UTC|newest] Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-08-18 3:50 [PATCH v2 0/3] Add Mediatek thermal driver for mt2701 Dawei Chien 2016-08-18 3:50 ` Dawei Chien 2016-08-18 3:50 ` Dawei Chien 2016-08-18 3:50 ` [PATCH v2 1/3] dt-bindings: thermal: Add binding document for Mediatek thermal controller Dawei Chien 2016-08-18 3:50 ` Dawei Chien 2016-08-18 3:50 ` Dawei Chien 2016-08-18 4:58 ` Keerthy 2016-08-18 4:58 ` Keerthy 2016-08-18 4:58 ` Keerthy 2016-08-19 13:44 ` Rob Herring 2016-08-19 13:44 ` Rob Herring 2016-08-19 13:44 ` Rob Herring 2016-08-18 3:50 ` [PATCH v2 2/3] thermal: Add Mediatek thermal driver for mt2701 Dawei Chien 2016-08-18 3:50 ` Dawei Chien 2016-08-18 3:50 ` Dawei Chien 2016-08-18 3:50 ` [PATCH v2 3/3] arm: dts: thermal: add thermal/auxadc node Dawei Chien 2016-08-18 3:50 ` Dawei Chien 2016-08-18 3:50 ` Dawei Chien 2016-09-06 0:24 ` Zhang Rui 2016-09-06 0:24 ` Zhang Rui 2016-09-06 0:24 ` Zhang Rui 2016-09-06 0:30 ` Zhang Rui [this message] 2016-09-06 0:30 ` Zhang Rui 2016-09-06 3:28 ` Dawei Chien 2016-09-06 3:28 ` Dawei Chien 2016-09-06 3:28 ` Dawei Chien 2016-09-07 9:47 ` Dawei Chien 2016-09-07 9:47 ` Dawei Chien 2016-09-07 9:47 ` Dawei Chien 2016-09-13 8:48 ` Matthias Brugger 2016-09-13 8:48 ` Matthias Brugger
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=1473121807.2201.23.camel@intel.com \ --to=rui.zhang@intel.com \ --cc=dawei.chien@mediatek.com \ --cc=devicetree@vger.kernel.org \ --cc=eddie.huang@mediatek.com \ --cc=edubezval@gmail.com \ --cc=erin.lo@mediatek.com \ --cc=fan.chen@mediatek.com \ --cc=galak@codeaurora.org \ --cc=ijc+devicetree@hellion.org.uk \ --cc=kernel@pengutronix.de \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mediatek@lists.infradead.org \ --cc=linux-pm@vger.kernel.org \ --cc=linux@armlinux.org.uk \ --cc=mark.rutland@arm.com \ --cc=matthias.bgg@gmail.com \ --cc=pawel.moll@arm.com \ --cc=robh+dt@kernel.org \ --cc=srv_heupstream@mediatek.com \ --cc=yingjoe.chen@mediatek.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.