[PATCH 1/2] dt-bindings: display: vc4: dpi: Remove unneeded 'simple-panel' property

[PATCH 1/2] dt-bindings: display: vc4: dpi: Remove unneeded 'simple-panel' property

[Fabio Estevam]

Remove the unneeded "simple-panel" property to fix the following warning
seen with 'make dt_binding_check': 

Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.example.dt.yaml: panel: compatible: Additional items are not allowed ('simple-panel' was unexpected)
Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.example.dt.yaml: panel: compatible: ['ontat,yx700wv03', 'simple-panel'] is too long

Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
 Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml b/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml
index 58213c564e03..24119411b007 100644
--- a/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml
+++ b/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml
@@ -46,7 +46,7 @@ examples:
     #include <dt-bindings/clock/bcm2835.h>
 
     panel: panel {
-        compatible = "ontat,yx700wv03", "simple-panel";
+        compatible = "ontat,yx700wv03";
 
         port {
             panel_in: endpoint {
-- 
2.17.1

[PATCH 2/2] dt-bindings: display: vc4: dpi: Pass the required 'power-supply' prorperty

[Fabio Estevam]

Pass the required 'power-supply' prorperty to fix the following warning
seen with 'make dt_binding_check': 

Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.example.dt.yaml: panel: 'power-supply' is a required property

Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
 Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml b/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml
index 24119411b007..7ab03614a8ea 100644
--- a/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml
+++ b/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml
@@ -47,6 +47,7 @@ examples:
 
     panel: panel {
         compatible = "ontat,yx700wv03";
+        power-supply = <&panel_reg>;
 
         port {
             panel_in: endpoint {
-- 
2.17.1

Re: [PATCH 1/2] dt-bindings: display: vc4: dpi: Remove unneeded 'simple-panel' property

[Rob Herring]

On Mon, Jun 29, 2020 at 07:13:26PM -0300, Fabio Estevam wrote:
> Remove the unneeded "simple-panel" property to fix the following warning
> seen with 'make dt_binding_check': 
> 
> Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.example.dt.yaml: panel: compatible: Additional items are not allowed ('simple-panel' was unexpected)
> Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.example.dt.yaml: panel: compatible: ['ontat,yx700wv03', 'simple-panel'] is too long

There's already a fix for this from Maxime which just removes the 
example as it doesn't add much value.

> 
> Signed-off-by: Fabio Estevam <festevam@gmail.com>
> ---
>  Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml b/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml
> index 58213c564e03..24119411b007 100644
> --- a/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml
> +++ b/Documentation/devicetree/bindings/display/brcm,bcm2835-dpi.yaml
> @@ -46,7 +46,7 @@ examples:
>      #include <dt-bindings/clock/bcm2835.h>
>  
>      panel: panel {
> -        compatible = "ontat,yx700wv03", "simple-panel";
> +        compatible = "ontat,yx700wv03";
>  
>          port {
>              panel_in: endpoint {
> -- 
> 2.17.1
>