Received-SPF: pass (google.com: domain of linux-kernel+bounces-215727-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
From: "Alim Akhtar" <alim.akhtar@samsung.com>
To: "'Krzysztof Kozlowski'" <krzysztof.kozlowski@linaro.org>, "'Daniel
 Lezcano'" <daniel.lezcano@linaro.org>, "'Zhang Rui'" <rui.zhang@intel.com>,
	"'Lukasz Luba'" <lukasz.luba@arm.com>, "'Rob Herring'" <robh@kernel.org>,
	"'Conor Dooley'" <conor+dt@kernel.org>, "'Guillaume La Roque'"
	<glaroque@baylibre.com>, "'Krzysztof Kozlowski'" <krzk+dt@kernel.org>,
	"'Vasily Khoruzhick'" <anarsoul@gmail.com>, "'Chen-Yu Tsai'"
	<wens@csie.org>, "'Jernej Skrabec'" <jernej.skrabec@gmail.com>, "'Samuel
 Holland'" <samuel@sholland.org>, "'Shawn Guo'" <shawnguo@kernel.org>,
	"'Sascha Hauer'" <s.hauer@pengutronix.de>, "'Pengutronix	Kernel Team'"
	<kernel@pengutronix.de>, "'Fabio Estevam'" <festevam@gmail.com>, "'Anson
 Huang'" <Anson.Huang@nxp.com>, "'Thierry Reding'"
	<thierry.reding@gmail.com>, "'Jonathan Hunter'" <jonathanh@nvidia.com>,
	"'Dmitry	Baryshkov'" <dmitry.baryshkov@linaro.org>, "'Amit Kucheria'"
	<amitk@kernel.org>, =?utf-8?Q?'Niklas_S=C3=B6derlund'?=
	<niklas.soderlund@ragnatech.se>, "'Heiko	Stuebner'" <heiko@sntech.de>,
	"'Biju Das'" <biju.das.jz@bp.renesas.com>, "'Orson	Zhai'"
	<orsonzhai@gmail.com>, "'Baolin Wang'" <baolin.wang@linux.alibaba.com>,
	"'Chunyan Zhang'" <zhang.lyra@gmail.com>, "'Alexandre Torgue'"
	<alexandre.torgue@foss.st.com>, "'Pascal Paillet'" <p.paillet@foss.st.com>,
	"'Keerthy'" <j-keerthy@ti.com>, "'Broadcom internal kernel review list'"
	<bcm-kernel-feedback-list@broadcom.com>, "'Florian Fainelli'"
	<florian.fainelli@broadcom.com>, "'Scott Branden'" <sbranden@broadcom.com>,
	"'zhanghongchen'" <zhanghongchen@loongson.cn>, "'Matthias Brugger'"
	<matthias.bgg@gmail.com>, "'AngeloGioacchino Del Regno'"
	<angelogioacchino.delregno@collabora.com>, "'Bjorn Andersson'"
	<andersson@kernel.org>, "'Geert Uytterhoeven'" <geert+renesas@glider.be>
Cc: <linux-pm@vger.kernel.org>, <linux-samsung-soc@vger.kernel.org>,
	<devicetree@vger.kernel.org>, <linux-arm-kernel@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>, <linux-amlogic@lists.infradead.org>,
	<linux-sunxi@lists.linux.dev>, <imx@lists.linux.dev>,
	<linux-tegra@vger.kernel.org>, <linux-arm-msm@vger.kernel.org>,
	<linux-renesas-soc@vger.kernel.org>, <linux-rockchip@lists.infradead.org>,
	<linux-stm32@st-md-mailman.stormreply.com>, "'Florian Fainelli'"
	<f.fainelli@gmail.com>, <linux-rpi-kernel@lists.infradead.org>,
	<linux-mediatek@lists.infradead.org>, <cpgs@samsung.com>
In-Reply-To: <20240614-dt-bindings-thermal-allof-v1-1-30b25a6ae24e@linaro.org>
Subject: RE: [PATCH 01/22] dt-bindings: thermal: samsung,exynos: specify
 cells
Date: Fri, 14 Jun 2024 19:59:11 +0530
Message-ID: <1891546521.01718433481489.JavaMail.epsvc@epcpadp4>
Precedence: bulk
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Thread-Index: AQGhCdcVAXQc0tvKk5x9QOKQ/J0WewGh/L0BAiWHCW6yHH4lsA==
Content-Language: en-us
Content-Type: text/plain; charset="utf-8"
CMS-TYPE: 105P
References: <20240614-dt-bindings-thermal-allof-v1-0-30b25a6ae24e@linaro.org>
	<CGME20240614094638epcas5p115d52130f45e130652b6f1d946358d19@epcas5p1.samsung.com>
	<20240614-dt-bindings-thermal-allof-v1-1-30b25a6ae24e@linaro.org>

Hi Krzysztof,

> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Sent: Friday, June 14, 2024 3:16 PM
> To: Daniel Lezcano <daniel.lezcano@linaro.org
.stormreply.com;
> Subject: [PATCH 01/22] dt-bindings: thermal: samsung,exynos: specify cell=
s
>=20
> All Samsung Exynos SoCs Thermal Management Units have only one sensor, so
> make '#thermal-sensor-cells' fixed at 0.
>=20
This is not entirely true, there are SoCs which have multiple temp sensors.
It is true that currently only one sensor support is added though.

So we can leave this as is or you suggest to make it to support only one se=
nsor
(to match the current DT support), and later (in near future) change it aga=
in to
match what HW actually support?

> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  Documentation/devicetree/bindings/thermal/samsung,exynos-thermal.yaml | =
3
> ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>=20
> diff --git a/Documentation/devicetree/bindings/thermal/samsung,exynos-
> thermal.yaml b/Documentation/devicetree/bindings/thermal/samsung,exynos-
> thermal.yaml
> index 1344df708e2d..29a08b0729ee 100644
> --- a/Documentation/devicetree/bindings/thermal/samsung,exynos-
> thermal.yaml
> +++ b/Documentation/devicetree/bindings/thermal/samsung,exynos-thermal.y
> +++ aml
> @@ -61,7 +61,8 @@ properties:
>            TRIMINFO at 0x10068000 contains data for TMU channel 2
>      minItems: 1
>=20
> -  '#thermal-sensor-cells': true
> +  '#thermal-sensor-cells':
> +    const: 0
>=20
>    vtmu-supply:
>      description: The regulator node supplying voltage to TMU.
>=20
> --
> 2.43.0