> -----Original Message----- > From: Krzysztof Kozlowski <krzk@xxxxxxxxxx> > Sent: Saturday, June 15, 2024 3:09 PM > To: Alim Akhtar <alim.akhtar@xxxxxxxxxxx>; 'Krzysztof Kozlowski' > <krzysztof.kozlowski@xxxxxxxxxx>; 'Daniel Lezcano' [snip] > On 14/06/2024 16:29, Alim Akhtar wrote: > > Hi Krzysztof, > > > >> -----Original Message----- > >> From: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> > >> Sent: Friday, June 14, 2024 3:16 PM > >> To: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx > > .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