* [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 1/5] ARM: dts: imx6sl: use tabs for code indent Marcel Ziswiler
` (4 more replies)
0 siblings, 5 replies; 16+ 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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v1 1/5] ARM: dts: imx6sl: use tabs for code indent
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-09-05 1:57 ` Shawn Guo
2022-08-26 19:22 ` [PATCH v1 2/5] ARM: dts: imx6sx-udoo-neo: don't use multiple blank lines Marcel Ziswiler
` (3 subsequent siblings)
4 siblings, 1 reply; 16+ messages in thread
From: Marcel Ziswiler @ 2022-08-26 19:22 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Marcel Ziswiler, Fabio Estevam, Krzysztof Kozlowski,
NXP Linux Team, Pengutronix Kernel Team, Rob Herring,
Sascha Hauer, Shawn Guo, devicetree, linux-kernel
From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
This fixes the following error:
arch/arm/boot/dts/imx6sl.dtsi:714: error: code indent should use tabs
where possible
Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
---
arch/arm/boot/dts/imx6sl.dtsi | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/arch/arm/boot/dts/imx6sl.dtsi b/arch/arm/boot/dts/imx6sl.dtsi
index 06a515121dfc..271f4b971a83 100644
--- a/arch/arm/boot/dts/imx6sl.dtsi
+++ b/arch/arm/boot/dts/imx6sl.dtsi
@@ -61,10 +61,10 @@ cpu0: cpu@0 {
<792000 1175000>,
<396000 975000>;
fsl,soc-operating-points =
- /* ARM kHz SOC-PU uV */
- <996000 1225000>,
- <792000 1175000>,
- <396000 1175000>;
+ /* ARM kHz SOC-PU uV */
+ <996000 1225000>,
+ <792000 1175000>,
+ <396000 1175000>;
clock-latency = <61036>; /* two CLK32 periods */
#cooling-cells = <2>;
clocks = <&clks IMX6SL_CLK_ARM>, <&clks IMX6SL_CLK_PLL2_PFD2>,
@@ -222,7 +222,7 @@ ecspi4: spi@2014000 {
uart5: serial@2018000 {
compatible = "fsl,imx6sl-uart",
- "fsl,imx6q-uart", "fsl,imx21-uart";
+ "fsl,imx6q-uart", "fsl,imx21-uart";
reg = <0x02018000 0x4000>;
interrupts = <0 30 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clks IMX6SL_CLK_UART>,
@@ -235,7 +235,7 @@ uart5: serial@2018000 {
uart1: serial@2020000 {
compatible = "fsl,imx6sl-uart",
- "fsl,imx6q-uart", "fsl,imx21-uart";
+ "fsl,imx6q-uart", "fsl,imx21-uart";
reg = <0x02020000 0x4000>;
interrupts = <0 26 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clks IMX6SL_CLK_UART>,
@@ -248,7 +248,7 @@ uart1: serial@2020000 {
uart2: serial@2024000 {
compatible = "fsl,imx6sl-uart",
- "fsl,imx6q-uart", "fsl,imx21-uart";
+ "fsl,imx6q-uart", "fsl,imx21-uart";
reg = <0x02024000 0x4000>;
interrupts = <0 27 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clks IMX6SL_CLK_UART>,
@@ -309,7 +309,7 @@ ssi3: ssi@2030000 {
uart3: serial@2034000 {
compatible = "fsl,imx6sl-uart",
- "fsl,imx6q-uart", "fsl,imx21-uart";
+ "fsl,imx6q-uart", "fsl,imx21-uart";
reg = <0x02034000 0x4000>;
interrupts = <0 28 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clks IMX6SL_CLK_UART>,
@@ -322,7 +322,7 @@ uart3: serial@2034000 {
uart4: serial@2038000 {
compatible = "fsl,imx6sl-uart",
- "fsl,imx6q-uart", "fsl,imx21-uart";
+ "fsl,imx6q-uart", "fsl,imx21-uart";
reg = <0x02038000 0x4000>;
interrupts = <0 29 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clks IMX6SL_CLK_UART>,
@@ -711,7 +711,7 @@ pd_pu: power-domain@1 {
#power-domain-cells = <0>;
power-supply = <®_pu>;
clocks = <&clks IMX6SL_CLK_GPU2D_OVG>,
- <&clks IMX6SL_CLK_GPU2D_PODF>;
+ <&clks IMX6SL_CLK_GPU2D_PODF>;
};
pd_disp: power-domain@2 {
--
2.36.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v1 2/5] ARM: dts: imx6sx-udoo-neo: don't use multiple blank lines
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 1/5] ARM: dts: imx6sl: use tabs for code indent Marcel Ziswiler
@ 2022-08-26 19:22 ` Marcel Ziswiler
2022-09-05 2:01 ` Shawn Guo
2022-08-26 19:22 ` [PATCH v1 3/5] arm64: dts: imx8ulp: no executable source file permission Marcel Ziswiler
` (2 subsequent siblings)
4 siblings, 1 reply; 16+ messages in thread
From: Marcel Ziswiler @ 2022-08-26 19:22 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Marcel Ziswiler, Fabio Estevam, Krzysztof Kozlowski,
NXP Linux Team, Pengutronix Kernel Team, Rob Herring,
Sascha Hauer, Shawn Guo, devicetree, linux-kernel
From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
This fixes the following warning:
arch/arm/boot/dts/imx6sx-udoo-neo.dtsi:309: check: Please don't use multiple
blank lines
Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
---
arch/arm/boot/dts/imx6sx-udoo-neo.dtsi | 14 ++++++--------
1 file changed, 6 insertions(+), 8 deletions(-)
diff --git a/arch/arm/boot/dts/imx6sx-udoo-neo.dtsi b/arch/arm/boot/dts/imx6sx-udoo-neo.dtsi
index 35861bbea94e..c84ea1fac5e9 100644
--- a/arch/arm/boot/dts/imx6sx-udoo-neo.dtsi
+++ b/arch/arm/boot/dts/imx6sx-udoo-neo.dtsi
@@ -226,7 +226,7 @@ lcdc: endpoint {
&iomuxc {
pinctrl_bt_reg: btreggrp {
fsl,pins =
- <MX6SX_PAD_KEY_ROW2__GPIO2_IO_17 0x15059>;
+ <MX6SX_PAD_KEY_ROW2__GPIO2_IO_17 0x15059>;
};
pinctrl_enet1: enet1grp {
@@ -306,7 +306,6 @@ MX6SX_PAD_LCD1_RESET__GPIO3_IO_27 0x4001b0b0
>;
};
-
pinctrl_uart1: uart1grp {
fsl,pins =
<MX6SX_PAD_GPIO1_IO04__UART1_DCE_TX 0x1b0b1>,
@@ -347,24 +346,23 @@ pinctrl_uart6: uart6grp {
pinctrl_otg1_reg: otg1grp {
fsl,pins =
- <MX6SX_PAD_GPIO1_IO09__GPIO1_IO_9 0x10b0>;
+ <MX6SX_PAD_GPIO1_IO09__GPIO1_IO_9 0x10b0>;
};
-
pinctrl_otg2_reg: otg2grp {
fsl,pins =
- <MX6SX_PAD_NAND_RE_B__GPIO4_IO_12 0x10b0>;
+ <MX6SX_PAD_NAND_RE_B__GPIO4_IO_12 0x10b0>;
};
pinctrl_usb_otg1: usbotg1grp {
fsl,pins =
- <MX6SX_PAD_GPIO1_IO10__ANATOP_OTG1_ID 0x17059>,
- <MX6SX_PAD_GPIO1_IO08__USB_OTG1_OC 0x10b0>;
+ <MX6SX_PAD_GPIO1_IO10__ANATOP_OTG1_ID 0x17059>,
+ <MX6SX_PAD_GPIO1_IO08__USB_OTG1_OC 0x10b0>;
};
pinctrl_usb_otg2: usbot2ggrp {
fsl,pins =
- <MX6SX_PAD_QSPI1A_DATA0__USB_OTG2_OC 0x10b0>;
+ <MX6SX_PAD_QSPI1A_DATA0__USB_OTG2_OC 0x10b0>;
};
pinctrl_usdhc2: usdhc2grp {
--
2.36.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v1 3/5] arm64: dts: imx8ulp: no executable source file permission
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 1/5] ARM: dts: imx6sl: use tabs for code indent Marcel Ziswiler
2022-08-26 19:22 ` [PATCH v1 2/5] ARM: dts: imx6sx-udoo-neo: don't use multiple blank lines Marcel Ziswiler
@ 2022-08-26 19:22 ` Marcel Ziswiler
2022-08-29 5:41 ` Peng Fan
2022-09-05 1:53 ` Shawn Guo
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
4 siblings, 2 replies; 16+ messages in thread
From: Marcel Ziswiler @ 2022-08-26 19:22 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Marcel Ziswiler, Dong Aisheng, Fabio Estevam, Jacky Bai,
Krzysztof Kozlowski, NXP Linux Team, Peng Fan,
Pengutronix Kernel Team, Rob Herring, Sascha Hauer, Shawn Guo,
devicetree, linux-kernel
From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
This fixes the following error:
arch/arm64/boot/dts/freescale/imx8ulp-pinfunc.h: error: do not set
execute permissions for source files
Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
---
arch/arm64/boot/dts/freescale/imx8ulp-pinfunc.h | 0
1 file changed, 0 insertions(+), 0 deletions(-)
mode change 100755 => 100644 arch/arm64/boot/dts/freescale/imx8ulp-pinfunc.h
diff --git a/arch/arm64/boot/dts/freescale/imx8ulp-pinfunc.h b/arch/arm64/boot/dts/freescale/imx8ulp-pinfunc.h
old mode 100755
new mode 100644
--
2.36.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 16+ 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
` (2 preceding siblings ...)
2022-08-26 19:22 ` [PATCH v1 3/5] arm64: dts: imx8ulp: no executable source file permission 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
4 siblings, 3 replies; 16+ 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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 16+ 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
` (3 preceding siblings ...)
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)
4 siblings, 3 replies; 16+ 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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 16+ 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; 16+ 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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 16+ messages in thread
* RE: [PATCH v1 3/5] arm64: dts: imx8ulp: no executable source file permission
2022-08-26 19:22 ` [PATCH v1 3/5] arm64: dts: imx8ulp: no executable source file permission Marcel Ziswiler
@ 2022-08-29 5:41 ` Peng Fan
2022-09-05 1:53 ` Shawn Guo
1 sibling, 0 replies; 16+ messages in thread
From: Peng Fan @ 2022-08-29 5:41 UTC (permalink / raw)
To: Marcel Ziswiler, linux-arm-kernel
Cc: Marcel Ziswiler, Aisheng Dong, Fabio Estevam, Jacky Bai,
Krzysztof Kozlowski, dl-linux-imx, Pengutronix Kernel Team,
Rob Herring, Sascha Hauer, Shawn Guo, devicetree, linux-kernel
> Subject: [PATCH v1 3/5] arm64: dts: imx8ulp: no executable source file
> permission
>
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
>
> This fixes the following error:
>
> arch/arm64/boot/dts/freescale/imx8ulp-pinfunc.h: error: do not set
> execute permissions for source files
>
> Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
Acked-by: Peng Fan <peng.fan@nxp.com>
> ---
>
> arch/arm64/boot/dts/freescale/imx8ulp-pinfunc.h | 0
> 1 file changed, 0 insertions(+), 0 deletions(-) mode change 100755 =>
> 100644 arch/arm64/boot/dts/freescale/imx8ulp-pinfunc.h
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8ulp-pinfunc.h
> b/arch/arm64/boot/dts/freescale/imx8ulp-pinfunc.h
> old mode 100755
> new mode 100644
> --
> 2.36.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 16+ 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; 16+ 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
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 16+ 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; 16+ 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
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 16+ 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; 16+ 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>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v1 3/5] arm64: dts: imx8ulp: no executable source file permission
2022-08-26 19:22 ` [PATCH v1 3/5] arm64: dts: imx8ulp: no executable source file permission Marcel Ziswiler
2022-08-29 5:41 ` Peng Fan
@ 2022-09-05 1:53 ` Shawn Guo
1 sibling, 0 replies; 16+ messages in thread
From: Shawn Guo @ 2022-09-05 1:53 UTC (permalink / raw)
To: Marcel Ziswiler
Cc: linux-arm-kernel, Marcel Ziswiler, Dong Aisheng, Fabio Estevam,
Jacky Bai, Krzysztof Kozlowski, NXP Linux Team, Peng Fan,
Pengutronix Kernel Team, Rob Herring, Sascha Hauer, devicetree,
linux-kernel
On Fri, Aug 26, 2022 at 09:22:50PM +0200, Marcel Ziswiler wrote:
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
>
> This fixes the following error:
>
> arch/arm64/boot/dts/freescale/imx8ulp-pinfunc.h: error: do not set
> execute permissions for source files
>
> Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
Applied, thanks!
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v1 1/5] ARM: dts: imx6sl: use tabs for code indent
2022-08-26 19:22 ` [PATCH v1 1/5] ARM: dts: imx6sl: use tabs for code indent Marcel Ziswiler
@ 2022-09-05 1:57 ` Shawn Guo
0 siblings, 0 replies; 16+ messages in thread
From: Shawn Guo @ 2022-09-05 1:57 UTC (permalink / raw)
To: Marcel Ziswiler
Cc: linux-arm-kernel, Marcel Ziswiler, Fabio Estevam,
Krzysztof Kozlowski, NXP Linux Team, Pengutronix Kernel Team,
Rob Herring, Sascha Hauer, devicetree, linux-kernel
On Fri, Aug 26, 2022 at 09:22:48PM +0200, Marcel Ziswiler wrote:
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
>
> This fixes the following error:
>
> arch/arm/boot/dts/imx6sl.dtsi:714: error: code indent should use tabs
> where possible
>
> Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
Applied, thanks!
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v1 2/5] ARM: dts: imx6sx-udoo-neo: don't use multiple blank lines
2022-08-26 19:22 ` [PATCH v1 2/5] ARM: dts: imx6sx-udoo-neo: don't use multiple blank lines Marcel Ziswiler
@ 2022-09-05 2:01 ` Shawn Guo
0 siblings, 0 replies; 16+ messages in thread
From: Shawn Guo @ 2022-09-05 2:01 UTC (permalink / raw)
To: Marcel Ziswiler
Cc: linux-arm-kernel, Marcel Ziswiler, Fabio Estevam,
Krzysztof Kozlowski, NXP Linux Team, Pengutronix Kernel Team,
Rob Herring, Sascha Hauer, devicetree, linux-kernel
On Fri, Aug 26, 2022 at 09:22:49PM +0200, Marcel Ziswiler wrote:
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
>
> This fixes the following warning:
>
> arch/arm/boot/dts/imx6sx-udoo-neo.dtsi:309: check: Please don't use multiple
> blank lines
It includes changes more than multiple blank lines fix. I mentioned
those indent changes a bit in the commit log.
>
> Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
Applied, thanks!
Shawn
> ---
>
> arch/arm/boot/dts/imx6sx-udoo-neo.dtsi | 14 ++++++--------
> 1 file changed, 6 insertions(+), 8 deletions(-)
>
> diff --git a/arch/arm/boot/dts/imx6sx-udoo-neo.dtsi b/arch/arm/boot/dts/imx6sx-udoo-neo.dtsi
> index 35861bbea94e..c84ea1fac5e9 100644
> --- a/arch/arm/boot/dts/imx6sx-udoo-neo.dtsi
> +++ b/arch/arm/boot/dts/imx6sx-udoo-neo.dtsi
> @@ -226,7 +226,7 @@ lcdc: endpoint {
> &iomuxc {
> pinctrl_bt_reg: btreggrp {
> fsl,pins =
> - <MX6SX_PAD_KEY_ROW2__GPIO2_IO_17 0x15059>;
> + <MX6SX_PAD_KEY_ROW2__GPIO2_IO_17 0x15059>;
> };
>
> pinctrl_enet1: enet1grp {
> @@ -306,7 +306,6 @@ MX6SX_PAD_LCD1_RESET__GPIO3_IO_27 0x4001b0b0
> >;
> };
>
> -
> pinctrl_uart1: uart1grp {
> fsl,pins =
> <MX6SX_PAD_GPIO1_IO04__UART1_DCE_TX 0x1b0b1>,
> @@ -347,24 +346,23 @@ pinctrl_uart6: uart6grp {
>
> pinctrl_otg1_reg: otg1grp {
> fsl,pins =
> - <MX6SX_PAD_GPIO1_IO09__GPIO1_IO_9 0x10b0>;
> + <MX6SX_PAD_GPIO1_IO09__GPIO1_IO_9 0x10b0>;
> };
>
> -
> pinctrl_otg2_reg: otg2grp {
> fsl,pins =
> - <MX6SX_PAD_NAND_RE_B__GPIO4_IO_12 0x10b0>;
> + <MX6SX_PAD_NAND_RE_B__GPIO4_IO_12 0x10b0>;
> };
>
> pinctrl_usb_otg1: usbotg1grp {
> fsl,pins =
> - <MX6SX_PAD_GPIO1_IO10__ANATOP_OTG1_ID 0x17059>,
> - <MX6SX_PAD_GPIO1_IO08__USB_OTG1_OC 0x10b0>;
> + <MX6SX_PAD_GPIO1_IO10__ANATOP_OTG1_ID 0x17059>,
> + <MX6SX_PAD_GPIO1_IO08__USB_OTG1_OC 0x10b0>;
> };
>
> pinctrl_usb_otg2: usbot2ggrp {
> fsl,pins =
> - <MX6SX_PAD_QSPI1A_DATA0__USB_OTG2_OC 0x10b0>;
> + <MX6SX_PAD_QSPI1A_DATA0__USB_OTG2_OC 0x10b0>;
> };
>
> pinctrl_usdhc2: usdhc2grp {
> --
> 2.36.1
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 16+ 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; 16+ messages in thread
From: Marcel Ziswiler @ 2022-09-23 13:28 UTC (permalink / raw)
To: linux-arm-kernel
Cc: abel.vesa, linux-clk, abelvesa, kernel, festevam, s.hauer, sboyd,
shawnguo, linux-imx, robh, linux-kernel, mturquette,
krzysztof.kozlowski+dt, robh+dt, devicetree
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?
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 16+ 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; 16+ messages in thread
From: Marcel Ziswiler @ 2022-09-23 13:29 UTC (permalink / raw)
To: linux-arm-kernel
Cc: linux-clk, abelvesa, kernel, festevam, s.hauer, sboyd, shawnguo,
linux-imx, linux-kernel, mturquette, krzysztof.kozlowski+dt,
robh+dt, devicetree
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?
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2022-09-23 13:31 UTC | newest]
Thread overview: 16+ 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 1/5] ARM: dts: imx6sl: use tabs for code indent Marcel Ziswiler
2022-09-05 1:57 ` Shawn Guo
2022-08-26 19:22 ` [PATCH v1 2/5] ARM: dts: imx6sx-udoo-neo: don't use multiple blank lines Marcel Ziswiler
2022-09-05 2:01 ` Shawn Guo
2022-08-26 19:22 ` [PATCH v1 3/5] arm64: dts: imx8ulp: no executable source file permission Marcel Ziswiler
2022-08-29 5:41 ` Peng Fan
2022-09-05 1:53 ` Shawn Guo
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;
as well as URLs for NNTP newsgroup(s).