Hello, On Wed, May 13, 2009 at 12:39 AM, <[email protected]> wrote: > From: Alessio Igor Bogani <[email protected]> > > radio-mr800.c uses struct mutex, so while <linux/mutex.h> seems to be > pulled in indirectly by one of the headers it already includes, the right > thing is to include it directly. > > Signed-off-by: Alessio Igor Bogani <[email protected]> > Cc: Mauro Carvalho Chehab <[email protected]> > Signed-off-by: Andrew Morton <[email protected]> > --- > > drivers/media/radio/radio-mr800.c | 1 + > 1 file changed, 1 insertion(+) > > diff -puN > drivers/media/radio/radio-mr800.c~radio-mr800c-missing-mutex-include > drivers/media/radio/radio-mr800.c > --- a/drivers/media/radio/radio-mr800.c~radio-mr800c-missing-mutex-include > +++ a/drivers/media/radio/radio-mr800.c > @@ -64,6 +64,7 @@ > #include <media/v4l2-ioctl.h> > #include <linux/usb.h> > #include <linux/version.h> /* for KERNEL_VERSION MACRO */ > +#include <linux/mutex.h> > > /* driver and module definitions */ > #define DRIVER_AUTHOR "Alexey Klimov <[email protected]>" > _
There was discussion about this patch. http://www.mail-archive.com/[email protected]/msg03556.html Well, i'm not against this patch. -- Best regards, Klimov Alexey -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
