From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 8927FCD1296 for ; Wed, 10 Apr 2024 09:50:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=td7PovVZ3nvZLRE8QBBLwUOTsJzT3zwrBCasAyOYeOc=; b=Dn3Wea2KchRfqmbqa3iWrF77jk EX8PhJmLQfxPIhD7XWUu0Jv4deleC0/ejpaBiBQq7E/exbZ6El9dVo0avpIokYlH3C6aAYCWd8JQr q9QghLHUJ7o3dXlfofCgyrVabcERvHg6ahQK/I84pYwAbmpa2755/6IBMGDld/xz29dUef5YApAq3 d0BWATqO8zPPsD74lNAhPjqOqKEsmgqNGcBTJNrrj8AzgW6RRhnC78+V/p5++bUNlRUq3GvOjX2yi G40UezX+m0Cnl2sywSP74hpRWHLDVZvGeCS0cc12++oiWAdysZXfDkB/c49HPkg8ZWO8v9ASFlRs/ DDZlH/jg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1ruUao-00000006E03-19vq; Wed, 10 Apr 2024 09:50:10 +0000 Received: from madrid.collaboradmins.com ([2a00:1098:ed:100::25]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1ruUak-00000006DxB-0Q13; Wed, 10 Apr 2024 09:50:07 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1712742603; bh=Ii71cBKKIW141cZWCc8RUZXkNgzA4tr4OR6L3mud6Dw=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=eih+DVQBahmd/5DRwuBsmzb797GZaSLdemKLXmgKi0pgdwvdsZ2TMBKjXicySQZvi KR3pbCwPybIVbvaL7gHcvncfa+PfXh7r08he4U08V1AnoAhzImpx7ks9fWSbg/HOU/ AhjZnWHDaIHajFyd4SH6zr8k/V0D5ZxuWawhPw5IG9J/QdEl3xrm5mfo2FwwoMGXtd SfU1Yo+w+D6aGSnJ6CrJVHeLQI6OGdK6jL439lJTZ00zZLete9CwdsIPYfmvok5LUo h6pqI46uWULKb50a/j+lWcXb9jrv/PSWn9vpB35e7AY/GfPksBNuteJroTsLKSsZoF Br1kKBLAtlOTg== Received: from [100.113.186.2] (cola.collaboradmins.com [195.201.22.229]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: kholk11) by madrid.collaboradmins.com (Postfix) with ESMTPSA id 4C95B378000E; Wed, 10 Apr 2024 09:50:01 +0000 (UTC) Message-ID: Date: Wed, 10 Apr 2024 11:49:59 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2] arm64: dts: mediatek: mt8192: Add missing trip point in thermal zone To: Hsin-Te Yuan , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Matthias Brugger , =?UTF-8?B?TsOtY29sYXMgRi4gUi4gQS4gUHJhZG8=?= , =?UTF-8?Q?Bernhard_Rosenkr=C3=A4nzer?= , Balsam CHIHI , Alexandre Mergnat Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org References: <20240410-upstream-torvalds-master-v2-1-679777847b63@chromium.org> From: AngeloGioacchino Del Regno Content-Language: en-US In-Reply-To: <20240410-upstream-torvalds-master-v2-1-679777847b63@chromium.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240410_025006_360773_AFF89B2D X-CRM114-Status: GOOD ( 20.04 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org Il 10/04/24 10:40, Hsin-Te Yuan ha scritto: > According to Documentation/driver-api/thermal/power_allocator.rst, there > should be two passive trip points. Adding the missing trip point to > ensure that the governor works optimally. > > Fixes: c7a728051f4e ("arm64: dts: mediatek: mt8192: Add thermal nodes and thermal zones") > Signed-off-by: Hsin-Te Yuan That's better - but while I can agree about setting a general temperature for that, I still have a few questions and requests: 1. Why is this 68°C? Was that value provided by MediaTek, or was it calculated? 1b. If this was calculated, please, can you explain how? 2. The power allocator documentation also says that the governor works good when a "sustainable dissipatable power" parameter is fed to it through the `sustainable-power` devicetree property (with that being strictly board specific and never SoC-global, as that depends on the form factor and on the cooling method of the machine), can you please also add the right sustainable power indication to the Chromebook devicetrees? In the MT8192 specific case, that's mt8192-asurada.dtsi. 3. I just noticed that MT8192 is not the only one that would be affected by the issue that you're describing in this commit; can you please perform a similar change on the others, if parameters are known? Thanks, Angelo > --- > Changes in v2: > - Clearify the reason of adding another passive trip point > - Link to v1: https://lore.kernel.org/r/20240410-upstream-torvalds-master-v1-1-852e903f0cec@chromium.org > --- > arch/arm64/boot/dts/mediatek/mt8192.dtsi | 40 ++++++++++++++++++++++++++++++++ > 1 file changed, 40 insertions(+) > > diff --git a/arch/arm64/boot/dts/mediatek/mt8192.dtsi b/arch/arm64/boot/dts/mediatek/mt8192.dtsi > index 05e401670bced..08d8bccc84669 100644 > --- a/arch/arm64/boot/dts/mediatek/mt8192.dtsi > +++ b/arch/arm64/boot/dts/mediatek/mt8192.dtsi > @@ -1959,6 +1959,11 @@ cpu0-thermal { > thermal-sensors = <&lvts_mcu MT8192_MCU_LITTLE_CPU0>; > > trips { > + cpu0_thres: trip-point { > + temperature = <68000>; > + hysteresis = <2000>; > + type = "passive"; > + }; > cpu0_alert: trip-alert { > temperature = <85000>; > hysteresis = <2000>; > @@ -1989,6 +1994,11 @@ cpu1-thermal { > thermal-sensors = <&lvts_mcu MT8192_MCU_LITTLE_CPU1>; > > trips { > + cpu1_thres: trip-point { > + temperature = <68000>; > + hysteresis = <2000>; > + type = "passive"; > + }; > cpu1_alert: trip-alert { > temperature = <85000>; > hysteresis = <2000>; > @@ -2019,6 +2029,11 @@ cpu2-thermal { > thermal-sensors = <&lvts_mcu MT8192_MCU_LITTLE_CPU2>; > > trips { > + cpu2_thres: trip-point { > + temperature = <68000>; > + hysteresis = <2000>; > + type = "passive"; > + }; > cpu2_alert: trip-alert { > temperature = <85000>; > hysteresis = <2000>; > @@ -2049,6 +2064,11 @@ cpu3-thermal { > thermal-sensors = <&lvts_mcu MT8192_MCU_LITTLE_CPU3>; > > trips { > + cpu3_thres: trip-point { > + temperature = <68000>; > + hysteresis = <2000>; > + type = "passive"; > + }; > cpu3_alert: trip-alert { > temperature = <85000>; > hysteresis = <2000>; > @@ -2079,6 +2099,11 @@ cpu4-thermal { > thermal-sensors = <&lvts_mcu MT8192_MCU_BIG_CPU0>; > > trips { > + cpu4_thres: trip-point { > + temperature = <68000>; > + hysteresis = <2000>; > + type = "passive"; > + }; > cpu4_alert: trip-alert { > temperature = <85000>; > hysteresis = <2000>; > @@ -2109,6 +2134,11 @@ cpu5-thermal { > thermal-sensors = <&lvts_mcu MT8192_MCU_BIG_CPU1>; > > trips { > + cpu5_thres: trip-point { > + temperature = <68000>; > + hysteresis = <2000>; > + type = "passive"; > + }; > cpu5_alert: trip-alert { > temperature = <85000>; > hysteresis = <2000>; > @@ -2139,6 +2169,11 @@ cpu6-thermal { > thermal-sensors = <&lvts_mcu MT8192_MCU_BIG_CPU2>; > > trips { > + cpu6_thres: trip-point { > + temperature = <68000>; > + hysteresis = <2000>; > + type = "passive"; > + }; > cpu6_alert: trip-alert { > temperature = <85000>; > hysteresis = <2000>; > @@ -2169,6 +2204,11 @@ cpu7-thermal { > thermal-sensors = <&lvts_mcu MT8192_MCU_BIG_CPU3>; > > trips { > + cpu7_thres: trip-point { > + temperature = <68000>; > + hysteresis = <2000>; > + type = "passive"; > + }; > cpu7_alert: trip-alert { > temperature = <85000>; > hysteresis = <2000>; > > --- > base-commit: 20cb38a7af88dc40095da7c2c9094da3873fea23 > change-id: 20240410-upstream-torvalds-master-40aeff5416c7 > > Best regards,