Re: [PATCH] ARC: dts: Remove leading 0x from node unit addresses

[Joe Perches]

On Mon, 2018-06-18 at 21:16 +0200, Krzysztof Kozlowski wrote:
> Remove leading 0x from all node unit addresses with a sed command:
> 
>     sed -E -i -e "s/@0x([0-9a-fA-F\.]+)\s?\{/@\L\1 \{/g" arch/arc/boot/dts/*.dts*
> 
> This fixes DTC warnings like:
> 
>     Warning (unit_address_format): /cpu_card/dw-apb-gpio@0x2000: unit name should not have leading "0x"

Do all of these need fixing?

Aren't there dozens of these in arch/arm and elsewhere too?

$ git grep -i -P "@0x([0-9a-f]+)\s*\{" -- "*.dt*"|cat
arch/arc/boot/dts/abilis_tb10x.dtsi:		spi0: spi@0xFE010000 {
arch/arc/boot/dts/abilis_tb10x.dtsi:		spi1: spi@0xFE011000 {
arch/arc/boot/dts/axc001.dtsi:		dw-apb-gpio@0x2000 {
arch/arc/boot/dts/axc001.dtsi:		debug_uart: dw-apb-uart@0x5000 {
arch/arc/boot/dts/axc001.dtsi:	mb_intc: dw-apb-ictl@0xe0012000 {
arch/arc/boot/dts/axc003.dtsi:		dw-apb-gpio@0x2000 {
arch/arc/boot/dts/axc003.dtsi:		debug_uart: dw-apb-uart@0x5000 {
arch/arc/boot/dts/axc003.dtsi:	mb_intc: dw-apb-ictl@0xe0012000 {
arch/arc/boot/dts/axc003_idu.dtsi:		dw-apb-gpio@0x2000 {
arch/arc/boot/dts/axc003_idu.dtsi:		debug_uart: dw-apb-uart@0x5000 {
arch/arc/boot/dts/axc003_idu.dtsi:	mb_intc: dw-apb-ictl@0xe0012000 {
arch/arc/boot/dts/axs10x_mb.dtsi:		ethernet@0x18000 {
arch/arc/boot/dts/axs10x_mb.dtsi:		ehci@0x40000 {
arch/arc/boot/dts/axs10x_mb.dtsi:		ohci@0x60000 {
arch/arc/boot/dts/axs10x_mb.dtsi:		mmc@0x15000 {
arch/arc/boot/dts/axs10x_mb.dtsi:		uart@0x20000 {
arch/arc/boot/dts/axs10x_mb.dtsi:		uart@0x21000 {
arch/arc/boot/dts/axs10x_mb.dtsi:		uart@0x22000 {
arch/arc/boot/dts/axs10x_mb.dtsi:		i2c@0x1d000 {
arch/arc/boot/dts/axs10x_mb.dtsi:		i2c@0x1f000 {
arch/arc/boot/dts/axs10x_mb.dtsi:			eeprom@0x54{
arch/arc/boot/dts/axs10x_mb.dtsi:			eeprom@0x57{
arch/arc/boot/dts/vdk_axc003.dtsi:		debug_uart: dw-apb-uart@0x5000 {
arch/arc/boot/dts/vdk_axc003.dtsi:	mb_intc: dw-apb-ictl@0xe0012000 {
arch/arc/boot/dts/vdk_axc003_idu.dtsi:		debug_uart: dw-apb-uart@0x5000 {
arch/arc/boot/dts/vdk_axc003_idu.dtsi:	mb_intc: dw-apb-ictl@0xe0012000 {
arch/arc/boot/dts/vdk_axs10x_mb.dtsi:		ethernet@0x18000 {
arch/arc/boot/dts/vdk_axs10x_mb.dtsi:		ehci@0x40000 {
arch/arc/boot/dts/vdk_axs10x_mb.dtsi:		uart@0x20000 {
arch/arc/boot/dts/vdk_axs10x_mb.dtsi:		uart@0x21000 {
arch/arc/boot/dts/vdk_axs10x_mb.dtsi:		uart@0x22000 {
arch/arc/boot/dts/vdk_axs10x_mb.dtsi:		mmc@0x15000 {
arch/arc/boot/dts/vdk_axs10x_mb.dtsi:	uio_ev: uio@0xD0000000 {
arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts:						bootloaderenv@0x100000 {
arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts:						bootloaderenvred@0x140000 {
arch/arm/boot/dts/atlas7.dtsi:			pmipc@0x13240000 {
arch/arm/boot/dts/atlas7.dtsi:			gnssmfw@0x18100000 {
arch/arm/boot/dts/atlas7.dtsi:			gpum@0x13000000 {
arch/arm/boot/dts/atlas7.dtsi:			sdr@0x13010000 {
arch/arm/boot/dts/atlas7.dtsi:			memory-controller@0x10800000 {
arch/arm/boot/dts/atlas7.dtsi:			retain@0x188D0000 {
arch/arm/boot/dts/dm8148-evm.dts:		partition@0x80000 {
arch/arm/boot/dts/dm8148-evm.dts:		partition@0x1c0000 {
arch/arm/boot/dts/dm8148-evm.dts:		partition@0x280000 {
arch/arm/boot/dts/dm8148-evm.dts:		partition@0x780000 {
arch/arm/boot/dts/dm8168-evm.dts:		partition@0x80000 {
arch/arm/boot/dts/dm8168-evm.dts:		partition@0x1c0000 {
arch/arm/boot/dts/dm8168-evm.dts:		partition@0x280000 {
arch/arm/boot/dts/dm8168-evm.dts:		partition@0x780000 {
arch/arm/boot/dts/keystone-k2e-netcp.dtsi:		pdsp0@0x2a10000 {
arch/arm/boot/dts/keystone-k2hk-netcp.dtsi:		pdsp0@0x2a10000 {
arch/arm/boot/dts/keystone-k2l-netcp.dtsi:		pdsp0@0x2a10000 {
arch/arm/boot/dts/socfpga.dtsi:		l3regs@0xff800000 {
arch/arm/boot/dts/spear300.dtsi:		shirq: interrupt-controller@0x50000000 {
arch/arm/boot/dts/spear310.dtsi:		shirq: interrupt-controller@0xb4000000 {
arch/arm/boot/dts/spear320.dtsi:		shirq: interrupt-controller@0xb3000000 {
arch/arm64/boot/dts/exynos/exynos5433.dtsi:		sysmmu_scaler_0: sysmmu@0x15040000 {
arch/arm64/boot/dts/exynos/exynos5433.dtsi:		sysmmu_scaler_1: sysmmu@0x15050000 {
arch/xtensa/boot/dts/xtfpga-flash-128m.dtsi:			partition@0x0 {
arch/xtensa/boot/dts/xtfpga-flash-128m.dtsi:			partition@0x6000000 {
arch/xtensa/boot/dts/xtfpga-flash-128m.dtsi:			partition@0x6800000 {
arch/xtensa/boot/dts/xtfpga-flash-128m.dtsi:			partition@0x7fe0000 {
arch/xtensa/boot/dts/xtfpga-flash-16m.dtsi:			partition@0x0 {
arch/xtensa/boot/dts/xtfpga-flash-16m.dtsi:			partition@0x400000 {
arch/xtensa/boot/dts/xtfpga-flash-16m.dtsi:			partition@0xa00000 {
arch/xtensa/boot/dts/xtfpga-flash-16m.dtsi:			partition@0xfe0000 {
arch/xtensa/boot/dts/xtfpga-flash-4m.dtsi:			partition@0x0 {
arch/xtensa/boot/dts/xtfpga-flash-4m.dtsi:			partition@0x3f0000 {