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 55058C4829A for ; Sun, 11 Feb 2024 12:54:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=JtpLYSZV6VYv3jXIqKFuJb6YJjTC++MYQtxfUMKXV3s=; b=122rhFoWCef+DQ Lpq0srPhorsK5xxFnbjZq87J+sp+OxTccC24bNWIHxW1oiWxppWMffAxEegYbCOdb475aoGrP/73U iAhna4vpxspPPkHVxEVUvBenMATo4+SZiNepJrjZ50zFN/xyLIekQljtxGQf8mjQKxWgfB1cYNdLJ gR1eufzUwiKdpS5fa1BFmi/ANitHNpFORIU+cWgFjDm8CEz0i9n2sp0vX1C+z1pjTGS+vSfv4zuiN v84SjadDhVUnB0FLP8FEOs7hFpC1HUKkHfKQBMpGhl9H1dzCfXlFhAZFNchT4sRGgBusAhANweVdG VozUARx0Z/TEU18cPeEQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rZ9LN-00000002lO3-3kfp; Sun, 11 Feb 2024 12:54:01 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rZ9LK-00000002lNc-2BQL for linux-arm-kernel@lists.infradead.org; Sun, 11 Feb 2024 12:54:00 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 028FCDA7; Sun, 11 Feb 2024 04:54:32 -0800 (PST) Received: from minigeek.lan (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 1CE4C3F5A1; Sun, 11 Feb 2024 04:53:47 -0800 (PST) Date: Sun, 11 Feb 2024 12:52:26 +0000 From: Andre Przywara To: Andrey Skvortsov Cc: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org, Arnaud Ferraris , Alexey Klimov , Daniel Lezcano Subject: Re: [PATCH] arm64: dts: allwinner: a64: Add thermal trip points for GPU Message-ID: <20240211125209.15d58462@minigeek.lan> In-Reply-To: <20240211105326.506049-1-andrej.skvortzov@gmail.com> References: <20240211105326.506049-1-andrej.skvortzov@gmail.com> Organization: Arm Ltd. X-Mailer: Claws Mail 4.2.0 (GTK 3.24.31; x86_64-slackware-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240211_045358_700116_3CE8E743 X-CRM114-Status: GOOD ( 26.25 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Sun, 11 Feb 2024 13:53:26 +0300 Andrey Skvortsov wrote: Hi Andrey, > From: Alexey Klimov > > Without trip points for GPU, the following errors are printed in the > dmesg log and the sun8i-thermal driver fails to load: So how does that post differ from Alexey one's a few weeks back: https://lore.kernel.org/linux-arm-kernel/20240101000008.65747-1-alexey.klimov@linaro.org/ It seems like the same patch? And Jernej and I had some comments (no mentioning of "Linux" in commit message, add cooling maps, source of trip temperature values), can you please try to address them? > thermal_sys: Failed to find 'trips' node > thermal_sys: Failed to find trip points for thermal-sensor id=1 > sun8i-thermal: probe of 1c25000.thermal-sensor failed with error -22 I think it's pretty obvious that the trip points are missing when they shouldn't, so this does not need too much explanation or rationale in the commit message, so you can cut this short. > When thermal zones are defined, trip points definitions are mandatory. > Trip values for the GPU are assumed to be the same values as the CPU > ones. The available specs do not provide any hints about thermal regimes > for the GPU and it seems GPU is implemented on the same die as the CPU. > > 'make dtbs_check' complains about problem in dts for 18 A64-based boards > supported by the kernel: > > sun50i-a64-pine64.dtb: thermal-zones: gpu0-thermal: 'trips' is a required property > from schema $id: http://devicetree.org/schemas/thermal/thermal-zones.yaml# > sun50i-a64-pine64.dtb: thermal-zones: gpu1-thermal: 'trips' is a required property > from schema $id: http://devicetree.org/schemas/thermal/thermal-zones.yaml# > > Tested on Pine a64+ and PinePhone 1.2. > > Cc: Samuel Holland > Cc: Jernej Skrabec > Cc: Chen-Yu Tsai > Cc: Daniel Lezcano > Cc: devicetree@vger.kernel.org > Signed-off-by: Alexey Klimov > Tested-by: Andrey Skvortsov You would need your Signed-off-by: here, since you send this, even when on Alexey's behalf. Cheers, Andre > > --- > arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi | 46 +++++++++++++++++++ > 1 file changed, 46 insertions(+) > > diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi > index 57ac18738c99..c25da7229e42 100644 > --- a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi > +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi > @@ -244,6 +244,29 @@ gpu0_thermal: gpu0-thermal { > polling-delay-passive = <0>; > polling-delay = <0>; > thermal-sensors = <&ths 1>; > + > + trips { > + gpu0_alert0: gpu0_alert0 { > + /* milliCelsius */ > + temperature = <75000>; > + hysteresis = <2000>; > + type = "passive"; > + }; > + > + gpu0_alert1: gpu0_alert1 { > + /* milliCelsius */ > + temperature = <90000>; > + hysteresis = <2000>; > + type = "hot"; > + }; > + > + gpu0_crit: gpu0_crit { > + /* milliCelsius */ > + temperature = <110000>; > + hysteresis = <2000>; > + type = "critical"; > + }; > + }; > }; > > gpu1_thermal: gpu1-thermal { > @@ -251,6 +274,29 @@ gpu1_thermal: gpu1-thermal { > polling-delay-passive = <0>; > polling-delay = <0>; > thermal-sensors = <&ths 2>; > + > + trips { > + gpu1_alert0: gpu1_alert0 { > + /* milliCelsius */ > + temperature = <75000>; > + hysteresis = <2000>; > + type = "passive"; > + }; > + > + gpu1_alert1: gpu1_alert1 { > + /* milliCelsius */ > + temperature = <90000>; > + hysteresis = <2000>; > + type = "hot"; > + }; > + > + gpu1_crit: gpu1_crit { > + /* milliCelsius */ > + temperature = <110000>; > + hysteresis = <2000>; > + type = "critical"; > + }; > + }; > }; > }; > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel