Discussion:
[meta-freescale] [PATCH] imx-vpu-hantro: Fix h1 encoder build issue
Carol Zhu
2018-10-09 02:16:19 UTC
Permalink
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 <***@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

--
Gary Bisson
2018-10-10 10:48:42 UTC
Permalink
Hi Carol,
Post by 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.
---
.../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 <***@boundarydevices.com>

Regards,
Gary
--

Loading...