On Monday 23 April 2012 13:29:11 Laurent Pinchart wrote: > This fixes the following warning: > > In file included from drivers/media/video/v4l2-subdev.c:29: > include/media/v4l2-ctrls.h:501: warning: 'struct file' declared inside > parameter list > include/media/v4l2-ctrls.h:501: warning: its scope is only this > definition or declaration, which is probably not what you want > include/media/v4l2-ctrls.h:509: warning: 'struct file' declared inside > parameter list Ping ? Should I include this in my next pull request ? > Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > include/media/v4l2-ctrls.h | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/include/media/v4l2-ctrls.h b/include/media/v4l2-ctrls.h > index 33907a9..9022e1c 100644 > --- a/include/media/v4l2-ctrls.h > +++ b/include/media/v4l2-ctrls.h > @@ -25,6 +25,7 @@ > #include <linux/videodev2.h> > > /* forward references */ > +struct file; > struct v4l2_ctrl_handler; > struct v4l2_ctrl_helper; > struct v4l2_ctrl; -- 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
