Re: [meta-intel] [PATCH v2 5/5] rmc: add database extraction functionality

2017-02-10 Thread Jianxun Zhang
Todor, Appreciate the V2 series. I have only one real concern on using system() in this patch and the whole series. The other comments are more for corner cases which have less impact. I could miss some of your feedbacks in V1 threads. Sorry for not to point out things earlier before the V2.

Re: [meta-intel] [PATCH v2 4/5] rmc: remove unnecessary return variable

2017-02-10 Thread Jianxun Zhang
Nice. Review +1 > On Feb 9, 2017, at 11:17 AM, Todor Minchev > wrote: > > Signed-off-by: Todor Minchev > --- > src/rmc.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/src/rmc.c b/src/rmc.c > index

Re: [meta-intel] [PATCH v2 3/5] rmc: Enable reading the contents of an existing fingerprint file

2017-02-10 Thread Jianxun Zhang
> On Feb 9, 2017, at 11:17 AM, Todor Minchev > wrote: > > The contents of an existing fingerprint file can be read and output on > the command line with the following options: > > rmc -E -f input_fingerprint_file > > Signed-off-by: Todor Minchev

[meta-intel] [PATCH 02/10] libva-intel-driver: Add wayland-native to DEPENDS list

2017-02-10 Thread Saul Wold
Wayland-native is needed for header file generation, since the addition of Recipe Specific Sysroot, the needed wayland-scanner is not available Signed-off-by: Saul Wold --- common/recipes-multimedia/libva/libva-intel-driver_1.7.3.bb | 2 +- 1 file changed, 1 insertion(+),

[meta-intel] [PATCH 10/10] linux-intel: Use base branch

2017-02-10 Thread Saul Wold
This branch will contain additional patches on top of the linux-stable to support the Intel platforms Updated the SRCREV accordingly also Signed-off-by: Saul Wold --- common/recipes-kernel/linux/linux-intel_4.9.bb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

[meta-intel] [PATCH 08/10] dpdk: Disable building for musl

2017-02-10 Thread Saul Wold
Currently dpdk requires the execinfo.h header from glibc which is not part of MUSL, so for now disable this package. Signed-off-by: Saul Wold --- common/recipes-extended/dpdk/dpdk_16.11.bb | 2 ++ 1 file changed, 2 insertions(+) diff --git

[meta-intel] [PATCH 09/10] intel-gpu-tools: Disbale build for MUSL

2017-02-10 Thread Saul Wold
Currently this program uses a SIGEV_THREAD_ID and structure that is not fully supported by MUSL, so disable this target for MUSL Signed-off-by: Saul Wold --- common/recipes-graphics/intel-gpu-tools/intel-gpu-tools_1.16.bb | 1 + 1 file changed, 1 insertion(+) diff --git

[meta-intel] [PATCH 07/10] zlib-qat: Add patch to support MUSL

2017-02-10 Thread Saul Wold
MUSL has a stricter policy for header inclusion, so add patch to explictly include pthread.h Signed-off-by: Saul Wold --- .../0001-qat_zlib.h-Add-pthread.h-for-MUSL.patch | 29 ++ .../zlib-qat/zlib-qat_0.4.7-002.bb | 4 +++ 2 files

[meta-intel] [PATCH 03/10] iucode-tool: Add support for MUSL

2017-02-10 Thread Saul Wold
iucode-tool needs the arpg API which is not part of MUSL, so add argp-standalone as a dependency. The arpg library is also required for linking, so add a patch to the Makefile.am, which is added again based on tclib being MUSL Signed-off-by: Saul Wold ---

[meta-intel] [PATCH 04/10] lms7: Use patches from lms8 that fixed MUSL build

2017-02-10 Thread Saul Wold
The same patches from LSM8 can be applied to LMS7 for enabling world build with MUSL in meta-intel Signed-off-by: Saul Wold --- common/recipes-bsp/amt/lms7_7.1.20.bb | 2 ++ 1 file changed, 2 insertions(+) diff --git a/common/recipes-bsp/amt/lms7_7.1.20.bb

[meta-intel] [PATCH 00/10] Get meta-intel working with MUSL & some other patches

2017-02-10 Thread Saul Wold
This patch set is mostly to get meta-intel to be MUSL compatible. I have also included a couple of other patches for libva-intel-driver and more importantly to shift linux-intel to use the base branch. Sau! The following changes since commit e8fd58ccf221b071508c1c168e4d9bcee92c3a5d:

[meta-intel] [PATCH 05/10] qat16: add additionl patch with guard for MUSL build

2017-02-10 Thread Saul Wold
This just disables the use of bits/time.h Signed-off-by: Saul Wold --- ...salServices-Only-use-bits-time-with-GLIBC.patch | 27 ++ common/recipes-extended/qat/qat16_2.6.0-65.bb | 2 ++ 2 files changed, 29 insertions(+) create mode 100644

[meta-intel] [PATCH 01/10] libva-intel-driver: Update to 1.7.3

2017-02-10 Thread Saul Wold
The upstream has moved their source base from FreeDesktop to github.com/01org. Set $S to WORKDIR/git since this is a git repo now. Signed-off-by: Saul Wold --- .../{libva-intel-driver_1.7.2.bb => libva-intel-driver_1.7.3.bb} | 7 --- 1 file changed, 4 insertions(+),

Re: [meta-intel] [PATCH v2 2/5] Makefile: add debug target

2017-02-10 Thread Jianxun Zhang
> On Feb 9, 2017, at 11:17 AM, Todor Minchev > wrote: > > A debug version of the rmc binary can be built by using the debug > Makefile target. This will include debug symbols and will disable > compiler optimizations. > > Example: > > make debug > >

Re: [meta-intel] [PATCH v2 1/5] Makefile: disable silent mode in Makefiles

2017-02-10 Thread Jianxun Zhang
Review +1, Thanks to address my comment > On Feb 9, 2017, at 11:17 AM, Todor Minchev > wrote: > > By default make will output the commands that are executed for each > target. Silent mode can be enabled with the '-s' option. > > Example: > make -s > >