[morty/krogoth][PATCH 0/6] Add barcode-roi demo

[morty/krogoth][PATCH 0/6] Add barcode-roi demo

[Hongmei Gou]

Add barcode demo to detect region of interest (ROI)

Hongmei Gou (6):
  barcode-roi: add barcode ROI detection demo
  arago-source-ipk: add SRCIPK for barcode-roi demo
  ti-tisdk-makefile: add barcode-roi demo
  matrix-gui-machinevision-demos: add barcode-roi demo
  packagegroup-arago-tisdk-matrix: add barcode-roi demo
  packagegroup-arago-tisdk-multimedia-sdk*: add barcode-roi demo

 .../conf/distro/arago-source-ipk.conf              |  3 +++
 .../packagegroup-arago-tisdk-matrix.bb             |  6 ++++-
 ...packagegroup-arago-tisdk-multimedia-sdk-host.bb |  9 ++++++-
 ...ckagegroup-arago-tisdk-multimedia-sdk-target.bb |  8 +++++-
 .../ti-tisdk-makefile/Makefile_barcode-roi         | 19 ++++++++++++++
 .../ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb     |  7 +++++-
 .../recipes-apps/barcode-demo/barcode-roi_git.bb   | 29 ++++++++++++++++++++++
 .../matrix/matrix-gui-machinevision-demos_2.0.bb   | 24 +++++++++++++++---
 8 files changed, 98 insertions(+), 7 deletions(-)
 create mode 100644 meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile/Makefile_barcode-roi
 create mode 100644 meta-arago-extras/recipes-apps/barcode-demo/barcode-roi_git.bb

-- 
1.9.1

[morty/krogoth][PATCH 1/6] barcode-roi: add barcode ROI detection demo

[Hongmei Gou]

Signed-off-by: Hongmei Gou <h-gou@ti.com>
---
 .../recipes-apps/barcode-demo/barcode-roi_git.bb   | 29 ++++++++++++++++++++++
 1 file changed, 29 insertions(+)
 create mode 100644 meta-arago-extras/recipes-apps/barcode-demo/barcode-roi_git.bb

diff --git a/meta-arago-extras/recipes-apps/barcode-demo/barcode-roi_git.bb b/meta-arago-extras/recipes-apps/barcode-demo/barcode-roi_git.bb
new file mode 100644
index 0000000..d828ef7
--- /dev/null
+++ b/meta-arago-extras/recipes-apps/barcode-demo/barcode-roi_git.bb
@@ -0,0 +1,29 @@
+DESCRIPTION = "Barcode demo to detect region of interest"
+LICENSE = "MIT"
+LIC_FILES_CHKSUM = "file://detect_barcode.cpp;beginline=1;endline=25;md5=babab64d39715b3671dd16bcb4f4917b"
+
+DEPENDS += "opencv"
+
+PV = "01.00.00"
+PR = "r1"
+
+BRANCH = "master"
+SRC_URI = "git://git.ti.com/apps/barcode-roi-detection.git;protocol=git;branch=${BRANCH}"
+
+SRCREV = "50fa051f66012ebaa10eba1cb289287c4c5e7432"
+
+S = "${WORKDIR}/git"
+
+do_install() {
+    install -d ${D}${bindir}
+    install -m 755 ${S}/detect_barcode ${D}${bindir}
+
+    install -d ${D}${datadir}/ti/image
+    install -m 644 ${S}/test/sample_barcode.jpg ${D}${datadir}/ti/image
+}
+
+FILES_${PN} += "\
+    ${datadir}/ti/image \
+"
+
+INSANE_SKIP_${PN} = "ldflags"
-- 
1.9.1

[morty/krogoth][PATCH 2/6] arago-source-ipk: add SRCIPK for barcode-roi demo

[Hongmei Gou]

Signed-off-by: Hongmei Gou <h-gou@ti.com>
---
 meta-arago-distro/conf/distro/arago-source-ipk.conf | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta-arago-distro/conf/distro/arago-source-ipk.conf b/meta-arago-distro/conf/distro/arago-source-ipk.conf
