2019-03-25 16:09:20

by Guido Günther

[permalink] [raw]

Subject: [PATCH] dt-bindings: display/panel: Add missing unit names

---

Some examples were missing the unit names triggering

Warning (unit_address_vs_reg): .../panel: node has a reg or ranges property, but no unit name

warnings when used verbatim in DTs and running dtc with W=1.

Signed-off-by: Guido Günther <[email protected]>
---
.../devicetree/bindings/display/panel/innolux,p079zca.txt | 2 +-
.../devicetree/bindings/display/panel/innolux,p097pfg.txt | 2 +-
.../devicetree/bindings/display/panel/kingdisplay,kd097d04.txt | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/display/panel/innolux,p079zca.txt b/Documentation/devicetree/bindings/display/panel/innolux,p079zca.txt
index d0f55161579a..3ab8c7412cf6 100644
--- a/Documentation/devicetree/bindings/display/panel/innolux,p079zca.txt
+++ b/Documentation/devicetree/bindings/display/panel/innolux,p079zca.txt
@@ -12,7 +12,7 @@ Optional properties:
Example:

&mipi_dsi {
- panel {
+ panel@0 {
compatible = "innolux,p079zca";
reg = <0>;
power-supply = <...>;
diff --git a/Documentation/devicetree/bindings/display/panel/innolux,p097pfg.txt b/Documentation/devicetree/bindings/display/panel/innolux,p097pfg.txt
index 595d9dfeffd3..d1cab3a8f0fb 100644
--- a/Documentation/devicetree/bindings/display/panel/innolux,p097pfg.txt
+++ b/Documentation/devicetree/bindings/display/panel/innolux,p097pfg.txt
@@ -13,7 +13,7 @@ Optional properties:
Example:

&mipi_dsi {
- panel {
+ panel@0 {
compatible = "innolux,p079zca";
reg = <0>;
avdd-supply = <...>;
diff --git a/Documentation/devicetree/bindings/display/panel/kingdisplay,kd097d04.txt b/Documentation/devicetree/bindings/display/panel/kingdisplay,kd097d04.txt
index 164a5fa236da..cfefff688614 100644
--- a/Documentation/devicetree/bindings/display/panel/kingdisplay,kd097d04.txt
+++ b/Documentation/devicetree/bindings/display/panel/kingdisplay,kd097d04.txt
@@ -12,7 +12,7 @@ Optional properties:
Example:

&mipi_dsi {
- panel {
+ panel@0 {
compatible = "kingdisplay,kd097d04";
reg = <0>;
power-supply = <...>;
--
2.20.1

2019-03-31 06:43:38

by Rob Herring (Arm)

[permalink] [raw]

Subject: Re: [PATCH] dt-bindings: display/panel: Add missing unit names

---

On Mon, 25 Mar 2019 17:08:10 +0100, =?UTF-8?q?Guido=20G=C3=BCnther?= wrote:
> Some examples were missing the unit names triggering
>
> Warning (unit_address_vs_reg): .../panel: node has a reg or ranges property, but no unit name
>
> warnings when used verbatim in DTs and running dtc with W=1.
>
> Signed-off-by: Guido Günther <[email protected]>
> ---
> .../devicetree/bindings/display/panel/innolux,p079zca.txt | 2 +-
> .../devicetree/bindings/display/panel/innolux,p097pfg.txt | 2 +-
> .../devicetree/bindings/display/panel/kingdisplay,kd097d04.txt | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
>

Reviewed-by: Rob Herring <[email protected]>

2019-04-03 16:16:30

by Thierry Reding

[permalink] [raw]

Subject: Re: [PATCH] dt-bindings: display/panel: Add missing unit names

---

On Mon, Mar 25, 2019 at 05:08:10PM +0100, Guido Günther wrote:
> Some examples were missing the unit names triggering
>
> Warning (unit_address_vs_reg): .../panel: node has a reg or ranges property, but no unit name
>
> warnings when used verbatim in DTs and running dtc with W=1.
>
> Signed-off-by: Guido Günther <[email protected]>
> ---
> .../devicetree/bindings/display/panel/innolux,p079zca.txt | 2 +-
> .../devicetree/bindings/display/panel/innolux,p097pfg.txt | 2 +-
> .../devicetree/bindings/display/panel/kingdisplay,kd097d04.txt | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)

Applied, thanks.

Thierry

---

Attachments:

(No filename) (662.00 B)
signature.asc (849.00 B)
Download all attachments