From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from relay1.mentorg.com ([192.94.38.131]:62289 "EHLO relay1.mentorg.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754801AbcCCCDA (ORCPT ); Wed, 2 Mar 2016 21:03:00 -0500 Subject: Re: i.mx6 camera interface (CSI) and mainline kernel To: Laurent Pinchart , Philippe De Muyter References: <20160223114943.GA10944@frolo.macqel> <20160223141258.GA5097@frolo.macqel> <4956050.OLrYA1VK2G@avalon> CC: , Philipp Zabel From: Steve Longerbeam Message-ID: <56D79B49.50009@mentor.com> Date: Wed, 2 Mar 2016 18:02:49 -0800 MIME-Version: 1.0 In-Reply-To: <4956050.OLrYA1VK2G@avalon> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: On 02/25/2016 02:05 PM, Laurent Pinchart wrote: > Hello Philippe, > > CC'ing Philipp and Steve. > > Philipp, Steve, are you still interested in getting a driver for the i.MX6 > camera interface upstreamed ? Hi Laurent, Philippe(s), I spent a few days updating my mx6-media-staging branch at git@github.com:slongerbeam/linux-meibp-314.git, moved forward to latest master at 4.5-rc3. So far I have retested video capture with the SabreAuto/ADV7180 and the SabreSD/OV5640-mipi-csi2, and video capture is working fine on those platforms. There is also a mem2mem that should work fine, but haven't tested yet. I removed camera preview support. At Mentor Graphics we have made quite a few changes to ipu-v3 driver to allow camera preview to initialize and control an overlay display plane independently of imx-drm, by adding a subsystem independent ipu-plane sub-unit. Note we also have a video output overlay driver that also makes use of ipu-plane. But those changes are extensive and touch imx-drm as well as ipu-v3, so I am leaving camera preview and the output overlay driver out (in fact, camera preview is not of much utility so I probably won't bring it back in upstream version). The video capture driver is not quite ready for upstream review yet. It still: - uses the old cropping APIs but should move forward to selection APIs. - uses custom sensor subdev drivers for ADV7180 and OV564x. Still need to switch to upstream subdevs. - still does not implement the media device framework. Steve > > On Tuesday 23 February 2016 15:12:58 Philippe De Muyter wrote: >> Update. >> >> On Tue, Feb 23, 2016 at 12:49:43PM +0100, Philippe De Muyter wrote: >>> Hello, >>> >>> We use a custom imx6 based board with a canera sensor on it. >>> I have written the driver for the camera sensor, based on >>> the freescale so-called "3.10" and even "3.14" linux versions. >>> >>> The camera works perfectly, but we would like to switch to >>> a mainline kernel for all the usual reasons (including being >>> able to contribute our fixes). >>> >>> >From an old mail thread (*), I have found two git repositories >>> >>> that used to contain not-yet-approved versions of mainline >>> imx6 ipu-v3 drivers : >>> >>> git://git.pengutronix.de/git/pza/linux.git test/nitrogen6x-ipu-media >>> https://github.com:slongerbeam/mediatree.git, mx6-camera-staging >>> >>> I have tried to compile them with the imx_v6_v7_defconfig, but both >>> fail directly at compile time. because of later changes in the >>> v4l2_subdev infrastructure, not ported to the those branches. >> What I wrote is true for Steve Longerbeam's branch, but for Philipp Zabel's >> branch the problem (so far) was only that CONFIG_MEDIA_CONTROLLER >> is not defined in imx_v6_v7_defconfig, but is required for a succesfull >> compilation of Philipp's tree. >> >>> Can someone point me to compilable versions (either not rebased >>> versions of those branches, or updated versions of those branches, >>> or yet another place to look at). ? >>> >>> Thanks in advance >>> >>> Philippe >>> >>> (*) >>> http://linux-media.vger.kernel.narkive.com/cZQ8NrZ2/i-mx6-status-for-ipu-> > vpu-gpu