Hi Steve, On Thu, Mar 03, 2016 at 08:19:55AM +0100, Hans Verkuil wrote: > On 03/03/2016 03:02 AM, Steve Longerbeam wrote: > > 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 First of all, thank you very much, Steve. > > git@xxxxxxxxxx:slongerbeam/linux-meibp-314.git, moved forward > > to latest master at 4.5-rc3. Just to be sure : do you mean https://github.com/slongerbeam/mediatree.git or something else ? > > > > 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. Is it only a problem of those sensor drivers (which exact files ?) or is it a problem of the capture driver itself ? I must update a sensor driver I wrote for the intdev interface found in the freescale kernel, and I'd like to start from a working subdev example. Which driver should I choose as an example ? > > > > - still does not implement the media device framework. > > After fixing the first two items on that list, would that be a good time to > add this driver to staging? > > i.MX6 is quite popular, so having support for this device upstream would be > very nice indeed. > > I'd really like to see some upstream support for this SoC soon. There are a bunch of people expecting that (and trying to help) :) Best regards Philippe > > Regards, > > Hans > > > > > > > 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 > > > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-media" in > > the body of a message to majordomo@xxxxxxxxxxxxxxx > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- Philippe De Muyter +32 2 6101532 Macq SA rue de l'Aeronef 2 B-1140 Bruxelles -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html