Hi Felipe, On Monday 27 June 2011 10:20:37 Felipe Balbi wrote: > On Fri, Jun 24, 2011 at 12:31:43AM +0200, Jesper Juhl wrote: > > It was pointed out by 'make versioncheck' that linux/version.h was not > > always being included where needed and sometimes included needlessly > > in drivers/usb/. > > This patch fixes up the includes. > > > > Signed-off-by: Jesper Juhl <jj@xxxxxxxxxxxxx> > > Laurent, I guess Jesper needs your Ack for the uvc part. It looks good > to me, but I'll leave it to you. For the UVC gadget driver, Acked-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> Sorry for the delay. > > --- > > > > drivers/usb/gadget/uvc.h | 1 + > > drivers/usb/gadget/uvc_v4l2.c | 1 - > > drivers/usb/storage/realtek_cr.c | 1 - > > 3 files changed, 1 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/usb/gadget/uvc.h b/drivers/usb/gadget/uvc.h > > index 5b79194..81971fb 100644 > > --- a/drivers/usb/gadget/uvc.h > > +++ b/drivers/usb/gadget/uvc.h > > @@ -56,6 +56,7 @@ struct uvc_event > > > > #include <linux/usb.h> /* For usb_endpoint_* */ > > #include <linux/usb/gadget.h> > > #include <linux/videodev2.h> > > > > +#include <linux/version.h> > > > > #include <media/v4l2-fh.h> > > > > #include "uvc_queue.h" > > > > diff --git a/drivers/usb/gadget/uvc_v4l2.c > > b/drivers/usb/gadget/uvc_v4l2.c index 5e807f0..a715805 100644 > > --- a/drivers/usb/gadget/uvc_v4l2.c > > +++ b/drivers/usb/gadget/uvc_v4l2.c > > @@ -16,7 +16,6 @@ > > > > #include <linux/errno.h> > > #include <linux/list.h> > > #include <linux/mutex.h> > > > > -#include <linux/version.h> > > > > #include <linux/videodev2.h> > > #include <linux/vmalloc.h> > > #include <linux/wait.h> > > > > diff --git a/drivers/usb/storage/realtek_cr.c > > b/drivers/usb/storage/realtek_cr.c index d509a4a..fe21104 100644 > > --- a/drivers/usb/storage/realtek_cr.c > > +++ b/drivers/usb/storage/realtek_cr.c > > @@ -26,7 +26,6 @@ > > > > #include <linux/sched.h> > > #include <linux/workqueue.h> > > #include <linux/kernel.h> > > > > -#include <linux/version.h> > > > > #include <scsi/scsi.h> > > #include <scsi/scsi_cmnd.h> -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html