Re: [PATCH 1/2] media: dt-bindings: imx908: Add Sony IMX908 sensor bindings

[Lachlan.Michael@xxxxxxxx]

Dear Kyrsztof,

Thank-you for your review.

<snip>

>>> To spare you one sending cycle:
>>>
>>> A nit, subject: drop second/last, redundant "bindings". The
>>> "dt-bindings" prefix is already stating that these are bindings.

Ok, will fix this in v2.

<snip>

>>>> +SONY IMX908 SENSOR DRIVER
>>>> +M:	Lachlan Michael <lachlan.michael@xxxxxxxx>
>>>> +L:	linux-media@xxxxxxxxxxxxxxx
>>>
>>> Drop
>> 
>> Why so ?
>
>It's just redundant and bloat. The address will ALWAYS appear through
>the subsystem maintainers entry, thus duplicating exactly the same list
>is not necessary. Many drivers make the same mistake (I fixed Samsung
>now) resulting in quite big amount of redundant data.

Ok, I will drop this line in v2.

Best Regards,

Lachlan