[Mesa-dev] [PATCH] radeon/vce: fix vui time_scale zero error

Alex Deucher alexdeucher at gmail.com
Fri Sep 25 10:40:58 PDT 2015


On Wed, Sep 23, 2015 at 3:57 PM, Leo Liu <leo.liu at amd.com> wrote:
> if app pass 0 as frame_rate_num, it should not be encoded to the VUI.
>
> Signed-off-by: Leo Liu <leo.liu at amd.com>
> Reviewed-by: Alex Deucher <alexander.deucher at amd.com>
> Reviewed-by: Christian König <christian.koenig at amd.com>
> Cc: "10.6" <mesa-stable at lists.freedesktop.org>

11.0 should be cc'ed as well.

Alex

> ---
>  src/gallium/drivers/radeon/radeon_vce_40_2_2.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/src/gallium/drivers/radeon/radeon_vce_40_2_2.c b/src/gallium/drivers/radeon/radeon_vce_40_2_2.c
> index e64fbc7..c005659 100644
> --- a/src/gallium/drivers/radeon/radeon_vce_40_2_2.c
> +++ b/src/gallium/drivers/radeon/radeon_vce_40_2_2.c
> @@ -233,6 +233,9 @@ static void vui(struct rvce_encoder *enc)
>  {
>         int i;
>
> +       if (!enc->pic.rate_ctrl.frame_rate_num)
> +               return;
> +
>         RVCE_BEGIN(0x04000009); // vui
>         RVCE_CS(0x00000000); //aspectRatioInfoPresentFlag
>         RVCE_CS(0x00000000); //aspectRatioInfo.aspectRatioIdc
> --
> 2.1.4
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list