Re: [hg:v4l-dvb] gspca - main: Fix a compile error when CONFIG_INPUT is not set

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 03 Mar 2010 00:45:02 +0100
Patch from Jean-Fran?ois Moine  <hg-commit@xxxxxxxxxxx> wrote:

> The patch number 14343 was added via Douglas Schilling Landgraf
> <dougsland@xxxxxxxxxx> to http://linuxtv.org/hg/v4l-dvb master
> development tree.
> 
> Kernel patches in this development tree may be modified to be backward
> compatible with older kernels. Compatibility modifications will be
> removed before inclusion into the mainstream Kernel
> 
> If anyone has any objections, please let us know by sending a message
> to: Linux Media Mailing List <linux-media@xxxxxxxxxxxxxxx>
> 
> ------
> 
> From: Jean-Fran?ois Moine  <moinejf@xxxxxxx>
> gspca - main: Fix a compile error when CONFIG_INPUT is not set
> 
> 
> Reported-by: Randy Dunlap <randy.dunlap@xxxxxxxxxx>
> 
> Priority: normal
> 
> [dougsland@xxxxxxxxxx: patch backported to hg tree]
> Signed-off-by: Jean-Fran?ois Moine <moinejf@xxxxxxx>
> Signed-off-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>
> Signed-off-by: Douglas Schilling Landgraf <dougsland@xxxxxxxxxx>
> 
> 
> ---
> 
>  linux/drivers/media/video/gspca/gspca.c |    6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff -r c533329e3c41 -r 6519c63ecf6d
> linux/drivers/media/video/gspca/gspca.c ---
> a/linux/drivers/media/video/gspca/gspca.c	Tue Mar 02 20:16:17
> 2010 -0300 +++ b/linux/drivers/media/video/gspca/gspca.c	Tue
> Mar 02 20:38:01 2010 -0300 @@ -44,10 +44,12 @@ 
>  #include "gspca.h"
>  
> +#ifdef CONFIG_INPUT
>  #if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 19)
>  #include <linux/input.h>
>  #include <linux/usb/input.h>
>  #endif
> +#endif
>  
>  /* global values */
>  #define DEF_NURBS 3		/* default number of URBs */
> @@ -2371,9 +2373,11 @@
>  void gspca_disconnect(struct usb_interface *intf)
>  {
>  	struct gspca_dev *gspca_dev = usb_get_intfdata(intf);
> +#ifdef CONFIG_INPUT
>  #if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 19)
>  	struct input_dev *input_dev;
>  #endif
> +#endif
>  
>  	PDEBUG(D_PROBE, "%s disconnect",
>  		video_device_node_name(&gspca_dev->vdev));
> @@ -2385,6 +2389,7 @@
>  		wake_up_interruptible(&gspca_dev->wq);
>  	}
>  
> +#ifdef CONFIG_INPUT
>  #if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 19)
>  	gspca_input_destroy_urb(gspca_dev);
>  	input_dev = gspca_dev->input_dev;
> @@ -2393,6 +2398,7 @@
>  		input_unregister_device(input_dev);
>  	}
>  #endif
> +#endif
>  
>  	/* the device is freed at exit of this function */
>  	gspca_dev->dev = NULL;
> 
> 
> ---
> 
> Patch is available at:
> http://linuxtv.org/hg/v4l-dvb/rev/6519c63ecf6d4e7e2c1c3d46ac2a161da8d6c6f4

Hello Douglas,

I do not understand your patch. Do you mean that the input events
cannot be used with kernel < 2.6.19, while CONFIG_INPUT can be set?

Anyway, this patch seems complex. It would have been easier to simply
unset CONFIG_INPUT when kernel < 2.6.19.

I join the diff of gspca.c between v4l-dvb and my repository. This last
one is closer to the git version and there are still other changes done
in git. How do you think I should merge?

Cheers.

-- 
Ken ar c'hentañ	|	      ** Breizh ha Linux atav! **
Jef		|		http://moinejf.free.fr/

Attachment: gspca.dif
Description: Binary data


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux