From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rui Miguel Silva Subject: Re: [PATCH v7 00/12] media: staging/imx7: add i.MX7 media driver Date: Wed, 07 Nov 2018 10:04:33 +0000 Message-ID: References: <20180810142045.27657-1-rui.silva@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-reply-to: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: driverdev-devel-bounces@linuxdriverproject.org Sender: "devel" To: Hans Verkuil Cc: devel@driverdev.osuosl.org, devicetree@vger.kernel.org, sakari.ailus@linux.intel.com, Greg Kroah-Hartman , Ryan Harkin , Rob Herring , Philipp Zabel , Fabio Estevam , mchehab@kernel.org, linux-media@vger.kernel.org List-Id: devicetree@vger.kernel.org Hi Hans, On Wed 07 Nov 2018 at 09:58, Hans Verkuil wrote: > Hi Rui, > > On 08/10/18 16:20, Rui Miguel Silva wrote: >> Hi, >> This series introduces the Media driver to work with the i.MX7 >> SoC. it uses the >> already existing imx media core drivers but since the i.MX7, >> contrary to >> i.MX5/6, do not have an IPU and because of that some changes in >> the imx media >> core are made along this series to make it support that case. >> >> This patches adds CSI and MIPI-CSI2 drivers for i.MX7, along >> with several >> configurations changes for this to work as a capture subsystem. >> Some bugs are >> also fixed along the line. And necessary documentation. >> >> For a more detailed view of the capture paths, pads links in >> the i.MX7 please >> take a look at the documentation in PATCH 14. >> >> The system used to test and develop this was the Warp7 board >> with an OV2680 >> sensor, which output format is 10-bit bayer. So, only MIPI >> interface was >> tested, a scenario with an parallel input would nice to have. >> >> >> Bellow goes an example of the output of the pads and links and >> the output of >> v4l2-compliance testing. >> >> The v4l-utils version used is: >> v4l2-compliance SHA : >> 90905c2e4b17d7595256f3824e2d30d19b0df1a1 from Aug 6th >> >> The Media Driver fail some tests but this failures are coming >> from code out of >> scope of this series (imx-capture), and some from the sensor >> OV2680 >> but that I think not related with the sensor driver but with >> the testing and >> core. >> >> The csi and mipi-csi entities pass all compliance tests. > > Can you rebase and repost? Please re-run v4l2-compliance as well > with the latest > compliance code. Yup, I may take one week or so, other things in hand, but will try to send it sooner than later. > > We should be able to merge this for 4.21 (finally!). Yeah!! Thanks. --- Cheers, Rui