lkml.org 
[lkml]   [2022]   [Aug]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.19 0460/1157] media: mediatek: vcodec: decoder: Skip alignment for default resolution
    Date
    From: Chen-Yu Tsai <wenst@chromium.org>

    [ Upstream commit 3b6a81a31370539f1fd0e9bdd315503aa154285e ]

    The default resolution of 64x64 is already aligned, according to the
    call to v4l_bound_align_image() in mtk_vcodec_dec_set_default_params().

    Drop the redundant v4l_bound_align_image() call. This also removes one
    usage of ctx->max_{width,height}.

    Signed-off-by: Chen-Yu Tsai <wenst@chromium.org>
    Tested-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
    Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
    Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec.c | 7 -------
    1 file changed, 7 deletions(-)

    diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec.c
    index 42b29f0a7436..7d654efabdfe 100644
    --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec.c
    +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec.c
    @@ -142,13 +142,6 @@ void mtk_vcodec_dec_set_default_params(struct mtk_vcodec_ctx *ctx)
    ctx->max_width = MTK_VDEC_MAX_W;
    ctx->max_height = MTK_VDEC_MAX_H;

    - v4l_bound_align_image(&q_data->coded_width,
    - MTK_VDEC_MIN_W,
    - ctx->max_width, 4,
    - &q_data->coded_height,
    - MTK_VDEC_MIN_H,
    - ctx->max_height, 5, 6);
    -
    q_data->sizeimage[0] = q_data->coded_width * q_data->coded_height;
    q_data->bytesperline[0] = q_data->coded_width;
    q_data->sizeimage[1] = q_data->sizeimage[0] / 2;
    --
    2.35.1


    \
     
     \ /
      Last update: 2022-08-16 01:55    [W:4.018 / U:2.376 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site