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

"[email protected]" <[email protected]>
Newsgroups org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel,org.kernel.vger.linux-media
Message-ID <TY6PR01MB17839A212637F4865314A3FE292D32@TY6PR01MB17839.jpnprd01.prod.outlook.com>
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 <[email protected]>
>>>> +L:	[email protected]
>>>
>>> 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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.