* [PATCH] imx-vpu-hantro: Fix h1 encoder build issue
@ 2018-10-09 2:16 Carol Zhu
2018-10-10 10:48 ` Gary Bisson
0 siblings, 1 reply; 2+ messages in thread
From: Carol Zhu @ 2018-10-09 2:16 UTC (permalink / raw)
To: meta-freescale; +Cc: Carol Zhu
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@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
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] imx-vpu-hantro: Fix h1 encoder build issue
2018-10-09 2:16 [PATCH] imx-vpu-hantro: Fix h1 encoder build issue Carol Zhu
@ 2018-10-10 10:48 ` Gary Bisson
0 siblings, 0 replies; 2+ messages in thread
From: Gary Bisson @ 2018-10-10 10:48 UTC (permalink / raw)
To: Yuqing Zhu; +Cc: meta-freescale Mailing List
Hi Carol,
On Tue, Oct 9, 2018 at 4:16 AM Carol Zhu <carol.zhu@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@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@boundarydevices.com>
Regards,
Gary
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-10-10 10:48 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-10-09 2:16 [PATCH] imx-vpu-hantro: Fix h1 encoder build issue Carol Zhu
2018-10-10 10:48 ` Gary Bisson
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.