Re: [PATCH v4 6/8] media: dt-bindings: Add bindings for JH7110 Camera Subsystem

Jack Zhu
2023-04-17T09:22:51Z

On 2023/4/15 16:59, Krzysztof Kozlowski wrote:
> On 13/04/2023 05:55, Jack Zhu wrote:
>> Add the bindings documentation for Starfive JH7110 Camera Subsystem
>> which is used for handing image sensor data.
>>
>> Signed-off-by: Jack Zhu <jack.zhu@starfivetech.com>
>
> Subject: drop second/last, redundant "bindings for". The "dt-bindings"
> prefix is already stating that these are bindings.
>

OK, I will fix it. Thank you.

>> ---
>> .../bindings/media/starfive,jh7110-camss.yaml | 164 ++++++++++++++++++
>> MAINTAINERS | 7 +
>> 2 files changed, 171 insertions(+)
>> create mode 100644 Documentation/devicetree/bindings/media/starfive,jh7110-camss.yaml
>>
>> diff --git a/Documentation/devicetree/bindings/media/starfive,jh7110-camss.yaml b/Documentation/devicetree/bindings/media/starfive,jh7110-camss.yaml
>> new file mode 100644
>> index 000000000000..4cd144f1b845
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/media/starfive,jh7110-camss.yaml
>> @@ -0,0 +1,164 @@
>> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>> +
>
> No need for blank line.
>

OK, I will delete the blank line.

> With two fixes above:
>
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>
> Best regards,
> Krzysztof
>