Re: [PATCH] ARC: Remove 0x prefix from unit-address of node names

2018-12-10 Thread Alexey Brodkin
Hi Rob,

On Mon, 2018-12-10 at 07:30 -0600, Rob Herring wrote:
> On Sat, Dec 8, 2018 at 1:01 AM Alexey Brodkin
>  wrote:
> > Done automatically with help of:
> > --->8
> > sed -i 's/@0x/@/g' arch/arc/boot/dts/*.dts*
> > --->8
> > 
> > Inspired by [1] and the like.
> > 
> > [1] 
> > https://urldefense.proofpoint.com/v2/url?u=http-3A__kisskb.ellerman.id.au_kisskb_buildresult_13612017_=DwIBaQ=DPL6_X_6JkXFx7AXWqB0tg=lqdeeSSEes0GFDDl656eViXO7breS55ytWkhpk5R81I=zx_mJ5GzAMI32-iqcXZiL8Qzrg79Dz4cxNF72VYhQl0=JwbFcFmnJRECl_z7jQGl0687UpORq9395mgRbo1OwFE=
> > 
> > Signed-off-by: Alexey Brodkin 
> > Cc: Rob Herring 
> > 
> > Signed-off-by: Alexey Brodkin 
> > ---
> >  arch/arc/boot/dts/abilis_tb10x.dtsi   |  4 ++--
> >  arch/arc/boot/dts/axc001.dtsi |  6 +++---
> >  arch/arc/boot/dts/axc003.dtsi | 14 +++---
> >  arch/arc/boot/dts/axc003_idu.dtsi | 14 +++---
> >  arch/arc/boot/dts/axs10x_mb.dtsi  | 22 +++---
> >  arch/arc/boot/dts/vdk_axc003.dtsi |  4 ++--
> >  arch/arc/boot/dts/vdk_axc003_idu.dtsi |  4 ++--
> >  arch/arc/boot/dts/vdk_axs10x_mb.dtsi  | 14 +++---
> >  8 files changed, 41 insertions(+), 41 deletions(-)
> > 
> > diff --git a/arch/arc/boot/dts/abilis_tb10x.dtsi 
> > b/arch/arc/boot/dts/abilis_tb10x.dtsi
> > index 3121536b25a3..77ac1368a65b 100644
> > --- a/arch/arc/boot/dts/abilis_tb10x.dtsi
> > +++ b/arch/arc/boot/dts/abilis_tb10x.dtsi
> > @@ -178,7 +178,7 @@
> > clocks = <_clk>;
> > };
> > 
> > -   spi0: spi@0xFE01 {
> > +   spi0: spi@FE01 {
> 
> This should be lowercase too.
> 
> > #address-cells = <1>;
> > #size-cells = <0>;
> > cell-index = <0>;
> 
> [...]
> 
> > @@ -119,7 +119,7 @@
> >  * This node is intentionally put outside of MB above becase
> >  * it maps areas outside of MB's 0xEz-0xFz.
> >  */
> > -   uio_ev: uio@0xD000 {
> > +   uio_ev: uio@D000 {
> 
> lowercase
> 
> > compatible = "generic-uio";
> > reg = <0xD000 0x2000 0xD100 0x2000 0x9000 
> > 0x1000 0xC000 0x1000>;

I guess real addresses should be lower-case too, right?

-Alexey
___
linux-snps-arc mailing list
linux-snps-arc@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-snps-arc


Re: [PATCH] ARC: Remove 0x prefix from unit-address of node names

2018-12-10 Thread Rob Herring
On Sat, Dec 8, 2018 at 1:01 AM Alexey Brodkin
 wrote:
>
> Done automatically with help of:
> --->8
> sed -i 's/@0x/@/g' arch/arc/boot/dts/*.dts*
> --->8
>
> Inspired by [1] and the like.
>
> [1] http://kisskb.ellerman.id.au/kisskb/buildresult/13612017/
>
> Signed-off-by: Alexey Brodkin 
> Cc: Rob Herring 
>
> Signed-off-by: Alexey Brodkin 
> ---
>  arch/arc/boot/dts/abilis_tb10x.dtsi   |  4 ++--
>  arch/arc/boot/dts/axc001.dtsi |  6 +++---
>  arch/arc/boot/dts/axc003.dtsi | 14 +++---
>  arch/arc/boot/dts/axc003_idu.dtsi | 14 +++---
>  arch/arc/boot/dts/axs10x_mb.dtsi  | 22 +++---
>  arch/arc/boot/dts/vdk_axc003.dtsi |  4 ++--
>  arch/arc/boot/dts/vdk_axc003_idu.dtsi |  4 ++--
>  arch/arc/boot/dts/vdk_axs10x_mb.dtsi  | 14 +++---
>  8 files changed, 41 insertions(+), 41 deletions(-)
>
> diff --git a/arch/arc/boot/dts/abilis_tb10x.dtsi 
> b/arch/arc/boot/dts/abilis_tb10x.dtsi
> index 3121536b25a3..77ac1368a65b 100644
> --- a/arch/arc/boot/dts/abilis_tb10x.dtsi
> +++ b/arch/arc/boot/dts/abilis_tb10x.dtsi
> @@ -178,7 +178,7 @@
> clocks = <_clk>;
> };
>
> -   spi0: spi@0xFE01 {
> +   spi0: spi@FE01 {

This should be lowercase too.

> #address-cells = <1>;
> #size-cells = <0>;
> cell-index = <0>;

[...]

> @@ -119,7 +119,7 @@
>  * This node is intentionally put outside of MB above becase
>  * it maps areas outside of MB's 0xEz-0xFz.
>  */
> -   uio_ev: uio@0xD000 {
> +   uio_ev: uio@D000 {

lowercase

> compatible = "generic-uio";
> reg = <0xD000 0x2000 0xD100 0x2000 0x9000 
> 0x1000 0xC000 0x1000>;
> reg-names = "ev_gsa", "ev_ctrl", "ev_shared_mem", 
> "ev_code_mem";
> --
> 2.16.2
>

___
linux-snps-arc mailing list
linux-snps-arc@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-snps-arc


[PATCH] ARC: Remove 0x prefix from unit-address of node names

2018-12-07 Thread Alexey Brodkin
Done automatically with help of:
--->8
sed -i 's/@0x/@/g' arch/arc/boot/dts/*.dts*
--->8

Inspired by [1] and the like.

[1] http://kisskb.ellerman.id.au/kisskb/buildresult/13612017/

Signed-off-by: Alexey Brodkin 
Cc: Rob Herring 

Signed-off-by: Alexey Brodkin 
---
 arch/arc/boot/dts/abilis_tb10x.dtsi   |  4 ++--
 arch/arc/boot/dts/axc001.dtsi |  6 +++---
 arch/arc/boot/dts/axc003.dtsi | 14 +++---
 arch/arc/boot/dts/axc003_idu.dtsi | 14 +++---
 arch/arc/boot/dts/axs10x_mb.dtsi  | 22 +++---
 arch/arc/boot/dts/vdk_axc003.dtsi |  4 ++--
 arch/arc/boot/dts/vdk_axc003_idu.dtsi |  4 ++--
 arch/arc/boot/dts/vdk_axs10x_mb.dtsi  | 14 +++---
 8 files changed, 41 insertions(+), 41 deletions(-)

diff --git a/arch/arc/boot/dts/abilis_tb10x.dtsi 
b/arch/arc/boot/dts/abilis_tb10x.dtsi
index 3121536b25a3..77ac1368a65b 100644
--- a/arch/arc/boot/dts/abilis_tb10x.dtsi
+++ b/arch/arc/boot/dts/abilis_tb10x.dtsi
@@ -178,7 +178,7 @@
clocks = <_clk>;
};
 
-   spi0: spi@0xFE01 {
+   spi0: spi@FE01 {
#address-cells = <1>;
#size-cells = <0>;
cell-index = <0>;
@@ -189,7 +189,7 @@
interrupts = <26 8>;
clocks = <_clk>;
};
-   spi1: spi@0xFE011000 {
+   spi1: spi@FE011000 {
#address-cells = <1>;
#size-cells = <0>;
cell-index = <1>;
diff --git a/arch/arc/boot/dts/axc001.dtsi b/arch/arc/boot/dts/axc001.dtsi
index fdc266504ada..37be3bf03ad6 100644
--- a/arch/arc/boot/dts/axc001.dtsi
+++ b/arch/arc/boot/dts/axc001.dtsi
@@ -41,7 +41,7 @@
 * this GPIO block ORs all interrupts on CPU card (creg,..)
 * to uplink only 1 IRQ to ARC core intc
 */
-   dw-apb-gpio@0x2000 {
+   dw-apb-gpio@2000 {
compatible = "snps,dw-apb-gpio";
reg = < 0x2000 0x80 >;
#address-cells = <1>;
@@ -60,7 +60,7 @@
};
};
 
-   debug_uart: dw-apb-uart@0x5000 {
+   debug_uart: dw-apb-uart@5000 {
compatible = "snps,dw-apb-uart";
reg = <0x5000 0x100>;
clock-frequency = <3000>;
@@ -88,7 +88,7 @@
 * avoid duplicating the MB dtsi file given that IRQ from
 * this intc to cpu intc are different for axs101 and axs103
 */
-   mb_intc: dw-apb-ictl@0xe0012000 {
+   mb_intc: dw-apb-ictl@e0012000 {
#interrupt-cells = <1>;
compatible = "snps,dw-apb-ictl";
reg = < 0x0 0xe0012000 0x0 0x200 >;
diff --git a/arch/arc/boot/dts/axc003.dtsi b/arch/arc/boot/dts/axc003.dtsi
index d75d65ddf8e3..ffa208a57a0a 100644
--- a/arch/arc/boot/dts/axc003.dtsi
+++ b/arch/arc/boot/dts/axc003.dtsi
@@ -55,7 +55,7 @@
 * this GPIO block ORs all interrupts on CPU card (creg,..)
 * to uplink only 1 IRQ to ARC core intc
 */
-   dw-apb-gpio@0x2000 {
+   dw-apb-gpio@2000 {
compatible = "snps,dw-apb-gpio";
reg = < 0x2000 0x80 >;
#address-cells = <1>;
@@ -74,7 +74,7 @@
};
};
 
-   debug_uart: dw-apb-uart@0x5000 {
+   debug_uart: dw-apb-uart@5000 {
compatible = "snps,dw-apb-uart";
reg = <0x5000 0x100>;
clock-frequency = <3000>;
@@ -102,19 +102,19 @@
 * external DMA buffer located outside of IOC aperture.
 */
axs10x_mb {
-   ethernet@0x18000 {
+   ethernet@18000 {
dma-coherent;
};
 
-   ehci@0x4 {
+   ehci@4 {
dma-coherent;
};
 
-   ohci@0x6 {
+   ohci@6 {
dma-coherent;
};
 
-   mmc@0x15000 {
+   mmc@15000 {
dma-coherent;
};
};
@@ -132,7 +132,7 @@
 * avoid duplicating the MB dtsi file given that IRQ from
 * this intc to cpu intc are different for axs101 and axs103
 */
-   mb_intc: dw-apb-ictl@0xe0012000 {
+   mb_intc: dw-apb-ictl@e0012000 {
#interrupt-cells = <1>;
compatible = "snps,dw-apb-ictl";
reg = < 0x0 0xe0012000 0x0 0x200 >;
diff --git a/arch/arc/boot/dts/axc003_idu.dtsi 
b/arch/arc/boot/dts/axc003_idu.dtsi
index a05bb737ea63..ad795748ac63 100644
---