Re: [PATCH v6 1/5] dt-bindings: media: Document STM32MP25 VDEC & VENC video codecs

From: Hugues FRUCHET
Date: Wed Jan 10 2024 - 04:16:11 EST


Hi Krzysztof,

On 1/9/24 22:49, Krzysztof Kozlowski wrote:
On 09/01/2024 09:51, Hugues Fruchet wrote:
Add STM32MP25 VDEC video decoder & VENC video encoder bindings.

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
Signed-off-by: Hugues Fruchet <hugues.fruchet@xxxxxxxxxxx>

Please use scripts/get_maintainers.pl to get a list of necessary people
and lists to CC. It might happen, that command when run on an older
kernel, gives you outdated entries. Therefore please be sure you base
your patches on recent Linux kernel.

You missed at least devicetree list (maybe more), so this won't be
tested by automated tooling.

Please kindly resend and include all necessary To/Cc entries.

My fault when sending this exact v6, sorry for that.. It was done the right way for previous versions and no issues raised by automated tools.


While resending drop redundant blank line after SPDX. See all other
bindings and example-schema.
I have double checked and I don't see any problem related to redundant blank lines in yaml after SPDX:

+++ b/Documentation/devicetree/bindings/media/st,stm32mp25-video-codec.yaml
@@ -0,0 +1,50 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+
+%YAML 1.2

Could you point me out what is the exact issue ?


Best regards,
Krzysztof