Messages in this thread | | | From | Laurent Pinchart <> | Subject | Re: [PATCH 1/6] usb: uvc: include videodev2.h in g_uvc.h | Date | Wed, 10 Oct 2018 16:42:51 +0300 |
| |
Hi Paul,
Thank you for the patch.
On Wednesday, 10 October 2018 05:48:58 EEST Paul Elder wrote: > V4L2_EVENT_PRIVATE_START is used in g_uvc.h but is defined in > videodev2.h, which is not included and causes a compiler warning: > > linux/usb/g_uvc.h:15:28: error: ‘V4L2_EVENT_PRIVATE_START’ undeclared here > (not in a function) #define UVC_EVENT_FIRST (V4L2_EVENT_PRIVATE_START + > 0) > > Include videodev2.h in g_uvc.h. > > Signed-off-by: Paul Elder <paul.elder@ideasonboard.com> > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
This is independent of the rest of the series so I've taken it in my tree already, with the subject line modified to use "usb: gadget: uvc:" as a prefix.
> --- > include/uapi/linux/usb/g_uvc.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/include/uapi/linux/usb/g_uvc.h b/include/uapi/linux/usb/g_uvc.h > index 3c9ee3020cbb..6698c3263ae8 100644 > --- a/include/uapi/linux/usb/g_uvc.h > +++ b/include/uapi/linux/usb/g_uvc.h > @@ -11,6 +11,7 @@ > #include <linux/ioctl.h> > #include <linux/types.h> > #include <linux/usb/ch9.h> > +#include <linux/videodev2.h> > > #define UVC_EVENT_FIRST (V4L2_EVENT_PRIVATE_START + 0) > #define UVC_EVENT_CONNECT (V4L2_EVENT_PRIVATE_START + 0)
-- Regards,
Laurent Pinchart
| |