On Mon, Feb 19, 2018 at 11:37:56AM +0100, Hans Verkuil wrote:
> The VIDIOC_DBG_G/S_REGISTER ioctls imply that VIDIOC_DBG_G_CHIP_INFO is also
> present, since without that you cannot use v4l2-dbg.
> 
> Just like the implementation in v4l2-ioctl.c this can be implemented in the
> core and no drivers need to be modified.
> 
> It also makes it possible for v4l2-compliance to properly test the
> VIDIOC_DBG_G/S_REGISTER ioctls.
> 
> Signed-off-by: Hans Verkuil <hans.verk...@cisco.com>

Considering the interface already is there and as debugfs isn't a drop-in
placement,

Acked-by: Sakari Ailus <sakari.ai...@linux.intel.com>

It'd be still good to have generic, non V4L2-specific solution to this
issue in the future.

-- 
Sakari Ailus
e-mail: sakari.ai...@iki.fi

Reply via email to