mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH 1/2] arm64: dts: microchip: minor whitespace cleanup around '='
@ 2023-07-02 18:51 Krzysztof Kozlowski
  2023-07-02 18:51 ` [PATCH 2/2] ARM: " Krzysztof Kozlowski
  2023-07-03  6:38 ` [PATCH 1/2] arm64: " Steen Hegelund
  0 siblings, 2 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2023-07-02 18:51 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Nicolas Ferre,
	Claudiu Beznea, Lars Povlsen, Steen Hegelund, Daniel Machon,
	UNGLinuxDriver, devicetree, linux-kernel, linux-arm-kernel
  Cc: Krzysztof Kozlowski

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

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 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


^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH 2/2] ARM: dts: microchip: minor whitespace cleanup around '='
  2023-07-02 18:51 [PATCH 1/2] arm64: dts: microchip: minor whitespace cleanup around '=' Krzysztof Kozlowski
@ 2023-07-02 18:51 ` Krzysztof Kozlowski
  2023-07-03  6:39   ` Steen Hegelund
  2023-07-10  8:35   ` Conor Dooley
  2023-07-03  6:38 ` [PATCH 1/2] arm64: " Steen Hegelund
  1 sibling, 2 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2023-07-02 18:51 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Nicolas Ferre,
	Claudiu Beznea, Lars Povlsen, Steen Hegelund, Daniel Machon,
	UNGLinuxDriver, devicetree, linux-kernel, linux-arm-kernel
  Cc: Krzysztof Kozlowski

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

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 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


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 1/2] arm64: dts: microchip: minor whitespace cleanup around '='
  2023-07-02 18:51 [PATCH 1/2] arm64: dts: microchip: minor whitespace cleanup around '=' Krzysztof Kozlowski
  2023-07-02 18:51 ` [PATCH 2/2] ARM: " Krzysztof Kozlowski
@ 2023-07-03  6:38 ` Steen Hegelund
  1 sibling, 0 replies; 5+ messages in thread
From: Steen Hegelund @ 2023-07-03  6:38 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Nicolas Ferre, Claudiu Beznea, Lars Povlsen,
	Daniel Machon, UNGLinuxDriver, devicetree, linux-kernel,
	linux-arm-kernel

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 <krzysztof.kozlowski@linaro.org>
> ---
>  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 <Steen.Hegelund@microchip.com>

BR
Steen

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 2/2] ARM: dts: microchip: minor whitespace cleanup around '='
  2023-07-02 18:51 ` [PATCH 2/2] ARM: " Krzysztof Kozlowski
@ 2023-07-03  6:39   ` Steen Hegelund
  2023-07-10  8:35   ` Conor Dooley
  1 sibling, 0 replies; 5+ messages in thread
From: Steen Hegelund @ 2023-07-03  6:39 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Nicolas Ferre, Claudiu Beznea, Lars Povlsen,
	Daniel Machon, UNGLinuxDriver, devicetree, linux-kernel,
	linux-arm-kernel

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 <krzysztof.kozlowski@linaro.org>
> ---
>  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
> 
Reviewed-by: Steen Hegelund <Steen.Hegelund@microchip.com>

BR
Steen

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 2/2] ARM: dts: microchip: minor whitespace cleanup around '='
  2023-07-02 18:51 ` [PATCH 2/2] ARM: " Krzysztof Kozlowski
  2023-07-03  6:39   ` Steen Hegelund
@ 2023-07-10  8:35   ` Conor Dooley
  1 sibling, 0 replies; 5+ messages in thread
From: Conor Dooley @ 2023-07-10  8:35 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Nicolas Ferre,
	Claudiu Beznea, Lars Povlsen, Steen Hegelund, Daniel Machon,
	UNGLinuxDriver, devicetree, linux-kernel, linux-arm-kernel

[-- Attachment #1: Type: text/plain, Size: 337 bytes --]

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 <krzysztof.kozlowski@linaro.org>

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

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2023-07-10  8:36 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-07-02 18:51 [PATCH 1/2] arm64: dts: microchip: minor whitespace cleanup around '=' Krzysztof Kozlowski
2023-07-02 18:51 ` [PATCH 2/2] ARM: " Krzysztof Kozlowski
2023-07-03  6:39   ` Steen Hegelund
2023-07-10  8:35   ` Conor Dooley
2023-07-03  6:38 ` [PATCH 1/2] arm64: " Steen Hegelund

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®