linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [RESEND PATCH v1 0/2] ARM: arm64: dts/clk: imx: indentation permission whitespace cleanup 2
@ 2023-01-19  8:54 Marcel Ziswiler
  2023-01-19  8:54 ` [RESEND PATCH v1 1/2] clk: imx6sll: add proper spdx license identifier Marcel Ziswiler
                   ` (2 more replies)
  0 siblings, 3 replies; 8+ messages in thread
From: Marcel Ziswiler @ 2023-01-19  8:54 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>


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 (2):
  clk: imx6sll: add proper spdx license identifier
  dt-bindings: imx8ulp: clock: no spaces before tabs

 include/dt-bindings/clock/imx6sll-clock.h | 2 +-
 include/dt-bindings/clock/imx8ulp-clock.h | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

-- 
2.36.1


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

* [RESEND PATCH v1 1/2] clk: imx6sll: add proper spdx license identifier
  2023-01-19  8:54 [RESEND PATCH v1 0/2] ARM: arm64: dts/clk: imx: indentation permission whitespace cleanup 2 Marcel Ziswiler
@ 2023-01-19  8:54 ` Marcel Ziswiler
  2023-01-29 17:37   ` Abel Vesa
  2023-01-19  8:54 ` [RESEND PATCH v1 2/2] dt-bindings: imx8ulp: clock: no spaces before tabs Marcel Ziswiler
  2023-01-29 17:42 ` [RESEND PATCH v1 0/2] ARM: arm64: dts/clk: imx: indentation permission whitespace cleanup 2 Abel Vesa
  2 siblings, 1 reply; 8+ messages in thread
From: Marcel Ziswiler @ 2023-01-19  8:54 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] 8+ messages in thread

* [RESEND PATCH v1 2/2] dt-bindings: imx8ulp: clock: no spaces before tabs
  2023-01-19  8:54 [RESEND PATCH v1 0/2] ARM: arm64: dts/clk: imx: indentation permission whitespace cleanup 2 Marcel Ziswiler
  2023-01-19  8:54 ` [RESEND PATCH v1 1/2] clk: imx6sll: add proper spdx license identifier Marcel Ziswiler
@ 2023-01-19  8:54 ` Marcel Ziswiler
  2023-01-19 10:32   ` Krzysztof Kozlowski
                     ` (2 more replies)
  2023-01-29 17:42 ` [RESEND PATCH v1 0/2] ARM: arm64: dts/clk: imx: indentation permission whitespace cleanup 2 Abel Vesa
  2 siblings, 3 replies; 8+ messages in thread
From: Marcel Ziswiler @ 2023-01-19  8:54 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] 8+ messages in thread

* Re: [RESEND PATCH v1 2/2] dt-bindings: imx8ulp: clock: no spaces before tabs
  2023-01-19  8:54 ` [RESEND PATCH v1 2/2] dt-bindings: imx8ulp: clock: no spaces before tabs Marcel Ziswiler
@ 2023-01-19 10:32   ` Krzysztof Kozlowski
  2023-01-29 17:43   ` Abel Vesa
  2023-01-29 17:44   ` Abel Vesa
  2 siblings, 0 replies; 8+ messages in thread
From: Krzysztof Kozlowski @ 2023-01-19 10:32 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 19/01/2023 09:54, 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
> 


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

Best regards,
Krzysztof


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

* Re: [RESEND PATCH v1 1/2] clk: imx6sll: add proper spdx license identifier
  2023-01-19  8:54 ` [RESEND PATCH v1 1/2] clk: imx6sll: add proper spdx license identifier Marcel Ziswiler
@ 2023-01-29 17:37   ` Abel Vesa
  0 siblings, 0 replies; 8+ messages in thread
From: Abel Vesa @ 2023-01-29 17:37 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 23-01-19 09:54:20, 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] 8+ messages in thread

* Re: [RESEND PATCH v1 0/2] ARM: arm64: dts/clk: imx: indentation permission whitespace cleanup 2
  2023-01-19  8:54 [RESEND PATCH v1 0/2] ARM: arm64: dts/clk: imx: indentation permission whitespace cleanup 2 Marcel Ziswiler
  2023-01-19  8:54 ` [RESEND PATCH v1 1/2] clk: imx6sll: add proper spdx license identifier Marcel Ziswiler
  2023-01-19  8:54 ` [RESEND PATCH v1 2/2] dt-bindings: imx8ulp: clock: no spaces before tabs Marcel Ziswiler
@ 2023-01-29 17:42 ` Abel Vesa
  2 siblings, 0 replies; 8+ messages in thread
From: Abel Vesa @ 2023-01-29 17:42 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 23-01-19 09:54:19, Marcel Ziswiler wrote:
> 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.

Nitpick: The subject line should not contain "ARM: arm64:" prefix, as it makes it
confusing for maintainers through which tree the patches should be
applied.

> 
> 
> Marcel Ziswiler (2):
>   clk: imx6sll: add proper spdx license identifier
>   dt-bindings: imx8ulp: clock: no spaces before tabs
> 
>  include/dt-bindings/clock/imx6sll-clock.h | 2 +-
>  include/dt-bindings/clock/imx8ulp-clock.h | 4 ++--
>  2 files changed, 3 insertions(+), 3 deletions(-)
> 
> -- 
> 2.36.1
> 

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

* Re: [RESEND PATCH v1 2/2] dt-bindings: imx8ulp: clock: no spaces before tabs
  2023-01-19  8:54 ` [RESEND PATCH v1 2/2] dt-bindings: imx8ulp: clock: no spaces before tabs Marcel Ziswiler
  2023-01-19 10:32   ` Krzysztof Kozlowski
@ 2023-01-29 17:43   ` Abel Vesa
  2023-01-29 17:44   ` Abel Vesa
  2 siblings, 0 replies; 8+ messages in thread
From: Abel Vesa @ 2023-01-29 17:43 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 23-01-19 09:54:21, 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] 8+ messages in thread

* Re: [RESEND PATCH v1 2/2] dt-bindings: imx8ulp: clock: no spaces before tabs
  2023-01-19  8:54 ` [RESEND PATCH v1 2/2] dt-bindings: imx8ulp: clock: no spaces before tabs Marcel Ziswiler
  2023-01-19 10:32   ` Krzysztof Kozlowski
  2023-01-29 17:43   ` Abel Vesa
@ 2023-01-29 17:44   ` Abel Vesa
  2 siblings, 0 replies; 8+ messages in thread
From: Abel Vesa @ 2023-01-29 17:44 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 23-01-19 09:54:21, 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>

Applied both.

Thanks.

> 
> ---
> 
>  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] 8+ messages in thread

end of thread, other threads:[~2023-01-29 17:45 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-19  8:54 [RESEND PATCH v1 0/2] ARM: arm64: dts/clk: imx: indentation permission whitespace cleanup 2 Marcel Ziswiler
2023-01-19  8:54 ` [RESEND PATCH v1 1/2] clk: imx6sll: add proper spdx license identifier Marcel Ziswiler
2023-01-29 17:37   ` Abel Vesa
2023-01-19  8:54 ` [RESEND PATCH v1 2/2] dt-bindings: imx8ulp: clock: no spaces before tabs Marcel Ziswiler
2023-01-19 10:32   ` Krzysztof Kozlowski
2023-01-29 17:43   ` Abel Vesa
2023-01-29 17:44   ` Abel Vesa
2023-01-29 17:42 ` [RESEND PATCH v1 0/2] ARM: arm64: dts/clk: imx: indentation permission whitespace cleanup 2 Abel Vesa

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).