From: Marcel Ziswiler <[email protected]>
While synchronising them imx device trees with U-Boot I stumbled over
various checkpatch warnings. This series addresses those trivial
indentation and/or whitespace cleanups.
Marcel Ziswiler (12):
ARM: dts: imx6-sabrelite: change to use SPDX identifiers
ARM: dts: imx6qdl-mba6: don't use multiple blank lines
ARM: dts: imx6qdl: phytec: no spaces at start of line, indent use tabs
ARM: dts: imx6qdl-sabre: change to use SPDX identifiers
ARM: dts: imx7d-pico: indent use tabs, no spaces at start of line
ARM: dts: vf610: no spaces in indent but tabs
ARM: dts: vf610-twr: indent use tabs, no spaces at start of line
ARM: dts: vf610: don't use multiple blank lines
arm64: dts: imx8mm-venice-gw72xx-0x: blank line at end of file
arm64: dts: imx8mp-verdin: don't use multiple blank lines
arm64: dts: mnt-reform2: don't use multiple blank lines
clk: imx8mm: don't use multiple blank lines
arch/arm/boot/dts/imx6q-sabrelite.dts | 37 +------------------
arch/arm/boot/dts/imx6qdl-mba6.dtsi | 1 -
.../dts/imx6qdl-phytec-mira-peb-av-02.dtsi | 2 +-
arch/arm/boot/dts/imx6qdl-sabrelite.dtsi | 37 +------------------
arch/arm/boot/dts/imx7d-pico.dtsi | 10 ++---
arch/arm/boot/dts/vf610-pinfunc.h | 2 +-
arch/arm/boot/dts/vf610-twr.dts | 2 +-
arch/arm/boot/dts/vf610.dtsi | 1 -
.../dts/freescale/imx8mm-venice-gw72xx-0x.dts | 1 -
.../boot/dts/freescale/imx8mp-verdin.dtsi | 1 -
.../boot/dts/freescale/imx8mq-mnt-reform2.dts | 1 -
include/dt-bindings/clock/imx8mm-clock.h | 1 -
12 files changed, 10 insertions(+), 86 deletions(-)
--
2.35.1
From: Marcel Ziswiler <[email protected]>
Avoid the following checkpatch warnings:
arch/arm/boot/dts/imx6qdl-phytec-mira-peb-av-02.dtsi:67: error: code
indent should use tabs where possible
arch/arm/boot/dts/imx6qdl-phytec-mira-peb-av-02.dtsi:67: warning:
please, no spaces at the start of a line
Signed-off-by: Marcel Ziswiler <[email protected]>
---
arch/arm/boot/dts/imx6qdl-phytec-mira-peb-av-02.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/imx6qdl-phytec-mira-peb-av-02.dtsi b/arch/arm/boot/dts/imx6qdl-phytec-mira-peb-av-02.dtsi
index 393475cb0f4d..0020dbb1722c 100644
--- a/arch/arm/boot/dts/imx6qdl-phytec-mira-peb-av-02.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-phytec-mira-peb-av-02.dtsi
@@ -64,7 +64,7 @@ edt_ft5x06: touchscreen@38 {
interrupt-parent = <&gpio3>;
interrupts = <2 IRQ_TYPE_NONE>;
status = "disabled";
- };
+ };
};
&ipu1_di0_disp0 {
--
2.35.1
From: Marcel Ziswiler <[email protected]>
Avoid spaces in indent but rather use tabs.
Signed-off-by: Marcel Ziswiler <[email protected]>
---
arch/arm/boot/dts/vf610-pinfunc.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/vf610-pinfunc.h b/arch/arm/boot/dts/vf610-pinfunc.h
index f1e5a7cf58a9..8bb970b0dff8 100644
--- a/arch/arm/boot/dts/vf610-pinfunc.h
+++ b/arch/arm/boot/dts/vf610-pinfunc.h
@@ -420,7 +420,7 @@
#define VF610_PAD_PTD29__FTM3_CH2 0x104 0x000 ALT4 0x0
#define VF610_PAD_PTD29__DSPI2_SIN 0x104 0x000 ALT5 0x0
#define VF610_PAD_PTD29__DEBUG_OUT11 0x104 0x000 ALT7 0x0
-#define VF610_PAD_PTD28__GPIO_66 0x108 0x000 ALT0 0x0
+#define VF610_PAD_PTD28__GPIO_66 0x108 0x000 ALT0 0x0
#define VF610_PAD_PTD28__FB_AD28 0x108 0x000 ALT1 0x0
#define VF610_PAD_PTD28__NF_IO12 0x108 0x000 ALT2 0x0
#define VF610_PAD_PTD28__I2C2_SCL 0x108 0x34C ALT3 0x1
--
2.35.1
From: Marcel Ziswiler <[email protected]>
Avoid the following checkpatch warning:
arch/arm/boot/dts/imx8mp-verdin.dtsi:281: check: Please don't use
multiple blank lines
Signed-off-by: Marcel Ziswiler <[email protected]>
---
arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi
index c5987bdbb383..f062fdbb2719 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi
@@ -286,7 +286,6 @@ &flexcan1 {
status = "disabled";
};
-
/* Verdin CAN_2 */
&flexcan2 {
pinctrl-names = "default";
--
2.35.1
From: Marcel Ziswiler <[email protected]>
Avoid the following checkpatch warning:
arch/arm/boot/dts/imx8mq-mnt-reform2.dts:213: check: Please don't use
multiple blank lines
Signed-off-by: Marcel Ziswiler <[email protected]>
---
arch/arm64/boot/dts/freescale/imx8mq-mnt-reform2.dts | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mq-mnt-reform2.dts b/arch/arm64/boot/dts/freescale/imx8mq-mnt-reform2.dts
index 8956a46788fa..055031bba8c4 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq-mnt-reform2.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mq-mnt-reform2.dts
@@ -210,7 +210,6 @@ &pwm2 {
status = "okay";
};
-
®_1p8v {
vin-supply = <®_main_5v>;
};
--
2.35.1
From: Marcel Ziswiler <[email protected]>
Avoid the following checkpatch warning:
Found possible blank line(s) at end of file
'arch/arm/boot/dts/imx8mm-venice-gw72xx-0x.dts'
Signed-off-by: Marcel Ziswiler <[email protected]>
---
arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x.dts | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x.dts
index b1e7540f0281..641be3af989d 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x.dts
@@ -17,4 +17,3 @@ chosen {
stdout-path = &uart2;
};
};
-
--
2.35.1
From: Marcel Ziswiler <[email protected]>
Avoid the following checkpatch warning:
include/dt-bindings/clock/imx8mm-clock.h:284: check: Please don't use
multiple blank lines
Signed-off-by: Marcel Ziswiler <[email protected]>
---
include/dt-bindings/clock/imx8mm-clock.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/include/dt-bindings/clock/imx8mm-clock.h b/include/dt-bindings/clock/imx8mm-clock.h
index 47c6f7f9582c..1f768b2eeb1a 100644
--- a/include/dt-bindings/clock/imx8mm-clock.h
+++ b/include/dt-bindings/clock/imx8mm-clock.h
@@ -281,7 +281,6 @@
#define IMX8MM_CLK_CLKOUT2_DIV 256
#define IMX8MM_CLK_CLKOUT2 257
-
#define IMX8MM_CLK_END 258
#endif
--
2.35.1
From: Marcel Ziswiler <[email protected]>
Avoid the following checkpatch warning:
arch/arm/boot/dts/imx7d-pico.dtsi:44: error: code indent should use tabs
where possible
arch/arm/boot/dts/imx7d-pico.dtsi:44: warning: please, no spaces at the
start of a line
Signed-off-by: Marcel Ziswiler <[email protected]>
---
arch/arm/boot/dts/imx7d-pico.dtsi | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/arch/arm/boot/dts/imx7d-pico.dtsi b/arch/arm/boot/dts/imx7d-pico.dtsi
index e519897fae08..e0bff39e8d3e 100644
--- a/arch/arm/boot/dts/imx7d-pico.dtsi
+++ b/arch/arm/boot/dts/imx7d-pico.dtsi
@@ -41,7 +41,7 @@ reg_lcd_3v3: regulator-lcd-3v3 {
regulator-max-microvolt = <3300000>;
gpio = <&gpio1 6 GPIO_ACTIVE_HIGH>;
enable-active-high;
- };
+ };
reg_wlreg_on: regulator-wlreg_on {
compatible = "regulator-fixed";
@@ -432,7 +432,7 @@ MX7D_PAD_ENET1_RGMII_RD1__ENET1_RGMII_RD1 0x1
MX7D_PAD_ENET1_RGMII_RD2__ENET1_RGMII_RD2 0x1
MX7D_PAD_ENET1_RGMII_RD3__ENET1_RGMII_RD3 0x1
MX7D_PAD_ENET1_RGMII_RX_CTL__ENET1_RGMII_RX_CTL 0x1
- MX7D_PAD_SD3_RESET_B__GPIO6_IO11 0x1 /* Ethernet reset */
+ MX7D_PAD_SD3_RESET_B__GPIO6_IO11 0x1 /* Ethernet reset */
>;
};
@@ -493,19 +493,19 @@ MX7D_PAD_LCD_RESET__GPIO3_IO4 0x14
pinctrl_pwm1: pwm1 {
fsl,pins = <
- MX7D_PAD_GPIO1_IO08__PWM1_OUT 0x7f
+ MX7D_PAD_GPIO1_IO08__PWM1_OUT 0x7f
>;
};
pinctrl_pwm2: pwm2 {
fsl,pins = <
- MX7D_PAD_GPIO1_IO09__PWM2_OUT 0x7f
+ MX7D_PAD_GPIO1_IO09__PWM2_OUT 0x7f
>;
};
pinctrl_pwm3: pwm3 {
fsl,pins = <
- MX7D_PAD_GPIO1_IO10__PWM3_OUT 0x7f
+ MX7D_PAD_GPIO1_IO10__PWM3_OUT 0x7f
>;
};
--
2.35.1
From: Marcel Ziswiler <[email protected]>
Avoid the following checkpatch warning:
arch/arm/boot/dts/vf610.dtsi:5: check: Please don't use multiple blank
lines
Signed-off-by: Marcel Ziswiler <[email protected]>
---
arch/arm/boot/dts/vf610.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/boot/dts/vf610.dtsi b/arch/arm/boot/dts/vf610.dtsi
index 956182d08e74..2fba923821d0 100644
--- a/arch/arm/boot/dts/vf610.dtsi
+++ b/arch/arm/boot/dts/vf610.dtsi
@@ -2,7 +2,6 @@
//
// Copyright 2013 Freescale Semiconductor, Inc.
-
#include "vf500.dtsi"
&a5_cpu {
--
2.35.1
From: Marcel Ziswiler <[email protected]>
Replace the license text with SPDX identifiers.
Avoid the following checkpatch warning:
arch/arm/boot/dts/imx6qdl-sabrelite.dtsi:1: warning: Missing or
malformed SPDX-License-Identifier tag in line 1
Signed-off-by: Marcel Ziswiler <[email protected]>
---
arch/arm/boot/dts/imx6qdl-sabrelite.dtsi | 37 +-----------------------
1 file changed, 1 insertion(+), 36 deletions(-)
diff --git a/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi b/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi
index 901b9a761b66..32f3c92bcc52 100644
--- a/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi
@@ -1,43 +1,8 @@
+// SPDX-License-Identifier: GPL-2.0-or-later OR X11
/*
* Copyright 2011 Freescale Semiconductor, Inc.
* Copyright 2011 Linaro Ltd.
*
- * This file is dual-licensed: you can use it either under the terms
- * of the GPL or the X11 license, at your option. Note that this dual
- * licensing only applies to this file, and not this project as a
- * whole.
- *
- * a) This file is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License
- * version 2 as published by the Free Software Foundation.
- *
- * This file is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * Or, alternatively,
- *
- * b) Permission is hereby granted, free of charge, to any person
- * obtaining a copy of this software and associated documentation
- * files (the "Software"), to deal in the Software without
- * restriction, including without limitation the rights to use,
- * copy, modify, merge, publish, distribute, sublicense, and/or
- * sell copies of the Software, and to permit persons to whom the
- * Software is furnished to do so, subject to the following
- * conditions:
- *
- * The above copyright notice and this permission notice shall be
- * included in all copies or substantial portions of the Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
- * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
- * OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
- * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
- * HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
- * WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
- * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
- * OTHER DEALINGS IN THE SOFTWARE.
*/
#include <dt-bindings/clock/imx6qdl-clock.h>
--
2.35.1
Am Donnerstag, dem 21.07.2022 um 21:59 +0200 schrieb Marcel Ziswiler:
> From: Marcel Ziswiler <[email protected]>
>
> Avoid the following checkpatch warning:
>
> arch/arm/boot/dts/imx8mq-mnt-reform2.dts:213: check: Please don't use
> multiple blank lines
>
> Signed-off-by: Marcel Ziswiler <[email protected]>
Acked-by: Lucas Stach <[email protected]>
> ---
>
> arch/arm64/boot/dts/freescale/imx8mq-mnt-reform2.dts | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mq-mnt-reform2.dts b/arch/arm64/boot/dts/freescale/imx8mq-mnt-reform2.dts
> index 8956a46788fa..055031bba8c4 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mq-mnt-reform2.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mq-mnt-reform2.dts
> @@ -210,7 +210,6 @@ &pwm2 {
> status = "okay";
> };
>
> -
> ®_1p8v {
> vin-supply = <®_main_5v>;
> };