Re: [PATCH 2/4] dt-bindings: thermal: Add Tegra264 header

[Conor Dooley <conor@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1751 bytes --]

On Thu, Aug 27, 2026 at 05:07:51PM +0200, Thierry Reding wrote:
> From: Thierry Reding <treding@nvidia.com>
> 
> This header file defines the set of thermal zones exposed by the BPMP on
> Tegra264.

Does this need to be a binding, since you're just using it in the dts?
Can it just go into your dts directory insteaD?

Cheers,
Conor.

> 
> Signed-off-by: Thierry Reding <treding@nvidia.com>
> ---
>  include/dt-bindings/thermal/nvidia,tegra264-bpmp.h | 20 ++++++++++++++++++++
>  1 file changed, 20 insertions(+)
> 
> diff --git a/include/dt-bindings/thermal/nvidia,tegra264-bpmp.h b/include/dt-bindings/thermal/nvidia,tegra264-bpmp.h
> new file mode 100644
> index 000000000000..b32739691385
> --- /dev/null
> +++ b/include/dt-bindings/thermal/nvidia,tegra264-bpmp.h
> @@ -0,0 +1,20 @@
> +/* SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) */
> +/* SPDX-FileCopyrightText: Copyright (c) 2023-2025 NVIDIA CORPORATION & AFFILIATES. All rights reserved. */
> +
> +#ifndef DT_BINDINGS_THERMAL_NVIDIA_TEGRA264_BPMP_H
> +#define DT_BINDINGS_THERMAL_NVIDIA_TEGRA264_BPMP_H
> +
> +#define TEGRA264_THERMAL_ZONE_TJ_MAX		0
> +#define TEGRA264_THERMAL_ZONE_TJ_MIN		1
> +#define TEGRA264_THERMAL_ZONE_GPU_AVG		2
> +#define TEGRA264_THERMAL_ZONE_CPU_AVG		3
> +#define TEGRA264_THERMAL_ZONE_SOC_012_AVG	4
> +#define TEGRA264_THERMAL_ZONE_SOC_45_AVG	5
> +#define TEGRA264_THERMAL_ZONE_SOC_3_AVG		6
> +#define TEGRA264_THERMAL_ZONE_GPU_MAX		7
> +#define TEGRA264_THERMAL_ZONE_CPU_MAX		8
> +#define TEGRA264_THERMAL_ZONE_SOC_012_MAX	9
> +#define TEGRA264_THERMAL_ZONE_SOC_345_MAX	10
> +#define TEGRA264_THERMAL_ZONE_TJ_AVG		11
> +
> +#endif /* DT_BINDINGS_THERMAL_NVIDIA_TEGAR264_BPMP_H */
> 
> -- 
> 2.55.0
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]