[meta-freescale] [PATCH] imx-vpu-hantro: Fix h1 encoder build issue

Gary Bisson gary.bisson at boundarydevices.com
Wed Oct 10 03:48:42 PDT 2018


Hi Carol,

On Tue, Oct 9, 2018 at 4:16 AM Carol Zhu <carol.zhu at nxp.com> wrote:
>
> Modify the environment setting of h1 encoder to
> correct include header directory, make LINUX_VERSION_CODE
> come from kernel build folder instead of toolchain headers.
>
> Signed-off-by: Yuqing Zhu <carol.zhu at nxp.com>
> ---
>  .../0002-Fix-version.h-inclusion-to-be-from-kernel-build-fold.patch     | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/recipes-bsp/imx-vpu-hantro/imx-vpu-hantro/0002-Fix-version.h-inclusion-to-be-from-kernel-build-fold.patch b/recipes-bsp/imx-vpu-hantro/imx-vpu-hantro/0002-Fix-version.h-inclusion-to-be-from-kernel-build-fold.patch
> index e9bf925..9d54223 100644
> --- a/recipes-bsp/imx-vpu-hantro/imx-vpu-hantro/0002-Fix-version.h-inclusion-to-be-from-kernel-build-fold.patch
> +++ b/recipes-bsp/imx-vpu-hantro/imx-vpu-hantro/0002-Fix-version.h-inclusion-to-be-from-kernel-build-fold.patch
> @@ -34,7 +34,7 @@ index 56b4332..0be43ce 100755
>   ENV += -I$(LINUX_KERNEL_ROOT)/drivers/staging/android/uapi
>
>  +# LINUX_VERSION_CODE from kernel build folder instead of toolchain headers
> -+INCLUDE_HEADERS += -I$(LINUX_KERNEL_BUILD)/include/generated/uapi
> ++ENV += -I$(LINUX_KERNEL_BUILD)/include/generated/uapi
>  +
>   LIBENCNAME = libcodec_enc
>   LIBSENC = -L./ -lhantro_h1 -lpthread
> --
> 1.9.1

I now see what was the issue, I copied the same fix on both Makefile
although one is using INCLUDE_HEADERS and the other ENV.
Thanks for the fix!
Reviewed-by: Gary Bisson <gary.bisson at boundarydevices.com>

Regards,
Gary


More information about the meta-freescale mailing list