media: hantro: Use output buffer width and height for H264 decoding
authorJonas Karlman <jonas@kwiboo.se>
Wed, 6 Nov 2019 22:34:22 +0000 (23:34 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Sun, 10 Nov 2019 16:58:45 +0000 (17:58 +0100)
Calculations for motion vector buffer offset is based on width and height
from the configured output format, lets use the same values for macroblock
width and height hw regs.

Signed-off-by: Jonas Karlman <jonas@kwiboo.se>
Reviewed-by: Boris Brezillon <boris.brezillon@collabora.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/staging/media/hantro/hantro_g1_h264_dec.c

index 30d977c3d529cde2afe45a73d9fb7f5cea82db83..27d40d8d3728968cb191595b50172b38a4dbbde8 100644 (file)
@@ -51,8 +51,8 @@ static void set_params(struct hantro_ctx *ctx)
        vdpu_write_relaxed(vpu, reg, G1_REG_DEC_CTRL0);
 
        /* Decoder control register 1. */
-       reg = G1_REG_DEC_CTRL1_PIC_MB_WIDTH(sps->pic_width_in_mbs_minus1 + 1) |
-             G1_REG_DEC_CTRL1_PIC_MB_HEIGHT_P(sps->pic_height_in_map_units_minus1 + 1) |
+       reg = G1_REG_DEC_CTRL1_PIC_MB_WIDTH(MB_WIDTH(ctx->src_fmt.width)) |
+             G1_REG_DEC_CTRL1_PIC_MB_HEIGHT_P(MB_HEIGHT(ctx->src_fmt.height)) |
              G1_REG_DEC_CTRL1_REF_FRAMES(sps->max_num_ref_frames);
        vdpu_write_relaxed(vpu, reg, G1_REG_DEC_CTRL1);