Re: [PATCH v2 1/4] dt-bindings: clock: mediatek: Fix wrong compatible list for hifsys YAML
From: Rob Herring <robh@kernel.org>
Date: 2025-09-24 14:03:58
Also in:
linux-arm-kernel, linux-clk, linux-mediatek, linux-pci, lkml
On Tue, Sep 23, 2025 at 10:12:29PM +0200, Christian Marangi wrote:
While converting the hifsys to YAML schema, the "syscon" compatible was dropped for the mt7623 and the mt2701 compatible.
Is "syscon" really needed? AFAICT, the clock and reset drivers don't need it.
quoted hunk ↗ jump to hunk
Add back the compatible to mute DTBs warning on "make dtbs_check" and reflect real state of the .dtsi. Signed-off-by: Christian Marangi <ansuelsmth@gmail.com> --- .../devicetree/bindings/clock/mediatek,mt2701-hifsys.yaml | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-)diff --git a/Documentation/devicetree/bindings/clock/mediatek,mt2701-hifsys.yaml b/Documentation/devicetree/bindings/clock/mediatek,mt2701-hifsys.yaml index 9e7c725093aa..aa3345ea8283 100644 --- a/Documentation/devicetree/bindings/clock/mediatek,mt2701-hifsys.yaml +++ b/Documentation/devicetree/bindings/clock/mediatek,mt2701-hifsys.yaml@@ -16,13 +16,15 @@ maintainers: properties: compatible: oneOf: - - enum: - - mediatek,mt2701-hifsys - - mediatek,mt7622-hifsys + - items: + - const: mediatek,mt2701-hifsys + - const: syscon + - const: mediatek,mt7622-hifsys - items: - enum: - mediatek,mt7623-hifsys - const: mediatek,mt2701-hifsys + - const: syscon reg: maxItems: 1-- 2.51.0