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). Best regards, Krzysztof
