From: Eduardo Valentin <eduardo.valentin@ti.com> To: Amit Daniel Kachhap <amit.daniel@samsung.com> Cc: <linux-pm@vger.kernel.org>, Thomas Abraham <thomas.abraham@linaro.org>, Zhang Rui <rui.zhang@intel.com>, <linux-samsung-soc@vger.kernel.org>, <linux-kernel@vger.kernel.org>, <amit.kachhap@gmail.com>, Kukjin Kim <kgene.kim@samsung.com>, <grant.likely@secretlab.ca>, <eduardo.valentin@ti.com> Subject: Re: [9/9] ARM: dts: Add device tree node for exynos5440 TMU controller Date: Thu, 11 Apr 2013 17:15:50 -0400 [thread overview] Message-ID: <51672806.5040501@ti.com> (raw) In-Reply-To: <1364297642-2746-10-git-send-email-amit.daniel@samsung.com> Amit, Copying Grant for DT. On 26-03-2013 07:34, Amit Daniel Kachhap wrote: > This patch adds device node for TMU controller. There are 3 > instances of the controllers so 3 nodes are created. > > Signed-off-by: Amit Daniel Kachhap <amit.daniel@samsung.com> > > --- > arch/arm/boot/dts/exynos5440.dtsi | 43 +++++++++++++++++++++++++++++++++++++ > 1 files changed, 43 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/boot/dts/exynos5440.dtsi b/arch/arm/boot/dts/exynos5440.dtsi > index 5f3562a..b74ce9f 100644 > --- a/arch/arm/boot/dts/exynos5440.dtsi > +++ b/arch/arm/boot/dts/exynos5440.dtsi > @@ -16,6 +16,12 @@ > > interrupt-parent = <&gic>; > > + aliases { > + tmuctrl0 = &tmuctrl_0; > + tmuctrl1 = &tmuctrl_1; > + tmuctrl2 = &tmuctrl_2; > + }; > + > gic:interrupt-controller@2E0000 { > compatible = "arm,cortex-a15-gic"; > #interrupt-cells = <3>; > @@ -156,4 +162,41 @@ > reg = <0x130000 0x1000>; > interrupts = <0 17 0>, <0 16 0>; > }; > + > + tmu_ctrl_info: tmu-ctrl-info { > + gain = <5>; > + reference-voltage = <16>; > + noise-cancel-mode = <4>; > + cal-type = <2>; > + cal-mode = <0>; > + efuse-value = <0x5b2c>; > + threshold-falling = <5>; > + }; > + > + tmuctrl_0: tmuctrl@160118 { > + compatible = "samsung,exynos5440-tmu"; > + reg = <0x160118 0x300>; > + interrupts = <0 58 0>; > + clocks = <&clock 8>; > + clock-names = "tmu_apbif"; > + tmu-ctrl-data = <&tmu_ctrl_info>; > + }; > + > + tmuctrl_1: tmuctrl@16011C { > + compatible = "samsung,exynos5440-tmu"; > + reg = <0x16011C 0x300>; > + interrupts = <0 58 0>; > + clocks = <&clock 8>; > + clock-names = "tmu_apbif"; > + tmu-ctrl-data = <&tmu_ctrl_info>; > + }; > + > + tmuctrl_2: tmuctrl@160120 { > + compatible = "samsung,exynos5440-tmu"; > + reg = <0x160120 0x300>; > + interrupts = <0 58 0>; > + clocks = <&clock 8>; > + clock-names = "tmu_apbif"; > + tmu-ctrl-data = <&tmu_ctrl_info>; > + }; Grant, how the above thermal data must be handled in DT? Is it fine to create private phandles like that? > }; > Please consider the comments on patch 8.
WARNING: multiple messages have this Message-ID (diff)
From: Eduardo Valentin <eduardo.valentin@ti.com> To: Amit Daniel Kachhap <amit.daniel@samsung.com> Cc: linux-pm@vger.kernel.org, Thomas Abraham <thomas.abraham@linaro.org>, Zhang Rui <rui.zhang@intel.com>, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, amit.kachhap@gmail.com, Kukjin Kim <kgene.kim@samsung.com>, grant.likely@secretlab.ca, eduardo.valentin@ti.com Subject: Re: [9/9] ARM: dts: Add device tree node for exynos5440 TMU controller Date: Thu, 11 Apr 2013 17:15:50 -0400 [thread overview] Message-ID: <51672806.5040501@ti.com> (raw) In-Reply-To: <1364297642-2746-10-git-send-email-amit.daniel@samsung.com> Amit, Copying Grant for DT. On 26-03-2013 07:34, Amit Daniel Kachhap wrote: > This patch adds device node for TMU controller. There are 3 > instances of the controllers so 3 nodes are created. > > Signed-off-by: Amit Daniel Kachhap <amit.daniel@samsung.com> > > --- > arch/arm/boot/dts/exynos5440.dtsi | 43 +++++++++++++++++++++++++++++++++++++ > 1 files changed, 43 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/boot/dts/exynos5440.dtsi b/arch/arm/boot/dts/exynos5440.dtsi > index 5f3562a..b74ce9f 100644 > --- a/arch/arm/boot/dts/exynos5440.dtsi > +++ b/arch/arm/boot/dts/exynos5440.dtsi > @@ -16,6 +16,12 @@ > > interrupt-parent = <&gic>; > > + aliases { > + tmuctrl0 = &tmuctrl_0; > + tmuctrl1 = &tmuctrl_1; > + tmuctrl2 = &tmuctrl_2; > + }; > + > gic:interrupt-controller@2E0000 { > compatible = "arm,cortex-a15-gic"; > #interrupt-cells = <3>; > @@ -156,4 +162,41 @@ > reg = <0x130000 0x1000>; > interrupts = <0 17 0>, <0 16 0>; > }; > + > + tmu_ctrl_info: tmu-ctrl-info { > + gain = <5>; > + reference-voltage = <16>; > + noise-cancel-mode = <4>; > + cal-type = <2>; > + cal-mode = <0>; > + efuse-value = <0x5b2c>; > + threshold-falling = <5>; > + }; > + > + tmuctrl_0: tmuctrl@160118 { > + compatible = "samsung,exynos5440-tmu"; > + reg = <0x160118 0x300>; > + interrupts = <0 58 0>; > + clocks = <&clock 8>; > + clock-names = "tmu_apbif"; > + tmu-ctrl-data = <&tmu_ctrl_info>; > + }; > + > + tmuctrl_1: tmuctrl@16011C { > + compatible = "samsung,exynos5440-tmu"; > + reg = <0x16011C 0x300>; > + interrupts = <0 58 0>; > + clocks = <&clock 8>; > + clock-names = "tmu_apbif"; > + tmu-ctrl-data = <&tmu_ctrl_info>; > + }; > + > + tmuctrl_2: tmuctrl@160120 { > + compatible = "samsung,exynos5440-tmu"; > + reg = <0x160120 0x300>; > + interrupts = <0 58 0>; > + clocks = <&clock 8>; > + clock-names = "tmu_apbif"; > + tmu-ctrl-data = <&tmu_ctrl_info>; > + }; Grant, how the above thermal data must be handled in DT? Is it fine to create private phandles like that? > }; > Please consider the comments on patch 8.
next prev parent reply other threads:[~2013-04-11 21:16 UTC|newest] Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top 2013-03-26 11:33 [PATCH 0/9] thermal: exynos: Add thermal driver for exynos5440 Amit Daniel Kachhap 2013-03-26 11:33 ` [PATCH 1/9] thermal: exynos: Adapt to temperature emulation core thermal framework Amit Daniel Kachhap 2013-04-11 19:33 ` [1/9] " Eduardo Valentin 2013-04-11 19:33 ` Eduardo Valentin 2013-04-12 10:57 ` amit kachhap 2013-03-26 11:33 ` [PATCH 2/9] thermal: exynos: Add support for instance based register/unregister Amit Daniel Kachhap 2013-04-11 20:09 ` [2/9] " Eduardo Valentin 2013-04-11 20:09 ` Eduardo Valentin 2013-04-12 11:03 ` amit kachhap 2013-03-26 11:33 ` [PATCH 3/9] thermal: exynos: Moving into samsung directory for easy maintenance Amit Daniel Kachhap 2013-04-11 20:25 ` [3/9] " Eduardo Valentin 2013-04-11 20:25 ` Eduardo Valentin 2013-03-26 11:33 ` [PATCH 4/9] thermal: exynos: Bifurcate exynos thermal common and tmu controller code Amit Daniel Kachhap 2013-04-11 20:30 ` [4/9] " Eduardo Valentin 2013-04-11 20:30 ` Eduardo Valentin 2013-04-12 11:06 ` amit daniel kachhap 2013-04-11 20:42 ` Eduardo Valentin 2013-04-11 20:42 ` Eduardo Valentin 2013-04-12 11:09 ` amit daniel kachhap 2013-04-12 12:42 ` Eduardo Valentin 2013-04-12 12:42 ` Eduardo Valentin 2013-03-26 11:33 ` [PATCH 5/9] thermal: exynos: Make the zone handling dependent on trip count Amit Daniel Kachhap 2013-04-11 20:48 ` [5/9] " Eduardo Valentin 2013-04-11 20:48 ` Eduardo Valentin 2013-04-12 11:16 ` amit daniel kachhap 2013-04-12 12:45 ` Eduardo Valentin 2013-04-12 12:45 ` Eduardo Valentin 2013-03-26 11:33 ` [PATCH 6/9] thermal: exynos: small cleanups to prepare for adding exynos5440 driver Amit Daniel Kachhap 2013-04-11 20:54 ` [6/9] " Eduardo Valentin 2013-04-11 20:54 ` Eduardo Valentin 2013-04-12 11:18 ` amit daniel kachhap 2013-03-26 11:34 ` [PATCH 7/9] thermal: exynos: Add support for exynos5440 TMU sensor Amit Daniel Kachhap 2013-04-11 21:04 ` [7/9] " Eduardo Valentin 2013-04-11 21:04 ` Eduardo Valentin 2013-04-12 11:32 ` amit daniel kachhap 2013-03-26 11:34 ` [PATCH 8/9] thermal: exynos: Parse the platform data from the device tree Amit Daniel Kachhap 2013-04-11 21:13 ` [8/9] " Eduardo Valentin 2013-04-11 21:13 ` Eduardo Valentin 2013-03-26 11:34 ` [PATCH 9/9] ARM: dts: Add device tree node for exynos5440 TMU controller Amit Daniel Kachhap 2013-04-08 12:24 ` Kukjin Kim 2013-04-11 21:15 ` Eduardo Valentin [this message] 2013-04-11 21:15 ` [9/9] " Eduardo Valentin 2013-04-02 10:26 ` [PATCH 0/9] thermal: exynos: Add thermal driver for exynos5440 Kukjin Kim 2013-04-09 5:24 ` amit daniel kachhap
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=51672806.5040501@ti.com \ --to=eduardo.valentin@ti.com \ --cc=amit.daniel@samsung.com \ --cc=amit.kachhap@gmail.com \ --cc=grant.likely@secretlab.ca \ --cc=kgene.kim@samsung.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-pm@vger.kernel.org \ --cc=linux-samsung-soc@vger.kernel.org \ --cc=rui.zhang@intel.com \ --cc=thomas.abraham@linaro.org \ /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.