public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Matt Doran <matt.doran@papercut.com>
To: Andy Walls <awalls@radix.net>
Cc: David Ward <david.ward@gatech.edu>, linux-media@vger.kernel.org
Subject: Re: videodev: Unknown symbol i2c_unregister_device (in kernels older than 2.6.26)
Date: Tue, 26 May 2009 11:42:18 +1000	[thread overview]
Message-ID: <4A1B48FA.8010909@papercut.com> (raw)
In-Reply-To: <1243276377.3167.15.camel@palomino.walls.org>

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


  reply	other threads:[~2009-05-26  1:42 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-24 23:10 videodev: Unknown symbol i2c_unregister_device (in kernels older than 2.6.26) Matt Doran
2009-05-25  0:52 ` David Ward
2009-05-25  1:04   ` Matt Doran
2009-05-25 18:32   ` Andy Walls
2009-05-26  1:42     ` Matt Doran [this message]
2009-05-27 18:41 ` Mauro Carvalho Chehab
2009-05-27 18:55   ` [PATCH v2] " Randy Dunlap
2009-06-03 11:51   ` Matt Doran
     [not found]   ` <4A26637D.1070009@papercut.com>
2009-06-03 12:06     ` Mauro Carvalho Chehab

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4A1B48FA.8010909@papercut.com \
    --to=matt.doran@papercut.com \
    --cc=awalls@radix.net \
    --cc=david.ward@gatech.edu \
    --cc=linux-media@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox