From: Dmitry Torokhov <dmitry.torokhov@gmail.com> To: Caesar Wang <caesar.wang@rock-chips.com> Cc: heiko@sntech.de, rui.zhang@intel.com, edubezval@gmail.com, zyf@rock-chips.com, dianders@chromium.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-doc@vger.kernel.org, cf@rock-chips.com, dbasehore@chromium.org, huangtao@rock-chips.com, cjf@rock-chips.com, zhengsq@rock-chips.com, vladimir_zapolskiy@mentor.com Subject: Re: [PATCH v11 4/5] ARM: dts: add main Thermal info to rk3288 Date: Thu, 16 Oct 2014 14:53:16 -0700 [thread overview] Message-ID: <20141016215316.GA30009@dtor-ws> (raw) In-Reply-To: <1413466301-25952-5-git-send-email-caesar.wang@rock-chips.com> Hi Caesar, On Thu, Oct 16, 2014 at 09:31:40PM +0800, Caesar Wang wrote: > This patch is depend on rk3288-thermal.dtsi,or > it will compile error. > > If the temperature over a period of time High,over 120C > the resulting TSHUT gave CRU module,let it reset > the entire chip,or via GPIO give PMIC. > > Signed-off-by: Caesar Wang <caesar.wang@rock-chips.com> > --- > arch/arm/boot/dts/rk3288.dtsi | 20 ++++++++++++++++++++ > 1 file changed, 20 insertions(+) > > diff --git a/arch/arm/boot/dts/rk3288.dtsi b/arch/arm/boot/dts/rk3288.dtsi > index cb18bb4..bf34111 100644 > --- a/arch/arm/boot/dts/rk3288.dtsi > +++ b/arch/arm/boot/dts/rk3288.dtsi > @@ -15,6 +15,7 @@ > #include <dt-bindings/interrupt-controller/arm-gic.h> > #include <dt-bindings/pinctrl/rockchip.h> > #include <dt-bindings/clock/rk3288-cru.h> > +#include <dt-bindings/thermal/thermal.h> > #include "skeleton.dtsi" > > / { > @@ -346,6 +347,19 @@ > status = "disabled"; > }; > > + tsadc: tsadc@ff280000 { > + compatible = "rockchip,rk3288-tsadc"; > + reg = <0xff280000 0x100>; > + interrupts = <GIC_SPI 37 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cru SCLK_TSADC>, <&cru PCLK_TSADC>; > + clock-names = "tsadc", "apb_pclk"; > + pinctrl-names = "default"; > + pinctrl-0 = <&otp_out>; > + #thermal-sensor-cells = <1>; > + hw-shut-temp = <120000>; > + status = "disabled" It looks like you are missing semicolon here. > + }; > + > usb_host0_ehci: usb@ff500000 { > compatible = "generic-ehci"; > reg = <0xff500000 0x100>; > @@ -965,6 +979,12 @@ > }; > }; > > + tsadc { > + otp_out: otp-out { > + rockchip,pins = <0 10 RK_FUNC_1 &pcfg_pull_none>; > + }; > + }; > + > pwm0 { > pwm0_pin: pwm0-pin { > rockchip,pins = <7 0 RK_FUNC_1 &pcfg_pull_none>; > -- > 1.9.1 > > -- Dmitry
WARNING: multiple messages have this Message-ID (diff)
From: dmitry.torokhov@gmail.com (Dmitry Torokhov) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v11 4/5] ARM: dts: add main Thermal info to rk3288 Date: Thu, 16 Oct 2014 14:53:16 -0700 [thread overview] Message-ID: <20141016215316.GA30009@dtor-ws> (raw) In-Reply-To: <1413466301-25952-5-git-send-email-caesar.wang@rock-chips.com> Hi Caesar, On Thu, Oct 16, 2014 at 09:31:40PM +0800, Caesar Wang wrote: > This patch is depend on rk3288-thermal.dtsi,or > it will compile error. > > If the temperature over a period of time High,over 120C > the resulting TSHUT gave CRU module,let it reset > the entire chip,or via GPIO give PMIC. > > Signed-off-by: Caesar Wang <caesar.wang@rock-chips.com> > --- > arch/arm/boot/dts/rk3288.dtsi | 20 ++++++++++++++++++++ > 1 file changed, 20 insertions(+) > > diff --git a/arch/arm/boot/dts/rk3288.dtsi b/arch/arm/boot/dts/rk3288.dtsi > index cb18bb4..bf34111 100644 > --- a/arch/arm/boot/dts/rk3288.dtsi > +++ b/arch/arm/boot/dts/rk3288.dtsi > @@ -15,6 +15,7 @@ > #include <dt-bindings/interrupt-controller/arm-gic.h> > #include <dt-bindings/pinctrl/rockchip.h> > #include <dt-bindings/clock/rk3288-cru.h> > +#include <dt-bindings/thermal/thermal.h> > #include "skeleton.dtsi" > > / { > @@ -346,6 +347,19 @@ > status = "disabled"; > }; > > + tsadc: tsadc at ff280000 { > + compatible = "rockchip,rk3288-tsadc"; > + reg = <0xff280000 0x100>; > + interrupts = <GIC_SPI 37 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cru SCLK_TSADC>, <&cru PCLK_TSADC>; > + clock-names = "tsadc", "apb_pclk"; > + pinctrl-names = "default"; > + pinctrl-0 = <&otp_out>; > + #thermal-sensor-cells = <1>; > + hw-shut-temp = <120000>; > + status = "disabled" It looks like you are missing semicolon here. > + }; > + > usb_host0_ehci: usb at ff500000 { > compatible = "generic-ehci"; > reg = <0xff500000 0x100>; > @@ -965,6 +979,12 @@ > }; > }; > > + tsadc { > + otp_out: otp-out { > + rockchip,pins = <0 10 RK_FUNC_1 &pcfg_pull_none>; > + }; > + }; > + > pwm0 { > pwm0_pin: pwm0-pin { > rockchip,pins = <7 0 RK_FUNC_1 &pcfg_pull_none>; > -- > 1.9.1 > > -- Dmitry
next prev parent reply other threads:[~2014-10-16 21:53 UTC|newest] Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top 2014-10-16 13:31 [PATCH v11 0/5] Rockchip soc thermal driver Caesar Wang 2014-10-16 13:31 ` Caesar Wang 2014-10-16 13:31 ` [PATCH v11 1/5] thermal: rockchip: add driver for thermal Caesar Wang 2014-10-16 13:31 ` Caesar Wang 2014-10-16 22:06 ` Dmitry Torokhov 2014-10-16 22:06 ` Dmitry Torokhov 2014-10-16 22:52 ` Dmitry Torokhov 2014-10-16 22:52 ` Dmitry Torokhov 2014-10-16 23:15 ` Vladimir Zapolskiy 2014-10-16 23:15 ` Vladimir Zapolskiy 2014-10-16 23:15 ` Vladimir Zapolskiy 2014-10-16 13:31 ` [PATCH v11 2/5] dt-bindings: document Rockchip thermal Caesar Wang 2014-10-16 13:31 ` Caesar Wang 2014-10-16 13:31 ` [PATCH v11 3/5] ARM: dts: add RK3288 Thermal data Caesar Wang 2014-10-16 13:31 ` Caesar Wang 2014-10-16 13:31 ` [PATCH v11 4/5] ARM: dts: add main Thermal info to rk3288 Caesar Wang 2014-10-16 13:31 ` Caesar Wang 2014-10-16 21:53 ` Dmitry Torokhov [this message] 2014-10-16 21:53 ` Dmitry Torokhov 2014-10-16 13:31 ` [PATCH v11 5/5] ARM: dts: enable Thermal on rk3288-evb board Caesar Wang 2014-10-16 13:31 ` Caesar Wang
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=20141016215316.GA30009@dtor-ws \ --to=dmitry.torokhov@gmail.com \ --cc=caesar.wang@rock-chips.com \ --cc=cf@rock-chips.com \ --cc=cjf@rock-chips.com \ --cc=dbasehore@chromium.org \ --cc=devicetree@vger.kernel.org \ --cc=dianders@chromium.org \ --cc=edubezval@gmail.com \ --cc=heiko@sntech.de \ --cc=huangtao@rock-chips.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-doc@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-pm@vger.kernel.org \ --cc=linux-rockchip@lists.infradead.org \ --cc=rui.zhang@intel.com \ --cc=vladimir_zapolskiy@mentor.com \ --cc=zhengsq@rock-chips.com \ --cc=zyf@rock-chips.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.