* [PATCH 1/2] dt-bindings: display: vc4: dpi: Remove unneeded 'simple-panel' property
@ 2020-06-29 22:13 Fabio Estevam
2020-06-29 22:13 ` [PATCH 2/2] dt-bindings: display: vc4: dpi: Pass the required 'power-supply' prorperty Fabio Estevam
2020-06-30 2:04 ` [PATCH 1/2] dt-bindings: display: vc4: dpi: Remove unneeded 'simple-panel' property Rob Herring
0 siblings, 2 replies; 3+ messages in thread
From: Fabio Estevam @ 2020-06-29 22:13 UTC (permalink / raw)
To: maxime; +Cc: eric, robh+dt, devicetree, 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
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH 2/2] dt-bindings: display: vc4: dpi: Pass the required 'power-supply' prorperty
2020-06-29 22:13 [PATCH 1/2] dt-bindings: display: vc4: dpi: Remove unneeded 'simple-panel' property Fabio Estevam
@ 2020-06-29 22:13 ` Fabio Estevam
2020-06-30 2:04 ` [PATCH 1/2] dt-bindings: display: vc4: dpi: Remove unneeded 'simple-panel' property Rob Herring
1 sibling, 0 replies; 3+ messages in thread
From: Fabio Estevam @ 2020-06-29 22:13 UTC (permalink / raw)
To: maxime; +Cc: eric, robh+dt, devicetree, 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
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 1/2] dt-bindings: display: vc4: dpi: Remove unneeded 'simple-panel' property
2020-06-29 22:13 [PATCH 1/2] dt-bindings: display: vc4: dpi: Remove unneeded 'simple-panel' property Fabio Estevam
2020-06-29 22:13 ` [PATCH 2/2] dt-bindings: display: vc4: dpi: Pass the required 'power-supply' prorperty Fabio Estevam
@ 2020-06-30 2:04 ` Rob Herring
1 sibling, 0 replies; 3+ messages in thread
From: Rob Herring @ 2020-06-30 2:04 UTC (permalink / raw)
To: Fabio Estevam; +Cc: maxime, eric, devicetree
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
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-06-30 2:04 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-06-29 22:13 [PATCH 1/2] dt-bindings: display: vc4: dpi: Remove unneeded 'simple-panel' property Fabio Estevam
2020-06-29 22:13 ` [PATCH 2/2] dt-bindings: display: vc4: dpi: Pass the required 'power-supply' prorperty Fabio Estevam
2020-06-30 2:04 ` [PATCH 1/2] dt-bindings: display: vc4: dpi: Remove unneeded 'simple-panel' property Rob Herring
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).