linux-kernel - Re: [PATCH 0/7] DRA7: DT thermal support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <524B7AC9.3070906@ti.com>
Date:	Tue, 1 Oct 2013 21:45:45 -0400
From:	Eduardo Valentin <eduardo.valentin@...com>
To:	Nishanth Menon <nm@...com>
CC:	Eduardo Valentin <eduardo.valentin@...com>,
	<bcousson@...libre.com>, <tony@...mide.com>,
	<linux-omap@...r.kernel.org>, <swarren@...dotorg.org>,
	<pawel.moll@....com>, <mark.rutland@....com>,
	<ian.campbell@...rix.com>, <rob.herring@...xeda.com>,
	<linux@...ck-us.net>, <rui.zhang@...el.com>,
	<grant.likely@...aro.org>, <linux-pm@...r.kernel.org>,
	<devicetree@...r.kernel.org>, <lm-sensors@...sensors.org>,
	<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 0/7] DRA7: DT thermal support

On 01-10-2013 18:58, Nishanth Menon wrote:
> On 14:32-20131001, Eduardo Valentin wrote:
>> Hello all,
>>
>> This is a complementary patch series with themal DT support for DRA7.
>>
>> Although this work depends on the thermal dt parser work [1], I decided
>> to share it before hand. It also depends on DRA7 DT base port support,
>> which I fetched from Rajendra's tree [2].
>>
>> This patch series has been runtime tested on DRA7-evm.
>>
>> All best,
>>
>> [1] - https://lkml.org/lkml/2013/9/26/787
>> [2] - git://github.com/rrnayak/linux.git out-of-tree/dra-integrated-v3
>>
>> Eduardo Valentin (7):
>>   arm: dts: add dra7 DSPEVE thermal data
>>   arm: dts: add dra7 IVA thermal data
>>   arm: dts: dra7: add bandgap entry
>>   arm: dts: add cooling properties on dra7 cpu node
>>   arm: dts: dra7: add thermal data
>>   arm: dts: add tmp102 i2c sensor node on dra7-evm
>>   arm: dts: add thermal zones info on tmp102 for DRA7-EVM
>>
>>  arch/arm/boot/dts/dra7-dspeve-thermal.dtsi | 28 ++++++++++++++++++++++++++++
>>  arch/arm/boot/dts/dra7-evm.dts             | 23 +++++++++++++++++++++++
>>  arch/arm/boot/dts/dra7-iva-thermal.dtsi    | 28 ++++++++++++++++++++++++++++
>>  arch/arm/boot/dts/dra7.dtsi                | 27 ++++++++++++++++++++++++++-
>>  4 files changed, 105 insertions(+), 1 deletion(-)
>>  create mode 100644 arch/arm/boot/dts/dra7-dspeve-thermal.dtsi
>>  create mode 100644 arch/arm/boot/dts/dra7-iva-thermal.dtsi
> 
> +Benoit, Tony, Linux-omap.
> Ref: http://marc.info/?l=linux-pm&m=138065243027014&w=2
> 
> Tony, Benoit,
> The following diff[1] is probably needed to ensure that right audience
> is addressed.

Well, yeah. thinking of this series now I probably should have mentioned
clear that it is not ready for merge. Not only because it depends on a
working in progress I have referred at [1], but because we lack
completely dra7 files (at linus tree we have no files under
arch/arm/boot/dts/*dra7*).  Therefore, I mentioned in this cover letter
that I actually based this work on top of [2] and [1]. Apologies if it
was not clear enough for you.

Just to make clear, the intention of this patch set was to share and use
yet another example of the usage of the work at [1]. Thus the audience
was directed to the cc and to lists I used.

> 
> Eduardo,
> could you send an updated series with Tony,
> Benoit and linux-omap in the loop?

Well sure, once it has the right dependency in place.

> 
> [1]
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 284969f..3442671 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -5969,6 +5969,7 @@ M:	Tony Lindgren <tony@...mide.com>
>  L:	linux-omap@...r.kernel.org
>  L:	devicetree@...r.kernel.org
>  S:	Maintained
> +F:	arch/arm/boot/dts/*dra7*

I believe, once we have them merged, then I suppose this diff is applicable.

>  F:	arch/arm/boot/dts/*omap*
>  F:	arch/arm/boot/dts/*am3*
>  
> 


-- 
You have got to be excited about what you are doing. (L. Lamport)

Eduardo Valentin


Download attachment "signature.asc" of type "application/pgp-signature" (296 bytes)