Re: [PATCH v1] media: mtk-vcodec: Align width and height to 64

["yunfei.dong@xxxxxxxxxxxx" <yunfei.dong@xxxxxxxxxxxx>]

Hi steve,

Thanks for your suggestion.

On Tue, 2021-11-02 at 09:43 -0700, Steve Cho wrote:

Thank you Yunfei for following up with this change. 

This change is the last missing piece to enable VD on Kukui with Chromium. 

This patch fixed the corruption we were seeing on Kukui with certain tests.

One comment from me is just to use defined macro or variable instead of hard coding 64. 

"User get width and height are 64 align when set format."

This sentence might need to be reworded. It is not clear to me. 

Maybe something like "Width and height need to be 64 bytes aligned when setting the format."

Thanks,

Steve

Fix it and send patch v2.

Thanks,

Yunfei Dong

On Fri, Oct 29, 2021 at 2:45 AM Yunfei Dong <yunfei.dong@xxxxxxxxxxxx> wrote:

User get width and height are 64 align when set format. Need to make
sure all is 64 align when use width and height to calculate buffer size.

Signed-off-by: Yunfei Dong <yunfei.dong@xxxxxxxxxxxx>
---
 drivers/media/platform/mtk-vcodec/vdec/vdec_h264_req_if.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/media/platform/mtk-vcodec/vdec/vdec_h264_req_if.c b/drivers/media/platform/mtk-vcodec/vdec/vdec_h264_req_if.c
index 946c23088308..28c17204f9a1 100644
--- a/drivers/media/platform/mtk-vcodec/vdec/vdec_h264_req_if.c
+++ b/drivers/media/platform/mtk-vcodec/vdec/vdec_h264_req_if.c
@@ -562,8 +562,8 @@ static void get_pic_info(struct vdec_h264_slice_inst *inst,
 {
        struct mtk_vcodec_ctx *ctx = inst->ctx;

-       ctx->picinfo.buf_w = (ctx->picinfo.pic_w + 15) & 0xFFFFFFF0;
-       ctx->picinfo.buf_h = (ctx->picinfo.pic_h + 31) & 0xFFFFFFE0;
+       ctx->picinfo.buf_w = ALIGN(ctx->picinfo.pic_w, 64);
+       ctx->picinfo.buf_h = ALIGN(ctx->picinfo.pic_h, 64);
        ctx->picinfo.fb_sz[0] = ctx->picinfo.buf_w * ctx->picinfo.buf_h;
        ctx->picinfo.fb_sz[1] = ctx->picinfo.fb_sz[0] >> 1;
        inst->vsi_ctx.dec.cap_num_planes =