Re: [PATCH v2 3/8] arm64: dts: exynos5433: Override thermal zones by phandle

Peter Griffin <[email protected]>
Newsgroups gmane.linux.kernel,gmane.linux.ports.arm.kernel,gmane.linux.kernel.samsung-soc,gmane.linux.drivers.devicetree
Message-ID <CADrjBPqc7jsrfYJXecG-PYt_Pp0vE0mA6GhOhb54u1p0SUFCzw@mail.gmail.com>
On Sat, 1 Aug 2026 at 16:07, Krzysztof Kozlowski
<[email protected]> wrote:
>
> Fix indentation of DTSI file with thermal zones (two opening { brackets,
> but only one indent) by using a safer choice of phandle/label override.
>
> Reviewed-by: André Draszik <[email protected]>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---

Reviewed-by: Peter Griffin <[email protected]>

>  arch/arm64/boot/dts/exynos/exynos5433-tmu.dtsi | 4 +---
>  arch/arm64/boot/dts/exynos/exynos5433.dtsi     | 3 +++
>  2 files changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/exynos/exynos5433-tmu.dtsi b/arch/arm64/boot/dts/exynos/exynos5433-tmu.dtsi
> index 57b335f579d8..a3ebc49fcd36 100644
> --- a/arch/arm64/boot/dts/exynos/exynos5433-tmu.dtsi
> +++ b/arch/arm64/boot/dts/exynos/exynos5433-tmu.dtsi
> @@ -7,8 +7,7 @@
>
>  #include <dt-bindings/thermal/thermal.h>
>
> -/ {
> -thermal-zones {
> +&thermal_zones {
>         atlas0_thermal: atlas0-thermal {
>                 thermal-sensors = <&tmu_atlas0>;
>                 polling-delay-passive = <0>;
> @@ -347,4 +346,3 @@ isp_alert_6: isp-alert-6 {
>                 };
>         };
>  };
> -};
> diff --git a/arch/arm64/boot/dts/exynos/exynos5433.dtsi b/arch/arm64/boot/dts/exynos/exynos5433.dtsi
> index 5fc3ca233ac6..ebd63b9e244d 100644
> --- a/arch/arm64/boot/dts/exynos/exynos5433.dtsi
> +++ b/arch/arm64/boot/dts/exynos/exynos5433.dtsi
> @@ -2073,6 +2073,9 @@ pcie: pcie@15700000 {
>                 };
>         };
>
> +       thermal_zones: thermal-zones {
> +       };
> +
>         timer: timer {
>                 compatible = "arm,armv8-timer";
>                 interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(8) | IRQ_TYPE_LEVEL_HIGH)>,
>
> --
> 2.53.0
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.