Hi Jonas,

On Wed, 2020-07-01 at 21:56 +0000, Jonas Karlman wrote:
> Use bytesperline and buffer height to calculate the strides configured.
> 
> This does not really change anything other than ensuring the bytesperline
> that is signaled to userspace matches was is configured in HW.
> 

Are you seeing any issue due to this?

> Signed-off-by: Jonas Karlman <jo...@kwiboo.se>
> ---
>  drivers/staging/media/rkvdec/rkvdec-h264.c | 27 +++++++++++++---------
>  1 file changed, 16 insertions(+), 11 deletions(-)
> 
> diff --git a/drivers/staging/media/rkvdec/rkvdec-h264.c 
> b/drivers/staging/media/rkvdec/rkvdec-h264.c
> index 9c8e49642cd9..1cb6af590138 100644
> --- a/drivers/staging/media/rkvdec/rkvdec-h264.c
> +++ b/drivers/staging/media/rkvdec/rkvdec-h264.c
> @@ -891,10 +891,11 @@ static void config_registers(struct rkvdec_ctx *ctx,
>       dma_addr_t rlc_addr;
>       dma_addr_t refer_addr;
>       u32 rlc_len;
> -     u32 hor_virstride = 0;
> -     u32 ver_virstride = 0;
> -     u32 y_virstride = 0;
> -     u32 yuv_virstride = 0;
> +     u32 hor_virstride;
> +     u32 ver_virstride;
> +     u32 y_virstride;
> +     u32 uv_virstride;
> +     u32 yuv_virstride;
>       u32 offset;
>       dma_addr_t dst_addr;
>       u32 reg, i;
> @@ -904,16 +905,20 @@ static void config_registers(struct rkvdec_ctx *ctx,
>  
>       f = &ctx->decoded_fmt;
>       dst_fmt = &f->fmt.pix_mp;
> -     hor_virstride = (sps->bit_depth_luma_minus8 + 8) * dst_fmt->width / 8;
> -     ver_virstride = round_up(dst_fmt->height, 16);
> +     hor_virstride = dst_fmt->plane_fmt[0].bytesperline;
> +     ver_virstride = dst_fmt->height;
>       y_virstride = hor_virstride * ver_virstride;
>  

So far so good.

> -     if (sps->chroma_format_idc == 0)
> -             yuv_virstride = y_virstride;
> -     else if (sps->chroma_format_idc == 1)
> -             yuv_virstride += y_virstride + y_virstride / 2;
> +     if (sps->chroma_format_idc == 1)
> +             uv_virstride = y_virstride / 2;
>       else if (sps->chroma_format_idc == 2)
> -             yuv_virstride += 2 * y_virstride;
> +             uv_virstride = y_virstride;
> +     else if (sps->chroma_format_idc == 3)
> +             uv_virstride = 2 * y_virstride;
> +     else
> +             uv_virstride = 0;
> +
> +     yuv_virstride = y_virstride + uv_virstride;
>  

Is the chunk above related to the patch, or mostly
cleaning/improving the code?

Thanks,
Ezequiel

>       reg = RKVDEC_Y_HOR_VIRSTRIDE(hor_virstride / 16) |
>             RKVDEC_UV_HOR_VIRSTRIDE(hor_virstride / 16) |


Reply via email to