From: Thierry Reding <treding@xxxxxxxxxx> For PWM controller device tree bindings, make sure that they include the pwm.yaml controller core bindings explicitly. This prevents the tooling from matching on the $nodename pattern, which can falsely match things like pinmux nodes, etc. Signed-off-by: Thierry Reding <treding@xxxxxxxxxx> --- .../bindings/pwm/allwinner,sun4i-a10-pwm.yaml | 51 ++++++++++--------- .../bindings/pwm/google,cros-ec-pwm.yaml | 4 +- .../devicetree/bindings/pwm/iqs620a-pwm.yaml | 4 +- .../devicetree/bindings/pwm/pwm-samsung.yaml | 1 - .../devicetree/bindings/pwm/pwm.yaml | 2 + .../bindings/pwm/renesas,pwm-rcar.yaml | 4 +- .../bindings/pwm/renesas,tpu-pwm.yaml | 4 +- 7 files changed, 40 insertions(+), 30 deletions(-) diff --git a/Documentation/devicetree/bindings/pwm/allwinner,sun4i-a10-pwm.yaml b/Documentation/devicetree/bindings/pwm/allwinner,sun4i-a10-pwm.yaml index 7dcab2bf8128..0a7e70114af0 100644 --- a/Documentation/devicetree/bindings/pwm/allwinner,sun4i-a10-pwm.yaml +++ b/Documentation/devicetree/bindings/pwm/allwinner,sun4i-a10-pwm.yaml @@ -50,33 +50,34 @@ properties: resets: maxItems: 1 -if: - properties: - compatible: - contains: - const: allwinner,sun50i-h6-pwm - -then: - properties: - clocks: - maxItems: 2 - - clock-names: - items: - - const: mod - - const: bus - - required: - - clock-names - - resets - -else: - properties: - clocks: - maxItems: 1 +allOf: + - $ref: pwm.yaml# + - if: + properties: + compatible: + contains: + const: allwinner,sun50i-h6-pwm + + then: + properties: + clocks: + maxItems: 2 + + clock-names: + items: + - const: mod + - const: bus + + required: + - clock-names + - resets + + else: + properties: + clocks: + maxItems: 1 required: - - "#pwm-cells" - compatible - reg - clocks diff --git a/Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.yaml b/Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.yaml index 41ece1d85315..a5a1e4f3df48 100644 --- a/Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.yaml +++ b/Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.yaml @@ -25,7 +25,9 @@ properties: required: - compatible - - '#pwm-cells' + +allOf: + - $ref: pwm.yaml# additionalProperties: false diff --git a/Documentation/devicetree/bindings/pwm/iqs620a-pwm.yaml b/Documentation/devicetree/bindings/pwm/iqs620a-pwm.yaml index 1d7c27be50da..a375cc9e8fd0 100644 --- a/Documentation/devicetree/bindings/pwm/iqs620a-pwm.yaml +++ b/Documentation/devicetree/bindings/pwm/iqs620a-pwm.yaml @@ -25,7 +25,9 @@ properties: required: - compatible - - "#pwm-cells" + +allOf: + - $ref: pwm.yaml# additionalProperties: false diff --git a/Documentation/devicetree/bindings/pwm/pwm-samsung.yaml b/Documentation/devicetree/bindings/pwm/pwm-samsung.yaml index fc799b0577d4..e1487194d9a7 100644 --- a/Documentation/devicetree/bindings/pwm/pwm-samsung.yaml +++ b/Documentation/devicetree/bindings/pwm/pwm-samsung.yaml @@ -89,7 +89,6 @@ required: - clock-names - compatible - interrupts - - "#pwm-cells" - reg additionalProperties: false diff --git a/Documentation/devicetree/bindings/pwm/pwm.yaml b/Documentation/devicetree/bindings/pwm/pwm.yaml index fa4f9de92090..b0ebd2881a91 100644 --- a/Documentation/devicetree/bindings/pwm/pwm.yaml +++ b/Documentation/devicetree/bindings/pwm/pwm.yaml @@ -9,6 +9,8 @@ title: PWM controllers (providers) maintainers: - Thierry Reding <thierry.reding@xxxxxxxxx> +select: false + properties: $nodename: pattern: "^pwm(@.*|-[0-9a-f])*$" diff --git a/Documentation/devicetree/bindings/pwm/renesas,pwm-rcar.yaml b/Documentation/devicetree/bindings/pwm/renesas,pwm-rcar.yaml index daadde9ff9c4..d2cc1226f309 100644 --- a/Documentation/devicetree/bindings/pwm/renesas,pwm-rcar.yaml +++ b/Documentation/devicetree/bindings/pwm/renesas,pwm-rcar.yaml @@ -57,9 +57,11 @@ properties: required: - compatible - reg - - '#pwm-cells' - clocks +allOf: + - $ref: pwm.yaml# + additionalProperties: false examples: diff --git a/Documentation/devicetree/bindings/pwm/renesas,tpu-pwm.yaml b/Documentation/devicetree/bindings/pwm/renesas,tpu-pwm.yaml index 4bf62a3d5bba..0db96f38ab0a 100644 --- a/Documentation/devicetree/bindings/pwm/renesas,tpu-pwm.yaml +++ b/Documentation/devicetree/bindings/pwm/renesas,tpu-pwm.yaml @@ -56,7 +56,9 @@ properties: required: - compatible - reg - - '#pwm-cells' + +allOf: + - $ref: pwm.yaml# additionalProperties: false -- 2.24.1