On Wed, Jul 13, 2016 at 1:41 AM, Xiang, Haihao <[email protected]>
wrote:

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

Good catch, lgtm.

Sean


>
> Tested-by: Xu, Guangxin <[email protected]>
> Signed-off-by: Xiang, Haihao <[email protected]>
> ---
>  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
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/libva
>



-- 
Sean V. Kelley <[email protected]>
Open Source Technology Center / SSG
Intel Corp.
_______________________________________________
Libva mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/libva

Reply via email to