Commit 9977a8c3 authored by Mathieu Malaterre's avatar Mathieu Malaterre Committed by Arnd Bergmann

arm64: dts: Remove leading 0x and 0s from bindings notation

Improve the DTS files by removing all the leading "0x" and zeros to fix the
following dtc warnings:

Warning (unit_address_format): Node /XXX unit name should not have leading "0x"

and

Warning (unit_address_format): Node /XXX unit name should not have leading 0s

Converted using the following command:

find . -type f \( -iname *.dts -o -iname *.dtsi \) -exec sed -E -i -e "s/@0x([0-9a-fA-F\.]+)\s?\{/@\L\1 \{/g" -e "s/@0+([0-9a-fA-F\.]+)\s?\{/@\L\1 \{/g" {} +

For simplicity, two sed expressions were used to solve each warnings separately.

To make the regex expression more robust a few other issues were resolved,
namely setting unit-address to lower case, and adding a whitespace before the
the opening curly brace:

https://elinux.org/Device_Tree_Linux#Linux_conventions

This is a follow up to commit 4c9847b7 ("dt-bindings: Remove leading 0x from bindings notation")
Reported-by: default avatarDavid Daney <ddaney@caviumnetworks.com>
Suggested-by: default avatarRob Herring <robh@kernel.org>
Signed-off-by: default avatarMathieu Malaterre <malat@debian.org>
Acked-by: default avatarMatthias Brugger <matthias.bgg@gmail.com>
Acked-by: default avatarAndy Gross <andy.gross@linaro.org>
Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
parent 713bb31c
...@@ -51,7 +51,7 @@ reserved-memory { ...@@ -51,7 +51,7 @@ reserved-memory {
#size-cells = <2>; #size-cells = <2>;
ranges; ranges;
ramoops@0x21f00000 { ramoops@21f00000 {
compatible = "ramoops"; compatible = "ramoops";
reg = <0x0 0x21f00000 0x0 0x00100000>; reg = <0x0 0x21f00000 0x0 0x00100000>;
record-size = <0x00020000>; record-size = <0x00020000>;
......
...@@ -341,7 +341,7 @@ syscfg_pctl_a: syscfg_pctl_a@10005000 { ...@@ -341,7 +341,7 @@ syscfg_pctl_a: syscfg_pctl_a@10005000 {
reg = <0 0x10005000 0 0x1000>; reg = <0 0x10005000 0 0x1000>;
}; };
pio: pinctrl@0x10005000 { pio: pinctrl@10005000 {
compatible = "mediatek,mt8173-pinctrl"; compatible = "mediatek,mt8173-pinctrl";
reg = <0 0x1000b000 0 0x1000>; reg = <0 0x1000b000 0 0x1000>;
mediatek,pctl-regmap = <&syscfg_pctl_a>; mediatek,pctl-regmap = <&syscfg_pctl_a>;
......
...@@ -140,16 +140,16 @@ usb2_id: usb2-id { ...@@ -140,16 +140,16 @@ usb2_id: usb2-id {
}; };
agnoc@0 { agnoc@0 {
qcom,pcie@00600000 { qcom,pcie@600000 {
perst-gpio = <&msmgpio 35 GPIO_ACTIVE_LOW>; perst-gpio = <&msmgpio 35 GPIO_ACTIVE_LOW>;
}; };
qcom,pcie@00608000 { qcom,pcie@608000 {
status = "okay"; status = "okay";
perst-gpio = <&msmgpio 130 GPIO_ACTIVE_LOW>; perst-gpio = <&msmgpio 130 GPIO_ACTIVE_LOW>;
}; };
qcom,pcie@00610000 { qcom,pcie@610000 {
status = "okay"; status = "okay";
perst-gpio = <&msmgpio 114 GPIO_ACTIVE_LOW>; perst-gpio = <&msmgpio 114 GPIO_ACTIVE_LOW>;
}; };
......
...@@ -840,7 +840,7 @@ agnoc@0 { ...@@ -840,7 +840,7 @@ agnoc@0 {
#size-cells = <1>; #size-cells = <1>;
ranges; ranges;
pcie0: qcom,pcie@00600000 { pcie0: qcom,pcie@600000 {
compatible = "qcom,pcie-msm8996", "snps,dw-pcie"; compatible = "qcom,pcie-msm8996", "snps,dw-pcie";
status = "disabled"; status = "disabled";
power-domains = <&gcc PCIE0_GDSC>; power-domains = <&gcc PCIE0_GDSC>;
...@@ -893,7 +893,7 @@ pcie0: qcom,pcie@00600000 { ...@@ -893,7 +893,7 @@ pcie0: qcom,pcie@00600000 {
}; };
pcie1: qcom,pcie@00608000 { pcie1: qcom,pcie@608000 {
compatible = "qcom,pcie-msm8996", "snps,dw-pcie"; compatible = "qcom,pcie-msm8996", "snps,dw-pcie";
power-domains = <&gcc PCIE1_GDSC>; power-domains = <&gcc PCIE1_GDSC>;
bus-range = <0x00 0xff>; bus-range = <0x00 0xff>;
...@@ -946,7 +946,7 @@ pcie1: qcom,pcie@00608000 { ...@@ -946,7 +946,7 @@ pcie1: qcom,pcie@00608000 {
"bus_slave"; "bus_slave";
}; };
pcie2: qcom,pcie@00610000 { pcie2: qcom,pcie@610000 {
compatible = "qcom,pcie-msm8996", "snps,dw-pcie"; compatible = "qcom,pcie-msm8996", "snps,dw-pcie";
power-domains = <&gcc PCIE2_GDSC>; power-domains = <&gcc PCIE2_GDSC>;
bus-range = <0x00 0xff>; bus-range = <0x00 0xff>;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment