2024-02-08 10:56:48

by Krzysztof Kozlowski

Subject: [PATCH] arm64: dts: exynos: gs101: minor whitespace cleanup

The DTS code coding style expects exactly one space before '{' and
around '=' characters.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
.../boot/dts/exynos/google/gs101-pinctrl.dtsi | 2 +-
arch/arm64/boot/dts/exynos/google/gs101.dtsi | 16 ++++++++--------
2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/arch/arm64/boot/dts/exynos/google/gs101-pinctrl.dtsi b/arch/arm64/boot/dts/exynos/google/gs101-pinctrl.dtsi
index e6a9776d4d62..a675f822acec 100644
--- a/arch/arm64/boot/dts/exynos/google/gs101-pinctrl.dtsi
+++ b/arch/arm64/boot/dts/exynos/google/gs101-pinctrl.dtsi
@@ -251,7 +251,7 @@ gph1: gph1-gpio-bank {
#interrupt-cells = <2>;
};

- pcie0_clkreq: pcie0-clkreq-pins{
+ pcie0_clkreq: pcie0-clkreq-pins {
samsung,pins = "gph0-1";
samsung,pin-function = <GS101_PIN_FUNC_2>;
samsung,pin-pud = <GS101_PIN_PULL_UP>;
diff --git a/arch/arm64/boot/dts/exynos/google/gs101.dtsi b/arch/arm64/boot/dts/exynos/google/gs101.dtsi
index c7a9e81d23b9..80cc933cca3d 100644
--- a/arch/arm64/boot/dts/exynos/google/gs101.dtsi
+++ b/arch/arm64/boot/dts/exynos/google/gs101.dtsi
@@ -73,7 +73,7 @@ cpu0: cpu@0 {
compatible = "arm,cortex-a55";
reg = <0x0000>;
enable-method = "psci";
- cpu-idle-states = <&ANANKE_CPU_SLEEP>;
+ cpu-idle-states = <&ANANKE_CPU_SLEEP>;
capacity-dmips-mhz = <250>;
dynamic-power-coefficient = <70>;
};
@@ -83,7 +83,7 @@ cpu1: cpu@100 {
compatible = "arm,cortex-a55";
reg = <0x0100>;
enable-method = "psci";
- cpu-idle-states = <&ANANKE_CPU_SLEEP>;
+ cpu-idle-states = <&ANANKE_CPU_SLEEP>;
capacity-dmips-mhz = <250>;
dynamic-power-coefficient = <70>;
};
@@ -93,7 +93,7 @@ cpu2: cpu@200 {
compatible = "arm,cortex-a55";
reg = <0x0200>;
enable-method = "psci";
- cpu-idle-states = <&ANANKE_CPU_SLEEP>;
+ cpu-idle-states = <&ANANKE_CPU_SLEEP>;
capacity-dmips-mhz = <250>;
dynamic-power-coefficient = <70>;
};
@@ -103,7 +103,7 @@ cpu3: cpu@300 {
compatible = "arm,cortex-a55";
reg = <0x0300>;
enable-method = "psci";
- cpu-idle-states = <&ANANKE_CPU_SLEEP>;
+ cpu-idle-states = <&ANANKE_CPU_SLEEP>;
capacity-dmips-mhz = <250>;
dynamic-power-coefficient = <70>;
};
@@ -113,7 +113,7 @@ cpu4: cpu@400 {
compatible = "arm,cortex-a76";
reg = <0x0400>;
enable-method = "psci";
- cpu-idle-states = <&ENYO_CPU_SLEEP>;
+ cpu-idle-states = <&ENYO_CPU_SLEEP>;
capacity-dmips-mhz = <620>;
dynamic-power-coefficient = <284>;
};
@@ -123,7 +123,7 @@ cpu5: cpu@500 {
compatible = "arm,cortex-a76";
reg = <0x0500>;
enable-method = "psci";
- cpu-idle-states = <&ENYO_CPU_SLEEP>;
+ cpu-idle-states = <&ENYO_CPU_SLEEP>;
capacity-dmips-mhz = <620>;
dynamic-power-coefficient = <284>;
};
@@ -133,7 +133,7 @@ cpu6: cpu@600 {
compatible = "arm,cortex-x1";
reg = <0x0600>;
enable-method = "psci";
- cpu-idle-states = <&HERA_CPU_SLEEP>;
+ cpu-idle-states = <&HERA_CPU_SLEEP>;
capacity-dmips-mhz = <1024>;
dynamic-power-coefficient = <650>;
};
@@ -143,7 +143,7 @@ cpu7: cpu@700 {
compatible = "arm,cortex-x1";
reg = <0x0700>;
enable-method = "psci";
- cpu-idle-states = <&HERA_CPU_SLEEP>;
+ cpu-idle-states = <&HERA_CPU_SLEEP>;
capacity-dmips-mhz = <1024>;
dynamic-power-coefficient = <650>;
};
--
2.34.1

2024-02-08 11:15:03

by Peter Griffin

Subject: Re: [PATCH] arm64: dts: exynos: gs101: minor whitespace cleanup

On Thu, 8 Feb 2024 at 10:52, Krzysztof Kozlowski
<[email protected]> wrote:
>
> The DTS code coding style expects exactly one space before '{' and
> around '=' characters.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---

Reviewed-by: Peter Griffin <[email protected]>

> .../boot/dts/exynos/google/gs101-pinctrl.dtsi | 2 +-
> arch/arm64/boot/dts/exynos/google/gs101.dtsi | 16 ++++++++--------
> 2 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/exynos/google/gs101-pinctrl.dtsi b/arch/arm64/boot/dts/exynos/google/gs101-pinctrl.dtsi
> index e6a9776d4d62..a675f822acec 100644
> --- a/arch/arm64/boot/dts/exynos/google/gs101-pinctrl.dtsi
> +++ b/arch/arm64/boot/dts/exynos/google/gs101-pinctrl.dtsi
> @@ -251,7 +251,7 @@ gph1: gph1-gpio-bank {
> #interrupt-cells = <2>;
> };
>
> - pcie0_clkreq: pcie0-clkreq-pins{
> + pcie0_clkreq: pcie0-clkreq-pins {
> samsung,pins = "gph0-1";
> samsung,pin-function = <GS101_PIN_FUNC_2>;
> samsung,pin-pud = <GS101_PIN_PULL_UP>;
> diff --git a/arch/arm64/boot/dts/exynos/google/gs101.dtsi b/arch/arm64/boot/dts/exynos/google/gs101.dtsi
> index c7a9e81d23b9..80cc933cca3d 100644
> --- a/arch/arm64/boot/dts/exynos/google/gs101.dtsi
> +++ b/arch/arm64/boot/dts/exynos/google/gs101.dtsi
> @@ -73,7 +73,7 @@ cpu0: cpu@0 {
> compatible = "arm,cortex-a55";
> reg = <0x0000>;
> enable-method = "psci";
> - cpu-idle-states = <&ANANKE_CPU_SLEEP>;
> + cpu-idle-states = <&ANANKE_CPU_SLEEP>;
> capacity-dmips-mhz = <250>;
> dynamic-power-coefficient = <70>;
> };
> @@ -83,7 +83,7 @@ cpu1: cpu@100 {
> compatible = "arm,cortex-a55";
> reg = <0x0100>;
> enable-method = "psci";
> - cpu-idle-states = <&ANANKE_CPU_SLEEP>;
> + cpu-idle-states = <&ANANKE_CPU_SLEEP>;
> capacity-dmips-mhz = <250>;
> dynamic-power-coefficient = <70>;
> };
> @@ -93,7 +93,7 @@ cpu2: cpu@200 {
> compatible = "arm,cortex-a55";
> reg = <0x0200>;
> enable-method = "psci";
> - cpu-idle-states = <&ANANKE_CPU_SLEEP>;
> + cpu-idle-states = <&ANANKE_CPU_SLEEP>;
> capacity-dmips-mhz = <250>;
> dynamic-power-coefficient = <70>;
> };
> @@ -103,7 +103,7 @@ cpu3: cpu@300 {
> compatible = "arm,cortex-a55";
> reg = <0x0300>;
> enable-method = "psci";
> - cpu-idle-states = <&ANANKE_CPU_SLEEP>;
> + cpu-idle-states = <&ANANKE_CPU_SLEEP>;
> capacity-dmips-mhz = <250>;
> dynamic-power-coefficient = <70>;
> };
> @@ -113,7 +113,7 @@ cpu4: cpu@400 {
> compatible = "arm,cortex-a76";
> reg = <0x0400>;
> enable-method = "psci";
> - cpu-idle-states = <&ENYO_CPU_SLEEP>;
> + cpu-idle-states = <&ENYO_CPU_SLEEP>;
> capacity-dmips-mhz = <620>;
> dynamic-power-coefficient = <284>;
> };
> @@ -123,7 +123,7 @@ cpu5: cpu@500 {
> compatible = "arm,cortex-a76";
> reg = <0x0500>;
> enable-method = "psci";
> - cpu-idle-states = <&ENYO_CPU_SLEEP>;
> + cpu-idle-states = <&ENYO_CPU_SLEEP>;
> capacity-dmips-mhz = <620>;
> dynamic-power-coefficient = <284>;
> };
> @@ -133,7 +133,7 @@ cpu6: cpu@600 {
> compatible = "arm,cortex-x1";
> reg = <0x0600>;
> enable-method = "psci";
> - cpu-idle-states = <&HERA_CPU_SLEEP>;
> + cpu-idle-states = <&HERA_CPU_SLEEP>;
> capacity-dmips-mhz = <1024>;
> dynamic-power-coefficient = <650>;
> };
> @@ -143,7 +143,7 @@ cpu7: cpu@700 {
> compatible = "arm,cortex-x1";
> reg = <0x0700>;
> enable-method = "psci";
> - cpu-idle-states = <&HERA_CPU_SLEEP>;
> + cpu-idle-states = <&HERA_CPU_SLEEP>;
> capacity-dmips-mhz = <1024>;
> dynamic-power-coefficient = <650>;
> };
> --
> 2.34.1
>

2024-02-08 12:41:35

by Sam Protsenko

Subject: Re: [PATCH] arm64: dts: exynos: gs101: minor whitespace cleanup

On Thu, Feb 8, 2024 at 4:55 AM Krzysztof Kozlowski
<[email protected]> wrote:
>
> The DTS code coding style expects exactly one space before '{' and
> around '=' characters.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---

Reviewed-by: Sam Protsenko <[email protected]>

> .../boot/dts/exynos/google/gs101-pinctrl.dtsi | 2 +-
> arch/arm64/boot/dts/exynos/google/gs101.dtsi | 16 ++++++++--------
> 2 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/exynos/google/gs101-pinctrl.dtsi b/arch/arm64/boot/dts/exynos/google/gs101-pinctrl.dtsi
> index e6a9776d4d62..a675f822acec 100644
> --- a/arch/arm64/boot/dts/exynos/google/gs101-pinctrl.dtsi
> +++ b/arch/arm64/boot/dts/exynos/google/gs101-pinctrl.dtsi
> @@ -251,7 +251,7 @@ gph1: gph1-gpio-bank {
> #interrupt-cells = <2>;
> };
>
> - pcie0_clkreq: pcie0-clkreq-pins{
> + pcie0_clkreq: pcie0-clkreq-pins {
> samsung,pins = "gph0-1";
> samsung,pin-function = <GS101_PIN_FUNC_2>;
> samsung,pin-pud = <GS101_PIN_PULL_UP>;
> diff --git a/arch/arm64/boot/dts/exynos/google/gs101.dtsi b/arch/arm64/boot/dts/exynos/google/gs101.dtsi
> index c7a9e81d23b9..80cc933cca3d 100644
> --- a/arch/arm64/boot/dts/exynos/google/gs101.dtsi
> +++ b/arch/arm64/boot/dts/exynos/google/gs101.dtsi
> @@ -73,7 +73,7 @@ cpu0: cpu@0 {
> compatible = "arm,cortex-a55";
> reg = <0x0000>;
> enable-method = "psci";
> - cpu-idle-states = <&ANANKE_CPU_SLEEP>;
> + cpu-idle-states = <&ANANKE_CPU_SLEEP>;
> capacity-dmips-mhz = <250>;
> dynamic-power-coefficient = <70>;
> };
> @@ -83,7 +83,7 @@ cpu1: cpu@100 {
> compatible = "arm,cortex-a55";
> reg = <0x0100>;
> enable-method = "psci";
> - cpu-idle-states = <&ANANKE_CPU_SLEEP>;
> + cpu-idle-states = <&ANANKE_CPU_SLEEP>;
> capacity-dmips-mhz = <250>;
> dynamic-power-coefficient = <70>;
> };
> @@ -93,7 +93,7 @@ cpu2: cpu@200 {
> compatible = "arm,cortex-a55";
> reg = <0x0200>;
> enable-method = "psci";
> - cpu-idle-states = <&ANANKE_CPU_SLEEP>;
> + cpu-idle-states = <&ANANKE_CPU_SLEEP>;
> capacity-dmips-mhz = <250>;
> dynamic-power-coefficient = <70>;
> };
> @@ -103,7 +103,7 @@ cpu3: cpu@300 {
> compatible = "arm,cortex-a55";
> reg = <0x0300>;
> enable-method = "psci";
> - cpu-idle-states = <&ANANKE_CPU_SLEEP>;
> + cpu-idle-states = <&ANANKE_CPU_SLEEP>;
> capacity-dmips-mhz = <250>;
> dynamic-power-coefficient = <70>;
> };
> @@ -113,7 +113,7 @@ cpu4: cpu@400 {
> compatible = "arm,cortex-a76";
> reg = <0x0400>;
> enable-method = "psci";
> - cpu-idle-states = <&ENYO_CPU_SLEEP>;
> + cpu-idle-states = <&ENYO_CPU_SLEEP>;
> capacity-dmips-mhz = <620>;
> dynamic-power-coefficient = <284>;
> };
> @@ -123,7 +123,7 @@ cpu5: cpu@500 {
> compatible = "arm,cortex-a76";
> reg = <0x0500>;
> enable-method = "psci";
> - cpu-idle-states = <&ENYO_CPU_SLEEP>;
> + cpu-idle-states = <&ENYO_CPU_SLEEP>;
> capacity-dmips-mhz = <620>;
> dynamic-power-coefficient = <284>;
> };
> @@ -133,7 +133,7 @@ cpu6: cpu@600 {
> compatible = "arm,cortex-x1";
> reg = <0x0600>;
> enable-method = "psci";
> - cpu-idle-states = <&HERA_CPU_SLEEP>;
> + cpu-idle-states = <&HERA_CPU_SLEEP>;
> capacity-dmips-mhz = <1024>;
> dynamic-power-coefficient = <650>;
> };
> @@ -143,7 +143,7 @@ cpu7: cpu@700 {
> compatible = "arm,cortex-x1";
> reg = <0x0700>;
> enable-method = "psci";
> - cpu-idle-states = <&HERA_CPU_SLEEP>;
> + cpu-idle-states = <&HERA_CPU_SLEEP>;
> capacity-dmips-mhz = <1024>;
> dynamic-power-coefficient = <650>;
> };
> --
> 2.34.1
>
>

2024-02-09 07:29:45

by Krzysztof Kozlowski

Subject: Re: [PATCH] arm64: dts: exynos: gs101: minor whitespace cleanup

On Thu, 08 Feb 2024 11:52:43 +0100, Krzysztof Kozlowski wrote:
> The DTS code coding style expects exactly one space before '{' and
> around '=' characters.
>
>

Applied, thanks!

[1/1] arm64: dts: exynos: gs101: minor whitespace cleanup
https://git.kernel.org/krzk/linux/c/28fafd554d233ef2344ee4a8709175b69bc93787

Best regards,
--
Krzysztof Kozlowski <[email protected]>