Re: [REVIEW PATCH for v3.15 1/4] v4l2-subdev.h: fix sparse error with v4l2_subdev_notify

2014-03-17 Thread Laurent Pinchart
Hi Hans, On Monday 17 March 2014 12:45:16 Hans Verkuil wrote: > On 03/17/2014 12:44 PM, Laurent Pinchart wrote: > > On Saturday 15 March 2014 14:08:00 Hans Verkuil wrote: > >> From: Hans Verkuil > >> > >> The notify function is a void function, yet the v4l2_subdev_notify > >> define uses it in a

Re: [REVIEW PATCH for v3.15 1/4] v4l2-subdev.h: fix sparse error with v4l2_subdev_notify

2014-03-17 Thread Hans Verkuil
On 03/17/2014 12:44 PM, Laurent Pinchart wrote: > Hi Hans, > > Thank you for the patch. > > On Saturday 15 March 2014 14:08:00 Hans Verkuil wrote: >> From: Hans Verkuil >> >> The notify function is a void function, yet the v4l2_subdev_notify >> define uses it in a ? : construction, which causes

Re: [REVIEW PATCH for v3.15 1/4] v4l2-subdev.h: fix sparse error with v4l2_subdev_notify

2014-03-17 Thread Laurent Pinchart
Hi Hans, Thank you for the patch. On Saturday 15 March 2014 14:08:00 Hans Verkuil wrote: > From: Hans Verkuil > > The notify function is a void function, yet the v4l2_subdev_notify > define uses it in a ? : construction, which causes sparse warnings. > > Signed-off-by: Hans Verkuil > --- > i

[REVIEW PATCH for v3.15 1/4] v4l2-subdev.h: fix sparse error with v4l2_subdev_notify

2014-03-15 Thread Hans Verkuil
From: Hans Verkuil The notify function is a void function, yet the v4l2_subdev_notify define uses it in a ? : construction, which causes sparse warnings. Signed-off-by: Hans Verkuil --- include/media/v4l2-subdev.h | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/incl