* v4l-dvb rev 11757 broke building under Ubuntu Hardy
@ 2009-05-13 5:09 David Ward
0 siblings, 0 replies; 6+ messages in thread
From: David Ward @ 2009-05-13 5:09 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: linux-media
I am using v4l-dvb in order to add the cx18 driver under Ubuntu Hardy
(8.04).
The build is currently broken under Hardy, which uses kernel 2.6.24. I
have traced the origin of the problem to revision 11757. As seen in the
latest cron job output, the build produces the error when trying to
compile adv7343.c:
/usr/local/src/v4l-dvb/v4l/adv7343.c:506: error: array type has
incomplete element type
/usr/local/src/v4l-dvb/v4l/adv7343.c:518: warning: initialization from
incompatible pointer type
/usr/local/src/v4l-dvb/v4l/adv7343.c:520: error: unknown field
'id_table' specified in initializer
Thanks for resolving this.
David Ward
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: v4l-dvb rev 11757 broke building under Ubuntu Hardy
2009-05-14 23:53 Andy Walls
2009-05-15 13:01 ` chaithrika
@ 2009-05-24 6:00 ` David Ward
1 sibling, 0 replies; 6+ messages in thread
From: David Ward @ 2009-05-24 6:00 UTC (permalink / raw)
To: Andy Walls; +Cc: Chaithrika U S, linux-media
On 05/14/2009 07:53 PM, Andy Walls wrote:
> David Ward wrote:
>
>> I am using v4l-dvb in order to add the cx18 driver under Ubuntu Hardy
>> (8.04).
>>
>> The build is currently broken under Hardy, which uses kernel 2.6.24. I
>> have traced the origin of the problem to revision 11757. As seen in
>> the latest cron job output, the build produces the error when trying
>> to compile adv7343.c:
>>
>> /usr/local/src/v4l-dvb/v4l/adv7343.c:506: error: array type has incomplete element type
>> /usr/local/src/v4l-dvb/v4l/adv7343.c:518: warning: initialization from incompatible pointer type
>> /usr/local/src/v4l-dvb/v4l/adv7343.c:520: error: unknown field 'id_table' specified in initializer
>>
>> Thanks for resolving this.
>>
>> David Ward
>>
> David,
>
> Please try the patch below.
>
> Chaithrika,
>
> Please review (and test if it is OK) the patch below. It modifies
> adv7343.c to what the cs5345.c file does for backward compatability.
>
> It adds some checks against kernel version, which would not go into the
> actual kernel, and changes some code to use the v4l2 i2c module template
> from v4l2-i2c-drv.h, which *would* go into the actual kenrel.
>
>
> Regards,
> Andy
>
>
Andy and Chaithrika, sorry for the late reply. Is a different patch
being created to replace the ones you posted already, for adv7343.c and
ths7303.c? This is still broken in the repository.
Andy, your initial patch at least did resolve the build errors for
adv7343.c under the Ubuntu Hardy kernel (2.6.24).
Thanks,
David
> Signed-off-by: Andy Walls<awalls@radix.net>
>
> diff -r 0018ed9bbca3 linux/drivers/media/video/adv7343.c
> --- a/linux/drivers/media/video/adv7343.c Tue May 12 16:13:13 2009 +0000
> +++ b/linux/drivers/media/video/adv7343.c Thu May 14 19:51:10 2009 -0400
> @@ -29,6 +29,8 @@
> #include<media/adv7343.h>
> #include<media/v4l2-device.h>
> #include<media/v4l2-chip-ident.h>
> +#include<media/v4l2-i2c-drv.h>
> +#include "compat.h"
>
> #include "adv7343_regs.h"
>
> @@ -503,6 +505,7 @@
> return 0;
> }
>
> +#if LINUX_VERSION_CODE>= KERNEL_VERSION(2, 6, 26)
> static const struct i2c_device_id adv7343_id[] = {
> {"adv7343", 0},
> {},
> @@ -510,25 +513,12 @@
>
> MODULE_DEVICE_TABLE(i2c, adv7343_id);
>
> -static struct i2c_driver adv7343_driver = {
> - .driver = {
> - .owner = THIS_MODULE,
> - .name = "adv7343",
> - },
> +#endif
> +static struct v4l2_i2c_driver_data v4l2_i2c_data = {
> + .name = "adv7343",
> .probe = adv7343_probe,
> .remove = adv7343_remove,
> +#if LINUX_VERSION_CODE>= KERNEL_VERSION(2, 6, 26)
> .id_table = adv7343_id,
> +#endif
> };
> -
> -static __init int init_adv7343(void)
> -{
> - return i2c_add_driver(&adv7343_driver);
> -}
> -
> -static __exit void exit_adv7343(void)
> -{
> - i2c_del_driver(&adv7343_driver);
> -}
> -
> -module_init(init_adv7343);
> -module_exit(exit_adv7343)
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: v4l-dvb rev 11757 broke building under Ubuntu Hardy
2009-05-15 13:01 ` chaithrika
@ 2009-05-15 16:08 ` Andy Walls
0 siblings, 0 replies; 6+ messages in thread
From: Andy Walls @ 2009-05-15 16:08 UTC (permalink / raw)
To: chaithrika; +Cc: david.ward, linux-media
On Fri, 2009-05-15 at 18:31 +0530, chaithrika wrote:
> Andy,
>
> Thanks for the patch. Please see my response below.
>
> > -----Original Message-----
> > From: Andy Walls [mailto:awalls@radix.net]
> > Sent: Friday, May 15, 2009 5:24 AM
> > To: david.ward@gatech.edu
> > Cc: Chaithrika U S; linux-media@vger.kernel.org
> > Subject: Re: v4l-dvb rev 11757 broke building under Ubuntu Hardy
> >
> >
> > David Ward wrote:
> >
> > > I am using v4l-dvb in order to add the cx18 driver under Ubuntu Hardy
> > > (8.04).
> > >
> > > The build is currently broken under Hardy, which uses kernel 2.6.24.
> > I
> > > have traced the origin of the problem to revision 11757. As seen in
> > > the latest cron job output, the build produces the error when trying
> > > to compile adv7343.c:
> > >
> > > /usr/local/src/v4l-dvb/v4l/adv7343.c:506: error: array type has
> > incomplete element type
> > > /usr/local/src/v4l-dvb/v4l/adv7343.c:518: warning: initialization
> > from incompatible pointer type
> > > /usr/local/src/v4l-dvb/v4l/adv7343.c:520: error: unknown field
> > 'id_table' specified in initializer
> > >
> > > Thanks for resolving this.
> > >
> > > David Ward
> >
> > David,
> >
> > Please try the patch below.
> >
> > Chaithrika,
> >
> > Please review (and test if it is OK) the patch below. It modifies
> > adv7343.c to what the cs5345.c file does for backward compatability.
> >
> > It adds some checks against kernel version, which would not go into the
> > actual kernel, and changes some code to use the v4l2 i2c module
> > template
> > from v4l2-i2c-drv.h, which *would* go into the actual kenrel.
> >
> >
> > Regards,
> > Andy
> >
> >
> > Signed-off-by: Andy Walls <awalls@radix.net>
> >
> > diff -r 0018ed9bbca3 linux/drivers/media/video/adv7343.c
> > --- a/linux/drivers/media/video/adv7343.c Tue May 12 16:13:13 2009
> > +0000
> > +++ b/linux/drivers/media/video/adv7343.c Thu May 14 19:51:10 2009 -
> > 0400
> > @@ -29,6 +29,8 @@
> > #include <media/adv7343.h>
> > #include <media/v4l2-device.h>
> > #include <media/v4l2-chip-ident.h>
> > +#include <media/v4l2-i2c-drv.h>
> > +#include "compat.h"
> >
> > #include "adv7343_regs.h"
> >
> > @@ -503,6 +505,7 @@
> > return 0;
> > }
> >
> > +#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 26)
> > static const struct i2c_device_id adv7343_id[] = {
> > {"adv7343", 0},
> > {},
> > @@ -510,25 +513,12 @@
> >
> > MODULE_DEVICE_TABLE(i2c, adv7343_id);
> >
> > -static struct i2c_driver adv7343_driver = {
> > - .driver = {
> > - .owner = THIS_MODULE,
> > - .name = "adv7343",
> > - },
> > +#endif
> > +static struct v4l2_i2c_driver_data v4l2_i2c_data = {
> > + .name = "adv7343",
> > .probe = adv7343_probe,
> > .remove = adv7343_remove,
> > +#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 26)
> > .id_table = adv7343_id,
> > +#endif
> > };
> > -
> > -static __init int init_adv7343(void)
> > -{
> > - return i2c_add_driver(&adv7343_driver);
> > -}
> > -
> > -static __exit void exit_adv7343(void)
> > -{
> > - i2c_del_driver(&adv7343_driver);
> > -}
> > -
> > -module_init(init_adv7343);
> > -module_exit(exit_adv7343);
> >
>
> During my initial implementations, I did follow the v4l2_i2c_driver_data
> procedure.
> But after the review comment I got, I changed it. Please refer to the link
> below.
>
> http://www.mail-archive.com/linux-media@vger.kernel.org/msg02973.html
Oh, OK. What prompted me to suggest that was the adv7343_probe()
function had the wrong prototype (the incompatible pointer type warning)
for that 2.6.24 kernel. The v4l2_i2c_driver_data method was the easiest
way around it.
Perhaps it would just be easier to add the adv7343 driver to the
[2.6.26] section of v4l/versions.txt and forget about backward
compatibility for this module.
> Was wondering if this is related to kernel version and is there a
> possibility that in the
> above patch, the module init and exit functions can be included in kernel
> version checks
> instead of removing them altogether.
Well if Hans said v4l2-i2c-drv.h is going away, it's best to avoid
trying to use it.
If you want to provide backward compatibility for kernels older than
2.6.26, then the 2 version checks that I put in are needed. You'll need
to figure out the rest of the i2c structure differences and wrap those
in kernel version checks too.
If you don't want to provide backward compatibility, then you'll need to
add an entry in v4l/versions.txt so that builds on older kernels won't
try to compile the module.
Regards,
Andy
> Regards,
> Chaithrika
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: v4l-dvb rev 11757 broke building under Ubuntu Hardy
2009-05-14 23:53 Andy Walls
@ 2009-05-15 13:01 ` chaithrika
2009-05-15 16:08 ` Andy Walls
2009-05-24 6:00 ` David Ward
1 sibling, 1 reply; 6+ messages in thread
From: chaithrika @ 2009-05-15 13:01 UTC (permalink / raw)
To: 'Andy Walls', david.ward; +Cc: linux-media
Andy,
Thanks for the patch. Please see my response below.
> -----Original Message-----
> From: Andy Walls [mailto:awalls@radix.net]
> Sent: Friday, May 15, 2009 5:24 AM
> To: david.ward@gatech.edu
> Cc: Chaithrika U S; linux-media@vger.kernel.org
> Subject: Re: v4l-dvb rev 11757 broke building under Ubuntu Hardy
>
>
> David Ward wrote:
>
> > I am using v4l-dvb in order to add the cx18 driver under Ubuntu Hardy
> > (8.04).
> >
> > The build is currently broken under Hardy, which uses kernel 2.6.24.
> I
> > have traced the origin of the problem to revision 11757. As seen in
> > the latest cron job output, the build produces the error when trying
> > to compile adv7343.c:
> >
> > /usr/local/src/v4l-dvb/v4l/adv7343.c:506: error: array type has
> incomplete element type
> > /usr/local/src/v4l-dvb/v4l/adv7343.c:518: warning: initialization
> from incompatible pointer type
> > /usr/local/src/v4l-dvb/v4l/adv7343.c:520: error: unknown field
> 'id_table' specified in initializer
> >
> > Thanks for resolving this.
> >
> > David Ward
>
> David,
>
> Please try the patch below.
>
> Chaithrika,
>
> Please review (and test if it is OK) the patch below. It modifies
> adv7343.c to what the cs5345.c file does for backward compatability.
>
> It adds some checks against kernel version, which would not go into the
> actual kernel, and changes some code to use the v4l2 i2c module
> template
> from v4l2-i2c-drv.h, which *would* go into the actual kenrel.
>
>
> Regards,
> Andy
>
>
> Signed-off-by: Andy Walls <awalls@radix.net>
>
> diff -r 0018ed9bbca3 linux/drivers/media/video/adv7343.c
> --- a/linux/drivers/media/video/adv7343.c Tue May 12 16:13:13 2009
> +0000
> +++ b/linux/drivers/media/video/adv7343.c Thu May 14 19:51:10 2009 -
> 0400
> @@ -29,6 +29,8 @@
> #include <media/adv7343.h>
> #include <media/v4l2-device.h>
> #include <media/v4l2-chip-ident.h>
> +#include <media/v4l2-i2c-drv.h>
> +#include "compat.h"
>
> #include "adv7343_regs.h"
>
> @@ -503,6 +505,7 @@
> return 0;
> }
>
> +#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 26)
> static const struct i2c_device_id adv7343_id[] = {
> {"adv7343", 0},
> {},
> @@ -510,25 +513,12 @@
>
> MODULE_DEVICE_TABLE(i2c, adv7343_id);
>
> -static struct i2c_driver adv7343_driver = {
> - .driver = {
> - .owner = THIS_MODULE,
> - .name = "adv7343",
> - },
> +#endif
> +static struct v4l2_i2c_driver_data v4l2_i2c_data = {
> + .name = "adv7343",
> .probe = adv7343_probe,
> .remove = adv7343_remove,
> +#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 26)
> .id_table = adv7343_id,
> +#endif
> };
> -
> -static __init int init_adv7343(void)
> -{
> - return i2c_add_driver(&adv7343_driver);
> -}
> -
> -static __exit void exit_adv7343(void)
> -{
> - i2c_del_driver(&adv7343_driver);
> -}
> -
> -module_init(init_adv7343);
> -module_exit(exit_adv7343);
>
During my initial implementations, I did follow the v4l2_i2c_driver_data
procedure.
But after the review comment I got, I changed it. Please refer to the link
below.
http://www.mail-archive.com/linux-media@vger.kernel.org/msg02973.html
Was wondering if this is related to kernel version and is there a
possibility that in the
above patch, the module init and exit functions can be included in kernel
version checks
instead of removing them altogether.
Regards,
Chaithrika
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: v4l-dvb rev 11757 broke building under Ubuntu Hardy
@ 2009-05-14 23:53 Andy Walls
2009-05-15 13:01 ` chaithrika
2009-05-24 6:00 ` David Ward
0 siblings, 2 replies; 6+ messages in thread
From: Andy Walls @ 2009-05-14 23:53 UTC (permalink / raw)
To: david.ward; +Cc: Chaithrika U S, linux-media
David Ward wrote:
> I am using v4l-dvb in order to add the cx18 driver under Ubuntu Hardy
> (8.04).
>
> The build is currently broken under Hardy, which uses kernel 2.6.24. I
> have traced the origin of the problem to revision 11757. As seen in
> the latest cron job output, the build produces the error when trying
> to compile adv7343.c:
>
> /usr/local/src/v4l-dvb/v4l/adv7343.c:506: error: array type has incomplete element type
> /usr/local/src/v4l-dvb/v4l/adv7343.c:518: warning: initialization from incompatible pointer type
> /usr/local/src/v4l-dvb/v4l/adv7343.c:520: error: unknown field 'id_table' specified in initializer
>
> Thanks for resolving this.
>
> David Ward
David,
Please try the patch below.
Chaithrika,
Please review (and test if it is OK) the patch below. It modifies
adv7343.c to what the cs5345.c file does for backward compatability.
It adds some checks against kernel version, which would not go into the
actual kernel, and changes some code to use the v4l2 i2c module template
from v4l2-i2c-drv.h, which *would* go into the actual kenrel.
Regards,
Andy
Signed-off-by: Andy Walls <awalls@radix.net>
diff -r 0018ed9bbca3 linux/drivers/media/video/adv7343.c
--- a/linux/drivers/media/video/adv7343.c Tue May 12 16:13:13 2009 +0000
+++ b/linux/drivers/media/video/adv7343.c Thu May 14 19:51:10 2009 -0400
@@ -29,6 +29,8 @@
#include <media/adv7343.h>
#include <media/v4l2-device.h>
#include <media/v4l2-chip-ident.h>
+#include <media/v4l2-i2c-drv.h>
+#include "compat.h"
#include "adv7343_regs.h"
@@ -503,6 +505,7 @@
return 0;
}
+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 26)
static const struct i2c_device_id adv7343_id[] = {
{"adv7343", 0},
{},
@@ -510,25 +513,12 @@
MODULE_DEVICE_TABLE(i2c, adv7343_id);
-static struct i2c_driver adv7343_driver = {
- .driver = {
- .owner = THIS_MODULE,
- .name = "adv7343",
- },
+#endif
+static struct v4l2_i2c_driver_data v4l2_i2c_data = {
+ .name = "adv7343",
.probe = adv7343_probe,
.remove = adv7343_remove,
+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 26)
.id_table = adv7343_id,
+#endif
};
-
-static __init int init_adv7343(void)
-{
- return i2c_add_driver(&adv7343_driver);
-}
-
-static __exit void exit_adv7343(void)
-{
- i2c_del_driver(&adv7343_driver);
-}
-
-module_init(init_adv7343);
-module_exit(exit_adv7343);
^ permalink raw reply [flat|nested] 6+ messages in thread
* v4l-dvb rev 11757 broke building under Ubuntu Hardy
@ 2009-05-13 5:21 David Ward
0 siblings, 0 replies; 6+ messages in thread
From: David Ward @ 2009-05-13 5:21 UTC (permalink / raw)
To: linux-media
I am using v4l-dvb in order to add the cx18 driver under Ubuntu Hardy
(8.04).
The build is currently broken under Hardy, which uses kernel 2.6.24. I
have traced the origin of the problem to revision 11757. As seen in the
latest cron job output, the build produces the error when trying to
compile adv7343.c:
/usr/local/src/v4l-dvb/v4l/adv7343.c:506: error: array type has
incomplete element type
/usr/local/src/v4l-dvb/v4l/adv7343.c:518: warning: initialization from
incompatible pointer type
/usr/local/src/v4l-dvb/v4l/adv7343.c:520: error: unknown field
'id_table' specified in initializer
Thanks for resolving this.
David Ward
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2009-05-24 6:00 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-05-13 5:09 v4l-dvb rev 11757 broke building under Ubuntu Hardy David Ward
2009-05-13 5:21 David Ward
2009-05-14 23:53 Andy Walls
2009-05-15 13:01 ` chaithrika
2009-05-15 16:08 ` Andy Walls
2009-05-24 6:00 ` David Ward
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.