>-----Original Message-----
>From: Dmytriyenko, Denys
>Sent: Monday, November 19, 2018 3:32 PM
>To: Su, Qin
>Cc: [email protected]
>Subject: Re: [meta-arago] [PATCH] armnn: bump SRCREV and add a demo
>example
>
>On Fri, Nov 16, 2018 at 03:27:32PM -0500, Qin Su wrote:
>> updated to the latest version of 18.08 updated to generate versioned
>> library created a demo example with mobileNet
>
>Same comment about missing Upstream-Status for new patches.
>

Fixed in patch v2

>> Signed-off-by: Qin Su <[email protected]>
>> ---
>>  .../0003-add-more-test-command-arguments.patch     |  74 ++
>>  .../armnn/0004-generate-versioned-library.patch    |  62 ++
>>  .../0005-add-armnn-mobilenet-test-example.patch    |  68 ++
>>  .../armnn/0006-armnn-mobilenet-test-example.patch  | 862
>+++++++++++++++++++++
>>  .../recipes-support/armnn/armnn_git.bb             |  31 +-
>>  5 files changed, 1092 insertions(+), 5 deletions(-)  create mode
>> 100755
>> meta-arago-extras/recipes-support/armnn/armnn/0003-add-more-test-
>comma
>> nd-arguments.patch  create mode 100755
>> meta-arago-extras/recipes-support/armnn/armnn/0004-generate-
>versioned-
>> library.patch  create mode 100755
>> meta-arago-extras/recipes-support/armnn/armnn/0005-add-armnn-
>mobilenet
>> -test-example.patch  create mode 100755
>> meta-arago-extras/recipes-support/armnn/armnn/0006-armnn-mobilenet-
>tes
>> t-example.patch
>
>[snip]
>
>
>> diff --git a/meta-arago-extras/recipes-support/armnn/armnn_git.bb
>> b/meta-arago-extras/recipes-support/armnn/armnn_git.bb
>> index 726a9da..c31e4ae 100644
>> --- a/meta-arago-extras/recipes-support/armnn/armnn_git.bb
>> +++ b/meta-arago-extras/recipes-support/armnn/armnn_git.bb
>> @@ -1,12 +1,13 @@
>>  SUMMARY = "ARM Neural Network SDK"
>>  DESCRIPTION = "Linux software and tools to enable machine learning
>(Caffe/Tensorflow) workloads on power-efficient devices"
>> -LICENSE = "MIT"
>> +LICENSE = "MIT & Apache-2.0"
>>  LIC_FILES_CHKSUM =
>"file://LICENSE;md5=3e14a924c16f7d828b8335a59da64074"
>
>Did the license change? Also, need a checksum for Apache-2.0 too.
>

No license change.
Added checksum for Apache-2.0 in patch v2

>> -PV = "20180502"
>> +PV = "18.08"
>> +PV_MAJOR = "${@d.getVar('PV',d,1).split('.')[0]}"
>>
>>  BRANCH = "master"
>> -SRCREV = "4c7098bfeab1ffe1cdc77f6c15548d3e73274746"
>> +SRCREV = "c577f2c6a3b4ddb6ba87a882723c53a248afbeba"
>>
>>  S = "${WORKDIR}/git"
>>
>> @@ -16,8 +17,20 @@ SRC_URI = " \
>>      git://github.com/ARM-software/armnn.git;branch=${BRANCH} \
>
>Might want to add "name=armnn" above since used in SRCREV_FORMAT.
>

Added "name=armnn" in patch v2

>>      file://0001-stdlib-issue-work-around.patch \
>>      file://0002-enable-use-of-boost-shared-library.patch \
>> +    file://0003-add-more-test-command-arguments.patch \
>> +    file://0004-generate-versioned-library.patch \
>> +    file://0005-add-armnn-mobilenet-test-example.patch \
>> +    file://0006-armnn-mobilenet-test-example.patch \
>> +
>> +
>http://download.tensorflow.org/models/mobilenet_v1_2018_02_22/mobile
>> + net_v1_1.0_224.tgz; \
>
>Might want to add "name=mobilenet" for use in
>SRC_URI[md5sum/sha256sum]
>

Added "name=mobilenet" in patch v2

>> +
>> + git://git.ti.com/tidl/tidl-api.git;name=tidl-api;branch=${BRANCH};su
>> + bdir=${WORKDIR}/tidl-api;destsuffix=tidl-api \
>>  "
>>
>> +SRC_URI[md5sum] = "d5f69cef81ad8afb335d9727a17c462a"
>> +SRC_URI[sha256sum] =
>"1ccb74dbd9c5f7aea879120614e91617db9534bdfaa53dfea54b7c14162e126b"
>> +
>> +SRCREV_tidl-api = "7e9a3942ec38efd64d45e34c10cba2f2938f5618"
>> +SRCREV_FORMAT = "armnn"
>> +
>>  DEPENDS = " \
>>      boost \
>>      protobuf \
>> @@ -25,6 +38,7 @@ DEPENDS = " \
>>      arm-compute-library \
>>      armnn-caffe \
>>      armnn-tensorflow \
>> +    opencv \
>>  "
>>
>>  RDEPENDS_${PN} = " arm-compute-library protobuf boost "
>> @@ -39,21 +53,28 @@ EXTRA_OECMAKE=" \
>>      -DARMCOMPUTENEON=1 \
>>      -DBUILD_TESTS=1 -DPROFILING=1 \
>>      -DTHIRD_PARTY_INCLUDE_DIRS=${STAGING_DIR_HOST}${includedir} \
>> +    -DBUILD_ARMNN_EXAMPLES=1 \
>> +    -DGENERIC_LIB_VERSION=${PV} -
>DGENERIC_LIB_SOVERSION=${PV_MAJOR} \
>>  "
>>
>> +TESTVECS_INSTALL_DIR = "${datadir}/arm/armnn"
>> +
>>  do_install_append() {
>>      CP_ARGS="-Prf --preserve=mode,timestamps --no-preserve=ownership"
>>      install -d ${D}${bindir}
>>      find ${WORKDIR}/build/tests -maxdepth 1 -type f -executable -exec cp
>$CP_ARGS {} ${D}${bindir} \;
>>      cp $CP_ARGS ${WORKDIR}/build/UnitTests  ${D}${bindir}
>> +    install -d ${D}${TESTVECS_INSTALL_DIR}/models
>> +    cp ${WORKDIR}/mobilenet_v1_1.0_224_frozen.pb
>${D}${TESTVECS_INSTALL_DIR}/models
>> +    cp ${WORKDIR}/git/tests/TfMobileNet-Armnn/labels.txt
>${D}${TESTVECS_INSTALL_DIR}/models
>> +    install -d ${D}${TESTVECS_INSTALL_DIR}/testvecs
>> +    cp ${WORKDIR}/tidl-api/examples/classification/clips/test2.mp4
>> + ${D}${TESTVECS_INSTALL_DIR}/testvecs
>>      chrpath -d ${D}${bindir}/*
>>  }
>>
>>  CXXFLAGS += "-fopenmp"
>>  LIBS += "-larmpl_lp64_mp"
>>
>> -SOLIBS = ".so"
>> -FILES_SOLIBSDEV = ""
>>  FILES_${PN} += "{bindir}/*"
>>  FILES_${PN} += "{libdir}/*"
>>  FILES_${PN}-dev += "{libdir}/cmake/*"
>> --
>> 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

Reply via email to