Re: [PATCH v11 13/17] media: controls: Log HEVC stateless control in .std_log

Ezequiel Garcia
2022-07-08T09:54:23Z

On Wed, Jul 06, 2022 at 11:37:59AM +0200, Benjamin Gaignard wrote:
> Simply print the type of the control.
>
> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@collabora.com>
> Acked-by: Nicolas Dufresne <nicolas.dufresne@collabora.com>
> Tested-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Reviewed-by: Ezequiel Garcia <ezequiel@vanguardiasur.com.ar>

> ---
> drivers/media/v4l2-core/v4l2-ctrls-core.c | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
>
> diff --git a/drivers/media/v4l2-core/v4l2-ctrls-core.c b/drivers/media/v4l2-core/v4l2-ctrls-core.c
> index ff8a61f24d0a..c5c5407584ff 100644
> --- a/drivers/media/v4l2-core/v4l2-ctrls-core.c
> +++ b/drivers/media/v4l2-core/v4l2-ctrls-core.c
> @@ -307,6 +307,21 @@ static void std_log(const struct v4l2_ctrl *ctrl)
> case V4L2_CTRL_TYPE_VP9_FRAME:
> pr_cont("VP9_FRAME");
> break;
> + case V4L2_CTRL_TYPE_HEVC_SPS:
> + pr_cont("HEVC_SPS");
> + break;
> + case V4L2_CTRL_TYPE_HEVC_PPS:
> + pr_cont("HEVC_PPS");
> + break;
> + case V4L2_CTRL_TYPE_HEVC_SLICE_PARAMS:
> + pr_cont("HEVC_SLICE_PARAMS");
> + break;
> + case V4L2_CTRL_TYPE_HEVC_SCALING_MATRIX:
> + pr_cont("HEVC_SCALING_MATRIX");
> + break;
> + case V4L2_CTRL_TYPE_HEVC_DECODE_PARAMS:
> + pr_cont("HEVC_DECODE_PARAMS");
> + break;
> default:
> pr_cont("unknown type %d", ctrl->type);
> break;
> --
> 2.32.0
>