RE: [PATCH 01/22] dt-bindings: thermal: samsung,exynos: specify cells

From: "Alim Akhtar" <alim.akhtar@samsung.com>
To: "'Krzysztof Kozlowski'" <krzk@kernel.org>,
	"'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>,
	"'Niklas Söderlund'" <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>
Subject: RE: [PATCH 01/22] dt-bindings: thermal: samsung,exynos: specify cells
Date: Mon, 24 Jun 2024 09:36:22 +0530	[thread overview]
Message-ID: <1296674576.21719202202469.JavaMail.epsvc@epcpadp3> (raw)
In-Reply-To: <003b73f2-3b5d-40b7-a87c-2fc937e81bcd@kernel.org>


> -----Original Message-----
> From: Krzysztof Kozlowski <krzk@kernel.org>
> Sent: Saturday, June 15, 2024 3:09 PM
> To: Alim Akhtar <alim.akhtar@samsung.com>; 'Krzysztof Kozlowski'
> <krzysztof.kozlowski@linaro.org>; 'Daniel Lezcano'
[snip]
> On 14/06/2024 16:29, Alim Akhtar wrote:
> > 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
> >> cells
> >>
> >> All Samsung Exynos SoCs Thermal Management Units have only one
> >> sensor, so make '#thermal-sensor-cells' fixed at 0.
> >>
> > 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.
> 
> All supported by mainline. Others do not exist now :)
> 
> >
> > So we can leave this as is or you suggest to make it to support only
> > one sensor (to match the current DT support), and later (in near
> > future) change it again to match what HW actually support?
> 
> Yes, different devices can have different value (and bindings).
Ok, this is fine for now.

> Best regards,
> Krzysztof