| Message ID | 20180117161717.14589-2-horms+renesas@verge.net.au (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi Simon, Thanks for your patch. On 2018-01-17 17:17:02 +0100, Simon Horman wrote: > Consistently use a single space after a =. > > This patch removes instances where a tab or multiple spaces are used > instead. It also avoids running over 80 columns in width in one of the > lines where whitespace is updated. > > This patch should not introduce any functional change. > > Signed-off-by: Simon Horman <horms+renesas@verge.net.au> Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> > --- > v2 > * Consistently use tabs for indentation as much as possible > --- > arch/arm/boot/dts/r8a7790.dtsi | 75 +++++++++++++++++++++--------------------- > 1 file changed, 38 insertions(+), 37 deletions(-) > > diff --git a/arch/arm/boot/dts/r8a7790.dtsi b/arch/arm/boot/dts/r8a7790.dtsi > index 13926fc7abfa..b4306f7c1bbb 100644 > --- a/arch/arm/boot/dts/r8a7790.dtsi > +++ b/arch/arm/boot/dts/r8a7790.dtsi > @@ -291,9 +291,9 @@ > }; > > thermal: thermal@e61f0000 { > - compatible = "renesas,thermal-r8a7790", > - "renesas,rcar-gen2-thermal", > - "renesas,rcar-thermal"; > + compatible = "renesas,thermal-r8a7790", > + "renesas,rcar-gen2-thermal", > + "renesas,rcar-thermal"; > reg = <0 0xe61f0000 0 0x10>, <0 0xe61f0100 0 0x38>; > interrupts = <GIC_SPI 69 IRQ_TYPE_LEVEL_HIGH>; > clocks = <&cpg CPG_MOD 522>; > @@ -423,20 +423,20 @@ > audma0: dma-controller@ec700000 { > compatible = "renesas,dmac-r8a7790", "renesas,rcar-dmac"; > reg = <0 0xec700000 0 0x10000>; > - interrupts = <GIC_SPI 346 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 320 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 321 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 322 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 323 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 324 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 325 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 326 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 327 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 328 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 329 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 330 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 331 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 332 IRQ_TYPE_LEVEL_HIGH>; > + interrupts = <GIC_SPI 346 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 320 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 321 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 322 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 323 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 324 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 325 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 326 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 327 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 328 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 329 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 330 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 331 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 332 IRQ_TYPE_LEVEL_HIGH>; > interrupt-names = "error", > "ch0", "ch1", "ch2", "ch3", > "ch4", "ch5", "ch6", "ch7", > @@ -453,20 +453,20 @@ > audma1: dma-controller@ec720000 { > compatible = "renesas,dmac-r8a7790", "renesas,rcar-dmac"; > reg = <0 0xec720000 0 0x10000>; > - interrupts = <GIC_SPI 347 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 333 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 334 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 335 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 336 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 337 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 338 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 339 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 340 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 341 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 342 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 343 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 344 IRQ_TYPE_LEVEL_HIGH > - GIC_SPI 345 IRQ_TYPE_LEVEL_HIGH>; > + interrupts = <GIC_SPI 347 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 333 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 334 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 335 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 336 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 337 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 338 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 339 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 340 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 341 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 342 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 343 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 344 IRQ_TYPE_LEVEL_HIGH > + GIC_SPI 345 IRQ_TYPE_LEVEL_HIGH>; > interrupt-names = "error", > "ch0", "ch1", "ch2", "ch3", > "ch4", "ch5", "ch6", "ch7", > @@ -1437,12 +1437,13 @@ > * Single DAI : #sound-dai-cells = <0>; <&rcar_sound>; > * Multi DAI : #sound-dai-cells = <1>; <&rcar_sound N>; > */ > - compatible = "renesas,rcar_sound-r8a7790", "renesas,rcar_sound-gen2"; > - reg = <0 0xec500000 0 0x1000>, /* SCU */ > - <0 0xec5a0000 0 0x100>, /* ADG */ > - <0 0xec540000 0 0x1000>, /* SSIU */ > - <0 0xec541000 0 0x280>, /* SSI */ > - <0 0xec740000 0 0x200>; /* Audio DMAC peri peri*/ > + compatible = "renesas,rcar_sound-r8a7790", > + "renesas,rcar_sound-gen2"; > + reg = <0 0xec500000 0 0x1000>, /* SCU */ > + <0 0xec5a0000 0 0x100>, /* ADG */ > + <0 0xec540000 0 0x1000>, /* SSIU */ > + <0 0xec541000 0 0x280>, /* SSI */ > + <0 0xec740000 0 0x200>; /* Audio DMAC peri peri*/ > reg-names = "scu", "adg", "ssiu", "ssi", "audmapp"; > > clocks = <&cpg CPG_MOD 1005>, > -- > 2.11.0 >
On Wed, Jan 17, 2018 at 5:17 PM, Simon Horman <horms+renesas@verge.net.au> wrote: > Consistently use a single space after a =. > > This patch removes instances where a tab or multiple spaces are used > instead. It also avoids running over 80 columns in width in one of the > lines where whitespace is updated. > > This patch should not introduce any functional change. > > Signed-off-by: Simon Horman <horms+renesas@verge.net.au> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
diff --git a/arch/arm/boot/dts/r8a7790.dtsi b/arch/arm/boot/dts/r8a7790.dtsi index 13926fc7abfa..b4306f7c1bbb 100644 --- a/arch/arm/boot/dts/r8a7790.dtsi +++ b/arch/arm/boot/dts/r8a7790.dtsi @@ -291,9 +291,9 @@ }; thermal: thermal@e61f0000 { - compatible = "renesas,thermal-r8a7790", - "renesas,rcar-gen2-thermal", - "renesas,rcar-thermal"; + compatible = "renesas,thermal-r8a7790", + "renesas,rcar-gen2-thermal", + "renesas,rcar-thermal"; reg = <0 0xe61f0000 0 0x10>, <0 0xe61f0100 0 0x38>; interrupts = <GIC_SPI 69 IRQ_TYPE_LEVEL_HIGH>; clocks = <&cpg CPG_MOD 522>; @@ -423,20 +423,20 @@ audma0: dma-controller@ec700000 { compatible = "renesas,dmac-r8a7790", "renesas,rcar-dmac"; reg = <0 0xec700000 0 0x10000>; - interrupts = <GIC_SPI 346 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 320 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 321 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 322 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 323 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 324 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 325 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 326 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 327 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 328 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 329 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 330 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 331 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 332 IRQ_TYPE_LEVEL_HIGH>; + interrupts = <GIC_SPI 346 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 320 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 321 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 322 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 323 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 324 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 325 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 326 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 327 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 328 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 329 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 330 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 331 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 332 IRQ_TYPE_LEVEL_HIGH>; interrupt-names = "error", "ch0", "ch1", "ch2", "ch3", "ch4", "ch5", "ch6", "ch7", @@ -453,20 +453,20 @@ audma1: dma-controller@ec720000 { compatible = "renesas,dmac-r8a7790", "renesas,rcar-dmac"; reg = <0 0xec720000 0 0x10000>; - interrupts = <GIC_SPI 347 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 333 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 334 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 335 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 336 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 337 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 338 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 339 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 340 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 341 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 342 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 343 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 344 IRQ_TYPE_LEVEL_HIGH - GIC_SPI 345 IRQ_TYPE_LEVEL_HIGH>; + interrupts = <GIC_SPI 347 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 333 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 334 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 335 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 336 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 337 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 338 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 339 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 340 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 341 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 342 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 343 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 344 IRQ_TYPE_LEVEL_HIGH + GIC_SPI 345 IRQ_TYPE_LEVEL_HIGH>; interrupt-names = "error", "ch0", "ch1", "ch2", "ch3", "ch4", "ch5", "ch6", "ch7", @@ -1437,12 +1437,13 @@ * Single DAI : #sound-dai-cells = <0>; <&rcar_sound>; * Multi DAI : #sound-dai-cells = <1>; <&rcar_sound N>; */ - compatible = "renesas,rcar_sound-r8a7790", "renesas,rcar_sound-gen2"; - reg = <0 0xec500000 0 0x1000>, /* SCU */ - <0 0xec5a0000 0 0x100>, /* ADG */ - <0 0xec540000 0 0x1000>, /* SSIU */ - <0 0xec541000 0 0x280>, /* SSI */ - <0 0xec740000 0 0x200>; /* Audio DMAC peri peri*/ + compatible = "renesas,rcar_sound-r8a7790", + "renesas,rcar_sound-gen2"; + reg = <0 0xec500000 0 0x1000>, /* SCU */ + <0 0xec5a0000 0 0x100>, /* ADG */ + <0 0xec540000 0 0x1000>, /* SSIU */ + <0 0xec541000 0 0x280>, /* SSI */ + <0 0xec740000 0 0x200>; /* Audio DMAC peri peri*/ reg-names = "scu", "adg", "ssiu", "ssi", "audmapp"; clocks = <&cpg CPG_MOD 1005>,
Consistently use a single space after a =. This patch removes instances where a tab or multiple spaces are used instead. It also avoids running over 80 columns in width in one of the lines where whitespace is updated. This patch should not introduce any functional change. Signed-off-by: Simon Horman <horms+renesas@verge.net.au> --- v2 * Consistently use tabs for indentation as much as possible --- arch/arm/boot/dts/r8a7790.dtsi | 75 +++++++++++++++++++++--------------------- 1 file changed, 38 insertions(+), 37 deletions(-)