* [PATCH v2 2/6] ARM: dts: imx51-ts4800: Fix 'port@0' node with unit name and no reg property
2017-10-18 19:29 [PATCH v2 1/6] ARM: dts: imx51-apf51dev: Fix 'backlight@bl1' node with unit name and no reg property Marco Franchi
@ 2017-10-18 19:29 ` Marco Franchi
2017-10-18 19:29 ` [PATCH v2 3/6] ARM: dts: imx53: Fix 'usbphy@x' " Marco Franchi
` (4 subsequent siblings)
5 siblings, 0 replies; 8+ messages in thread
From: Marco Franchi @ 2017-10-18 19:29 UTC (permalink / raw)
To: linux-arm-kernel
The following build warning is seen with W=1:
Warning (unit_address_vs_reg): Node /display-di0/port at 0 has a unit name,
but no reg property
Fix this warning by removing '@' from such node.
Signed-off-by: Marco Franchi <marco.franchi@nxp.com>
---
Changes since v1:
-rebased to imx/dt branch
arch/arm/boot/dts/imx51-ts4800.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/imx51-ts4800.dts b/arch/arm/boot/dts/imx51-ts4800.dts
index 35a1112..564233e 100644
--- a/arch/arm/boot/dts/imx51-ts4800.dts
+++ b/arch/arm/boot/dts/imx51-ts4800.dts
@@ -71,7 +71,7 @@
};
};
- port at 0 {
+ port {
display0_in: endpoint {
remote-endpoint = <&ipu_di0_disp1>;
};
--
2.7.4
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH v2 3/6] ARM: dts: imx53: Fix 'usbphy@x' node with unit name and no reg property
2017-10-18 19:29 [PATCH v2 1/6] ARM: dts: imx51-apf51dev: Fix 'backlight@bl1' node with unit name and no reg property Marco Franchi
2017-10-18 19:29 ` [PATCH v2 2/6] ARM: dts: imx51-ts4800: Fix 'port@0' " Marco Franchi
@ 2017-10-18 19:29 ` Marco Franchi
2017-10-18 19:29 ` [PATCH v2 4/6] ARM: dts: imx53-m53evk: Fix 'led_gpio@0' " Marco Franchi
` (3 subsequent siblings)
5 siblings, 0 replies; 8+ messages in thread
From: Marco Franchi @ 2017-10-18 19:29 UTC (permalink / raw)
To: linux-arm-kernel
The following build warnings are seen with W=1:
Warning (unit_address_vs_reg): Node /soc/aips at 50000000/usbphy at 0 has a unit name, but no reg property
Warning (unit_address_vs_reg): Node /soc/aips at 50000000/usbphy at 1 has a unit name, but no reg property
Fix these warnings by changing '@' to '-'.
Signed-off-by: Marco Franchi <marco.franchi@nxp.com>
---
Changes since v1:
-none
arch/arm/boot/dts/imx53.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/imx53.dtsi b/arch/arm/boot/dts/imx53.dtsi
index 8bf0d89..c937949 100644
--- a/arch/arm/boot/dts/imx53.dtsi
+++ b/arch/arm/boot/dts/imx53.dtsi
@@ -299,14 +299,14 @@
reg = <0x53f00000 0x60>;
};
- usbphy0: usbphy at 0 {
+ usbphy0: usbphy-0 {
compatible = "usb-nop-xceiv";
clocks = <&clks IMX5_CLK_USB_PHY1_GATE>;
clock-names = "main_clk";
status = "okay";
};
- usbphy1: usbphy at 1 {
+ usbphy1: usbphy-1 {
compatible = "usb-nop-xceiv";
clocks = <&clks IMX5_CLK_USB_PHY2_GATE>;
clock-names = "main_clk";
--
2.7.4
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH v2 4/6] ARM: dts: imx53-m53evk: Fix 'led_gpio@0' node with unit name and no reg property
2017-10-18 19:29 [PATCH v2 1/6] ARM: dts: imx51-apf51dev: Fix 'backlight@bl1' node with unit name and no reg property Marco Franchi
2017-10-18 19:29 ` [PATCH v2 2/6] ARM: dts: imx51-ts4800: Fix 'port@0' " Marco Franchi
2017-10-18 19:29 ` [PATCH v2 3/6] ARM: dts: imx53: Fix 'usbphy@x' " Marco Franchi
@ 2017-10-18 19:29 ` Marco Franchi
2017-10-18 19:29 ` [PATCH v2 5/6] ARM: dts: imx53-qsb-common: Fix 'led_gpio7_7@0' " Marco Franchi
` (2 subsequent siblings)
5 siblings, 0 replies; 8+ messages in thread
From: Marco Franchi @ 2017-10-18 19:29 UTC (permalink / raw)
To: linux-arm-kernel
The following build warning is seen with W=1:
Warning (unit_address_vs_reg): Node /soc/aips at 50000000/iomuxc at 53fa8000/imx53-m53evk/led_gpio at 0 has a unit name, but no reg property
Fix this warning by removing '@0' from such node.
Signed-off-by: Marco Franchi <marco.franchi@nxp.com>
---
Changes since v1:
-none
arch/arm/boot/dts/imx53-m53evk.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/imx53-m53evk.dts b/arch/arm/boot/dts/imx53-m53evk.dts
index 9794a04..fdbf525 100644
--- a/arch/arm/boot/dts/imx53-m53evk.dts
+++ b/arch/arm/boot/dts/imx53-m53evk.dts
@@ -183,7 +183,7 @@
>;
};
- led_pin_gpio: led_gpio at 0 {
+ led_pin_gpio: led_gpio {
fsl,pins = <
MX53_PAD_PATA_DATA8__GPIO2_8 0x80000000
MX53_PAD_PATA_DATA9__GPIO2_9 0x80000000
--
2.7.4
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH v2 5/6] ARM: dts: imx53-qsb-common: Fix 'led_gpio7_7@0' node with unit name and no reg property
2017-10-18 19:29 [PATCH v2 1/6] ARM: dts: imx51-apf51dev: Fix 'backlight@bl1' node with unit name and no reg property Marco Franchi
` (2 preceding siblings ...)
2017-10-18 19:29 ` [PATCH v2 4/6] ARM: dts: imx53-m53evk: Fix 'led_gpio@0' " Marco Franchi
@ 2017-10-18 19:29 ` Marco Franchi
2017-10-18 19:29 ` [PATCH v2 6/6] ARM: dts: imx53-tx53: Fix 'ethernet-phy@0' " Marco Franchi
2017-10-18 20:12 ` [PATCH v2 1/6] ARM: dts: imx51-apf51dev: Fix 'backlight@bl1' " Rob Herring
5 siblings, 0 replies; 8+ messages in thread
From: Marco Franchi @ 2017-10-18 19:29 UTC (permalink / raw)
To: linux-arm-kernel
The following build warning is seen with W=1:
Warning (unit_address_vs_reg): Node /soc/aips at 50000000/iomuxc at 53fa8000/imx53-qsb/led_gpio7_7 at 0 has a unit name, but no reg property
Fix this warning by removing '@0' from such node.
Signed-off-by: Marco Franchi <marco.franchi@nxp.com>
---
Changes since v1:
-none
arch/arm/boot/dts/imx53-qsb-common.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/imx53-qsb-common.dtsi b/arch/arm/boot/dts/imx53-qsb-common.dtsi
index d5adf33..3c71de8 100644
--- a/arch/arm/boot/dts/imx53-qsb-common.dtsi
+++ b/arch/arm/boot/dts/imx53-qsb-common.dtsi
@@ -172,7 +172,7 @@
>;
};
- led_pin_gpio7_7: led_gpio7_7 at 0 {
+ led_pin_gpio7_7: led_gpio7_7 {
fsl,pins = <
MX53_PAD_PATA_DA_1__GPIO7_7 0x80000000
>;
--
2.7.4
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH v2 6/6] ARM: dts: imx53-tx53: Fix 'ethernet-phy@0' node with unit name and no reg property
2017-10-18 19:29 [PATCH v2 1/6] ARM: dts: imx51-apf51dev: Fix 'backlight@bl1' node with unit name and no reg property Marco Franchi
` (3 preceding siblings ...)
2017-10-18 19:29 ` [PATCH v2 5/6] ARM: dts: imx53-qsb-common: Fix 'led_gpio7_7@0' " Marco Franchi
@ 2017-10-18 19:29 ` Marco Franchi
2017-10-18 20:15 ` Rob Herring
2017-10-18 20:12 ` [PATCH v2 1/6] ARM: dts: imx51-apf51dev: Fix 'backlight@bl1' " Rob Herring
5 siblings, 1 reply; 8+ messages in thread
From: Marco Franchi @ 2017-10-18 19:29 UTC (permalink / raw)
To: linux-arm-kernel
The following build warning is seen with W=1:
Warning (unit_address_vs_reg): Node /soc/aips at 60000000/ethernet at 63fec000/ethernet-phy at 0 has a unit name, but no reg property
Fix this warning by removing '@' from such node.
Signed-off-by: Marco Franchi <marco.franchi@nxp.com>
---
Changes since v1:
-rebased to imx/dt branch
arch/arm/boot/dts/imx53-tx53.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/imx53-tx53.dtsi b/arch/arm/boot/dts/imx53-tx53.dtsi
index 76e7186..5d93287 100644
--- a/arch/arm/boot/dts/imx53-tx53.dtsi
+++ b/arch/arm/boot/dts/imx53-tx53.dtsi
@@ -225,7 +225,7 @@
mac-address = [000000000000]; /* placeholder; will be overwritten by bootloader */
status = "okay";
- phy0: ethernet-phy at 0 {
+ phy0: ethernet-phy {
interrupts-extended = <&gpio2 4 IRQ_TYPE_EDGE_FALLING>;
device_type = "ethernet-phy";
};
--
2.7.4
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH v2 6/6] ARM: dts: imx53-tx53: Fix 'ethernet-phy@0' node with unit name and no reg property
2017-10-18 19:29 ` [PATCH v2 6/6] ARM: dts: imx53-tx53: Fix 'ethernet-phy@0' " Marco Franchi
@ 2017-10-18 20:15 ` Rob Herring
0 siblings, 0 replies; 8+ messages in thread
From: Rob Herring @ 2017-10-18 20:15 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Oct 18, 2017 at 2:29 PM, Marco Franchi <marco.franchi@nxp.com> wrote:
> The following build warning is seen with W=1:
>
> Warning (unit_address_vs_reg): Node /soc/aips at 60000000/ethernet at 63fec000/ethernet-phy at 0 has a unit name, but no reg property
>
> Fix this warning by removing '@' from such node.
>
> Signed-off-by: Marco Franchi <marco.franchi@nxp.com>
> ---
> Changes since v1:
> -rebased to imx/dt branch
> arch/arm/boot/dts/imx53-tx53.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/imx53-tx53.dtsi b/arch/arm/boot/dts/imx53-tx53.dtsi
> index 76e7186..5d93287 100644
> --- a/arch/arm/boot/dts/imx53-tx53.dtsi
> +++ b/arch/arm/boot/dts/imx53-tx53.dtsi
> @@ -225,7 +225,7 @@
> mac-address = [000000000000]; /* placeholder; will be overwritten by bootloader */
> status = "okay";
>
> - phy0: ethernet-phy at 0 {
> + phy0: ethernet-phy {
Is this MDIO bus? If so, then it should have an address.
> interrupts-extended = <&gpio2 4 IRQ_TYPE_EDGE_FALLING>;
> device_type = "ethernet-phy";
BTW, this should also go away.
> };
> --
> 2.7.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v2 1/6] ARM: dts: imx51-apf51dev: Fix 'backlight@bl1' node with unit name and no reg property
2017-10-18 19:29 [PATCH v2 1/6] ARM: dts: imx51-apf51dev: Fix 'backlight@bl1' node with unit name and no reg property Marco Franchi
` (4 preceding siblings ...)
2017-10-18 19:29 ` [PATCH v2 6/6] ARM: dts: imx53-tx53: Fix 'ethernet-phy@0' " Marco Franchi
@ 2017-10-18 20:12 ` Rob Herring
5 siblings, 0 replies; 8+ messages in thread
From: Rob Herring @ 2017-10-18 20:12 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Oct 18, 2017 at 2:29 PM, Marco Franchi <marco.franchi@nxp.com> wrote:
> The following build warning is seen with W=1:
>
> Warning (unit_address_vs_reg): Node /backlight at bl1 has a unit name, but no
> reg property
>
> Fix this warning by changing '@' to '-'.
>
> Signed-off-by: Marco Franchi <marco.franchi@nxp.com>
> ---
> Changes since v1:
> -none
> arch/arm/boot/dts/imx51-apf51dev.dts | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/imx51-apf51dev.dts b/arch/arm/boot/dts/imx51-apf51dev.dts
> index 2f1a9d2..a6c2c11 100644
> --- a/arch/arm/boot/dts/imx51-apf51dev.dts
> +++ b/arch/arm/boot/dts/imx51-apf51dev.dts
> @@ -16,7 +16,7 @@
> model = "Armadeus Systems APF51Dev docking/development board";
> compatible = "armadeus,imx51-apf51dev", "armadeus,imx51-apf51", "fsl,imx51";
>
> - backlight at bl1{
> + backlight-bl1{
Is there more that one? If not, just "backlight".
You also should have a space before the {.
> pinctrl-names = "default";
> pinctrl-0 = <&pinctrl_backlight>;
> compatible = "gpio-backlight";
> --
> 2.7.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 8+ messages in thread