diff mbox series

dt-bindings: display: vc4: dpi: Fix panel warning

Message ID 20200626121131.127192-1-maxime@cerno.tech
State Not Applicable
Headers show
Series dt-bindings: display: vc4: dpi: Fix panel warning | expand

Checks

Context Check Description
robh/checkpatch warning total: 0 errors, 1 warnings, 16 lines checked

Commit Message

Maxime Ripard June 26, 2020, 12:11 p.m. UTC
The example used in the DPI binding before the conversion to YAML had a
simple-panel example that got carried over to the YAML binding.

However, that example doesn't match the simple-panel binding and results in
validation errors. Since it's only marginally helpful, let's remove that
part of the example entirely.

Fixes: 094536003e06 ("dt-bindings: display: Convert VC4 bindings to schemas")
Signed-off-by: Maxime Ripard <maxime@cerno.tech>
---
 .../devicetree/bindings/display/brcm,bcm2835-dpi.yaml  | 10 ----------
 1 file changed, 10 deletions(-)

Comments

Sam Ravnborg June 27, 2020, 10:50 a.m. UTC | #1
On Fri, Jun 26, 2020 at 02:11:31PM +0200, Maxime Ripard wrote:
> The example used in the DPI binding before the conversion to YAML had a
> simple-panel example that got carried over to the YAML binding.
> 
> However, that example doesn't match the simple-panel binding and results in
> validation errors. Since it's only marginally helpful, let's remove that
> part of the example entirely.
> 
> Fixes: 094536003e06 ("dt-bindings: display: Convert VC4 bindings to schemas")
> Signed-off-by: Maxime Ripard <maxime@cerno.tech>
Acked-by: Sam Ravnborg <sam@ravnborg.org>
> ---
>  .../devicetree/bindings/display/brcm,bcm2835-dpi.yaml  | 10 ----------
>  1 file changed, 10 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml b/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml
> index 58213c564e03..5c1024bbc1b3 100644
> --- a/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml
> +++ b/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml
> @@ -45,16 +45,6 @@ examples:
>    - |
>      #include <dt-bindings/clock/bcm2835.h>
>  
> -    panel: panel {
> -        compatible = "ontat,yx700wv03", "simple-panel";
> -
> -        port {
> -            panel_in: endpoint {
> -                remote-endpoint = <&dpi_out>;
> -            };
> -        };
> -    };
> -
>      dpi: dpi@7e208000 {
>          compatible = "brcm,bcm2835-dpi";
>          reg = <0x7e208000 0x8c>;
> -- 
> 2.26.2
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
Maxime Ripard June 30, 2020, 7:42 a.m. UTC | #2
On Sat, Jun 27, 2020 at 12:50:10PM +0200, Sam Ravnborg wrote:
> On Fri, Jun 26, 2020 at 02:11:31PM +0200, Maxime Ripard wrote:
> > The example used in the DPI binding before the conversion to YAML had a
> > simple-panel example that got carried over to the YAML binding.
> > 
> > However, that example doesn't match the simple-panel binding and results in
> > validation errors. Since it's only marginally helpful, let's remove that
> > part of the example entirely.
> > 
> > Fixes: 094536003e06 ("dt-bindings: display: Convert VC4 bindings to schemas")
> > Signed-off-by: Maxime Ripard <maxime@cerno.tech>
>
> Acked-by: Sam Ravnborg <sam@ravnborg.org>

Thanks for the review, I just applied it's
Maxime
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml b/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml
index 58213c564e03..5c1024bbc1b3 100644
--- a/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml
+++ b/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml
@@ -45,16 +45,6 @@  examples:
   - |
     #include <dt-bindings/clock/bcm2835.h>
 
-    panel: panel {
-        compatible = "ontat,yx700wv03", "simple-panel";
-
-        port {
-            panel_in: endpoint {
-                remote-endpoint = <&dpi_out>;
-            };
-        };
-    };
-
     dpi: dpi@7e208000 {
         compatible = "brcm,bcm2835-dpi";
         reg = <0x7e208000 0x8c>;