Re: [PATCH v4 2/3] media: ov7670: Add the get_fmt callback

2017-09-18 Thread Yang, Wenyou
Hi Sakari, On 2017/9/18 15:36, Sakari Ailus wrote: Hi Wenyou, On Mon, Sep 18, 2017 at 02:45:13PM +0800, Wenyou Yang wrote: @@ -998,8 +1002,15 @@ static int ov7670_set_fmt(struct v4l2_subdev *sd, ret = ov7670_try_fmt_internal(sd, >format, NULL, NULL); if (ret)

Re: [PATCH v4 2/3] media: ov7670: Add the get_fmt callback

2017-09-18 Thread Sakari Ailus
Hi Wenyou, On Mon, Sep 18, 2017 at 02:45:13PM +0800, Wenyou Yang wrote: > @@ -998,8 +1002,15 @@ static int ov7670_set_fmt(struct v4l2_subdev *sd, > ret = ov7670_try_fmt_internal(sd, >format, NULL, NULL); > if (ret) > return ret; > -

Re: [PATCH v4 2/3] media: ov7670: Add the get_fmt callback

2017-09-18 Thread kbuild test robot
Hi Wenyou, [auto build test WARNING on linuxtv-media/master] [also build test WARNING on v4.14-rc1 next-20170918] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

[PATCH v4 2/3] media: ov7670: Add the get_fmt callback

2017-09-18 Thread Wenyou Yang
Add the get_fmt callback, also enable V4L2_SUBDEV_FL_HAS_DEVNODE flag to make this subdev has device node. Signed-off-by: Wenyou Yang --- Changes in v4: - Fix the build error when not enabling V4L2 sub-device userspace API option. Changes in v3: - Keep tried format