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

From: Krzysztof Kozlowski
Date: Wed Jan 10 2024 - 04:18:16 EST


On 10/01/2024 10:15, Hugues FRUCHET wrote:
>> 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:

I wonder how did you check worked... Open existing other file, open your
file. Look at yours. Then look at others. Then again look at yours.
>
> +++ 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 ?

You have a redundant blank line after SPDX.

Best regards,
Krzysztof