dri-devel.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: display: sun8i-mixer: Remove duplicated 'iommus'
@ 2020-07-10 20:31 Fabio Estevam
  2020-07-16  8:32 ` Maxime Ripard
  0 siblings, 1 reply; 2+ messages in thread
From: Fabio Estevam @ 2020-07-10 20:31 UTC (permalink / raw)
  To: mripard; +Cc: devicetree, robh+dt, dri-devel

Commit 13871279ff5c ("arm64: dts: allwinner: h6: Fix Cedrus IOMMU usage")
introduced a double instance of 'iommus' causing the following build
error with 'make dt_binding_check':

found duplicate key "iommus" with value "{}" (original value: "{}")
  in "<unicode string>", line 45, column 3

Remove the double occurrence to fix this problem.

Fixes: 13871279ff5c ("arm64: dts: allwinner: h6: Fix Cedrus IOMMU usage")
Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
 .../bindings/display/allwinner,sun8i-a83t-de2-mixer.yaml       | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-de2-mixer.yaml b/Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-de2-mixer.yaml
index c2fbf0b06d32..c040eef56518 100644
--- a/Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-de2-mixer.yaml
+++ b/Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-de2-mixer.yaml
@@ -42,9 +42,6 @@ properties:
   resets:
     maxItems: 1
 
-  iommus:
-    maxItems: 1
-
   ports:
     type: object
     description: |
-- 
2.17.1

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] dt-bindings: display: sun8i-mixer: Remove duplicated 'iommus'
  2020-07-10 20:31 [PATCH] dt-bindings: display: sun8i-mixer: Remove duplicated 'iommus' Fabio Estevam
@ 2020-07-16  8:32 ` Maxime Ripard
  0 siblings, 0 replies; 2+ messages in thread
From: Maxime Ripard @ 2020-07-16  8:32 UTC (permalink / raw)
  To: Fabio Estevam; +Cc: devicetree, robh+dt, dri-devel


[-- Attachment #1.1: Type: text/plain, Size: 694 bytes --]

Hi,

On Fri, Jul 10, 2020 at 05:31:24PM -0300, Fabio Estevam wrote:
> Commit 13871279ff5c ("arm64: dts: allwinner: h6: Fix Cedrus IOMMU usage")
> introduced a double instance of 'iommus' causing the following build
> error with 'make dt_binding_check':
> 
> found duplicate key "iommus" with value "{}" (original value: "{}")
>   in "<unicode string>", line 45, column 3
> 
> Remove the double occurrence to fix this problem.
> 
> Fixes: 13871279ff5c ("arm64: dts: allwinner: h6: Fix Cedrus IOMMU usage")
> Signed-off-by: Fabio Estevam <festevam@gmail.com>

Sorry for that, I've amended the original commit to move it into the
proper binding.

Thanks for reporting

Maxime

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-07-17  7:22 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-10 20:31 [PATCH] dt-bindings: display: sun8i-mixer: Remove duplicated 'iommus' Fabio Estevam
2020-07-16  8:32 ` Maxime Ripard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).