Re: [PATCH] Adding YUV input support for OMAP3ISP driver

[Gary Thomas <gary@mlbassoc.com>]

On 2012-01-19 07:11, Javier Martinez Canillas wrote:
> On Thu, Jan 19, 2012 at 2:35 PM, Gary Thomas<gary@mlbassoc.com>  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:
>>>>>>       <snip>
>>>>>>>
>>>>>>>
>>>>>>> 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.

Thanks


-- 
------------------------------------------------------------
Gary Thomas                 |  Consulting for the
MLB Associates              |    Embedded world
------------------------------------------------------------