Re: [PATCH v3 1/2] media: dt-bindings: Add ST VD56G3 camera sensor

[Sylvain Petinot]

Hi Krzysztof,

Thanks for the review.

On 9/5/2024 8:41 AM, Krzysztof Kozlowski wrote:
> On Wed, Sep 04, 2024 at 04:52:37PM +0200, Sylvain Petinot wrote:
>> Add devicetree bindings Documentation for ST VD56G3 & ST VD66GY camera
>> sensors. Update MAINTAINERS file.
>>
> 
>> +            port {
>> +                endpoint {
>> +                    data-lanes = <1 2>;
>> +                    link-frequencies = /bits/ 64 <402000000>;
>> +                    remote-endpoint = <&csiphy0_ep>;
>> +                };
>> +            };
>> +        };
>> +    };
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index e6d77bea5db5..e58deb5b3047 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -21561,6 +21561,14 @@ S:	Maintained
>>  F:	Documentation/hwmon/stpddc60.rst
>>  F:	drivers/hwmon/pmbus/stpddc60.c
>>  
>> +ST VD56G3 IMAGE SENSOR DRIVER
>> +M:	Benjamin Mugnier <benjamin.mugnier@xxxxxxxxxxx>
>> +M:	Sylvain Petinot <sylvain.petinot@xxxxxxxxxxx>
>> +L:	linux-media@xxxxxxxxxxxxxxx
>> +S:	Maintained
>> +F:	Documentation/devicetree/bindings/media/i2c/st,vd56g3.yaml
>> +F:	drivers/media/i2c/vd56g3.c
> 
> There is no such file.

My bad, fixed for V4.

> 
> 
>> +
>>  ST VGXY61 DRIVER
>>  M:	Benjamin Mugnier <benjamin.mugnier@xxxxxxxxxxx>
>>  M:	Sylvain Petinot <sylvain.petinot@xxxxxxxxxxx>
>> @@ -23947,6 +23955,7 @@ F:	drivers/media/i2c/mt*
>>  F:	drivers/media/i2c/og*
>>  F:	drivers/media/i2c/ov*
>>  F:	drivers/media/i2c/s5*
>> +F:	drivers/media/i2c/vd56g3.c
> 
> Neither this one.

Ditto

> 
> After fixing the paths:
> 
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> 
> Best regards,
> Krzysztof


--
Sylvain