lkml.org 
[lkml]   [2022]   [Apr]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.16 0336/1017] media: cedrus: H265: Fix neighbour info buffer size
    Date
    From: Jernej Skrabec <jernej.skrabec@gmail.com>

    [ Upstream commit ee8b887329c78971967506f3ac79b9302c9f83c1 ]

    Neighbour info buffer size needs to be 794 kiB in H6. This is actually
    already indirectly mentioned in the comment, but smaller size is used
    nevertheless.

    Increase buffer size to cover H6 needs. Since increase is not that big
    in absolute numbers, it doesn't make sense to complicate logic for older
    generations.

    Bug was discovered using iommu, which reported access error when trying
    to play H265 video.

    Fixes: 86caab29da78 ("media: cedrus: Add HEVC/H.265 decoding support")
    Signed-off-by: Jernej Skrabec <jernej.skrabec@gmail.com>
    Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/staging/media/sunxi/cedrus/cedrus_h265.c | 2 +-
    1 file changed, 1 insertion(+), 1 deletion(-)

    diff --git a/drivers/staging/media/sunxi/cedrus/cedrus_h265.c b/drivers/staging/media/sunxi/cedrus/cedrus_h265.c
    index 8829a7bab07e..ffade5cbd2e4 100644
    --- a/drivers/staging/media/sunxi/cedrus/cedrus_h265.c
    +++ b/drivers/staging/media/sunxi/cedrus/cedrus_h265.c
    @@ -23,7 +23,7 @@
    * Subsequent BSP implementations seem to double the neighbor info buffer size
    * for the H6 SoC, which may be related to 10 bit H265 support.
    */
    -#define CEDRUS_H265_NEIGHBOR_INFO_BUF_SIZE (397 * SZ_1K)
    +#define CEDRUS_H265_NEIGHBOR_INFO_BUF_SIZE (794 * SZ_1K)
    #define CEDRUS_H265_ENTRY_POINTS_BUF_SIZE (4 * SZ_1K)
    #define CEDRUS_H265_MV_COL_BUF_UNIT_CTB_SIZE 160

    --
    2.34.1


    \
     
     \ /
      Last update: 2022-04-05 14:56    [W:2.918 / U:1.384 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site