Re: [PATCH v2] dt-bindings: rcc: stm32: Sync with u-boot copy for STM32MP13 SoC

Conor Dooley
2023-05-10T17:02:51Z

[-- Attachment #1: Type: text/plain, Size: 1543 bytes --]

On Wed, May 10, 2023 at 06:51:40PM +0200, Patrick Delaunay wrote:
> Minor cosmetic change, aligned with files in U-Boot:
> - change obsolete SPDX id : GPL-2.0+ and use the same license
> GPL-2.0-only for the 2 files
> - use correct mail address gabriel.fernandez@foss.st.com
> - remove extra space

Rob's comments on v1 were about $subject, which you've fixed, although
the second prefix is a bit odd (I assume since you're changing clock and
reset headers) and about the change in license to "or-later" for the
resets header, which you've changed too, so...
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
> ---
>
> Changes in v2:
> - Change title "Fix STM32MP13 include file"
> - Use the same license for the 2 files GPL-2.0-only as recommended
> to avoid check-patch issue on next
>
> 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..0bd7b54c65ff 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-only OR BSD-3-Clause */

...since Gabriel does appear to be the only author of this file:

Acked-by: Conor Dooley <conor.dooley@microchip.com>

Thanks,
Conor.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]