<html dir="ltr">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body style="text-align:left; direction:ltr;">
<div>Hi Hans, I have updated v11 patch to fix this. Thanks! best regards xiaoyong Lu On Fri, 2023-05-26 at 14:04 +0200, Hans Verkuil wrote:</div>
<blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex">
<table class="MsoNormalTable" border="0" cellspacing="0" cellpadding="0" align="left" width="100%" style="width:100.0%;background:#77FFCC">
<tbody>
<tr>
<td width="9" style="width:7.0pt;background:#EA0621;padding:5.25pt 1.5pt 5.25pt 1.5pt">
</td>
<td width="100%" style="width:100.0%;background:#FFD4D4;padding:5.25pt 3.75pt 5.25pt 11.25pt">
<div>
<p class="MsoNormal" style="mso-element:frame;mso-element-frame-hspace:2.25pt;mso-element-wrap:around;mso-element-anchor-vertical:paragraph;mso-element-anchor-horizontal:column;mso-height-rule:exactly">
<span lang="EN-US" style="font-size:9.5pt;font-family:"Segoe UI",sans-serif;color:#212121">External email : Please do not click links or open attachments until you have verified the sender or the content.<o:p></o:p></span></p>
</div>
</td>
</tr>
</tbody>
</table>
<!--}-->
<pre>On 14/04/2023 10:30, Xiaoyong Lu wrote:</pre>
<pre>> Add mediatek av1 decoder linux driver which use the stateless API in</pre>
<pre>> MT8195.</pre>
<pre>> </pre>
<pre>> Signed-off-by: Xiaoyong Lu<xiaoyong.lu@mediatek.com></pre>
<pre>> Tested-by: Nicolas Dufresne <nicolas.dufresne@collabora.com></pre>
<pre>> Reviewed-by: Nicolas Dufresne <nicolas.dufresne@collabora.com></pre>
<pre>> Tested-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com></pre>
<pre>> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com></pre>
<pre><br></pre>
<pre>After rebasing on top of our media staging tree I get these compile errors:</pre>
<pre><br></pre>
<pre> CC [M] drivers/media/platform/mediatek/vcodec/vdec/vdec_av1_req_lat_if.o</pre>
<pre>drivers/media/platform/mediatek/vcodec/vdec/vdec_av1_req_lat_if.c: In function ‘vdec_av1_slice_lat_decode’:</pre>
<pre>drivers/media/platform/mediatek/vcodec/vdec/vdec_av1_req_lat_if.c:2075:46: error: ‘struct mtk_vcodec_dev’ has no member named ‘msg_queue_core_ctx’</pre>
<pre> 2075 | vdec_msg_queue_qbuf(&ctx->dev->msg_queue_core_ctx, lat_buf);</pre>
<pre> | ^~</pre>
<pre>drivers/media/platform/mediatek/vcodec/vdec/vdec_av1_req_lat_if.c:2114:46: error: ‘struct mtk_vcodec_dev’ has no member named ‘msg_queue_core_ctx’</pre>
<pre> 2114 | vdec_msg_queue_qbuf(&ctx->dev->msg_queue_core_ctx, lat_buf);</pre>
<pre> | ^~</pre>
<pre><br></pre>
<pre>That's due to the patch "media: mediatek: vcodec: move core context from device</pre>
<pre>to each instance" that has now been merged and that drops that queue.</pre>
<pre><br></pre>
<pre>Can you rebase v10? This is now the only remaining blocked for the av1 series</pre>
<pre>to be merged.</pre>
<pre><br></pre>
<pre>Regards,</pre>
<pre><br></pre>
<pre>Hans</pre>
<pre><br></pre>
</blockquote>
</body>
</html>
<!--type:text--><!--{--><pre>************* MEDIATEK Confidentiality Notice
********************
The information contained in this e-mail message (including any
attachments) may be confidential, proprietary, privileged, or otherwise
exempt from disclosure under applicable laws. It is intended to be
conveyed only to the designated recipient(s). Any use, dissemination,
distribution, printing, retaining or copying of this e-mail (including its
attachments) by unintended recipient(s) is strictly prohibited and may
be unlawful. If you are not an intended recipient of this e-mail, or believe
that you have received this e-mail in error, please notify the sender
immediately (by replying to this e-mail), delete any and all copies of
this e-mail (including any attachments) from your system, and do not
disclose the content of this e-mail to any other person. Thank you!
</pre><!--}-->