[PATCH 0/3] Maverick ti-omap4, Modularize syslink drivers

[Sebastien Jan]

Looping Mathieu and Nicolas

+ See my answers inlined

On 06/30/2010 11:21 AM, Bryan Wu wrote:
> On 06/30/2010 03:25 AM, Tim Gardner wrote:
>> Bryan,
>>
>> TI is interested in modularizing some of the syslink drivers. The first 2
>> patches are required before that can be done. Please have a look to make
>> sure its the right thing to do. I assume the right way to get everything loaded is to add
>> notify_ducatidriver to /etc/modules.
>>
> 
> Sebjan, will you confirm this request? IIRC, you mentioned we need built-in 
> syslink driver before. And if we modularize it, does TI notice this failure 
> which Tim fixed in this patchset?

Yes: up to now, our syslink driver used to be statically linked mainly because it did not support beeing built as a module. After working with our dev team, this is now fixed. I have just push some patches for that in our dev.omapzoom tree.

Now we were wondering which was the best way to auto install the syslink driver modules in the system, hence the discussion with Mathieu and Tim.

I did not try yet any auto install strategy, and have not seen the patches from Tim, so I cannot comment further.

> 
>> There is also a wicked symbol reference dependency between 2 modules that you ought to have a look at:
>>
>> (maverick-armel)root at tyler:/home/rtg/kern/maverick/armel/junk# depmod -b `pwd` 2.6.34-901-omap4
>> WARNING: Module /lib/modules/2.6.34-901-omap4/kernel/drivers/dsp/syslink/multicore_ipc/syslink_ipc.ko ignored, due to loop
>> WARNING: Loop detected: /home/rtg/kern/maverick/armel/junk/lib/modules/2.6.34-901-omap4/kernel/drivers/dsp/syslink/ipu_pm/syslink_ipu_pm.ko needs syslink_ipc.ko which needs syslink_ipu_pm.ko again!
>> WARNING: Module /lib/modules/2.6.34-901-omap4/kernel/drivers/dsp/syslink/ipu_pm/syslink_ipu_pm.ko ignored, due to loop
>>
> 
> That's weird. I will take a look.
> 
>> Anyways, try this out on your hardware.
>>
> 
> Sorry, I still don't have the hardware. But I built the kernel for testing:
> http://people.canonical.com/~roc/kernel/syslink/
> 
> Orga and Lee, could you please help this?
> 
> Thanks,
> -Bryan