[PATCH v2] OpenCSD: Support for Arm CoreSight decode lib

[PATCH v2] OpenCSD: Support for Arm CoreSight decode lib

[Leo Yan]

This patch is to add recipe for OpenCSD, which is an open source
CoreSight trace decode library and utility.

Also create a new folder recipes-devtools under meta-arm to place
this recipe.

v2: Refined recipe to use '-C' make option so can catch make error
    (Bertrand Marquis).

Signed-off-by: Leo Yan <leo.yan@linaro.org>
---
 .../recipes-devtools/opencsd/opencsd_git.bb   | 28 +++++++++++++++++++
 1 file changed, 28 insertions(+)
 create mode 100644 meta-arm/recipes-devtools/opencsd/opencsd_git.bb

diff --git a/meta-arm/recipes-devtools/opencsd/opencsd_git.bb b/meta-arm/recipes-devtools/opencsd/opencsd_git.bb
new file mode 100644
index 0000000..af96723
--- /dev/null
+++ b/meta-arm/recipes-devtools/opencsd/opencsd_git.bb
@@ -0,0 +1,28 @@
+SUMMARY = "OpenCSD - An open source CoreSight(tm) Trace Decode library"
+HOMEPAGE = "https://github.com/Linaro/OpenCSD"
+LICENSE = "BSD-3-Clause"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=ad8cb685eb324d2fa2530b985a43f3e5"
+
+SRC_URI = "git://github.com/Linaro/OpenCSD;protocol=http;branch=master"
+SRCREV = "03c194117971e4ad0598df29395757ced2e6e9bd"
+
+S = "${WORKDIR}/git"
+
+COMPATIBLE_HOST = "(x86_64.*|aarch64.*)-linux"
+
+EXTRA_OEMAKE = "ARCH='${TARGET_ARCH}' \
+                CROSS_COMPILE='${TARGET_SYS}-' \
+                CC='${CC}' \
+                CXX='${CXX}' \
+                LINKER='${CXX}' \
+                LINUX64=1 \
+                DEBUG=1 \
+                "
+
+do_compile() {
+    oe_runmake -C ${S}/decoder/build/linux ${EXTRA_OEMAKE}
+}
+
+do_install() {
+    oe_runmake -C ${S}/decoder/build/linux PREFIX=${D}/usr install
+}
-- 
2.17.1

Re: [PATCH v2] OpenCSD: Support for Arm CoreSight decode lib

[Bertrand Marquis]

> On 18 Mar 2020, at 04:28, Leo Yan <leo.yan@linaro.org> wrote:
> 
> This patch is to add recipe for OpenCSD, which is an open source
> CoreSight trace decode library and utility.
> 
> Also create a new folder recipes-devtools under meta-arm to place
> this recipe.
> 
> v2: Refined recipe to use '-C' make option so can catch make error
>    (Bertrand Marquis).
> 
> Signed-off-by: Leo Yan <leo.yan@linaro.org>
> ---
> .../recipes-devtools/opencsd/opencsd_git.bb   | 28 +++++++++++++++++++
> 1 file changed, 28 insertions(+)
> create mode 100644 meta-arm/recipes-devtools/opencsd/opencsd_git.bb
> 
> diff --git a/meta-arm/recipes-devtools/opencsd/opencsd_git.bb b/meta-arm/recipes-devtools/opencsd/opencsd_git.bb
> new file mode 100644
> index 0000000..af96723
> --- /dev/null
> +++ b/meta-arm/recipes-devtools/opencsd/opencsd_git.bb
> @@ -0,0 +1,28 @@
> +SUMMARY = "OpenCSD - An open source CoreSight(tm) Trace Decode library"
> +HOMEPAGE = "https://github.com/Linaro/OpenCSD"
> +LICENSE = "BSD-3-Clause"
> +LIC_FILES_CHKSUM = "file://LICENSE;md5=ad8cb685eb324d2fa2530b985a43f3e5"
> +
> +SRC_URI = "git://github.com/Linaro/OpenCSD;protocol=http;branch=master"
> +SRCREV = "03c194117971e4ad0598df29395757ced2e6e9bd"
> +
> +S = "${WORKDIR}/git"
> +
> +COMPATIBLE_HOST = "(x86_64.*|aarch64.*)-linux"
> +
> +EXTRA_OEMAKE = "ARCH='${TARGET_ARCH}' \
> +                CROSS_COMPILE='${TARGET_SYS}-' \
> +                CC='${CC}' \
> +                CXX='${CXX}' \
> +                LINKER='${CXX}' \
> +                LINUX64=1 \
> +                DEBUG=1 \
> +                "
> +
> +do_compile() {
> +    oe_runmake -C ${S}/decoder/build/linux ${EXTRA_OEMAKE}
> +}
> +
> +do_install() {
> +    oe_runmake -C ${S}/decoder/build/linux PREFIX=${D}/usr install
> +}
> — 
> 2.17.1
> 

Would it make sense to have this extending native ? This is a development tool so one might want to use this on his development host instead of the target.

Just add the following to your recipe:
BBCLASSEXTEND = "native"

This would allow to build this for the dev host using “bitbake opencsd-native"

Regards
Bertrand

Re: [PATCH v2] OpenCSD: Support for Arm CoreSight decode lib

[Leo Yan]

Hi Bertrand,

On Wed, Mar 18, 2020 at 06:52:11AM +0000, Bertrand Marquis wrote:

[...]

> > --- /dev/null
> > +++ b/meta-arm/recipes-devtools/opencsd/opencsd_git.bb
> > @@ -0,0 +1,28 @@
> > +SUMMARY = "OpenCSD - An open source CoreSight(tm) Trace Decode library"
> > +HOMEPAGE = "https://github.com/Linaro/OpenCSD"
> > +LICENSE = "BSD-3-Clause"
> > +LIC_FILES_CHKSUM = "file://LICENSE;md5=ad8cb685eb324d2fa2530b985a43f3e5"
> > +
> > +SRC_URI = "git://github.com/Linaro/OpenCSD;protocol=http;branch=master"
> > +SRCREV = "03c194117971e4ad0598df29395757ced2e6e9bd"
> > +
> > +S = "${WORKDIR}/git"
> > +
> > +COMPATIBLE_HOST = "(x86_64.*|aarch64.*)-linux"
> > +
> > +EXTRA_OEMAKE = "ARCH='${TARGET_ARCH}' \
> > +                CROSS_COMPILE='${TARGET_SYS}-' \
> > +                CC='${CC}' \
> > +                CXX='${CXX}' \
> > +                LINKER='${CXX}' \
> > +                LINUX64=1 \
> > +                DEBUG=1 \
> > +                "
> > +
> > +do_compile() {
> > +    oe_runmake -C ${S}/decoder/build/linux ${EXTRA_OEMAKE}
> > +}
> > +
> > +do_install() {
> > +    oe_runmake -C ${S}/decoder/build/linux PREFIX=${D}/usr install
> > +}
> > — 
> > 2.17.1
> > 
> 
> Would it make sense to have this extending native ? This is a development tool so one might want to use this on his development host instead of the target.
> 
> Just add the following to your recipe:
> BBCLASSEXTEND = "native"
> 
> This would allow to build this for the dev host using “bitbake opencsd-native"

OpenCSD can be built for x86; makes sense for me.

Will do it.  Thanks for suggestion!

Leo