From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-it0-f66.google.com ([209.85.214.66]:40576 "EHLO mail-it0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752826AbeFZJMN (ORCPT ); Tue, 26 Jun 2018 05:12:13 -0400 Received: by mail-it0-f66.google.com with SMTP id 188-v6so1257963ita.5 for ; Tue, 26 Jun 2018 02:12:13 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <20180626082330.pvq3gi6mmokky5vl@flea> References: <20180305100432.15009-1-maxime.ripard@bootlin.com> <20180626082330.pvq3gi6mmokky5vl@flea> From: Jagan Teki Date: Tue, 26 Jun 2018 14:42:11 +0530 Message-ID: Subject: Re: [PATCH 0/4] media: sun6i: Add support for the H3 CSI controller To: Maxime Ripard Cc: Mauro Carvalho Chehab , Yong Deng , Mark Rutland , Rob Herring , devicetree , linux-media@vger.kernel.org, Thomas Petazzoni , Mylene Josserand , Hans Verkuil , Sakari Ailus , linux-arm-kernel , Chen-Yu Tsai Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Sender: linux-media-owner@vger.kernel.org List-ID: On Tue, Jun 26, 2018 at 1:53 PM, Maxime Ripard 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 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=C3=A8ne Josserand (2): >> > ARM: dts: sun8i: Add the H3/H5 CSI controller >> > [DO NOT MERGE] ARM: dts: sun8i: Add CAM500B camera module to the Nan= o >> > 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.