Re: [PATCH 1/2] media: dt-bindings: imx908: Add Sony IMX908 sensor bindings
| Newsgroups | org.kernel.vger.linux-media,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel |
|---|---|
| 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