arch/arm64/boot/dts/qcom/hamoa.dtsi | 50 +-- arch/arm64/boot/dts/qcom/purwa.dtsi | 590 +++--------------------------------- 2 files changed, 72 insertions(+), 568 deletions(-)
Hamoa and Purwa have pretty close thermal zones definitions. The major
difference is that several zones are absent (because of the lack of the
CPU cluster) and several zones use tsens2 instead of tsens3.
Instead of completely duplicating thermal zones for Purwa, reuse themal
zones from Hamoa, patching them where required.
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
---
arch/arm64/boot/dts/qcom/hamoa.dtsi | 50 +--
arch/arm64/boot/dts/qcom/purwa.dtsi | 590 +++---------------------------------
2 files changed, 72 insertions(+), 568 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/hamoa.dtsi b/arch/arm64/boot/dts/qcom/hamoa.dtsi
index d7596ccf63b9..a8ecb433381b 100644
--- a/arch/arm64/boot/dts/qcom/hamoa.dtsi
+++ b/arch/arm64/boot/dts/qcom/hamoa.dtsi
@@ -9230,7 +9230,7 @@ aoss0-critical {
};
};
- cpu2-0-top-thermal {
+ thermal_cpu2_0_top: cpu2-0-top-thermal {
thermal-sensors = <&tsens2 1>;
trips {
@@ -9242,7 +9242,7 @@ cpu-critical {
};
};
- cpu2-0-btm-thermal {
+ thermal_cpu2_0_btm: cpu2-0-btm-thermal {
thermal-sensors = <&tsens2 2>;
trips {
@@ -9254,7 +9254,7 @@ cpu-critical {
};
};
- cpu2-1-top-thermal {
+ thermal_cpu2_1_top: cpu2-1-top-thermal {
thermal-sensors = <&tsens2 3>;
trips {
@@ -9266,7 +9266,7 @@ cpu-critical {
};
};
- cpu2-1-btm-thermal {
+ thermal_cpu2_1_btm: cpu2-1-btm-thermal {
thermal-sensors = <&tsens2 4>;
trips {
@@ -9278,7 +9278,7 @@ cpu-critical {
};
};
- cpu2-2-top-thermal {
+ thermal_cpu2_2_top: cpu2-2-top-thermal {
thermal-sensors = <&tsens2 5>;
trips {
@@ -9290,7 +9290,7 @@ cpu-critical {
};
};
- cpu2-2-btm-thermal {
+ thermal_cpu2_2_btm: cpu2-2-btm-thermal {
thermal-sensors = <&tsens2 6>;
trips {
@@ -9302,7 +9302,7 @@ cpu-critical {
};
};
- cpu2-3-top-thermal {
+ thermal_cpu2_3_top: cpu2-3-top-thermal {
thermal-sensors = <&tsens2 7>;
trips {
@@ -9314,7 +9314,7 @@ cpu-critical {
};
};
- cpu2-3-btm-thermal {
+ thermal_cpu2_3_btm: cpu2-3-btm-thermal {
thermal-sensors = <&tsens2 8>;
trips {
@@ -9326,7 +9326,7 @@ cpu-critical {
};
};
- cpuss2-top-thermal {
+ thermal_cpuss2_top: cpuss2-top-thermal {
thermal-sensors = <&tsens2 9>;
trips {
@@ -9338,7 +9338,7 @@ cpuss2-critical {
};
};
- cpuss2-btm-thermal {
+ thermal_cpuss2_btm: cpuss2-btm-thermal {
thermal-sensors = <&tsens2 10>;
trips {
@@ -9350,7 +9350,7 @@ cpuss2-critical {
};
};
- aoss3-thermal {
+ thermal_aoss3: aoss3-thermal {
thermal-sensors = <&tsens3 0>;
trips {
@@ -9368,7 +9368,7 @@ aoss0-critical {
};
};
- nsp0-thermal {
+ thermal_nsp0: nsp0-thermal {
thermal-sensors = <&tsens3 1>;
trips {
@@ -9386,7 +9386,7 @@ nsp0-critical {
};
};
- nsp1-thermal {
+ thermal_nsp1: nsp1-thermal {
thermal-sensors = <&tsens3 2>;
trips {
@@ -9404,7 +9404,7 @@ nsp1-critical {
};
};
- nsp2-thermal {
+ thermal_nsp2: nsp2-thermal {
thermal-sensors = <&tsens3 3>;
trips {
@@ -9422,7 +9422,7 @@ nsp2-critical {
};
};
- nsp3-thermal {
+ thermal_nsp3: nsp3-thermal {
thermal-sensors = <&tsens3 4>;
trips {
@@ -9440,7 +9440,7 @@ nsp3-critical {
};
};
- gpuss-0-thermal {
+ thermal_gpuss_0: gpuss-0-thermal {
polling-delay-passive = <200>;
thermal-sensors = <&tsens3 5>;
@@ -9467,7 +9467,7 @@ gpu-critical {
};
};
- gpuss-1-thermal {
+ thermal_gpuss_1: gpuss-1-thermal {
polling-delay-passive = <200>;
thermal-sensors = <&tsens3 6>;
@@ -9494,7 +9494,7 @@ gpu-critical {
};
};
- gpuss-2-thermal {
+ thermal_gpuss_2: gpuss-2-thermal {
polling-delay-passive = <200>;
thermal-sensors = <&tsens3 7>;
@@ -9521,7 +9521,7 @@ gpu-critical {
};
};
- gpuss-3-thermal {
+ thermal_gpuss_3: gpuss-3-thermal {
polling-delay-passive = <200>;
thermal-sensors = <&tsens3 8>;
@@ -9548,7 +9548,7 @@ gpu-critical {
};
};
- gpuss-4-thermal {
+ thermal_gpuss_4: gpuss-4-thermal {
polling-delay-passive = <200>;
thermal-sensors = <&tsens3 9>;
@@ -9575,7 +9575,7 @@ gpu-critical {
};
};
- gpuss-5-thermal {
+ thermal_gpuss_5: gpuss-5-thermal {
polling-delay-passive = <200>;
thermal-sensors = <&tsens3 10>;
@@ -9602,7 +9602,7 @@ gpu-critical {
};
};
- gpuss-6-thermal {
+ thermal_gpuss_6: gpuss-6-thermal {
polling-delay-passive = <200>;
thermal-sensors = <&tsens3 11>;
@@ -9629,7 +9629,7 @@ gpu-critical {
};
};
- gpuss-7-thermal {
+ thermal_gpuss_7: gpuss-7-thermal {
polling-delay-passive = <200>;
thermal-sensors = <&tsens3 12>;
@@ -9656,7 +9656,7 @@ gpu-critical {
};
};
- camera0-thermal {
+ thermal_camera0: camera0-thermal {
thermal-sensors = <&tsens3 13>;
trips {
@@ -9674,7 +9674,7 @@ camera0-critical {
};
};
- camera1-thermal {
+ thermal_camera1: camera1-thermal {
thermal-sensors = <&tsens3 14>;
trips {
diff --git a/arch/arm64/boot/dts/qcom/purwa.dtsi b/arch/arm64/boot/dts/qcom/purwa.dtsi
index 2cecd2dd0de8..9ab4f26b35f2 100644
--- a/arch/arm64/boot/dts/qcom/purwa.dtsi
+++ b/arch/arm64/boot/dts/qcom/purwa.dtsi
@@ -20,7 +20,21 @@
/delete-node/ &gpu_opp_table;
/delete-node/ &gpu_speed_bin;
/delete-node/ &pcie3_phy;
-/delete-node/ &thermal_zones;
+/delete-node/ &thermal_aoss3;
+/delete-node/ &thermal_cpu2_0_btm;
+/delete-node/ &thermal_cpu2_0_top;
+/delete-node/ &thermal_cpu2_1_btm;
+/delete-node/ &thermal_cpu2_1_top;
+/delete-node/ &thermal_cpu2_2_btm;
+/delete-node/ &thermal_cpu2_2_top;
+/delete-node/ &thermal_cpu2_3_btm;
+/delete-node/ &thermal_cpu2_3_top;
+/delete-node/ &thermal_cpuss2_btm;
+/delete-node/ &thermal_cpuss2_top;
+/delete-node/ &thermal_gpuss_4;
+/delete-node/ &thermal_gpuss_5;
+/delete-node/ &thermal_gpuss_6;
+/delete-node/ &thermal_gpuss_7;
&gcc {
compatible = "qcom,x1p42100-gcc", "qcom,x1e80100-gcc";
@@ -198,557 +212,47 @@ pcie3_phy: phy@1bd4000 {
};
};
-/* While physically present, this controller is left unconfigured and unused */
-&tsens3 {
- status = "disabled";
+&thermal_camera0 {
+ thermal-sensors = <&tsens2 9>;
};
-/ {
- thermal-zones {
- aoss0-thermal {
- thermal-sensors = <&tsens0 0>;
-
- trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
- type = "hot";
- };
-
- trip-point1 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpu0-0-top-thermal {
- thermal-sensors = <&tsens0 1>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpu0-0-btm-thermal {
- thermal-sensors = <&tsens0 2>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpu0-1-top-thermal {
- thermal-sensors = <&tsens0 3>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpu0-1-btm-thermal {
- thermal-sensors = <&tsens0 4>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpu0-2-top-thermal {
- thermal-sensors = <&tsens0 5>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpu0-2-btm-thermal {
- thermal-sensors = <&tsens0 6>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpu0-3-top-thermal {
- thermal-sensors = <&tsens0 7>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpu0-3-btm-thermal {
- thermal-sensors = <&tsens0 8>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpuss0-top-thermal {
- thermal-sensors = <&tsens0 9>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpuss0-btm-thermal {
- thermal-sensors = <&tsens0 10>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- mem-thermal {
- thermal-sensors = <&tsens0 11>;
-
- trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
- type = "hot";
- };
-
- trip-point1 {
- temperature = <115000>;
- hysteresis = <0>;
- type = "critical";
- };
- };
- };
-
- video-thermal {
- thermal-sensors = <&tsens0 12>;
-
- trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
- type = "hot";
- };
-
- trip-point1 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- aoss1-thermal {
- thermal-sensors = <&tsens1 0>;
-
- trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
- type = "hot";
- };
-
- trip-point1 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpu1-0-top-thermal {
- thermal-sensors = <&tsens1 1>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpu1-0-btm-thermal {
- thermal-sensors = <&tsens1 2>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpu1-1-top-thermal {
- thermal-sensors = <&tsens1 3>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpu1-1-btm-thermal {
- thermal-sensors = <&tsens1 4>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpu1-2-top-thermal {
- thermal-sensors = <&tsens1 5>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpu1-2-btm-thermal {
- thermal-sensors = <&tsens1 6>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpu1-3-top-thermal {
- thermal-sensors = <&tsens1 7>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpu1-3-btm-thermal {
- thermal-sensors = <&tsens1 8>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpuss1-top-thermal {
- thermal-sensors = <&tsens1 9>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- cpuss1-btm-thermal {
- thermal-sensors = <&tsens1 10>;
-
- trips {
- trip-point0 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- aoss2-thermal {
- thermal-sensors = <&tsens2 0>;
-
- trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
- type = "hot";
- };
-
- trip-point1 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
-
- nsp0-thermal {
- thermal-sensors = <&tsens2 1>;
-
- trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
- type = "hot";
- };
-
- trip-point1 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
+&thermal_camera1 {
+ thermal-sensors = <&tsens2 10>;
+};
- nsp1-thermal {
- thermal-sensors = <&tsens2 2>;
-
- trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
- type = "hot";
- };
-
- trip-point1 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
+&thermal_gpuss_0 {
+ thermal-sensors = <&tsens2 5>;
+};
- nsp2-thermal {
- thermal-sensors = <&tsens2 3>;
-
- trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
- type = "hot";
- };
-
- trip-point1 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
+&thermal_gpuss_1 {
+ thermal-sensors = <&tsens2 6>;
+};
- nsp3-thermal {
- thermal-sensors = <&tsens2 4>;
-
- trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
- type = "hot";
- };
-
- trip-point1 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
+&thermal_gpuss_2 {
+ thermal-sensors = <&tsens2 7>;
+};
- gpuss-0-thermal {
- polling-delay-passive = <200>;
-
- thermal-sensors = <&tsens2 5>;
-
- cooling-maps {
- map0 {
- trip = <&gpuss0_alert0>;
- cooling-device = <&gpu THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
- };
- };
-
- trips {
- gpuss0_alert0: trip-point0 {
- temperature = <95000>;
- hysteresis = <1000>;
- type = "passive";
- };
-
- trip-point1 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
+&thermal_gpuss_3 {
+ thermal-sensors = <&tsens2 8>;
+};
- gpuss-1-thermal {
- polling-delay-passive = <200>;
-
- thermal-sensors = <&tsens2 6>;
-
- cooling-maps {
- map0 {
- trip = <&gpuss1_alert0>;
- cooling-device = <&gpu THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
- };
- };
-
- trips {
- gpuss1_alert0: trip-point0 {
- temperature = <95000>;
- hysteresis = <1000>;
- type = "passive";
- };
-
- trip-point1 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
+&thermal_nsp0 {
+ thermal-sensors = <&tsens2 1>;
+};
- gpuss-2-thermal {
- polling-delay-passive = <200>;
-
- thermal-sensors = <&tsens2 7>;
-
- cooling-maps {
- map0 {
- trip = <&gpuss2_alert0>;
- cooling-device = <&gpu THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
- };
- };
-
- trips {
- gpuss2_alert0: trip-point0 {
- temperature = <95000>;
- hysteresis = <1000>;
- type = "passive";
- };
-
- trip-point1 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
+&thermal_nsp1 {
+ thermal-sensors = <&tsens2 2>;
+};
- gpuss-3-thermal {
- polling-delay-passive = <200>;
-
- thermal-sensors = <&tsens2 8>;
-
- cooling-maps {
- map0 {
- trip = <&gpuss3_alert0>;
- cooling-device = <&gpu THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
- };
- };
-
- trips {
- gpuss3_alert0: trip-point0 {
- temperature = <95000>;
- hysteresis = <1000>;
- type = "passive";
- };
-
- trip-point1 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
+&thermal_nsp2 {
+ thermal-sensors = <&tsens2 3>;
+};
- camera0-thermal {
- thermal-sensors = <&tsens2 9>;
-
- trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
- type = "hot";
- };
-
- trip-point1 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
+&thermal_nsp3 {
+ thermal-sensors = <&tsens2 4>;
+};
- camera1-thermal {
- thermal-sensors = <&tsens2 10>;
-
- trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
- type = "hot";
- };
-
- trip-point1 {
- temperature = <115000>;
- hysteresis = <1000>;
- type = "critical";
- };
- };
- };
- };
+/* While physically present, this controller is left unconfigured and unused */
+&tsens3 {
+ status = "disabled";
};
---
base-commit: 8e5a478b6d6a5bb0a3d52147862b15e4d826af19
change-id: 20260320-purwa-drop-thermals-19d1abe37758
Best regards,
--
With best wishes
Dmitry
On Fri, 20 Mar 2026 04:33:15 +0200, Dmitry Baryshkov wrote:
> Hamoa and Purwa have pretty close thermal zones definitions. The major
> difference is that several zones are absent (because of the lack of the
> CPU cluster) and several zones use tsens2 instead of tsens3.
>
> Instead of completely duplicating thermal zones for Purwa, reuse themal
> zones from Hamoa, patching them where required.
>
> [...]
Applied, thanks!
[1/1] arm64: dts: qcom: purwa: deduplicate thermal sensors with Hamoa
commit: 12b9fae5293e64022e0c09a4c106216fdfa308a2
Best regards,
--
Bjorn Andersson <andersson@kernel.org>
On 3/20/26 3:33 AM, Dmitry Baryshkov wrote: > Hamoa and Purwa have pretty close thermal zones definitions. The major > difference is that several zones are absent (because of the lack of the > CPU cluster) and several zones use tsens2 instead of tsens3. > > Instead of completely duplicating thermal zones for Purwa, reuse themal > zones from Hamoa, patching them where required. > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> > --- Slightly reluctantly: Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> (confirmed with dtx_diff) Konrad
On Mon, Mar 23, 2026 at 04:21:00PM +0100, Konrad Dybcio wrote: > On 3/20/26 3:33 AM, Dmitry Baryshkov wrote: > > Hamoa and Purwa have pretty close thermal zones definitions. The major > > difference is that several zones are absent (because of the lack of the > > CPU cluster) and several zones use tsens2 instead of tsens3. > > > > Instead of completely duplicating thermal zones for Purwa, reuse themal > > zones from Hamoa, patching them where required. > > > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> > > --- > > Slightly reluctantly: Why? I'd really interested here. -- With best wishes Dmitry
On 3/23/26 5:49 PM, Dmitry Baryshkov wrote: > On Mon, Mar 23, 2026 at 04:21:00PM +0100, Konrad Dybcio wrote: >> On 3/20/26 3:33 AM, Dmitry Baryshkov wrote: >>> Hamoa and Purwa have pretty close thermal zones definitions. The major >>> difference is that several zones are absent (because of the lack of the >>> CPU cluster) and several zones use tsens2 instead of tsens3. >>> >>> Instead of completely duplicating thermal zones for Purwa, reuse themal >>> zones from Hamoa, patching them where required. >>> >>> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> >>> --- >> >> Slightly reluctantly: > > Why? I'd really interested here. I suppose my reluctance comes from the decreased readability, but then I suppose the silicon is not going to change if we get it right once, so it's not a real concern.. Konrad
On Tue, Mar 24, 2026 at 10:25:30AM +0100, Konrad Dybcio wrote: > On 3/23/26 5:49 PM, Dmitry Baryshkov wrote: > > On Mon, Mar 23, 2026 at 04:21:00PM +0100, Konrad Dybcio wrote: > >> On 3/20/26 3:33 AM, Dmitry Baryshkov wrote: > >>> Hamoa and Purwa have pretty close thermal zones definitions. The major > >>> difference is that several zones are absent (because of the lack of the > >>> CPU cluster) and several zones use tsens2 instead of tsens3. > >>> > >>> Instead of completely duplicating thermal zones for Purwa, reuse themal > >>> zones from Hamoa, patching them where required. > >>> > >>> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> > >>> --- > >> > >> Slightly reluctantly: > > > > Why? I'd really interested here. > > I suppose my reluctance comes from the decreased readability, but then > I suppose the silicon is not going to change if we get it right once, so > it's not a real concern.. I see. My usual concern is opposite: to make sure that we don't need to fix another DT if we fix something. -- With best wishes Dmitry
On Wed, Mar 25, 2026 at 01:13:28AM +0200, Dmitry Baryshkov wrote: > On Tue, Mar 24, 2026 at 10:25:30AM +0100, Konrad Dybcio wrote: > > On 3/23/26 5:49 PM, Dmitry Baryshkov wrote: > > > On Mon, Mar 23, 2026 at 04:21:00PM +0100, Konrad Dybcio wrote: > > >> On 3/20/26 3:33 AM, Dmitry Baryshkov wrote: > > >>> Hamoa and Purwa have pretty close thermal zones definitions. The major > > >>> difference is that several zones are absent (because of the lack of the > > >>> CPU cluster) and several zones use tsens2 instead of tsens3. > > >>> > > >>> Instead of completely duplicating thermal zones for Purwa, reuse themal > > >>> zones from Hamoa, patching them where required. > > >>> > > >>> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> > > >>> --- > > >> > > >> Slightly reluctantly: > > > > > > Why? I'd really interested here. > > > > I suppose my reluctance comes from the decreased readability, but then > > I suppose the silicon is not going to change if we get it right once, so > > it's not a real concern.. > > I see. My usual concern is opposite: to make sure that we don't need to > fix another DT if we fix something. > Which means that if you make a change to that "something" you now have the burden to validate the additional users. Regards, Bjorn > -- > With best wishes > Dmitry
On 26-03-20 04:33:15, Dmitry Baryshkov wrote: > Hamoa and Purwa have pretty close thermal zones definitions. The major > difference is that several zones are absent (because of the lack of the > CPU cluster) and several zones use tsens2 instead of tsens3. > > Instead of completely duplicating thermal zones for Purwa, reuse themal > zones from Hamoa, patching them where required. > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> Reviewed-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
© 2016 - 2026 Red Hat, Inc.