Message ID | 20180618174216.24801-1-krzk@kernel.org (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Mon, 2018-06-18 at 19:42 +0200, Krzysztof Kozlowski wrote: > Remove leading 0x from recently introduced unit addresses to fix DTC > warnings: > > Warning (unit_address_format): /soc/sysmmu@0x15040000: unit name should not have leading "0x" [] > diff --git a/arch/arm64/boot/dts/exynos/exynos5433.dtsi b/arch/arm64/boot/dts/exynos/exynos5433.dtsi [] > @@ -1171,7 +1171,7 @@ > power-domains = <&pd_gscl>; > }; > > - sysmmu_scaler_0: sysmmu@0x15040000 { > + sysmmu_scaler_0: sysmmu@15040000 { > compatible = "samsung,exynos-sysmmu"; > reg = <0x15040000 0x1000>; > interrupts = <GIC_SPI 404 IRQ_TYPE_LEVEL_HIGH>; > @@ -1182,7 +1182,7 @@ > power-domains = <&pd_mscl>; > }; > > - sysmmu_scaler_1: sysmmu@0x15050000 { > + sysmmu_scaler_1: sysmmu@15050000 { > compatible = "samsung,exynos-sysmmu"; > reg = <0x15050000 0x1000>; > interrupts = <GIC_SPI 406 IRQ_TYPE_LEVEL_HIGH>; Presumably these 3 other entries too? $ git grep -P "\b\w+:\s+\w+@0x" -- "*.dt*" arch/arc/boot/dts/abilis_tb10x.dtsi: spi0: spi@0xFE010000 { arch/arc/boot/dts/abilis_tb10x.dtsi: spi1: spi@0xFE011000 { arch/arc/boot/dts/vdk_axs10x_mb.dtsi: uio_ev: uio@0xD0000000 { arch/arm64/boot/dts/exynos/exynos5433.dtsi: sysmmu_scaler_0: sysmmu@0x15040000 { arch/arm64/boot/dts/exynos/exynos5433.dtsi: sysmmu_scaler_1: sysmmu@0x15050000 {
On 18 June 2018 at 19:57, Joe Perches <joe@perches.com> wrote: > On Mon, 2018-06-18 at 19:42 +0200, Krzysztof Kozlowski wrote: >> Remove leading 0x from recently introduced unit addresses to fix DTC >> warnings: >> >> Warning (unit_address_format): /soc/sysmmu@0x15040000: unit name should not have leading "0x" > [] >> diff --git a/arch/arm64/boot/dts/exynos/exynos5433.dtsi b/arch/arm64/boot/dts/exynos/exynos5433.dtsi > [] >> @@ -1171,7 +1171,7 @@ >> power-domains = <&pd_gscl>; >> }; >> >> - sysmmu_scaler_0: sysmmu@0x15040000 { >> + sysmmu_scaler_0: sysmmu@15040000 { >> compatible = "samsung,exynos-sysmmu"; >> reg = <0x15040000 0x1000>; >> interrupts = <GIC_SPI 404 IRQ_TYPE_LEVEL_HIGH>; >> @@ -1182,7 +1182,7 @@ >> power-domains = <&pd_mscl>; >> }; >> >> - sysmmu_scaler_1: sysmmu@0x15050000 { >> + sysmmu_scaler_1: sysmmu@15050000 { >> compatible = "samsung,exynos-sysmmu"; >> reg = <0x15050000 0x1000>; >> interrupts = <GIC_SPI 406 IRQ_TYPE_LEVEL_HIGH>; > > Presumably these 3 other entries too? > > $ git grep -P "\b\w+:\s+\w+@0x" -- "*.dt*" > arch/arc/boot/dts/abilis_tb10x.dtsi: spi0: spi@0xFE010000 { > arch/arc/boot/dts/abilis_tb10x.dtsi: spi1: spi@0xFE011000 { > arch/arc/boot/dts/vdk_axs10x_mb.dtsi: uio_ev: uio@0xD0000000 { True. That's an "arc" but I can fix it as well with your reported-by. Best regards, Krzysztof
diff --git a/arch/arm64/boot/dts/exynos/exynos5433.dtsi b/arch/arm64/boot/dts/exynos/exynos5433.dtsi index 038c99792ccb..3a9b4c4b9c63 100644 --- a/arch/arm64/boot/dts/exynos/exynos5433.dtsi +++ b/arch/arm64/boot/dts/exynos/exynos5433.dtsi @@ -1171,7 +1171,7 @@ power-domains = <&pd_gscl>; }; - sysmmu_scaler_0: sysmmu@0x15040000 { + sysmmu_scaler_0: sysmmu@15040000 { compatible = "samsung,exynos-sysmmu"; reg = <0x15040000 0x1000>; interrupts = <GIC_SPI 404 IRQ_TYPE_LEVEL_HIGH>; @@ -1182,7 +1182,7 @@ power-domains = <&pd_mscl>; }; - sysmmu_scaler_1: sysmmu@0x15050000 { + sysmmu_scaler_1: sysmmu@15050000 { compatible = "samsung,exynos-sysmmu"; reg = <0x15050000 0x1000>; interrupts = <GIC_SPI 406 IRQ_TYPE_LEVEL_HIGH>;
Remove leading 0x from recently introduced unit addresses to fix DTC warnings: Warning (unit_address_format): /soc/sysmmu@0x15040000: unit name should not have leading "0x" Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> --- arch/arm64/boot/dts/exynos/exynos5433.dtsi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)