public inbox for linux-clk@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH v1 0/5] ARM: arm64: dts/clk: imx: indentation permission whitespace cleanup 2
@ 2022-08-26 19:22 Marcel Ziswiler
  2022-08-26 19:22 ` [PATCH v1 4/5] clk: imx6sll: add proper spdx license identifier Marcel Ziswiler
  2022-08-26 19:22 ` [PATCH v1 5/5] dt-bindings: imx8ulp: clock: no spaces before tabs Marcel Ziswiler
  0 siblings, 2 replies; 9+ messages in thread
From: Marcel Ziswiler @ 2022-08-26 19:22 UTC (permalink / raw)
  To: linux-arm-kernel
  Cc: Marcel Ziswiler, Abel Vesa, Dong Aisheng, Fabio Estevam,
	Jacky Bai, Krzysztof Kozlowski, Michael Turquette, NXP Linux Team,
	Peng Fan, Pengutronix Kernel Team, Rob Herring, Sascha Hauer,
	Shawn Guo, Stephen Boyd, devicetree, linux-clk, linux-kernel

From: Marcel Ziswiler <marcel.ziswiler@toradex.com>


While further synchronising them imx device trees with U-Boot I stumbled
over more checkpatch warnings. This series addresses those trivial
indentation, file permission and/or whitespace cleanups.


Marcel Ziswiler (5):
  ARM: dts: imx6sl: use tabs for code indent
  ARM: dts: imx6sx-udoo-neo: don't use multiple blank lines
  arm64: dts: imx8ulp: no executable source file permission
  clk: imx6sll: add proper spdx license identifier
  dt-bindings: imx8ulp: clock: no spaces before tabs

 arch/arm/boot/dts/imx6sl.dtsi                 | 20 +++++++++----------
 arch/arm/boot/dts/imx6sx-udoo-neo.dtsi        | 14 ++++++-------
 .../boot/dts/freescale/imx8ulp-pinfunc.h      |  0
 include/dt-bindings/clock/imx6sll-clock.h     |  2 +-
 include/dt-bindings/clock/imx8ulp-clock.h     |  4 ++--
 5 files changed, 19 insertions(+), 21 deletions(-)
 mode change 100755 => 100644 arch/arm64/boot/dts/freescale/imx8ulp-pinfunc.h

-- 
2.36.1


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

* [PATCH v1 4/5] clk: imx6sll: add proper spdx license identifier
  2022-08-26 19:22 [PATCH v1 0/5] ARM: arm64: dts/clk: imx: indentation permission whitespace cleanup 2 Marcel Ziswiler
@ 2022-08-26 19:22 ` Marcel Ziswiler
  2022-08-29  9:40   ` Abel Vesa
                     ` (2 more replies)
  2022-08-26 19:22 ` [PATCH v1 5/5] dt-bindings: imx8ulp: clock: no spaces before tabs Marcel Ziswiler
  1 sibling, 3 replies; 9+ messages in thread
From: Marcel Ziswiler @ 2022-08-26 19:22 UTC (permalink / raw)
  To: linux-arm-kernel
  Cc: Marcel Ziswiler, Abel Vesa, Fabio Estevam, Krzysztof Kozlowski,
	Michael Turquette, NXP Linux Team, Pengutronix Kernel Team,
	Rob Herring, Sascha Hauer, Shawn Guo, Stephen Boyd, devicetree,
	linux-clk, linux-kernel

From: Marcel Ziswiler <marcel.ziswiler@toradex.com>

This fixes the following error:

include/dt-bindings/clock/imx6sll-clock.h:1: warning: Improper SPDX
 comment style for 'include/dt-bindings/clock/imx6sll-clock.h', please
 use '/*' instead
include/dt-bindings/clock/imx6sll-clock.h:1: warning: Missing or
 malformed SPDX-License-Identifier tag in line 1

Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
---

 include/dt-bindings/clock/imx6sll-clock.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/dt-bindings/clock/imx6sll-clock.h b/include/dt-bindings/clock/imx6sll-clock.h
