Re: [PATCH 2/2] arm64: dts: mediatek: mt7988: add PWM controller

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

 



Il 14/02/24 10:24, Rafał Miłecki ha scritto:
On 14.02.2024 10:09, AngeloGioacchino Del Regno wrote:
Il 13/02/24 17:46, Rafał Miłecki ha scritto:
From: Rafał Miłecki <rafal@xxxxxxxxxx>

Add binding for on-SoC controller that can control up to 8 PWMs.

Signed-off-by: Rafał Miłecki <rafal@xxxxxxxxxx>
---
  arch/arm64/boot/dts/mediatek/mt7988a.dtsi | 21 ++++++++++++++++++++-
  1 file changed, 20 insertions(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/mediatek/mt7988a.dtsi b/arch/arm64/boot/dts/mediatek/mt7988a.dtsi
index bba97de4fb44..67007626b5cd 100644
--- a/arch/arm64/boot/dts/mediatek/mt7988a.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt7988a.dtsi
@@ -1,5 +1,6 @@
  // SPDX-License-Identifier: GPL-2.0-only OR MIT
+#include <dt-bindings/clock/mediatek,mt7988-clk.h>
  #include <dt-bindings/interrupt-controller/arm-gic.h>
  / {
@@ -78,7 +79,7 @@ gic: interrupt-controller@c000000 {
              #interrupt-cells = <3>;
          };
-        clock-controller@10001000 {
+        infracfg: clock-controller@10001000 {
              compatible = "mediatek,mt7988-infracfg", "syscon";
              reg = <0 0x10001000 0 0x1000>;
              #clock-cells = <1>;
@@ -103,6 +104,24 @@ clock-controller@1001e000 {
              #clock-cells = <1>;
          };
+        pwm@10048000 {
+            compatible = "mediatek,mt7988-pwm";

I can't take this unless there's a driver that supports your device.

I'd argue you should rather look for a documented binding rather than a
(Linux?) driver. Otherwise you would refuse changes that are not
strictly Linux related. DTS files are meant to describe hardware in a
generic way and not be driven by Linux drivers / design.


Of course, devicetree describes hardware - that is pretty much globally known.

As I wrote in the bindings patch, I still anyway want to see the driver part
for this block before deciding if your description of this hardware is correct.

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