On Tue, Jun 26, 2018 at 1:53 PM, Maxime Ripard <maxime.ripard@xxxxxxxxxxx> wrote: > Hi, > > On Tue, Jun 26, 2018 at 11:41:56AM +0530, Jagan Teki wrote: >> On Mon, Mar 5, 2018 at 3:34 PM, Maxime Ripard <maxime.ripard@xxxxxxxxxxx> wrote: >> > The H3 and H5 have a CSI controller based on the one previously found >> > in the A31, that is currently supported by the sun6i-csi driver. >> > >> > Add the compatibles to the device tree bindings and to the driver to >> > make it work properly. >> > >> > This obviously depends on the serie "Initial Allwinner V3s CSI >> > Support" by Yong Deng. >> > >> > Let me know what you think, >> > Maxime >> > >> > Maxime Ripard (2): >> > dt-bindings: media: sun6i: Add A31 and H3 compatibles >> > media: sun6i: Add A31 compatible >> > >> > Mylène Josserand (2): >> > ARM: dts: sun8i: Add the H3/H5 CSI controller >> > [DO NOT MERGE] ARM: dts: sun8i: Add CAM500B camera module to the Nano >> > Pi M1+ >> >> Just trying to understand what interface has been tested with npi-m1+, >> is it DVP (parallel) interface? > > Yes > >> I've Bananapi 5MP[1] and trying to test on top, and look like its >> MIPI CSI2. I guess Yong patch[2] doesn't support CSI2 yet, am I >> correct? > > This is what Yong's cover letter is saying yes :) Thanks, and this ov5640 tested on -next or any specific code? please provide if you have different code base. Jagan.