From mboxrd@z Thu Jan 1 00:00:00 1970 From: Krzysztof Kozlowski Subject: Re: [PATCH v2] ARM: dts: exynos: fix incomplete Odroid-XU3/4 thermal-zones definition Date: Fri, 8 Sep 2017 18:40:15 +0200 Message-ID: <20170908164015.rayfh2kluy45ktfi@kozik-lap> References: <1848394.K3C0KxojUN@amdc3058> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Return-path: Content-Disposition: inline In-Reply-To: <1848394.K3C0KxojUN@amdc3058> Sender: linux-kernel-owner@vger.kernel.org To: Bartlomiej Zolnierkiewicz Cc: Willy Wolff , Rob Herring , Mark Rutland , Russell King , Kukjin Kim , linux-samsung-soc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: devicetree@vger.kernel.org On Thu, Sep 07, 2017 at 06:10:00PM +0200, Bartlomiej Zolnierkiewicz wrote: > From: Willy Wolff > > Odroid XU3/4 boards have thermal sensors per 4 pairs of A7+A15 > cores but currently there is only one thermal-zone (including > cooling maps) defined (for the first pair of cores - the first > core of the A7 cluster and the first core of A15 cluster) so > i.e. if the task is running on any of A15 cores but the first > one, such core can reach high temperature without any proper > cooling action. > > Fix it by adding missing thermal-zones definitions. > > Also while at it fix the number of steps in cpufreq cooling for > cpu4 (11 steps for A15 corresponds to 700MHz, for 600MHz 12 steps > should be used). > > Signed-off-by: Willy Wolff > Cc: Rob Herring > Cc: Mark Rutland > Cc: Russell King > Cc: Kukjin Kim > [b.zolnierkie: rewrote patch subject & description + minor fixups] > Signed-off-by: Bartlomiej Zolnierkiewicz Thanks for taking care of this. I'll take it after merge window. Best regards, Krzysztof