From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nishanth Menon Subject: Re: [PATCH] arm: dts: AM57XX: Correct the thermal thresholds Date: Mon, 15 Feb 2016 11:25:38 -0600 Message-ID: <56C20A12.8080305@ti.com> References: <1454924705-8382-1-git-send-email-j-keerthy@ti.com> <20160209220901.GC28102@localhost.localdomain> <56BA6530.4030102@ti.com> <20160212222347.GO19432@atomide.com> <20160212224129.GP19432@atomide.com> <56C153A3.3030508@ti.com> <56C1F3A1.4080709@ti.com> <56C1F92A.1020803@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <56C1F92A.1020803@ti.com> Sender: linux-kernel-owner@vger.kernel.org To: Keerthy , Tony Lindgren Cc: Eduardo Valentin , Keerthy , Dmitry Lifshitz , "linux-arm-kernel@lists.infradead.org" , lkml , dt list , linux-omap , Russell King - ARM Linux , Kumar Gala , "ijc+devicetree@hellion.org.uk" , Mark Rutland List-Id: linux-omap@vger.kernel.org On 02/15/2016 10:13 AM, Keerthy wrote: [...] >> >> 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? Why would we want to throttle earlier on customer platforms or other boards using extended? at the very least -> if all upstream boards have only commercial grade, then just introduce am57xx-commercial-thermal.dtsi .. -- Regards, Nishanth Menon From mboxrd@z Thu Jan 1 00:00:00 1970 From: nm@ti.com (Nishanth Menon) Date: Mon, 15 Feb 2016 11:25:38 -0600 Subject: [PATCH] arm: dts: AM57XX: Correct the thermal thresholds In-Reply-To: <56C1F92A.1020803@ti.com> References: <1454924705-8382-1-git-send-email-j-keerthy@ti.com> <20160209220901.GC28102@localhost.localdomain> <56BA6530.4030102@ti.com> <20160212222347.GO19432@atomide.com> <20160212224129.GP19432@atomide.com> <56C153A3.3030508@ti.com> <56C1F3A1.4080709@ti.com> <56C1F92A.1020803@ti.com> Message-ID: <56C20A12.8080305@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 02/15/2016 10:13 AM, Keerthy wrote: [...] >> >> 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? Why would we want to throttle earlier on customer platforms or other boards using extended? at the very least -> if all upstream boards have only commercial grade, then just introduce am57xx-commercial-thermal.dtsi .. -- Regards, Nishanth Menon