Hi Hans, Em Mon, 10 Apr 2017 21:26:36 +0200 Hans Verkuil <hverkuil@xxxxxxxxx> escreveu: > From: Hans Verkuil <hans.verkuil@xxxxxxxxx> > > This patch series is the rebased version of this pull request: > > https://www.mail-archive.com/linux-media@xxxxxxxxxxxxxxx/msg111025.html > > It slightly modifies 'Add metadata buffer type and format' (remove > experimental note and add newline after label) and it adds support > for V4L2_CTRL_FLAG_MODIFY_LAYOUT, as requested by Mauro. > > No other changes were made. Patch series look ok. I found just one typo on one of the patches. What seems to be missing here is to set the GRABBED flag for controls that modify layout but don't allow control update while streaming. While it is OK to do such change for the existing drivers later (as this is actually a bug fix), I would be expecting such change for the controls used at the vsp1 driver, as, from what I'm seeing at the code, vsp1_wpf_s_ctrl() will block changing any controls in runtime. So, wpf_init_controls() should mark all such controls with V4L2_CTRL_FLAG_GRABBED[1]. Could you please add such patch at the end of this patchset? Thanks! Mauro [1] I don't see any reason why not allowing HFLIP/VFLIP controls to be handled in realtime (except if the hardware itself doesn't allow), but the current code doesn't allow such changes in realtime anymore. Perhaps the code could be less pick in the future. Thanks, Mauro
