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 21:43:30 +0530 [thread overview]
Message-ID: <56C1F92A.1020803@ti.com> (raw)
In-Reply-To: <56C1F3A1.4080709@ti.com>
On Monday 15 February 2016 09:19 PM, Nishanth Menon wrote:
> On 02/14/2016 10:27 PM, Keerthy wrote:
>>
>>
>> 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.
>
>
>
> This data is in the AM572x datasheet:
> http://www.ti.com/lit/ds/symlink/am5728.pdf
>
> Table 5-3 in the current version contains the maximum junction
> temperatures.
>
> this indicates that there are "Commercial" and "extended" -> we may
> need to introduce just 2 dtsis -> am57xx-commercial and
> am57xx-extended and reuse those based on the part on the board. just a
> suggestion..
Or use the minimum of the 2 to be on the safer side if we have to deal
with one dtsi?
Regards,
Keerthy
>
>
next prev parent reply other threads:[~2016-02-15 16:13 UTC|newest]
Thread overview: 10+ 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-09 22:09 ` Eduardo Valentin
2016-02-09 22:16 ` Nishanth Menon
2016-02-12 22:23 ` Tony Lindgren
2016-02-12 22:30 ` Nishanth Menon
2016-02-12 22:41 ` Tony Lindgren
2016-02-15 4:27 ` Keerthy
2016-02-15 15:49 ` Nishanth Menon
2016-02-15 16:13 ` Keerthy [this message]
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=56C1F92A.1020803@ti.com \
--to=a0393675@ti$(echo .)com \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox