linux-kernel - Re: [PATCH] dt-bindings: rcc: stm32: Fix STM32MP13 include file

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20230412151404.GA2372294-robh@kernel.org>
Date:   Wed, 12 Apr 2023 10:14:04 -0500
From:   Rob Herring <robh@...nel.org>
To:     Patrick Delaunay <patrick.delaunay@...s.st.com>
Cc:     Alexandre TORGUE <alexandre.torgue@...s.st.com>,
        Michael Turquette <mturquette@...libre.com>,
        Stephen Boyd <sboyd@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Maxime Coquelin <mcoquelin.stm32@...il.com>,
        Philipp Zabel <p.zabel@...gutronix.de>,
        gabriel.fernandez@...com, gabriel.fernandez@...s.st.com,
        devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        linux-clk@...r.kernel.org, linux-kernel@...r.kernel.org,
        linux-stm32@...md-mailman.stormreply.com
Subject: Re: [PATCH] dt-bindings: rcc: stm32: Fix STM32MP13 include file

On Fri, Apr 07, 2023 at 08:15:33PM +0200, Patrick Delaunay wrote:
> Minor cosmetic change, aligned with files in U-Boot:
> - change obsolete SPDX id : GPL-2.0+
> - use correct address gabriel.fernandez@...s.st.com
> - remove extra space

Make the subject reflect the purpose: 'Sync with u-boot copy'

> 
> Signed-off-by: Patrick Delaunay <patrick.delaunay@...s.st.com>
> ---
> 
>  include/dt-bindings/clock/stm32mp13-clks.h   | 6 +++---
>  include/dt-bindings/reset/stm32mp13-resets.h | 4 ++--
>  2 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/include/dt-bindings/clock/stm32mp13-clks.h b/include/dt-bindings/clock/stm32mp13-clks.h
> index 02befd25edce..da4cb7567430 100644
> --- a/include/dt-bindings/clock/stm32mp13-clks.h
> +++ b/include/dt-bindings/clock/stm32mp13-clks.h
> @@ -1,7 +1,7 @@
> -/* SPDX-License-Identifier: GPL-2.0+ or BSD-3-Clause */
> +/* SPDX-License-Identifier: GPL-2.0-or-later or BSD-3-Clause */
>  /*
>   * Copyright (C) STMicroelectronics 2020 - All Rights Reserved
> - * Author: Gabriel Fernandez <gabriel.fernandez@...com> for STMicroelectronics.
> + * Author: Gabriel Fernandez <gabriel.fernandez@...s.st.com> for STMicroelectronics.
>   */
>  
>  #ifndef _DT_BINDINGS_STM32MP13_CLKS_H_
> @@ -64,7 +64,7 @@
>  #define CK_MCO1		38
>  #define CK_MCO2		39
>  
> -/*  IP clocks */
> +/* IP clocks */
>  #define SYSCFG		40
>  #define VREF		41
>  #define DTS		42
> diff --git a/include/dt-bindings/reset/stm32mp13-resets.h b/include/dt-bindings/reset/stm32mp13-resets.h
> index 934864e90da6..1b83a01de8f0 100644
> --- a/include/dt-bindings/reset/stm32mp13-resets.h
> +++ b/include/dt-bindings/reset/stm32mp13-resets.h
> @@ -1,7 +1,7 @@
> -/* SPDX-License-Identifier: GPL-2.0 or BSD-3-Clause */
> +/* SPDX-License-Identifier: GPL-2.0-or-later or BSD-3-Clause */

Really, the Linux copy came first, so you should use its license. Your 
company's lawyers are good with GPLv3 and then v4?

>  /*
>   * Copyright (C) STMicroelectronics 2018 - All Rights Reserved
> - * Author: Gabriel Fernandez <gabriel.fernandez@...com> for STMicroelectronics.
> + * Author: Gabriel Fernandez <gabriel.fernandez@...s.st.com> for STMicroelectronics.
>   */
>  
>  #ifndef _DT_BINDINGS_STM32MP13_RESET_H_
> -- 
> 2.25.1
>