LinuxLists.cc - [PATCH 1/2] arm64: dts: microchip: minor whitespace cleanup around '='

2023-07-02 18:52:30

Subject: [PATCH 1/2] arm64: dts: microchip: minor whitespace cleanup around '='

The DTS code coding style expects exactly one space before and after '='
sign.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
arch/arm64/boot/dts/microchip/sparx5.dtsi | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/arch/arm64/boot/dts/microchip/sparx5.dtsi b/arch/arm64/boot/dts/microchip/sparx5.dtsi
index 4996499cc738..24075cd91420 100644
--- a/arch/arm64/boot/dts/microchip/sparx5.dtsi
+++ b/arch/arm64/boot/dts/microchip/sparx5.dtsi
@@ -469,14 +469,14 @@ serdes: serdes@10808000 {

switch: switch@600000000 {
compatible = "microchip,sparx5-switch";
- reg = <0x6 0 0x401000>,
- <0x6 0x10004000 0x7fc000>,
- <0x6 0x11010000 0xaf0000>;
+ reg = <0x6 0 0x401000>,
+ <0x6 0x10004000 0x7fc000>,
+ <0x6 0x11010000 0xaf0000>;
reg-names = "cpu", "dev", "gcb";
interrupt-names = "xtr", "fdma", "ptp";
- interrupts = <GIC_SPI 30 IRQ_TYPE_LEVEL_HIGH>,
- <GIC_SPI 25 IRQ_TYPE_LEVEL_HIGH>,
- <GIC_SPI 27 IRQ_TYPE_LEVEL_HIGH>;
+ interrupts = <GIC_SPI 30 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 25 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 27 IRQ_TYPE_LEVEL_HIGH>;
resets = <&reset 0>;
reset-names = "switch";
};
--
2.34.1

2023-07-02 18:55:20

by Krzysztof Kozlowski

[permalink] [raw]

Subject: [PATCH 2/2] ARM: dts: microchip: minor whitespace cleanup around '='

The DTS code coding style expects exactly one space before and after '='
sign.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
arch/arm/boot/dts/microchip/lan966x-pcb8290.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/microchip/lan966x-pcb8290.dts b/arch/arm/boot/dts/microchip/lan966x-pcb8290.dts
index 8804e8ba5370..3b7577e48b46 100644
--- a/arch/arm/boot/dts/microchip/lan966x-pcb8290.dts
+++ b/arch/arm/boot/dts/microchip/lan966x-pcb8290.dts
@@ -28,7 +28,7 @@ &aes {
&gpio {
miim_a_pins: mdio-pins {
/* MDC, MDIO */
- pins = "GPIO_28", "GPIO_29";
+ pins = "GPIO_28", "GPIO_29";
function = "miim_a";
};

--
2.34.1

2023-07-03 06:45:25

by Steen Hegelund

[permalink] [raw]

Subject: Re: [PATCH 2/2] ARM: dts: microchip: minor whitespace cleanup around '='

2023-07-03 07:19:39

by Steen Hegelund

[permalink] [raw]

Subject: Re: [PATCH 1/2] arm64: dts: microchip: minor whitespace cleanup around '='

On Sun, 2023-07-02 at 20:51 +0200, Krzysztof Kozlowski wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the
> content is safe
>
> The DTS code coding style expects exactly one space before and after '='
> sign.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> arch/arm64/boot/dts/microchip/sparx5.dtsi | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/microchip/sparx5.dtsi
> b/arch/arm64/boot/dts/microchip/sparx5.dtsi
> index 4996499cc738..24075cd91420 100644
> --- a/arch/arm64/boot/dts/microchip/sparx5.dtsi
> +++ b/arch/arm64/boot/dts/microchip/sparx5.dtsi
> @@ -469,14 +469,14 @@ serdes: serdes@10808000 {
>
>                 switch: switch@600000000 {
>                         compatible = "microchip,sparx5-switch";
> -                       reg =   <0x6 0 0x401000>,
> -                               <0x6 0x10004000 0x7fc000>,
> -                               <0x6 0x11010000 0xaf0000>;
> +                       reg = <0x6 0 0x401000>,
> +                             <0x6 0x10004000 0x7fc000>,
> +                             <0x6 0x11010000 0xaf0000>;
>                         reg-names = "cpu", "dev", "gcb";
>                         interrupt-names = "xtr", "fdma", "ptp";
> -                       interrupts =    <GIC_SPI 30 IRQ_TYPE_LEVEL_HIGH>,
> -                                       <GIC_SPI 25 IRQ_TYPE_LEVEL_HIGH>,
> -                                       <GIC_SPI 27 IRQ_TYPE_LEVEL_HIGH>;
> +                       interrupts = <GIC_SPI 30 IRQ_TYPE_LEVEL_HIGH>,
> +                                    <GIC_SPI 25 IRQ_TYPE_LEVEL_HIGH>,
> +                                    <GIC_SPI 27 IRQ_TYPE_LEVEL_HIGH>;
>                         resets = <&reset 0>;
>                         reset-names = "switch";
>                 };
> --
> 2.34.1
>
Reviewed-by: Steen Hegelund <[email protected]>

BR
Steen

2023-07-10 08:40:18

by Conor Dooley

[permalink] [raw]

Subject: Re: [PATCH 2/2] ARM: dts: microchip: minor whitespace cleanup around '='

On Sun, Jul 02, 2023 at 08:51:08PM +0200, Krzysztof Kozlowski wrote:
> The DTS code coding style expects exactly one space before and after '='
> sign.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

I was speaking to Claudiu on Friday & he didn't see this in his queue,
so I have applied both patches.

Attachments:

(No filename) (337.00 B)
signature.asc (235.00 B)
Download all attachments