Re: [meta-arago][scarthgap][PATCH] arm-ddr-bandwidth: Bandwidth script for testing

[Denys Dmytriyenko <denis@denix.org>]

On Tue, Jun 04, 2024 at 11:38:32AM -0500, Ryan Eatmon wrote:
> 
> 
> On 6/4/2024 11:05 AM, Denys Dmytriyenko wrote:
> >On Mon, Jun 03, 2024 at 11:10:39AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
> >>We need to pull a single script in from the matrix-gui packages since
> >>they are getting removed.  The runLmDDRBandwidth.sh is required for the
> >>ti-test flow.
> >>
> >>Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> >>---
> >>  .../arm-ddr-bandwidth/arm-ddr-bandwidth.bb    | 23 +++++++++++++++++++
> >>  .../recipes-core/packagegroups/ti-test.bb     |  1 +
> >>  2 files changed, 24 insertions(+)
> >>  create mode 100644 meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb
> >>
> >>diff --git a/meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb b/meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb
> >>new file mode 100644
> >>index 00000000..fc4aea94
> >>--- /dev/null
> >>+++ b/meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb
> >>@@ -0,0 +1,23 @@
> >>+SUMMARY = "Benchmark for checking DDR bandwidth"
> >>+HOMEPAGE = "https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/tree/arm_apps/arm_ddr_bandwidth/runLmDDRBandwidth.sh"
> >>+LICENSE = "CC-BY-SA-3.0"
> >>+LIC_FILES_CHKSUM = "file://${WORKDIR}/LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
> >
> >Isn't the default here ^^^ relative to ${S} anyway?
> 
> Could be.  I was making a number of changes to get this recipe
> "right". I'll test it and make sure it works without it.  It might
> be that master now requires the UNPACKDIR and when I backported this
> to scarthgap I didn't check that it was not needed.

Yeah, it should also work for master, as long as LICENSE is in ${S}
And I see there you set UNPACKDIR = "${S}"


> >>+PV = "1.0"
> >>+
> >>+SRC_URI = " \
> >>+    https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/plain/arm_apps/arm_ddr_bandwidth/runLmDDRBandwidth.sh;name=runLmDDRBandwidth \
> >>+    https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/plain/LICENSE;name=license \
> >>+"
> >>+SRC_URI[runLmDDRBandwidth.sha256sum] = "67997d721b351344d8788200a15bb3640093045bc25f2b5307dd331ebd1556a3"
> >>+SRC_URI[license.sha256sum] = "7febd1df714fa4b1e44fe0b0f73ceac7f9b9f97326695a0cc7074bd6c8d263e3"
> >>+
> >>+S = "${WORKDIR}"
> >>+
> >>+do_install() {
> >>+    install -d ${D}${datadir}/matrix-gui-2.0/apps
> >>+    install -m 0644 ${S}/runLmDDRBandwidth.sh ${D}${datadir}/matrix-gui-2.0/apps/runLmDDRBandwidth.sh
> >>+}
> >>+
> >>+FILES:${PN} = "${datadir}/matrix-gui-2.0/apps/runLmDDRBandwidth.sh"
> >>+
> >>diff --git a/meta-arago-test/recipes-core/packagegroups/ti-test.bb b/meta-arago-test/recipes-core/packagegroups/ti-test.bb
> >>index 4dbd9fca..b71c374b 100644
> >>--- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb
> >>+++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb
> >>@@ -69,6 +69,7 @@ TI_TEST_BASE:append:armv7ve = " \
> >>  #    timestamping
> >>  TI_TEST_TI_TOOLS = " \
> >>      arm-benchmarks \
> >>+    arm-ddr-bandwidth \
> >>      cpuloadgen \
> >>      input-utils \
> >>      ltp-ddt \
> >>-- 
> >>2.17.1