* [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
* 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
* [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 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
* 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
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).