Hongmei, I see. We may need a separate "opencv" flag in MACHINE_FEATURES in the future.
For now, can you please group barcode-roi in this packagegroup according to whether it uses opencl or not. Thanks. -- Denys On Mon, Feb 06, 2017 at 02:49:16PM -0500, Gou, Hongmei wrote: > Hi Denys, > > The barcode-roi demo is included for AM3/AM4/DRA7xx/Keystone. It does depend > on OpenCV, but not necessarily on OpenCL. > > For dra7xx and keystone, some OpenCV kernels are dispatched to DSP via > OpenCL, thus there can be dependency on OpenCL. > For AM3 and AM4, OpenCV kernels run on ARM only, and there is no dependency > on OpenCL. > > Thanks, > Hongmei > > > -----Original Message----- > > From: Dmytriyenko, Denys > > Sent: Monday, February 06, 2017 1:11 PM > > To: Gou, Hongmei > > Cc: [email protected] > > Subject: Re: [meta-arago] [morty/krogoth][PATCH 5/6] packagegroup-arago- > > tisdk-matrix: add barcode-roi demo > > > > Hongmei, > > > > Can you please re-do this patch due to this change: > > http://arago-project.org/git/?p=meta- > > arago.git;a=commitdiff;h=908f169117c7589eabc78a392e8c43eb099553cd > > > > barcode-roi depends on opencv/opencl and hence should be in > > MATRIX_OPENCL_APPS list, which gets conditionally included based on > > corresponding MACHINE_FEATURE. > > > > -- > > Denys > > > > > > On Fri, Feb 03, 2017 at 05:22:18PM -0500, Hongmei Gou wrote: > > > Signed-off-by: Hongmei Gou <[email protected]> > > > --- > > > .../recipes-core/packagegroups/packagegroup-arago-tisdk-matrix.bb | 6 > > +++++- > > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > > > diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup- > > arago-tisdk-matrix.bb b/meta-arago-distro/recipes- > > core/packagegroups/packagegroup-arago-tisdk-matrix.bb > > > index a851ed5..7358da4 100644 > > > --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago- > > tisdk-matrix.bb > > > +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup- > > arago-tisdk-matrix.bb > > > @@ -1,6 +1,6 @@ > > > DESCRIPTION = "Task to include Matrix v2" > > > LICENSE = "MIT" > > > -PR = "r61" > > > +PR = "r62" > > > > > > PACKAGE_ARCH = "${MACHINE_ARCH}" > > > > > > @@ -76,6 +76,7 @@ MATRIX_APPS_append_ti33x = " \ > > > matrix-gui-pm-demos-governor \ > > > \ > > > matrix-gui-ethernet-demos \ > > > + matrix-machinevision-demo-barcoderoi \ > > > " > > > > > > MATRIX_APPS_append_ti43x = " \ > > > @@ -91,6 +92,7 @@ MATRIX_APPS_append_ti43x = " \ > > > \ > > > matrix-gui-apps-dual-camera \ > > > matrix-gui-apps-image-gallery \ > > > + matrix-machinevision-demo-barcoderoi \ > > > " > > > > > > MATRIX_APPS_append_omap-a15 = " \ > > > @@ -105,6 +107,7 @@ MATRIX_APPS_append_omap-a15 = " \ > > > matrix-gui-generic-pm \ > > > matrix-gui-clocks \ > > > matrix-gui-pm-demos-governor \ > > > + matrix-machinevision-demo-barcoderoi \ > > > " > > > > > > MATRIX_APPS_append_dra7xx += " \ > > > @@ -116,6 +119,7 @@ MATRIX_APPS_append_keystone = " \ > > > matrix-gui-utility-demos \ > > > matrix-mc-demo-ipc \ > > > matrix-multimedia-demo-dsp66imgproc-f2f \ > > > + matrix-machinevision-demo-barcoderoi-f2f \ > > > " > > > > > > RDEPENDS_${PN} = " \ > > > -- > > > 1.9.1 > > > > > > _______________________________________________ > > > meta-arago mailing list > > > [email protected] > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago _______________________________________________ meta-arago mailing list [email protected] http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
