[Libva] [Libva-intel-driver][PATCH] Fix the condition used in 'else if()'

[Sean V Kelley]

On Wed, Jul 13, 2016 at 1:41 AM, Xiang, Haihao <haihao.xiang at intel.com>
wrote:

> This 'else if()' is used to check output surface format, not input surface
> format.
>

Good catch, lgtm.

Sean


>
> Tested-by: Xu, Guangxin <Guangxin.Xu at intel.com>
> Signed-off-by: Xiang, Haihao <haihao.xiang at intel.com>
> ---
>  src/gen75_vpp_vebox.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/gen75_vpp_vebox.c b/src/gen75_vpp_vebox.c
> index e124604..9773c24 100644
> --- a/src/gen75_vpp_vebox.c
> +++ b/src/gen75_vpp_vebox.c
> @@ -1450,8 +1450,8 @@ int hsw_veb_pre_format_convert(VADriverContextP ctx,
>          proc_ctx->format_convert_flags |= POST_FORMAT_CONVERT;
>      } else if(obj_surf_output->fourcc ==  VA_FOURCC_AYUV ||
>                obj_surf_output->fourcc ==  VA_FOURCC_YUY2 ||
> -                obj_surf_input->fourcc ==  VA_FOURCC_NV12 ||
> -                obj_surf_input->fourcc ==  VA_FOURCC_P010){
> +              obj_surf_output->fourcc ==  VA_FOURCC_NV12 ||
> +              obj_surf_output->fourcc ==  VA_FOURCC_P010) {
>
>                /* Nothing to do here */
>       } else {
> --
> 2.8.3
>
> _______________________________________________
> Libva mailing list
> Libva at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/libva
>



-- 
Sean V. Kelley <sean.v.kelley at intel.com>
Open Source Technology Center / SSG
Intel Corp.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/libva/attachments/20160714/5f47340e/attachment.html>