Hi Sakari, On Sun, 2019-08-18 at 00:32 +0300, Sakari Ailus wrote: > Hi Dongchun, > > On Sat, Aug 17, 2019 at 11:53:05AM +0800, Dongchun Zhu wrote: > ... > > > > + > > > > +The device node shall contain one 'port' child node with an > > > > +'endpoint' subnode for its digital output video port, in accordance > > > > +with the video interface bindings defined in > > > > +Documentation/devicetree/bindings/media/video-interfaces.txt. > > > > +The endpoint optional property 'data-lanes' shall be "<1>". > > > > > > How many lanes does the module (or the sensor) have? > > > > > > > From sensor datasheet, OV02A10 supports 1-lane MIPI interface. > > If only one lane is supported, the property should be omitted: there's > nothing that the driver needs to know here. > Thanks for the suggestion. But sorry that I didn't read this message this morning, so this issue still remains unsettled in v3. https://patchwork.kernel.org/patch/11100219/ I would fix this point in v4.
