Hi Yong, On Mon, Jan 14, 2019 at 09:37:44PM -0600, Yong Zhi wrote: > This addresses the below TODO item. > - Use V4L2_CTRL_TYPE_MENU for dual-pipe mode control. (Sakari) > > Signed-off-by: Yong Zhi <yong.zhi@xxxxxxxxx> > --- > drivers/staging/media/ipu3/TODO | 2 -- > drivers/staging/media/ipu3/include/intel-ipu3.h | 6 ------ > drivers/staging/media/ipu3/ipu3-v4l2.c | 18 +++++++++++++----- > 3 files changed, 13 insertions(+), 13 deletions(-) > > diff --git a/drivers/staging/media/ipu3/TODO b/drivers/staging/media/ipu3/TODO > index 905bbb190217..0dc9a2e79978 100644 > --- a/drivers/staging/media/ipu3/TODO > +++ b/drivers/staging/media/ipu3/TODO > @@ -11,8 +11,6 @@ staging directory. > - Prefix imgu for all public APIs, i.e. change ipu3_v4l2_register() to > imgu_v4l2_register(). (Sakari) > > -- Use V4L2_CTRL_TYPE_MENU for dual-pipe mode control. (Sakari) > - It's good to see TODO entries being addressed. :-) With Tomasz's comments addressed, this is good to go in IMO. -- Regards, Sakari Ailus sakari.ailus@xxxxxxxxxxxxxxx