Hi Alexey, On Sunday 15 January 2012 10:09:54 Oleksij Rempel (Alexey Fisher) wrote: > hi Laurent, > > this patch seem to create circular module dependency. I get this error: > WARNING: Module > /lib/modules/3.2.0-00660-g1801bbe-dirty/kernel/drivers/media/video/videodev > .ko ignored, due to loop > WARNING: Loop detected: > /lib/modules/3.2.0-00660-g1801bbe-dirty/kernel/drivers/media/video/v4l2-com > pat-ioctl32.ko needs videodev.ko which needs v4l2-compat-ioctl32.ko again! Thanks for the report. Hans, what do you think about the patch below ? diff --git a/drivers/media/video/Makefile b/drivers/media/video/Makefile index 3541388..8c4a94d 100644 --- a/drivers/media/video/Makefile +++ b/drivers/media/video/Makefile @@ -17,7 +17,7 @@ videodev-objs := v4l2-dev.o v4l2-ioctl.o v4l2-device.o v4l2-fh.o \ obj-$(CONFIG_VIDEO_DEV) += videodev.o v4l2-int-device.o ifeq ($(CONFIG_COMPAT),y) - obj-$(CONFIG_VIDEO_DEV) += v4l2-compat-ioctl32.o + videodev-objs += v4l2-compat-ioctl32.o endif obj-$(CONFIG_VIDEO_V4L2_COMMON) += v4l2-common.o I don't see a very compelling reason to put v4l2_compat_ioctl32() in a separate module. If that fine with you, I'll also remove the #ifdef CONFIG_COMPAT from v4l2-compat-ioctl32.c. > commit bf5aa456853816f807a46c0d944efb997142ffaf > Author: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > Date: Mon Dec 19 00:41:19 2011 +0100 > > v4l: Add custom compat_ioctl32 operation > > Drivers implementing custom ioctls need to handle 32-bit/64-bit > compatibility themselves. Provide them with a way to do so. > > Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > Reviewed-by: Hans Verkuil <hans.verkuil@xxxxxxxxx> -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
