2023-01-19 09:19:25

by Marcel Ziswiler

Subject: [RESEND PATCH v1 0/2] ARM: arm64: dts/clk: imx: indentation permission whitespace cleanup 2

From: Marcel Ziswiler <[email protected]>


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

2023-01-19 09:24:45

by Marcel Ziswiler

Subject: [RESEND PATCH v1 1/2] clk: imx6sll: add proper spdx license identifier

From: Marcel Ziswiler <[email protected]>

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 <[email protected]>
---

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

2023-01-29 17:37:56

by Abel Vesa

Subject: Re: [RESEND PATCH v1 1/2] clk: imx6sll: add proper spdx license identifier

On 23-01-19 09:54:20, Marcel Ziswiler wrote:
> From: Marcel Ziswiler <[email protected]>
>
> 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 <[email protected]>

Reviewed-by: Abel Vesa <[email protected]>

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

2023-01-29 17:42:40

by Abel Vesa

Subject: Re: [RESEND PATCH v1 0/2] ARM: arm64: dts/clk: imx: indentation permission whitespace cleanup 2

On 23-01-19 09:54:19, Marcel Ziswiler wrote:
> From: Marcel Ziswiler <[email protected]>
>
>
> 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
>