media: mediatek: vcodec: add description for vsi struct

The vsi (video shared information) struct needs to be synchronized
between firmware and host, as a change that is only done in the host
version of the struct but isn't synchronized to the firmware. This can
lead to decoding issues with H264 bitstreams. Highlight this requirement
within the struct descriptions.

Signed-off-by: Yunfei Dong <yunfei.dong@mediatek.com>
Reviewed-by: Chen-Yu Tsai <wenst@chromium.org>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Signed-off-by: Nicolas Dufresne <nicolas.dufresne@collabora.com>
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
This commit is contained in:
Yunfei Dong 2025-03-08 15:47:57 +08:00 committed by Hans Verkuil
parent c8c3bb1e54
commit 4c3596d7e8

View File

@ -30,6 +30,7 @@ enum vdec_h264_core_dec_err_type {
/**
* struct vdec_h264_slice_lat_dec_param - parameters for decode current frame
* (shared data between host and firmware)
*
* @sps: h264 sps syntax parameters
* @pps: h264 pps syntax parameters
@ -48,7 +49,7 @@ struct vdec_h264_slice_lat_dec_param {
};
/**
* struct vdec_h264_slice_info - decode information
* struct vdec_h264_slice_info - decode information (shared data between host and firmware)
*
* @nal_info: nal info of current picture
* @timeout: Decode timeout: 1 timeout, 0 no timeout
@ -72,7 +73,7 @@ struct vdec_h264_slice_info {
/**
* struct vdec_h264_slice_vsi - shared memory for decode information exchange
* between SCP and Host.
* between SCP and Host (shared data between host and firmware).
*
* @wdma_err_addr: wdma error dma address
* @wdma_start_addr: wdma start dma address