index f446710fe63d..494fd0c37fb5 100644
--- a/include/dt-bindings/clock/imx6sll-clock.h
+++ b/include/dt-bindings/clock/imx6sll-clock.h
@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-2.0
+/* SPDX-License-Identifier: GPL-2.0 */
 /*
  * Copyright (C) 2016 Freescale Semiconductor, Inc.
  * Copyright 2017-2018 NXP.
-- 
2.36.1


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

* [PATCH v1 5/5] dt-bindings: imx8ulp: clock: no spaces before tabs
  2022-08-26 19:22 [PATCH v1 0/5] ARM: arm64: dts/clk: imx: indentation permission whitespace cleanup 2 Marcel Ziswiler
  2022-08-26 19:22 ` [PATCH v1 4/5] clk: imx6sll: add proper spdx license identifier Marcel Ziswiler
@ 2022-08-26 19:22 ` Marcel Ziswiler
  2022-08-27 17:23   ` Krzysztof Kozlowski
                     ` (2 more replies)
  1 sibling, 3 replies; 9+ messages in thread
From: Marcel Ziswiler @ 2022-08-26 19:22 UTC (permalink / raw)
  To: linux-arm-kernel
  Cc: Marcel Ziswiler, Abel Vesa, Fabio Estevam, Krzysztof Kozlowski,
	Michael Turquette, NXP Linux Team, Pengutronix Kernel Team,
	Rob Herring, Sascha Hauer, Shawn Guo, Stephen Boyd, devicetree,
	linux-clk, linux-kernel

From: Marcel Ziswiler <marcel.ziswiler@toradex.com>

This fixes the following warnings:

include/dt-bindings/clock/imx8ulp-clock.h:204: warning: please, no space
 before tabs
include/dt-bindings/clock/imx8ulp-clock.h:215: warning: please, no space
 before tabs

Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>

---

 include/dt-bindings/clock/imx8ulp-clock.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/dt-bindings/clock/imx8ulp-clock.h b/include/dt-bindings/clock/imx8ulp-clock.h
index 953ecfe8ebcc..827404fadf5c 100644
--- a/include/dt-bindings/clock/imx8ulp-clock.h
+++ b/include/dt-bindings/clock/imx8ulp-clock.h
@@ -201,7 +201,7 @@
 #define IMX8ULP_CLK_SAI7		2
 #define IMX8ULP_CLK_SPDIF		3
 #define IMX8ULP_CLK_ISI			4
-#define IMX8ULP_CLK_CSI_REGS 		5
+#define IMX8ULP_CLK_CSI_REGS		5
 #define IMX8ULP_CLK_PCTLD		6
 #define IMX8ULP_CLK_CSI			7
 #define IMX8ULP_CLK_DSI			8
@@ -212,7 +212,7 @@
 #define IMX8ULP_CLK_GPU2D		13
 #define IMX8ULP_CLK_GPU3D		14
 #define IMX8ULP_CLK_DC_NANO		15
-#define IMX8ULP_CLK_CSI_CLK_UI 		16
+#define IMX8ULP_CLK_CSI_CLK_UI		16
 #define IMX8ULP_CLK_CSI_CLK_ESC		17
 #define IMX8ULP_CLK_RGPIOD		18
 #define IMX8ULP_CLK_DMA2_MP		19
-- 
2.36.1


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

* Re: [PATCH v1 5/5] dt-bindings: imx8ulp: clock: no spaces before tabs
  2022-08-26 19:22 ` [PATCH v1 5/5] dt-bindings: imx8ulp: clock: no spaces before tabs Marcel Ziswiler
@ 2022-08-27 17:23   ` Krzysztof Kozlowski
  2022-08-29  9:40   ` Abel Vesa
  2022-09-23 13:29   ` Marcel Ziswiler
  2 siblings, 0 replies; 9+ messages in thread
From: Krzysztof Kozlowski @ 2022-08-27 17:23 UTC (permalink / raw)
  To: Marcel Ziswiler, linux-arm-kernel
  Cc: Marcel Ziswiler, Abel Vesa, Fabio Estevam, Krzysztof Kozlowski,
	Michael Turquette, NXP Linux Team, Pengutronix Kernel Team,
	Rob Herring, Sascha Hauer, Shawn Guo, Stephen Boyd, devicetree,
	linux-clk, linux-kernel

On 26/08/2022 22:22, Marcel Ziswiler wrote:
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> 
> This fixes the following warnings:
> 
> include/dt-bindings/clock/imx8ulp-clock.h:204: warning: please, no space
>  before tabs
> include/dt-bindings/clock/imx8ulp-clock.h:215: warning: please, no space
>  before tabs
> 
> Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>


Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>


Best regards,
Krzysztof

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

* Re: [PATCH v1 5/5] dt-bindings: imx8ulp: clock: no spaces before tabs
  2022-08-26 19:22 ` [PATCH v1 5/5] dt-bindings: imx8ulp: clock: no spaces before tabs Marcel Ziswiler
  2022-08-27 17:23   ` Krzysztof Kozlowski
@ 2022-08-29  9:40   ` Abel Vesa
  2022-09-23 13:29   ` Marcel Ziswiler
  2 siblings, 0 replies; 9+ messages in thread
From: Abel Vesa @ 2022-08-29  9:40 UTC (permalink / raw)
  To: Marcel Ziswiler
  Cc: linux-arm-kernel, Marcel Ziswiler, Abel Vesa, Fabio Estevam,
	Krzysztof Kozlowski, Michael Turquette, NXP Linux Team,
	Pengutronix Kernel Team, Rob Herring, Sascha Hauer, Shawn Guo,
	Stephen Boyd, devicetree, linux-clk, linux-kernel

On 22-08-26 21:22:52, Marcel Ziswiler wrote:
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> 
> This fixes the following warnings:
> 
> include/dt-bindings/clock/imx8ulp-clock.h:204: warning: please, no space
>  before tabs
> include/dt-bindings/clock/imx8ulp-clock.h:215: warning: please, no space
>  before tabs
> 
> Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>

Reviewed-by: Abel Vesa <abel.vesa@linaro.org>

> 
> ---
> 
>  include/dt-bindings/clock/imx8ulp-clock.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/dt-bindings/clock/imx8ulp-clock.h b/include/dt-bindings/clock/imx8ulp-clock.h
> index 953ecfe8ebcc..827404fadf5c 100644
> --- a/include/dt-bindings/clock/imx8ulp-clock.h
> +++ b/include/dt-bindings/clock/imx8ulp-clock.h
> @@ -201,7 +201,7 @@
>  #define IMX8ULP_CLK_SAI7		2
>  #define IMX8ULP_CLK_SPDIF		3
>  #define IMX8ULP_CLK_ISI			4
> -#define IMX8ULP_CLK_CSI_REGS 		5
> +#define IMX8ULP_CLK_CSI_REGS		5
>  #define IMX8ULP_CLK_PCTLD		6
>  #define IMX8ULP_CLK_CSI			7
>  #define IMX8ULP_CLK_DSI			8
> @@ -212,7 +212,7 @@
>  #define IMX8ULP_CLK_GPU2D		13
>  #define IMX8ULP_CLK_GPU3D		14
>  #define IMX8ULP_CLK_DC_NANO		15
> -#define IMX8ULP_CLK_CSI_CLK_UI 		16
> +#define IMX8ULP_CLK_CSI_CLK_UI		16
>  #define IMX8ULP_CLK_CSI_CLK_ESC		17
>  #define IMX8ULP_CLK_RGPIOD		18
>  #define IMX8ULP_CLK_DMA2_MP		19
> -- 
> 2.36.1
> 

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

* Re: [PATCH v1 4/5] clk: imx6sll: add proper spdx license identifier
  2022-08-26 19:22 ` [PATCH v1 4/5] clk: imx6sll: add proper spdx license identifier Marcel Ziswiler
@ 2022-08-29  9:40   ` Abel Vesa
  2022-08-30 21:13   ` Rob Herring
  2022-09-23 13:28   ` Marcel Ziswiler
  2 siblings, 0 replies; 9+ messages in thread
From: Abel Vesa @ 2022-08-29  9:40 UTC (permalink / raw)
  To: Marcel Ziswiler
  Cc: linux-arm-kernel, Marcel Ziswiler, Abel Vesa, Fabio Estevam,
	Krzysztof Kozlowski, Michael Turquette, NXP Linux Team,
	Pengutronix Kernel Team, Rob Herring, Sascha Hauer, Shawn Guo,
	Stephen Boyd, devicetree, linux-clk, linux-kernel

On 22-08-26 21:22:51, Marcel Ziswiler wrote:
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> 
> This fixes the following error:
> 
> include/dt-bindings/clock/imx6sll-clock.h:1: warning: Improper SPDX
>  comment style for 'include/dt-bindings/clock/imx6sll-clock.h', please
>  use '/*' instead
> include/dt-bindings/clock/imx6sll-clock.h:1: warning: Missing or
>  malformed SPDX-License-Identifier tag in line 1
> 
> Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>

Reviewed-by: Abel Vesa <abel.vesa@linaro.org>

> ---
> 
>  include/dt-bindings/clock/imx6sll-clock.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/dt-bindings/clock/imx6sll-clock.h b/include/dt-bindings/clock/imx6sll-clock.h
> index f446710fe63d..494fd0c37fb5 100644
> --- a/include/dt-bindings/clock/imx6sll-clock.h
> +++ b/include/dt-bindings/clock/imx6sll-clock.h
> @@ -1,4 +1,4 @@
> -// SPDX-License-Identifier: GPL-2.0
> +/* SPDX-License-Identifier: GPL-2.0 */
>  /*
>   * Copyright (C) 2016 Freescale Semiconductor, Inc.
>   * Copyright 2017-2018 NXP.
> -- 
> 2.36.1
> 

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

* Re: [PATCH v1 4/5] clk: imx6sll: add proper spdx license identifier
  2022-08-26 19:22 ` [PATCH v1 4/5] clk: imx6sll: add proper spdx license identifier Marcel Ziswiler
  2022-08-29  9:40   ` Abel Vesa
@ 2022-08-30 21:13   ` Rob Herring
  2022-09-23 13:28   ` Marcel Ziswiler
  2 siblings, 0 replies; 9+ messages in thread
From: Rob Herring @ 2022-08-30 21:13 UTC (permalink / raw)
  To: Marcel Ziswiler
  Cc: Shawn Guo, devicetree, Krzysztof Kozlowski, Michael Turquette,
	NXP Linux Team, Pengutronix Kernel Team, Stephen Boyd, linux-clk,
	Fabio Estevam, Marcel Ziswiler, Sascha Hauer, Rob Herring,
	Abel Vesa, linux-kernel, linux-arm-kernel

On Fri, 26 Aug 2022 21:22:51 +0200, Marcel Ziswiler wrote:
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> 
> This fixes the following error:
> 
> include/dt-bindings/clock/imx6sll-clock.h:1: warning: Improper SPDX
>  comment style for 'include/dt-bindings/clock/imx6sll-clock.h', please
>  use '/*' instead
> include/dt-bindings/clock/imx6sll-clock.h:1: warning: Missing or
>  malformed SPDX-License-Identifier tag in line 1
> 
> Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> ---
> 
>  include/dt-bindings/clock/imx6sll-clock.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Acked-by: Rob Herring <robh@kernel.org>

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

* Re: [PATCH v1 4/5] clk: imx6sll: add proper spdx license identifier
  2022-08-26 19:22 ` [PATCH v1 4/5] clk: imx6sll: add proper spdx license identifier Marcel Ziswiler
  2022-08-29  9:40   ` Abel Vesa
  2022-08-30 21:13   ` Rob Herring
@ 2022-09-23 13:28   ` Marcel Ziswiler
  2 siblings, 0 replies; 9+ messages in thread
From: Marcel Ziswiler @ 2022-09-23 13:28 UTC (permalink / raw)
  To: linux-arm-kernel@lists.infradead.org
  Cc: abel.vesa@linaro.org, linux-clk@vger.kernel.org,
	abelvesa@kernel.org, kernel@pengutronix.de, festevam@gmail.com,
	s.hauer@pengutronix.de, sboyd@kernel.org, shawnguo@kernel.org,
	linux-imx@nxp.com, robh@kernel.org, linux-kernel@vger.kernel.org,
	mturquette@baylibre.com, krzysztof.kozlowski+dt@linaro.org,
	robh+dt@kernel.org, devicetree@vger.kernel.org

On Fri, 2022-08-26 at 21:22 +0200, Marcel Ziswiler wrote:
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> 
> This fixes the following error:
> 
> include/dt-bindings/clock/imx6sll-clock.h:1: warning: Improper SPDX
>  comment style for 'include/dt-bindings/clock/imx6sll-clock.h', please
>  use '/*' instead
> include/dt-bindings/clock/imx6sll-clock.h:1: warning: Missing or
>  malformed SPDX-License-Identifier tag in line 1
> 
> Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> ---
> 
>  include/dt-bindings/clock/imx6sll-clock.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/dt-bindings/clock/imx6sll-clock.h b/include/dt-bindings/clock/imx6sll-clock.h
> index f446710fe63d..494fd0c37fb5 100644
> --- a/include/dt-bindings/clock/imx6sll-clock.h
> +++ b/include/dt-bindings/clock/imx6sll-clock.h
> @@ -1,4 +1,4 @@
> -// SPDX-License-Identifier: GPL-2.0
> +/* SPDX-License-Identifier: GPL-2.0 */
>  /*
>   * Copyright (C) 2016 Freescale Semiconductor, Inc.
>   * Copyright 2017-2018 NXP.

I noticed that this one has not landed yet while patches 1-3 were pulled by Shawn. Who will take this one?

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

* Re: [PATCH v1 5/5] dt-bindings: imx8ulp: clock: no spaces before tabs
  2022-08-26 19:22 ` [PATCH v1 5/5] dt-bindings: imx8ulp: clock: no spaces before tabs Marcel Ziswiler
  2022-08-27 17:23   ` Krzysztof Kozlowski
  2022-08-29  9:40   ` Abel Vesa
@ 2022-09-23 13:29   ` Marcel Ziswiler
  2 siblings, 0 replies; 9+ messages in thread
From: Marcel Ziswiler @ 2022-09-23 13:29 UTC (permalink / raw)
  To: linux-arm-kernel@lists.infradead.org
  Cc: linux-clk@vger.kernel.org, abelvesa@kernel.org,
	kernel@pengutronix.de, festevam@gmail.com, s.hauer@pengutronix.de,
	sboyd@kernel.org, shawnguo@kernel.org, linux-imx@nxp.com,
	linux-kernel@vger.kernel.org, mturquette@baylibre.com,
	krzysztof.kozlowski+dt@linaro.org, robh+dt@kernel.org,
	devicetree@vger.kernel.org

On Fri, 2022-08-26 at 21:22 +0200, Marcel Ziswiler wrote:
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> 
> This fixes the following warnings:
> 
> include/dt-bindings/clock/imx8ulp-clock.h:204: warning: please, no space
>  before tabs
> include/dt-bindings/clock/imx8ulp-clock.h:215: warning: please, no space
>  before tabs
> 
> Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> 
> ---
> 
>  include/dt-bindings/clock/imx8ulp-clock.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/dt-bindings/clock/imx8ulp-clock.h b/include/dt-bindings/clock/imx8ulp-clock.h
> index 953ecfe8ebcc..827404fadf5c 100644
> --- a/include/dt-bindings/clock/imx8ulp-clock.h
> +++ b/include/dt-bindings/clock/imx8ulp-clock.h
> @@ -201,7 +201,7 @@
>  #define IMX8ULP_CLK_SAI7               2
>  #define IMX8ULP_CLK_SPDIF              3
>  #define IMX8ULP_CLK_ISI                        4
> -#define IMX8ULP_CLK_CSI_REGS           5
> +#define IMX8ULP_CLK_CSI_REGS           5
>  #define IMX8ULP_CLK_PCTLD              6
>  #define IMX8ULP_CLK_CSI                        7
>  #define IMX8ULP_CLK_DSI                        8
> @@ -212,7 +212,7 @@
>  #define IMX8ULP_CLK_GPU2D              13
>  #define IMX8ULP_CLK_GPU3D              14
>  #define IMX8ULP_CLK_DC_NANO            15
> -#define IMX8ULP_CLK_CSI_CLK_UI                 16
> +#define IMX8ULP_CLK_CSI_CLK_UI         16
>  #define IMX8ULP_CLK_CSI_CLK_ESC                17
>  #define IMX8ULP_CLK_RGPIOD             18
>  #define IMX8ULP_CLK_DMA2_MP            19

Same applies to this one. Any takers?

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

end of thread, other threads:[~2022-09-23 13:30 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-08-26 19:22 [PATCH v1 0/5] ARM: arm64: dts/clk: imx: indentation permission whitespace cleanup 2 Marcel Ziswiler
2022-08-26 19:22 ` [PATCH v1 4/5] clk: imx6sll: add proper spdx license identifier Marcel Ziswiler
2022-08-29  9:40   ` Abel Vesa
2022-08-30 21:13   ` Rob Herring
2022-09-23 13:28   ` Marcel Ziswiler
2022-08-26 19:22 ` [PATCH v1 5/5] dt-bindings: imx8ulp: clock: no spaces before tabs Marcel Ziswiler
2022-08-27 17:23   ` Krzysztof Kozlowski
2022-08-29  9:40   ` Abel Vesa
2022-09-23 13:29   ` Marcel Ziswiler

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