index 96f2db2..5d2796f 100644
--- a/meta-arago-distro/conf/distro/arago-source-ipk.conf
+++ b/meta-arago-distro/conf/distro/arago-source-ipk.conf
@@ -357,6 +357,9 @@ CREATE_SRCIPK_pn-linalg-examples = "1"
 SRCIPK_INSTALL_DIR_pn-linalg-examples = "example-applications/${PN}-${PV}"
 SRCIPK_SRC_DIR_pn-linalg-examples = "${EXAMPLES_ROOT}"
 
+CREATE_SRCIPK_pn-barcode-roi = "1"
+SRCIPK_INSTALL_DIR_pn-barcode-roi = "example-applications/${PN}-${PV}"
+
 CREATE_SRCIPK_pn-boot-monitor = "1"
 SRCIPK_INSTALL_DIR_pn-boot-monitor = "board-support/${PN}-${PV}"
 
-- 
1.9.1

[morty/krogoth][PATCH 3/6] ti-tisdk-makefile: add barcode-roi demo

[Hongmei Gou]

Signed-off-by: Hongmei Gou <h-gou@ti.com>
---
 .../ti-tisdk-makefile/Makefile_barcode-roi            | 19 +++++++++++++++++++
 .../ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb        |  7 ++++++-
 2 files changed, 25 insertions(+), 1 deletion(-)
 create mode 100644 meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile/Makefile_barcode-roi

diff --git a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile/Makefile_barcode-roi b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile/Makefile_barcode-roi
new file mode 100644
index 0000000..7d7ae56
--- /dev/null
+++ b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile/Makefile_barcode-roi
@@ -0,0 +1,19 @@
+# barcode-roi build targets
+
+barcode-roi:
+	@echo =============================
+	@echo    Building Barcode ROI
+	@echo =============================
+	@cd example-applications; cd `find . -name "*barcode-roi*"`; make -f Makefile.build release
+
+barcode-roi_clean:
+	@echo =============================
+	@echo    Cleaning Barcode ROI
+	@echo =============================
+	@cd example-applications; cd `find . -name "*barcode-roi*"`; make -f Makefile.build clean
+
+barcode-roi_install:
+	@echo ================================================
+	@echo   Installing Barcode ROI - Release version
+	@echo ================================================
+	@cd example-applications; cd `find . -name "*barcode-roi*"`; make -f Makefile.build install
diff --git a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb
index 25cde69..8cc0758 100644
--- a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb
+++ b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb
@@ -47,9 +47,10 @@ SRC_URI = "\
     file://Makefile_openmpacc-examples \
     file://Makefile_linalg-examples \
     file://Makefile_ti-gc320-driver \
+    file://Makefile_barcode-roi \
 "
 
-PR = "r76"
+PR = "r77"
 
 MAKEFILES_MATRIX_GUI = "matrix-gui-browser \
                         refresh-screen \
@@ -88,6 +89,7 @@ MAKEFILES_append_ti33x = " u-boot-spl \
                            cryptodev \
                            ti-sgx-ddk-km \
                            pru-icss \
+                           barcode-roi \
 "
 MAKEFILES_append_ti43x = " u-boot-spl \
                            ${QUICK_PLAYGROUND} \
@@ -98,6 +100,7 @@ MAKEFILES_append_ti43x = " u-boot-spl \
                            image-gallery \
                            ti-sgx-ddk-km \
                            pru-icss \
+                           barcode-roi \
 "
 
 MAKEFILES_append_dra7xx = " cryptodev \
@@ -116,6 +119,7 @@ MAKEFILES_append_omap-a15 = " u-boot-spl \
                               cmem-mod \
                               pru-icss \
                               ti-gc320-driver \
+                              barcode-roi \
 "
 MAKEFILES_append_am180x-evm = " pru \
                                 u-boot-legacy \
@@ -130,6 +134,7 @@ MAKEFILES_append_keystone = " u-boot-spl \
                               hplib-mod \
                               uio-module-drv \
                               ipsecmgr-mod \
