Re: videodev: Unknown symbol i2c_unregister_device (in kernels older than 2.6.26)

2009-06-03 Thread Matt Doran

Mauro Carvalho Chehab wrote:

Em Mon, 25 May 2009 09:10:17 +1000
Matt Doran matt.do...@papercut.com escreveu:

  

Hi there,

I tried using the latest v4l code on an Mythtv box running 2.6.20, but 
the v4l videodev module fails to load with the following warnings:


videodev: Unknown symbol i2c_unregister_device
v4l2_common: Unknown symbol v4l2_device_register_subdev


It seems the i2c_unregister_device function was added in 2.6.26.   
References to this function in v4l2-common.c are enclosed in an ifdef like:


#if LINUX_VERSION_CODE = KERNEL_VERSION(2, 6, 26)


However in v4l2_device_unregister() in v4l2-device.c, there is a 
reference to i2c_unregister_device without any ifdefs.   I am running 
a pretty old kernel, but I'd guess anyone running 2.6.25 or earlier will 
have this problem.   It seems this code was added by Mauro 3 weeks ago 
in this rev:


http://linuxtv.org/hg/v4l-dvb/rev/87afa7a4ccdf



I've just applied a patch at the tree that should fix this issue. It adds
several tests and the code, but, hopefully, it should be possible even to use
the IR's with kernels starting from 2.6.16.


  
Thanks Mauro. 

I've recompiled all drivers without compile error and I've been using 
everything for a few days now and it all works great.


Thanks again!
Matt

--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: videodev: Unknown symbol i2c_unregister_device (in kernels older than 2.6.26)

2009-05-25 Thread Matt Doran

Andy Walls wrote:

On Sun, 2009-05-24 at 20:52 -0400, David Ward wrote:
  

On 05/24/2009 07:10 PM, Matt Doran wrote:


Hi there,

I tried using the latest v4l code on an Mythtv box running 2.6.20, but
the v4l videodev module fails to load with the following warnings:

   videodev: Unknown symbol i2c_unregister_device
   v4l2_common: Unknown symbol v4l2_device_register_subdev


It seems the i2c_unregister_device function was added in 2.6.26.
References to this function in v4l2-common.c are enclosed in an ifdef
like:

   #if LINUX_VERSION_CODE = KERNEL_VERSION(2, 6, 26)


However in v4l2_device_unregister() in v4l2-device.c, there is a
reference to i2c_unregister_device without any ifdefs.   I am
running a pretty old kernel, but I'd guess anyone running 2.6.25 or
earlier will have this problem.   It seems this code was added by
Mauro 3 weeks ago in this rev:

   http://linuxtv.org/hg/v4l-dvb/rev/87afa7a4ccdf

  
I have been seeing the errors compiling adv7343.c and ths7303.c under 
2.6.24 as well.  Andy Walls and Chaithrika Subrahmanya had written 
patches for those two modules respectively, but there were some comments 
during the review of the patches, so I think they are still being worked on.



Well, just to manage expectations: I am not working on this.  I do not
advise waiting for something from me. ;)


As an end user, you work-around is to use make menuconfig (or
whatever) as Matt did: disable the modules that aren't compiling on
older kernels.

  
I agree, but the main problem I raise is the use of 
i2c_unregister_device in the main v4l module on Linux kernels that 
don't support it.


Regards,
Matt

--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


videodev: Unknown symbol i2c_unregister_device (in kernels older than 2.6.26)

2009-05-24 Thread Matt Doran

Hi there,

I tried using the latest v4l code on an Mythtv box running 2.6.20, but 
the v4l videodev module fails to load with the following warnings:


   videodev: Unknown symbol i2c_unregister_device
   v4l2_common: Unknown symbol v4l2_device_register_subdev


It seems the i2c_unregister_device function was added in 2.6.26.   
References to this function in v4l2-common.c are enclosed in an ifdef like:


   #if LINUX_VERSION_CODE = KERNEL_VERSION(2, 6, 26)


However in v4l2_device_unregister() in v4l2-device.c, there is a 
reference to i2c_unregister_device without any ifdefs.   I am running 
a pretty old kernel, but I'd guess anyone running 2.6.25 or earlier will 
have this problem.   It seems this code was added by Mauro 3 weeks ago 
in this rev:


   http://linuxtv.org/hg/v4l-dvb/rev/87afa7a4ccdf




I also had some other compile problems, but don't have all the details 
(sorry!).  I had to disable the following drivers to get it to compile:


   * CONFIG_VIDEO_PVRUSB2
   * CONFIG_VIDEO_THS7303
   * CONFIG_VIDEO_ADV7343
   * CONFIG_DVB_SIANO_SMS1XXX


Regards,
Matt


--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: videodev: Unknown symbol i2c_unregister_device (in kernels older than 2.6.26)

2009-05-24 Thread Matt Doran

David Ward wrote:

On 05/24/2009 07:10 PM, Matt Doran wrote:

Hi there,

I tried using the latest v4l code on an Mythtv box running 2.6.20, but
the v4l videodev module fails to load with the following warnings:

   videodev: Unknown symbol i2c_unregister_device
   v4l2_common: Unknown symbol v4l2_device_register_subdev



Matt, I checked out v4l-dvb today and am using it under 2.6.24 and so 
far so good.  When did the error appear -- when you were trying to 
load the module?
The error appeared when trying to load the module at boot time for my 
saa7134 based tuner card.   This card would no longer work after 
installing the latest v4l code, however another tuner card continued to 
work.Maybe because this the saa7134 card is an I2C based card and 
the other is USB based?? (but this is all a bit over my head).


I basically just commented out the i2c_unregister_device function in 
v4l2-device.c, recompiled and everything started working. :)I don't 
know the implication of removing this, so I didn't submit a patch ... I 
thought I'd leave that to the experts.


I have been seeing the errors compiling adv7343.c and ths7303.c under 
2.6.24 as well.  Andy Walls and Chaithrika Subrahmanya had written 
patches for those two modules respectively, but there were some 
comments during the review of the patches, so I think they are still 
being worked on.
Great, thanks for letting me know.   It's not a showstopper, but it's a 
pain repeatedly going through the reconfigure/compile/error loop each 
time you encounter a new compiler error. :)I'm just not game to 
upgrade the kernel on my mythtv box ... it took a long time to get 
stable, and I don't want to go there again. :PI'm recompiling v4l 
because I added a new tuner card.


Matt
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html