Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1334556

Re: [PATCH] arm: dts: AM57XX: Correct the thermal thresholds

Show all headers | View raw

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..


-- 
Regards,
Nishanth Menon

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH] arm: dts: AM57XX: Correct the thermal thresholds Eduardo Valentin <edubezval@gmail.com> - 2016-02-09 23:10 +0100
  Re: [PATCH] arm: dts: AM57XX: Correct the thermal thresholds Nishanth Menon <nm@ti.com> - 2016-02-09 23:20 +0100
    Re: [PATCH] arm: dts: AM57XX: Correct the thermal thresholds Tony Lindgren <tony@atomide.com> - 2016-02-12 23:30 +0100
      Re: [PATCH] arm: dts: AM57XX: Correct the thermal thresholds Nishanth Menon <nm@ti.com> - 2016-02-12 23:40 +0100
        Re: [PATCH] arm: dts: AM57XX: Correct the thermal thresholds Tony Lindgren <tony@atomide.com> - 2016-02-12 23:50 +0100
          Re: [PATCH] arm: dts: AM57XX: Correct the thermal thresholds Keerthy <a0393675@ti.com> - 2016-02-15 05:40 +0100
            Re: [PATCH] arm: dts: AM57XX: Correct the thermal thresholds Nishanth Menon <nm@ti.com> - 2016-02-15 17:00 +0100
              Re: [PATCH] arm: dts: AM57XX: Correct the thermal thresholds Keerthy <a0393675@ti.com> - 2016-02-15 17:20 +0100
                Re: [PATCH] arm: dts: AM57XX: Correct the thermal thresholds Nishanth Menon <nm@ti.com> - 2016-02-15 18:30 +0100

csiph-web