[Mesa-dev] [PATCH] radeonsi: fix radeon create encoder return

Alex Deucher alexdeucher at gmail.com
Tue Feb 27 22:48:30 UTC 2018


On Tue, Feb 27, 2018 at 5:43 PM,  <boyuan.zhang at amd.com> wrote:
> From: Boyuan Zhang <boyuan.zhang at amd.com>
>
> Previous patch missed a "return" when trying to modify the create encoder
> function, which made the whole logic fail. Therefore, add the return back.
>
> Signed-off-by: Boyuan Zhang <boyuan.zhang at amd.com>

Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  src/gallium/drivers/radeonsi/si_uvd.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/radeonsi/si_uvd.c b/src/gallium/drivers/radeonsi/si_uvd.c
> index 3906bbd..b6cb4cb 100644
> --- a/src/gallium/drivers/radeonsi/si_uvd.c
> +++ b/src/gallium/drivers/radeonsi/si_uvd.c
> @@ -150,7 +150,7 @@ struct pipe_video_codec *si_uvd_create_decoder(struct pipe_context *context,
>
>         if (templ->entrypoint == PIPE_VIDEO_ENTRYPOINT_ENCODE) {
>                 if (vcn) {
> -                       radeon_create_encoder(context, templ, ctx->b.ws, si_vce_get_buffer);
> +                       return radeon_create_encoder(context, templ, ctx->b.ws, si_vce_get_buffer);
>                 } else {
>                         if (u_reduce_video_profile(templ->profile) == PIPE_VIDEO_FORMAT_HEVC)
>                                 return radeon_uvd_create_encoder(context, templ, ctx->b.ws, si_vce_get_buffer);
> --
> 2.7.4
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list