+                              barcode-roi \
 "
 
 MAKEFILES_append_k2hk-evm = " opencl-examples \
-- 
1.9.1

[morty/krogoth][PATCH 4/6] matrix-gui-machinevision-demos: add barcode-roi demo

[Hongmei Gou]

* machinevision_barcode_roi for displaying the output
* machinevision_barcode_roi_f2f for saving the output to a file

Signed-off-by: Hongmei Gou <h-gou@ti.com>
---
 .../matrix/matrix-gui-machinevision-demos_2.0.bb   | 24 +++++++++++++++++++---
 1 file changed, 21 insertions(+), 3 deletions(-)

diff --git a/meta-arago-extras/recipes-core/matrix/matrix-gui-machinevision-demos_2.0.bb b/meta-arago-extras/recipes-core/matrix/matrix-gui-machinevision-demos_2.0.bb
index 4bd245d..9c7eb92 100644
--- a/meta-arago-extras/recipes-core/matrix/matrix-gui-machinevision-demos_2.0.bb
+++ b/meta-arago-extras/recipes-core/matrix/matrix-gui-machinevision-demos_2.0.bb
@@ -3,7 +3,7 @@ HOMEPAGE = "https://gitorious.org/matrix-gui-v2/matrix-gui-v2-apps"
 
 require recipes-core/matrix/matrix-gui-apps-git.inc
 
-PR = "${INC_PR}.1"
+PR = "${INC_PR}.2"
 
 inherit allarch
 
@@ -14,12 +14,30 @@ MACHINEVISION_RDEPENDS += "matrix-gui-apps-images matrix-gui-submenus-machinevis
 
 FILES_${PN} += "${MATRIX_BASE_DIR}/*"
 
-PACKAGES = "matrix-machinevision-demo-dlp3dscanner"
+PACKAGES = "matrix-machinevision-demo-dlp3dscanner \
+            matrix-machinevision-demo-barcoderoi \
+            matrix-machinevision-demo-barcoderoi-f2f \
+"
 
-RDEPENDS_matrix-machinevision-demo-dlp3dscanner = " \
+RDEPENDS_matrix-machinevision-demo-dlp3dscanner_dra7xx = " \
     ${MACHINEVISION_RDEPENDS} \
     point-cloud-viewer \
 "
 
+RDEPENDS_matrix-machinevision-demo-barcoderoi = " \
+    ${MACHINEVISION_RDEPENDS} \
+    barcode-roi \
+"
+
+RDEPENDS_matrix-machinevision-demo-barcoderoi-f2f = " \
+    ${MACHINEVISION_RDEPENDS} \
+    barcode-roi \
+"
+
 FILES_matrix-machinevision-demo-dlp3dscanner    = "${MATRIX_APP_DIR}/machinevision_dlp_3d_scanner/*"
 FILES_matrix-machinevision-demo-dlp3dscanner   += "${bindir}/runDlp3DScanner.sh"
+
+FILES_matrix-machinevision-demo-barcoderoi    = "${MATRIX_APP_DIR}/machinevision_barcode_roi/*"
+FILES_matrix-machinevision-demo-barcoderoi   += "${bindir}/runBarcodeRoi.sh"
+FILES_matrix-machinevision-demo-barcoderoi-f2f = "${MATRIX_APP_DIR}/machinevision_barcode_roi_f2f/*"
+FILES_matrix-machinevision-demo-barcoderoi-f2f += "${bindir}/runBarcodeRoiSave2File.sh"
-- 
1.9.1

[morty/krogoth][PATCH 5/6] packagegroup-arago-tisdk-matrix: add barcode-roi demo

[Hongmei Gou]

Signed-off-by: Hongmei Gou <h-gou@ti.com>
---
 .../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

[morty/krogoth][PATCH 6/6] packagegroup-arago-tisdk-multimedia-sdk*: add barcode-roi demo

[Hongmei Gou]

Signed-off-by: Hongmei Gou <h-gou@ti.com>
---
 .../packagegroup-arago-tisdk-multimedia-sdk-host.bb              | 9 ++++++++-
 .../packagegroup-arago-tisdk-multimedia-sdk-target.bb            | 8 +++++++-
 2 files changed, 15 insertions(+), 2 deletions(-)

diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia-sdk-host.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia-sdk-host.bb
index 9bb62fc..cded1aa 100644
--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia-sdk-host.bb
+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia-sdk-host.bb
@@ -1,6 +1,6 @@
 DESCRIPTION = "Task to add multimedia related sources into the SDK"
 LICENSE = "MIT"
-PR = "r12"
+PR = "r13"
 
 PACKAGE_ARCH = "${MACHINE_ARCH}"
 
@@ -11,12 +11,19 @@ MULTIMEDIA = ""
 MULTIMEDIA_append_ti43x = " \
     dual-camera-demo-src \
     image-gallery-src \
+    barcode-roi-src \
 "
 
+MULTIMEDIA_append_ti33x = " barcode-roi-src"
+
+MULTIMEDIA_append_omap-a15 = " barcode-roi-src"
+
 MULTIMEDIA_append_dra7xx = " \
     ${@bb.utils.contains('MACHINE_FEATURES', 'opencl', 'qt-opencv-opencl-opengl-multithreaded-src', '', d)} \
 "
 
+MULTIMEDIA_append_keystone = " barcode-roi-src"
+
 #Demo doesn't work on 3.14
 # ${@base_conditional('QT_PROVIDER', 'qt5', '', 'dual-camera-demo-src image-gallery-src', d)}
 
diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia-sdk-target.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia-sdk-target.bb
index e77f66d..3c72bf0 100644
--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia-sdk-target.bb
+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia-sdk-target.bb
@@ -1,6 +1,6 @@
 DESCRIPTION = "Task to build and install header and libs into sdk"
 LICENSE = "MIT"
-PR = "r9"
+PR = "r10"
 
 PACKAGE_ARCH = "${MACHINE_ARCH}"
 
@@ -14,6 +14,12 @@ MULTIMEDIA_append_dra7xx = " \
     ${@bb.utils.contains('MACHINE_FEATURES', 'opencl', 'qt-opencv-opencl-opengl-multithreaded-dev', '', d)} \
 "
 
+MULTIMEDIA_append_ti33x = " barcode-roi-dev"
+MULTIMEDIA_append_ti43x = " barcode-roi-dev"
+MULTIMEDIA_append_omap-a15 = " barcode-roi-dev"
+
+MULTIMEDIA_append_keystone = " barcode-roi-dev"
+
 RDEPENDS_${PN} = "\
     ${MULTIMEDIA} \
     packagegroup-arago-gst-sdk-target \
-- 
1.9.1

Re: [morty/krogoth][PATCH 5/6] packagegroup-arago-tisdk-matrix: add barcode-roi demo

[Denys Dmytriyenko]

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 <h-gou@ti.com>
> ---
>  .../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
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [morty/krogoth][PATCH 5/6] packagegroup-arago-tisdk-matrix: add barcode-roi demo

[Gou, Hongmei]

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: meta-arago@arago-project.org
> 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 <h-gou@ti.com>
> > ---
> >  .../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
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [morty/krogoth][PATCH 5/6] packagegroup-arago-tisdk-matrix: add barcode-roi demo

[Denys Dmytriyenko]

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: meta-arago@arago-project.org
> > 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 <h-gou@ti.com>
> > > ---
> > >  .../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
> > > meta-arago@arago-project.org
> > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [morty/krogoth][PATCH 5/6] packagegroup-arago-tisdk-matrix: add barcode-roi demo

[Gou, Hongmei]

V2 patch has been submitted.

Thanks,
Hongmei

> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Monday, February 06, 2017 3:15 PM
> To: Gou, Hongmei
> Cc: meta-arago@arago-project.org
> 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: meta-arago@arago-project.org
> > > 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 <h-gou@ti.com>
> > > > ---
> > > >  .../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
> > > > meta-arago@arago-project.org
> > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago