V2 patch has been submitted. Thanks, Hongmei
> -----Original Message----- > From: Dmytriyenko, Denys > Sent: Monday, February 06, 2017 3:15 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, > > 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
