mbox series

[v2,00/12] ARM: arm64: dts/clk: imx8mm: indentation whitespace cleanup

Message ID 20220722215445.3548530-1-marcel@ziswiler.com
Headers show
Series ARM: arm64: dts/clk: imx8mm: indentation whitespace cleanup | expand

Message

Marcel Ziswiler July 22, 2022, 9:54 p.m. UTC
From: Marcel Ziswiler <marcel.ziswiler@toradex.com>


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.

Changes in v2:
- The GPL part of the boilerplate license was actually GPL-2.0 only.
- Add Alexander's acked-by. Thanks!
- Add Lucas' acked-by. Thanks!

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(-)

Comments

Laurent Pinchart July 23, 2022, 11:08 p.m. UTC | #1
Hi Marcel,

Thank you for the patch.

On Fri, Jul 22, 2022 at 11:54:43PM +0200, Marcel Ziswiler wrote:
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> 
> 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 <marcel.ziswiler@toradex.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
> 
> (no changes since v1)
> 
>  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";
Fabio Estevam July 25, 2022, 10:55 a.m. UTC | #2
Hi Marcel,

On Fri, Jul 22, 2022 at 6:55 PM Marcel Ziswiler <marcel@ziswiler.com> wrote:
>
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
>
>
> 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.

Thanks for doing this cleanup:

Reviewed-by: Fabio Estevam <festevam@gmail.com>
Shawn Guo Aug. 21, 2022, 7:52 a.m. UTC | #3
On Fri, Jul 22, 2022 at 11:54:33PM +0200, Marcel Ziswiler wrote:
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> 
> 
> 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.
> 
> Changes in v2:
> - The GPL part of the boilerplate license was actually GPL-2.0 only.
> - Add Alexander's acked-by. Thanks!
> - Add Lucas' acked-by. Thanks!
> 
> 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

Applied all, thanks!