Re: [PATCH v8 2/6] dt-bindings: thermal: Add binding document for LVTS thermal controllers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Il 26/07/22 15:55, Balsam CHIHI ha scritto:
This patch adds dt-binding documents for mt8192 and mt8195 thermal controllers.

Signed-off-by: Alexandre Bailon <abailon@xxxxxxxxxxxx>
Signed-off-by: Balsam CHIHI <bchihi@xxxxxxxxxxxx>
---
  .../thermal/mediatek,mt8192-lvts.yaml         | 73 ++++++++++++++++++
  .../thermal/mediatek,mt8195-lvts.yaml         | 75 +++++++++++++++++++
  2 files changed, 148 insertions(+)
  create mode 100644 Documentation/devicetree/bindings/thermal/mediatek,mt8192-lvts.yaml
  create mode 100644 Documentation/devicetree/bindings/thermal/mediatek,mt8195-lvts.yaml


Hello Balsam,

I don't think that having two separated files is necessary (and I don't know if
that's correct, even).

You're referring to variants of the same hardware (and using the same driver, but
that's not really relevant here), so my suggestion here is to have only one YAML,
named like "mediatek,lvts-thermal.yaml", where we list all of the compatibles for
both MT8192 and MT8195 (and future SoCs).

Regards,
Angelo



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux