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 <[email protected]> Regards, Hans > > > Signed-off-by: Laurent Pinchart <[email protected]> > > --- > > 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
