From: Keerthy <a0393675@ti.com>
To: Tony Lindgren <tony@atomide.com>, Nishanth Menon <nm@ti.com>
Cc: Eduardo Valentin <edubezval@gmail.com>,
Keerthy <j-keerthy@ti.com>,
Dmitry Lifshitz <lifshitz@compulab.co.il>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
lkml <linux-kernel@vger.kernel.org>,
dt list <devicetree@vger.kernel.org>,
linux-omap <linux-omap@vger.kernel.org>,
Russell King - ARM Linux <linux@arm.linux.org.uk>,
Kumar Gala <galak@codeaurora.org>,
"ijc+devicetree@hellion.org.uk" <ijc+devicetree@hellion.org.uk>,
Mark Rutland <mark.rutland@arm.com>
Subject: Re: [PATCH] arm: dts: AM57XX: Correct the thermal thresholds
Date: Mon, 15 Feb 2016 09:57:15 +0530 [thread overview]
Message-ID: <56C153A3.3030508@ti.com> (raw)
In-Reply-To: <20160212224129.GP19432@atomide.com>
On Saturday 13 February 2016 04:11 AM, Tony Lindgren wrote:
> * Nishanth Menon <nm@ti.com> [160212 14:32]:
>> On Fri, Feb 12, 2016 at 4:23 PM, Tony Lindgren <tony@atomide.com> wrote:
>>> * Nishanth Menon <nm@ti.com> [160209 14:17]:
>>>> On 02/09/2016 04:09 PM, Eduardo Valentin wrote:
>>>>> On Mon, Feb 08, 2016 at 03:15:05PM +0530, Keerthy wrote:
>>>>>> The thermal threholds for am57xx are different from that of dra7.
>>>>>> Correcting the thresholds for the same.
>>>>>>
>>>>>> Signed-off-by: Keerthy <j-keerthy@ti.com>
>>>>>> ---
>>>>>>
>>>>>> Depends on: https://lkml.org/lkml/2016/2/8/90
>>>>>>
>>>>>> arch/arm/boot/dts/am57xx-beagle-x15.dts | 24 ++++++++++++++++++++++++
>>>>>> 1 file changed, 24 insertions(+)
>>>>>>
>>>>>> Index: linux/arch/arm/boot/dts/am57xx-beagle-x15.dts
>>>>>> ===================================================================
>>>>>> --- linux.orig/arch/arm/boot/dts/am57xx-beagle-x15.dts 2016-02-08 14:57:05.514959101 +0530
>>>>>> +++ linux/arch/arm/boot/dts/am57xx-beagle-x15.dts 2016-02-08 14:57:05.510959168 +0530
>>>>>
>>>>> Sorry, did not bother to check, but is this the only user of am57xx?
>>>>
>>>> we will have AM57xx-IDK as well coming down the line. we already have
>>>> arch/arm/boot/dts/am57xx-cl-som-am57x.dts
>>>> arch/arm/boot/dts/am57xx-sbc-am57x.dts - not really sure what grade of
>>>> silicon are used in cl-SOM and sbc boards.
>>>
>>> Hmm can this one wait for v4.6 or is it critical for v4.5-rc
>>> cycle?
>>
>> personally, I'd like to NAK this patch -> lack of reference
>> documentation, and guidance for various AM57xx boards in a generic
>> patch should probably be followed IMHO.
>
> OK let's wait for that then.
OK. I will send a v2 with better documentation.
>
> Regards,
>
> Tony
>
WARNING: multiple messages have this Message-ID (diff)
From: a0393675@ti.com (Keerthy)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] arm: dts: AM57XX: Correct the thermal thresholds
Date: Mon, 15 Feb 2016 09:57:15 +0530 [thread overview]
Message-ID: <56C153A3.3030508@ti.com> (raw)
In-Reply-To: <20160212224129.GP19432@atomide.com>
On Saturday 13 February 2016 04:11 AM, Tony Lindgren wrote:
> * Nishanth Menon <nm@ti.com> [160212 14:32]:
>> On Fri, Feb 12, 2016 at 4:23 PM, Tony Lindgren <tony@atomide.com> wrote:
>>> * Nishanth Menon <nm@ti.com> [160209 14:17]:
>>>> On 02/09/2016 04:09 PM, Eduardo Valentin wrote:
>>>>> On Mon, Feb 08, 2016 at 03:15:05PM +0530, Keerthy wrote:
>>>>>> The thermal threholds for am57xx are different from that of dra7.
>>>>>> Correcting the thresholds for the same.
>>>>>>
>>>>>> Signed-off-by: Keerthy <j-keerthy@ti.com>
>>>>>> ---
>>>>>>
>>>>>> Depends on: https://lkml.org/lkml/2016/2/8/90
>>>>>>
>>>>>> arch/arm/boot/dts/am57xx-beagle-x15.dts | 24 ++++++++++++++++++++++++
>>>>>> 1 file changed, 24 insertions(+)
>>>>>>
>>>>>> Index: linux/arch/arm/boot/dts/am57xx-beagle-x15.dts
>>>>>> ===================================================================
>>>>>> --- linux.orig/arch/arm/boot/dts/am57xx-beagle-x15.dts 2016-02-08 14:57:05.514959101 +0530
>>>>>> +++ linux/arch/arm/boot/dts/am57xx-beagle-x15.dts 2016-02-08 14:57:05.510959168 +0530
>>>>>
>>>>> Sorry, did not bother to check, but is this the only user of am57xx?
>>>>
>>>> we will have AM57xx-IDK as well coming down the line. we already have
>>>> arch/arm/boot/dts/am57xx-cl-som-am57x.dts
>>>> arch/arm/boot/dts/am57xx-sbc-am57x.dts - not really sure what grade of
>>>> silicon are used in cl-SOM and sbc boards.
>>>
>>> Hmm can this one wait for v4.6 or is it critical for v4.5-rc
>>> cycle?
>>
>> personally, I'd like to NAK this patch -> lack of reference
>> documentation, and guidance for various AM57xx boards in a generic
>> patch should probably be followed IMHO.
>
> OK let's wait for that then.
OK. I will send a v2 with better documentation.
>
> Regards,
>
> Tony
>
next prev parent reply other threads:[~2016-02-15 4:27 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-08 9:45 [PATCH] arm: dts: AM57XX: Correct the thermal thresholds Keerthy
2016-02-08 9:45 ` Keerthy
2016-02-08 9:45 ` Keerthy
2016-02-09 22:09 ` Eduardo Valentin
2016-02-09 22:09 ` Eduardo Valentin
2016-02-09 22:16 ` Nishanth Menon
2016-02-09 22:16 ` Nishanth Menon
2016-02-09 22:16 ` Nishanth Menon
[not found] ` <56BA6530.4030102-l0cyMroinI0@public.gmane.org>
2016-02-12 22:23 ` Tony Lindgren
2016-02-12 22:23 ` Tony Lindgren
2016-02-12 22:23 ` Tony Lindgren
2016-02-12 22:30 ` Nishanth Menon
2016-02-12 22:30 ` Nishanth Menon
[not found] ` <CAGo_u6r8-LJBWWrew0fV50Q+JnCxDj9PD57+9NoDm5TqCWL=jw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-02-12 22:41 ` Tony Lindgren
2016-02-12 22:41 ` Tony Lindgren
2016-02-12 22:41 ` Tony Lindgren
2016-02-15 4:27 ` Keerthy [this message]
2016-02-15 4:27 ` Keerthy
2016-02-15 15:49 ` Nishanth Menon
2016-02-15 15:49 ` Nishanth Menon
2016-02-15 16:13 ` Keerthy
2016-02-15 16:13 ` Keerthy
2016-02-15 17:25 ` Nishanth Menon
2016-02-15 17:25 ` Nishanth Menon
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=56C153A3.3030508@ti.com \
--to=a0393675@ti.com \
--cc=devicetree@vger.kernel.org \
--cc=edubezval@gmail.com \
--cc=galak@codeaurora.org \
--cc=ijc+devicetree@hellion.org.uk \
--cc=j-keerthy@ti.com \
--cc=lifshitz@compulab.co.il \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=mark.rutland@arm.com \
--cc=nm@ti.com \
--cc=tony@atomide.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: link
Be 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.