From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from hermes.mlbassoc.com ([64.234.241.98]:52834 "EHLO mail.chez-thomas.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752112Ab2ASRjl (ORCPT ); Thu, 19 Jan 2012 12:39:41 -0500 Message-ID: <4F185555.2030607@mlbassoc.com> Date: Thu, 19 Jan 2012 10:39:33 -0700 From: Gary Thomas MIME-Version: 1.0 To: Laurent Pinchart CC: linux-media@vger.kernel.org Subject: Re: [PATCH] Adding YUV input support for OMAP3ISP driver References: <201201191350.51761.laurent.pinchart@ideasonboard.com> <4F181711.1020201@mlbassoc.com> <201201191428.35340.laurent.pinchart@ideasonboard.com> <4F181C24.9030806@mlbassoc.com> <4F182A79.6000603@mlbassoc.com> <4F184E0B.3010402@mlbassoc.com> <4F185171.9080504@mlbassoc.com> In-Reply-To: <4F185171.9080504@mlbassoc.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: On 2012-01-19 10:22, Gary Thomas wrote: > On 2012-01-19 10:08, Gary Thomas wrote: >> On 2012-01-19 07:36, Gary Thomas wrote: >>> On 2012-01-19 07:11, Javier Martinez Canillas wrote: >>>> On Thu, Jan 19, 2012 at 2:35 PM, Gary Thomas wrote: >>>>> On 2012-01-19 06:28, Laurent Pinchart wrote: >>>>>> >>>>>> Hi Gary, >>>>>> >>>>>> On Thursday 19 January 2012 14:13:53 Gary Thomas wrote: >>>>>>> >>>>>>> On 2012-01-19 05:50, Laurent Pinchart wrote: >>>>>>>> >>>>>>>> On Thursday 19 January 2012 13:41:57 Gary Thomas wrote: >>>>>>>>> >>>>>>>>> On 2012-01-17 08:33, Laurent Pinchart wrote: >>>>>>>>> >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> I already had a couple of YUV support patches in my OMAP3 ISP tree at >>>>>>>>>> git.kernel.org. I've rebased them on top of the lastest V4L/DVB tree >>>>>>>>>> and pushed them to >>>>>>>>>> >>>>>>>>>> http://git.linuxtv.org/pinchartl/media.git/shortlog/refs/heads/omap3isp >>>>>>>>>> - omap3isp-yuv. Could you please try them, and see if they're usable >>>>>>>>>> with your sensor ? >>>>>>>>> >>>>>>>>> >>>>>>>>> I just tried this kernel with my board. The media control >>>>>>>>> infrastructure comes up and all of the devices are created, but I can't >>>>>>>>> access them. >>>>>>>>> >>>>>>>>> From the bootup log: >>>>>>>>> Linux media interface: v0.10 >>>>>>>>> Linux video capture interface: v2.00 >>>>>>>> >>>>>>>> >>>>>>>> Any message from the omap3isp driver and from the sensor driver ? >>>>>>> >>>>>>> >>>>>>> No, it doesn't appear that the sensor was probed (or maybe it failed but >>>>>>> no messages). I'll check into this. >>>>>> >>>>>> >>>>>> Is the omap3-isp driver compiled as a module ? If so, make sure iommu2.ko >>>>>> is >>>>>> loaded first. 'rmmod omap3-isp&& modprobe iommu2&& modprobe omap3-isp' >>>>>> is a >>>>>> >>>>>> quick way to test it. >>>>> >>>>> >>>>> I have everything compiled in - no modules. >>>>> >>>> >>>> At least for me, it only worked when compiled both the omap3-isp and >>>> tvp5150 drivers as a module. If I compile them built-in, it fails. >>> >>> Can you share your board/sensor init code from your board-init.c >>> so I can see how to manage this as a module? >>> >>> n.b. I really don't like messing with modules - it used to work >>> fine, so IMO it should continue to do so. >> >> I figured out part of the problem - I had tried to reuse my 3.0 kernel >> config. Sadly, this left out major chunks, in particular the OMAP3ISP >> was left out because OMAP_IOVMMU is new. I got this configured and it's >> starting to initialize, but now it fails during boot: >> kernel BUG at /local/pinchartl-media/drivers/media/media-entity.c:348 >> [] (media_entity_create_link+0x60/0x138) from [] (isp_probe+0x938/0xba4) >> [] (isp_probe+0x938/0xba4) from [] (platform_drv_probe+0x1c/0x24) >> [] (platform_drv_probe+0x1c/0x24) from [] (driver_probe_device+0xcc/0x1b4) >> [] (driver_probe_device+0xcc/0x1b4) from [] (bus_for_each_drv+0x4c/0x8c) >> [] (bus_for_each_drv+0x4c/0x8c) from [] (device_attach+0x74/0xa0) >> [] (device_attach+0x74/0xa0) from [] (bus_probe_device+0x28/0x50) >> [] (bus_probe_device+0x28/0x50) from [] (device_add+0x40c/0x590) >> [] (device_add+0x40c/0x590) from [] (platform_device_add+0x108/0x168) >> [] (platform_device_add+0x108/0x168) from [] (cobra3530p73_camera_init+0x13c/0x188) >> [] (cobra3530p73_camera_init+0x13c/0x188) from [] (do_one_initcall+0x94/0x15c) >> [] (do_one_initcall+0x94/0x15c) from [] (kernel_init+0x78/0x120) >> [] (kernel_init+0x78/0x120) from [] (kernel_thread_exit+0x0/0x8) >> >> Any ideas what else I might have missed? My kernel config is attached in case >> that helps. > > It turns out that drivers/media/video/tvp5150.c is not the most > recent one posted - it has no v4l2_subdev support in it at all :-( > > I copied the one I've been using from my 3.0+ kernel and it > now builds. I can run media-ctl and configure the pipeline, > etc, but sadly no data is captured at all :-( > It also looks like none of the BT656 support is in this driver either. -- ------------------------------------------------------------ Gary Thomas | Consulting for the MLB Associates | Embedded world ------------------------------------------------------------