On Sun 29 April 2012 18:30:24 Laurent Pinchart wrote: > 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 ? Acked-by: Hans Verkuil <hans.verkuil@xxxxxxxxx> Regards, Hans > > > 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; > > -- 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
