diff mbox series

dt-bindings: dma: pl08x: Fix unevaluatedProperties warnings

Message ID 20211206174231.2298349-1-robh@kernel.org (mailing list archive)
State Accepted
Commit a5480a3b0a97e1f41dfb637f251ac6a93e75fabd
Headers show
Series dt-bindings: dma: pl08x: Fix unevaluatedProperties warnings | expand

Commit Message

Rob Herring (Arm) Dec. 6, 2021, 5:42 p.m. UTC
With 'unevaluatedProperties' support implemented, the example has
warnings on primecell properties and 'resets':

Documentation/devicetree/bindings/dma/arm-pl08x.example.dt.yaml: dma-controller@67000000: Unevaluated properties are not allowed ('arm,primecell-periphid', 'resets' were unexpected)

Add the missing reference to primecell.yaml and definition for 'resets'.

Cc: Vinod Koul <vkoul@kernel.org>
Cc: dmaengine@vger.kernel.org
Signed-off-by: Rob Herring <robh@kernel.org>
---
 Documentation/devicetree/bindings/dma/arm-pl08x.yaml | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Thierry Reding Dec. 7, 2021, 8:41 a.m. UTC | #1
On Mon, Dec 06, 2021 at 11:42:31AM -0600, Rob Herring wrote:
> With 'unevaluatedProperties' support implemented, the example has
> warnings on primecell properties and 'resets':
> 
> Documentation/devicetree/bindings/dma/arm-pl08x.example.dt.yaml: dma-controller@67000000: Unevaluated properties are not allowed ('arm,primecell-periphid', 'resets' were unexpected)
> 
> Add the missing reference to primecell.yaml and definition for 'resets'.
> 
> Cc: Vinod Koul <vkoul@kernel.org>
> Cc: dmaengine@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
>  Documentation/devicetree/bindings/dma/arm-pl08x.yaml | 4 ++++
>  1 file changed, 4 insertions(+)

Reviewed-by: Thierry Reding <treding@nvidia.com>
Vinod Koul Dec. 13, 2021, 6:04 a.m. UTC | #2
On 06-12-21, 11:42, Rob Herring wrote:
> With 'unevaluatedProperties' support implemented, the example has
> warnings on primecell properties and 'resets':
> 
> Documentation/devicetree/bindings/dma/arm-pl08x.example.dt.yaml: dma-controller@67000000: Unevaluated properties are not allowed ('arm,primecell-periphid', 'resets' were unexpected)
> 
> Add the missing reference to primecell.yaml and definition for 'resets'.

Applied, thanks
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/dma/arm-pl08x.yaml b/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
index 3bd9eea543ca..9193b18fb75f 100644
--- a/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
+++ b/Documentation/devicetree/bindings/dma/arm-pl08x.yaml
@@ -10,6 +10,7 @@  maintainers:
   - Vinod Koul <vkoul@kernel.org>
 
 allOf:
+  - $ref: /schemas/arm/primecell.yaml#
   - $ref: "dma-controller.yaml#"
 
 # We need a select here so we don't match all nodes with 'arm,primecell'
@@ -89,6 +90,9 @@  properties:
       - 64
     description: bus width used for memcpy in bits. FTDMAC020 also accept 64 bits
 
+  resets:
+    maxItems: 1
+
 required:
   - reg
   - interrupts