Hi Dongchun, Rob, On Tue, May 12, 2020 at 4:41 AM Dongchun Zhu <dongchun.zhu@xxxxxxxxxxxx> wrote: > > Hi Rob, > > Thanks for the review. > > On Mon, 2020-05-11 at 11:02 -0500, Rob Herring wrote: > > On Sat, May 09, 2020 at 04:06:26PM +0800, Dongchun Zhu wrote: [snip] > > > + port { > > > + wcam_out: endpoint { > > > + remote-endpoint = <&mipi_in_wcam>; > > > + data-lanes = <1>; > > > > This doesn't match the schema which says this should be 4 entries. > > > > Property "data-lanes" shows the number of lanes that sensor supports. > If this property is omitted, four-lane operation is assumed. > For OV02A10, it is one-lane operation, so the property is supposed to be > set to <1>. > To clarify on this, the ov02a10 sensor supports only 1 lane. It's not a driver limitation. Best regards, Tomasz