2024-04-11 07:35:55

by Mihai Sain

[permalink] [raw]
Subject: [PATCH 03/13] ARM: dts: microchip: sama5d2: Remove the empty lines 47 and 63

Remove the empty line 47 from etb node.
Remove the empty line 63 from etm node.

Signed-off-by: Mihai Sain <[email protected]>
---
arch/arm/boot/dts/microchip/sama5d2.dtsi | 2 --
1 file changed, 2 deletions(-)

diff --git a/arch/arm/boot/dts/microchip/sama5d2.dtsi b/arch/arm/boot/dts/microchip/sama5d2.dtsi
index 5f8e297e19ed..7aea6c2368a2 100644
--- a/arch/arm/boot/dts/microchip/sama5d2.dtsi
+++ b/arch/arm/boot/dts/microchip/sama5d2.dtsi
@@ -44,7 +44,6 @@ pmu {
etb@740000 {
compatible = "arm,coresight-etb10", "arm,primecell";
reg = <0x740000 0x1000>;
-
clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
clock-names = "apb_pclk";

@@ -60,7 +59,6 @@ etb_in: endpoint {
etm@73c000 {
compatible = "arm,coresight-etm3x", "arm,primecell";
reg = <0x73c000 0x1000>;
-
clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
clock-names = "apb_pclk";

--
2.44.0



2024-04-11 08:08:04

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 03/13] ARM: dts: microchip: sama5d2: Remove the empty lines 47 and 63

On 11/04/2024 09:33, Mihai Sain wrote:
> Remove the empty line 47 from etb node.
> Remove the empty line 63 from etm node.
>
> Signed-off-by: Mihai Sain <[email protected]>
> ---
> arch/arm/boot/dts/microchip/sama5d2.dtsi | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/microchip/sama5d2.dtsi b/arch/arm/boot/dts/microchip/sama5d2.dtsi
> index 5f8e297e19ed..7aea6c2368a2 100644
> --- a/arch/arm/boot/dts/microchip/sama5d2.dtsi
> +++ b/arch/arm/boot/dts/microchip/sama5d2.dtsi
> @@ -44,7 +44,6 @@ pmu {
> etb@740000 {
> compatible = "arm,coresight-etb10", "arm,primecell";
> reg = <0x740000 0x1000>;
> -

Do not remove line by line in one patch. This is trivial. One patch per
all your boards doing such simple cleanuops.

Your subject already shows the ridiculousness of this split.

Best regards,
Krzysztof