From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2B498305660 for ; Thu, 8 Oct 2026 21:15:35 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791494137; cv=none; b=OpSsKGAbzXtc04CSTJOH+wlRIsFuKGrdalh1OLtrj4nSTTBosAQbGtUL7Pr0mFeRPKV8/gu+xXvq+Pk9lv7wFIyPM3IN4z1FBkHg18xgoxZe8PlAh2tE5iEezdKypQZ2FLFiGFoRAVk0ZaGgAgiLCNRs00XdGO9zVkLJHyemCYg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791494137; c=relaxed/simple; bh=XYYVAe5uKZwHERIrHXUdTsSPqrgNYTMBK8DqUkipDHg=; h=Message-ID:From:Subject:To:Cc:In-Reply-To:References:Content-Type: Date; b=EvisibVyCoR9XOoIF8XRKX2QVvhkAzVcN/DBTc8P3yLeUM/M9OGs3TWdmlx836d77V97y2r2StZCh3mQ6Yq4IOltPv9A9+AN5ijSTykH103NXxmGLQO4MYVjHZSQqszhYHhmZCPXspZc34BCetppi0M6c1wfrra4cbDT79LUkcE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=jHEZzyka; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="jHEZzyka" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 712821F000FF; Thu, 8 Oct 2026 21:15:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1791494135; bh=EUjJU4QS/nJLfIBzxcye2SlSFqIg3lZcc5WD5aB/RrM=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=jHEZzykaL8nevSGwEsqXYpG3n9TNIUFZEzsuCqbSTO2XQweUAFS9VEu0fsSHS4H6A EUMv/8G4a3bFnCTcHVVkOw1WIhVWfhMonP1IR4xSrmXf6Far8xAHjEbrY6XdVyqbZN 5PEA7TtaX0rAj9Kd5yOQmoIVcuSsGFZa2eipD5LiPOrTwFMdXLK2+XEtgA1AQ867eB e3Lo+BWrWbsQX8vqlr+7S8gRXkrSDg42iaaxJUIGiEH4eBIwOC/7GK+V5Hf8VyLMp2 n6r9x2yxikqabisrjTiuf1pSNbXFc6cj/oInzFjq2R1tDbNnrbhyJcxbN9+UQUIIFH wdk+oXiOKabBA== Message-ID: From: sashiko-bot@kernel.org Subject: Re: [PATCH 4/5] arm64: dts: qcom: nord: Add GPU cooling Reply-To: sashiko-reviews@lists.linux.dev To: "Akhil P Oommen" Cc: conor+dt@kernel.org, dri-devel@lists.freedesktop.org, robh@kernel.org, devicetree@vger.kernel.org In-Reply-To: <20261009-nord-gpu1-v1-4-a06a6c57bde7@oss.qualcomm.com> References: <20261009-nord-gpu1-v1-0-a06a6c57bde7@oss.qualcomm.com> <20261009-nord-gpu1-v1-4-a06a6c57bde7@oss.qualcomm.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Date: Thu, 08 Oct 2026 21:15:34 +0000 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Thank you for your contribution! Sashiko AI review found 1 potential issue(= s) to consider: - [Medium] dts: missing cooling-device mappings for 4 GPU thermal zones -- commit be1fe5e4df785d4f2fc715cc8d3e734547b71199 Author: Aastha Pandey arm64: dts: qcom: nord: Add GPU cooling This commit sets up GPU cooling on Nord by throttling the Adreno 753 GPU when it reaches 105C. It labels the GPU thermal zones and their passive trip points, maps them to the GPU as a cooling device, and adds polling delays. > diff --git a/arch/arm64/boot/dts/qcom/nord-embedded.dtsi b/arch/arm64/boo= t/dts/qcom/nord-embedded.dtsi > index 4379092b7bf8d..23370007c9748 100644 > --- a/arch/arm64/boot/dts/qcom/nord-embedded.dtsi > +++ b/arch/arm64/boot/dts/qcom/nord-embedded.dtsi [ ... ] > +&gpuss_2_thermal { > + polling-delay-passive =3D <10>; > + > + cooling-maps { > + map0 { > + trip =3D <&gpuss2_alert0>; > + cooling-device =3D <&gpu_0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>; > + }; > + }; > +}; [Severity: Medium] Are the cooling device mappings for the gpuss_1_0_thermal, gpuss_1_1_therma= l, gpuss_1_2_thermal, and gpu_1_0_thermal zones intentionally omitted from this list? If a sustained GPU workload creates hotspots near these unmapped sensors, localized heating will not trigger GPU throttling. This could allow the SoC to reach unsafe temperature thresholds and trigger a hardware shutdown before the adjacent mapped sensors become hot enough to intervene. These zones and their trip points are explicitly labeled in the nord.dtsi changes below, which suggests they should have cooling maps defined alongsi= de the other zones. > diff --git a/arch/arm64/boot/dts/qcom/nord.dtsi b/arch/arm64/boot/dts/qco= m/nord.dtsi > index 147d8e595a3a3..b032d1d2ad9b6 100644 > --- a/arch/arm64/boot/dts/qcom/nord.dtsi > +++ b/arch/arm64/boot/dts/qcom/nord.dtsi [ ... ] > - gpuss-1-0-thermal { > + gpuss_1_0_thermal: gpuss-1-0-thermal { > polling-delay-passive =3D <0>; > polling-delay =3D <0>; > thermal-sensors =3D <&tsens6 5>; [ ... ] > - gpuss-1-1-thermal { > + gpuss_1_1_thermal: gpuss-1-1-thermal { > polling-delay-passive =3D <0>; > polling-delay =3D <0>; > thermal-sensors =3D <&tsens6 6>; [ ... ] > - gpuss-1-2-thermal { > + gpuss_1_2_thermal: gpuss-1-2-thermal { > polling-delay-passive =3D <0>; > polling-delay =3D <0>; > thermal-sensors =3D <&tsens7 5>; [ ... ] > - gpu-1-0-thermal { > + gpu_1_0_thermal: gpu-1-0-thermal { > polling-delay-passive =3D <0>; > polling-delay =3D <0>; > thermal-sensors =3D <&tsens7 6>; --=20 Sashiko AI review =C2=B7 https://sashiko.dev/#/patchset/20261009-nord-gpu1-= v1-0-a06a6c57bde7@oss.qualcomm.com?part=3D4