dt-bindings: display: Fix dtc unit-address warnings in examples
Subsystems (6)
Rob Herring
maintainerPATCH 1/1 dt-bindings: display: Fix dtc unit-address warnings in examples
Extra dtc warnings (roughly what W=1 enables) are now enabled by default when building the binding examples. These were fixed treewide in 5.6-rc5, but some new display bindings have been added with new warnings:
Documentation/devicetree/bindings/display/panel/raydium,rm68200.example.dts:17.7-27.11: Warning (unit_address_vs_reg): /example-0/dsi@0: node has a unit name, but no reg property Documentation/devicetree/bindings/display/panel/panel-simple-dsi.example.dts:17.19-31.11: Warning (unit_address_vs_reg): /example-0/mdss_dsi@fd922800: node has a unit name, but no reg property Documentation/devicetree/bindings/display/panel/orisetech,otm8009a.example.dts:17.7-26.11: Warning (unit_address_vs_reg): /example-0/dsi@0: node has a unit name, but no reg property Documentation/devicetree/bindings/display/ti/ti,am65x-dss.example.dts:21.27-49.11: Warning (unit_address_format): /example-0/dss@04a00000: unit name should not have leading 0s Documentation/devicetree/bindings/display/ti/ti,j721e-dss.example.dts:21.27-72.11: Warning (unit_address_format): /example-0/dss@04a00000: unit name should not have leading 0s Documentation/devicetree/bindings/display/ti/ti,k2g-dss.example.dts:20.27-42.11: Warning (unit_address_format): /example-0/dss@02540000: unit name should not have leading 0s
--- .../devicetree/bindings/display/panel/orisetech,otm8009a.yaml | 3 +-- .../devicetree/bindings/display/panel/panel-simple-dsi.yaml | 2 +- .../devicetree/bindings/display/panel/raydium,rm68200.yaml | 2 +- Documentation/devicetree/bindings/display/ti/ti,am65x-dss.yaml | 2 +- Documentation/devicetree/bindings/display/ti/ti,j721e-dss.yaml | 2 +- Documentation/devicetree/bindings/display/ti/ti,k2g-dss.yaml | 2 +- 6 files changed, 6 insertions(+), 7 deletions(-)
Documentation/devicetree/bindings/display/panel/orisetech,otm8009a.yaml +1 -2
@@ -39,7 +39,7 @@ required: examples: - | - dsi@0 { + dsi { #address-cells = <1>; #size-cells = <0>; panel@0 {
@@ -50,4 +50,3 @@ examples: }; }; ... -
Documentation/devicetree/bindings/display/panel/panel-simple-dsi.yaml +1 -1
@@ -50,7 +50,7 @@ required: examples: - | - mdss_dsi@fd922800 { + dsi { #address-cells = <1>; #size-cells = <0>; panel@0 {
Documentation/devicetree/bindings/display/panel/raydium,rm68200.yaml +1 -1
@@ -42,7 +42,7 @@ required: examples: - | - dsi@0 { + dsi { #address-cells = <1>; #size-cells = <0>; panel@0 {
Documentation/devicetree/bindings/display/ti/ti,am65x-dss.yaml +1 -1
@@ -121,7 +121,7 @@ examples: #include <dt-bindings/interrupt-controller/irq.h> #include <dt-bindings/soc/ti,sci_pm_domain.h> - dss: dss@04a00000 { + dss: dss@4a00000 { compatible = "ti,am65x-dss"; reg = <0x0 0x04a00000 0x0 0x1000>, /* common */ <0x0 0x04a02000 0x0 0x1000>, /* vidl1 */
Documentation/devicetree/bindings/display/ti/ti,j721e-dss.yaml +1 -1
@@ -154,7 +154,7 @@ examples: #include <dt-bindings/interrupt-controller/irq.h> #include <dt-bindings/soc/ti,sci_pm_domain.h> - dss: dss@04a00000 { + dss: dss@4a00000 { compatible = "ti,j721e-dss"; reg = <0x00 0x04a00000 0x00 0x10000>, /* common_m */ <0x00 0x04a10000 0x00 0x10000>, /* common_s0*/
Documentation/devicetree/bindings/display/ti/ti,k2g-dss.yaml +1 -1
@@ -81,7 +81,7 @@ examples: #include <dt-bindings/interrupt-controller/arm-gic.h> #include <dt-bindings/interrupt-controller/irq.h> - dss: dss@02540000 { + dss: dss@2540000 { compatible = "ti,k2g-dss"; reg = <0x02540000 0x400>, <0x02550000 0x1000>,
Signature · 6 lines
2.20.1 _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
Quoted 12 lines · Rob Herring
Extra dtc warnings (roughly what W=1 enables) are now enabled by default when building the binding examples. These were fixed treewide in 5.6-rc5, but some new display bindings have been added with new warnings:
Documentation/devicetree/bindings/display/panel/raydium,rm68200.example.dts:17.7-27.11: Warning (
unit_address_vs_reg): /example-0/dsi@0: node has a unit name, but no reg property Documentation/devicetree/bindings/display/panel/panel-simple-dsi.example.dts:17.19-31.11: Warning (unit_address_vs_reg): /example-0/mdss_dsi@fd922800: node has a unit name, but no reg property Documentation/devicetree/bindings/display/panel/orisetech,otm8009a.example.dts:17.7-26.11: Warning (unit_address_vs_reg): /example-0/dsi@0: node has a unit name, but no reg property Documentation/devicetree/bindings/display/ti/ti,am65x-dss.example.dts:21.27-49.11: Warning (unit_address_format): /example-0/dss@04a00000: unit name should not have leading 0s Documentation/devicetree/bindings/display/ti/ti,j721e-dss.example.dts:21.27-72.11: Warning (unit_address_format): /example-0/dss@04a00000: unit name should not have leading 0s Documentation/devicetree/bindings/display/ti/ti,k2g-dss.example.dts:20.27-42.11: Warning (unit_address_format): /example-0/dss@02540000: unit name should not have leading 0s
Quoted 5 lines
Cc Thierry Reding <[email protected]>Cc Sam Ravnborg <[email protected]>Cc Jyri Sarha <[email protected]>Cc Tomi Valkeinen <[email protected]>Signed-off-by Rob Herring <[email protected]>
And I can drop my patch that only fixed the panel/ parts - good.
Sam
Quoted 94 lines
--- .../devicetree/bindings/display/panel/orisetech,otm8009a.yaml | 3 +-- .../devicetree/bindings/display/panel/panel-simple-dsi.yaml | 2 +- .../devicetree/bindings/display/panel/raydium,rm68200.yaml | 2 +- Documentation/devicetree/bindings/display/ti/ti,am65x-dss.yaml | 2 +- Documentation/devicetree/bindings/display/ti/ti,j721e-dss.yaml | 2 +- Documentation/devicetree/bindings/display/ti/ti,k2g-dss.yaml | 2 +- 6 files changed, 6 insertions(+), 7 deletions(-)Documentation/devicetree/bindings/display/panel/orisetech,otm8009a.yaml +1
-2@@ -39,7 +39,7 @@ required: examples: - | - dsi@0 { + dsi { #address-cells = <1>; #size-cells = <0>; panel@0 {@@ -50,4 +50,3 @@ examples: }; }; -Documentation/devicetree/bindings/display/panel/panel-simple-dsi.yaml +1
-1@@ -50,7 +50,7 @@ required: examples: - | - mdss_dsi@fd922800 { + dsi { #address-cells = <1>; #size-cells = <0>; panel@0 {Documentation/devicetree/bindings/display/panel/raydium,rm68200.yaml +1
-1@@ -42,7 +42,7 @@ required: examples: - | - dsi@0 { + dsi { #address-cells = <1>; #size-cells = <0>; panel@0 {Documentation/devicetree/bindings/display/ti/ti,am65x-dss.yaml +1
-1@@ -121,7 +121,7 @@ examples: #include <dt-bindings/interrupt-controller/irq.h> #include <dt-bindings/soc/ti,sci_pm_domain.h> - dss: dss@04a00000 { + dss: dss@4a00000 { compatible = "ti,am65x-dss"; reg = <0x0 0x04a00000 0x0 0x1000>, /* common */ <0x0 0x04a02000 0x0 0x1000>, /* vidl1 */Documentation/devicetree/bindings/display/ti/ti,j721e-dss.yaml +1
-1@@ -154,7 +154,7 @@ examples: #include <dt-bindings/interrupt-controller/irq.h> #include <dt-bindings/soc/ti,sci_pm_domain.h> - dss: dss@04a00000 { + dss: dss@4a00000 { compatible = "ti,j721e-dss"; reg = <0x00 0x04a00000 0x00 0x10000>, /* common_m */ <0x00 0x04a10000 0x00 0x10000>, /* common_s0*/Documentation/devicetree/bindings/display/ti/ti,k2g-dss.yaml +1
-1@@ -81,7 +81,7 @@ examples: #include <dt-bindings/interrupt-controller/arm-gic.h> #include <dt-bindings/interrupt-controller/irq.h> - dss: dss@02540000 { + dss: dss@2540000 { compatible = "ti,k2g-dss"; reg = <0x02540000 0x400>, <0x02550000 0x1000>,Signature · 1 line
2.20.1
_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel
Rob Herring
maintainerQuoted 14 lines · Sam Ravnborg
Quoted 12 lines · Rob Herring
Extra dtc warnings (roughly what W=1 enables) are now enabled by default when building the binding examples. These were fixed treewide in 5.6-rc5, but some new display bindings have been added with new warnings:
Documentation/devicetree/bindings/display/panel/raydium,rm68200.example.dts:17.7-27.11: Warning (
unit_address_vs_reg): /example-0/dsi@0: node has a unit name, but no reg property Documentation/devicetree/bindings/display/panel/panel-simple-dsi.example.dts:17.19-31.11: Warning (unit_address_vs_reg): /example-0/mdss_dsi@fd922800: node has a unit name, but no reg property Documentation/devicetree/bindings/display/panel/orisetech,otm8009a.example.dts:17.7-26.11: Warning (unit_address_vs_reg): /example-0/dsi@0: node has a unit name, but no reg property Documentation/devicetree/bindings/display/ti/ti,am65x-dss.example.dts:21.27-49.11: Warning (unit_address_format): /example-0/dss@04a00000: unit name should not have leading 0s Documentation/devicetree/bindings/display/ti/ti,j721e-dss.example.dts:21.27-72.11: Warning (unit_address_format): /example-0/dss@04a00000: unit name should not have leading 0s Documentation/devicetree/bindings/display/ti/ti,k2g-dss.example.dts:20.27-42.11: Warning (unit_address_format): /example-0/dss@02540000: unit name should not have leading 0s
Quoted 8 lines
Quoted 5 lines
Cc Thierry Reding <[email protected]>Cc Sam Ravnborg <[email protected]>Cc Jyri Sarha <[email protected]>Cc Tomi Valkeinen <[email protected]>Signed-off-by Rob Herring <[email protected]>Reviewed-by Sam Ravnborg <[email protected]>And I can drop my patch that only fixed the panel/ parts - good.
It seems I missed some panels. Perhaps ones that are in drm-misc, but not in linux-next yet? Keep yours and I'll respin with just the TI part.
Rob