All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/3] gdbc6x 1.1.0
@ 2015-05-18 17:32 Jacob Stiffler
  2015-05-18 17:32 ` [PATCH 1/3] gdbserverproxy-module-drv: Add version 1.1.0 Jacob Stiffler
                   ` (2 more replies)
  0 siblings, 3 replies; 14+ messages in thread
From: Jacob Stiffler @ 2015-05-18 17:32 UTC (permalink / raw)
  To: meta-ti

* The gdbc6x utility provides remote gdb capabilities for C6x DSP
  cores.

Jacob Stiffler (3):
  gdbserverproxy-module-drv: Add version 1.1.0
  gdbserver-c6x: Add version 1.1.0
  gdbc6x: Add version 1.1.0

 recipes-devtools/gdbc6x/gdbc6x.inc                 |    9 ++++
 recipes-devtools/gdbc6x/gdbc6x/init                |   48 ++++++++++++++++++++
 recipes-devtools/gdbc6x/gdbc6x_git.bb              |   48 ++++++++++++++++++++
 recipes-devtools/gdbc6x/gdbserver-c6x_git.bb       |   46 +++++++++++++++++++
 .../gdbc6x/gdbserverproxy-module-drv_git.bb        |   31 +++++++++++++
 5 files changed, 182 insertions(+)
 create mode 100644 recipes-devtools/gdbc6x/gdbc6x.inc
 create mode 100644 recipes-devtools/gdbc6x/gdbc6x/init
 create mode 100644 recipes-devtools/gdbc6x/gdbc6x_git.bb
 create mode 100644 recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
 create mode 100644 recipes-devtools/gdbc6x/gdbserverproxy-module-drv_git.bb

-- 
1.7.9.5



^ permalink raw reply	[flat|nested] 14+ messages in thread

* [PATCH 1/3] gdbserverproxy-module-drv: Add version 1.1.0
  2015-05-18 17:32 [PATCH 0/3] gdbc6x 1.1.0 Jacob Stiffler
@ 2015-05-18 17:32 ` Jacob Stiffler
  2015-05-18 17:40   ` Denys Dmytriyenko
  2015-05-18 17:32 ` [PATCH 2/3] gdbserver-c6x: " Jacob Stiffler
  2015-05-18 17:32 ` [PATCH 3/3] gdbc6x: " Jacob Stiffler
  2 siblings, 1 reply; 14+ messages in thread
From: Jacob Stiffler @ 2015-05-18 17:32 UTC (permalink / raw)
  To: meta-ti

Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
---
 recipes-devtools/gdbc6x/gdbc6x.inc                 |    9 ++++++
 .../gdbc6x/gdbserverproxy-module-drv_git.bb        |   31 ++++++++++++++++++++
 2 files changed, 40 insertions(+)
 create mode 100644 recipes-devtools/gdbc6x/gdbc6x.inc
 create mode 100644 recipes-devtools/gdbc6x/gdbserverproxy-module-drv_git.bb

diff --git a/recipes-devtools/gdbc6x/gdbc6x.inc b/recipes-devtools/gdbc6x/gdbc6x.inc
new file mode 100644
index 0000000..ecdc11e
--- /dev/null
+++ b/recipes-devtools/gdbc6x/gdbc6x.inc
@@ -0,0 +1,9 @@
+BRANCH = "master"
+
+PV = "1.1.0"
+INC_PR = "r0"
+
+# This corresponds to version 1.1.0    
+SRCREV = "1ab06f8bcfe7540754558bac331a0312c4b59142"
+
+SRC_URI = "git://git.ti.com/sdo-emu/gdbc6x.git;protocol=git;branch=${BRANCH}"
diff --git a/recipes-devtools/gdbc6x/gdbserverproxy-module-drv_git.bb b/recipes-devtools/gdbc6x/gdbserverproxy-module-drv_git.bb
new file mode 100644
index 0000000..2b02236
--- /dev/null
+++ b/recipes-devtools/gdbc6x/gdbserverproxy-module-drv_git.bb
@@ -0,0 +1,31 @@
+DESCRIPTION="Interface for GDB to commincate witha TI C66X DSP"
+
+LICENSE = "GPLv2"
+LIC_FILES_CHKSUM = "file://COPYING.txt;md5=75859989545e37968a99b631ef42722e"
+
+include gdbc6x.inc
+
+COMPATIBLE_MACHINE = "dra7xx"
+PACKAGE_ARCH = "${MACHINE_ARCH}"
+
+# This package builds a kernel module, use kernel PR as base and append a local
+MACHINE_KERNEL_PR_append = "a+gitr${SRCPV}"
+PR = "${MACHINE_KERNEL_PR}"
+
+PLATFORM = ""
+PLATFORM_dra7xx = "DRA7xx_PLATFORM"
+
+EXTRA_OEMAKE = "PLATFORM=${PLATFORM}"
+
+S = "${WORKDIR}/git/kernel_module/gdbproxy-mod"
+
+module_autoload_gdbserverproxy = "gdbserverproxy"
+
+inherit module
+
+do_configure[noexec] = "1"
+do_qa_configure[noexec] = "1"
+
+CREATE_SRCIPK = "1"
+SRCIPK_INSTALL_DIR = "board-support/extra-drivers/${PN}-${PV}"
+SRCIPK_PACKAGE_ARCH = "${MACHINE_ARCH}"
-- 
1.7.9.5



^ permalink raw reply related	[flat|nested] 14+ messages in thread

* [PATCH 2/3] gdbserver-c6x: Add version 1.1.0
  2015-05-18 17:32 [PATCH 0/3] gdbc6x 1.1.0 Jacob Stiffler
  2015-05-18 17:32 ` [PATCH 1/3] gdbserverproxy-module-drv: Add version 1.1.0 Jacob Stiffler
@ 2015-05-18 17:32 ` Jacob Stiffler
  2015-05-18 17:41   ` Denys Dmytriyenko
  2015-05-18 17:32 ` [PATCH 3/3] gdbc6x: " Jacob Stiffler
  2 siblings, 1 reply; 14+ messages in thread
From: Jacob Stiffler @ 2015-05-18 17:32 UTC (permalink / raw)
  To: meta-ti

* c6x DSP development files required to support remote gdb debugging.

Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
---
 recipes-devtools/gdbc6x/gdbserver-c6x_git.bb |   46 ++++++++++++++++++++++++++
 1 file changed, 46 insertions(+)
 create mode 100644 recipes-devtools/gdbc6x/gdbserver-c6x_git.bb

diff --git a/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb b/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
new file mode 100644
index 0000000..d21f03f
--- /dev/null
+++ b/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
@@ -0,0 +1,46 @@
+DESCRIPTION="TI gdb-server to be used with c6xgdb."
+
+LICENSE = "BSD"
+LIC_FILES_CHKSUM = "file://../debian/copyright;md5=82c616b6c8f9c11c46feaaf8f9a11495"
+
+include gdbc6x.inc
+
+COMPATIBLE_MACHINE = "dra7xx"
+PACKAGE_ARCH = "${MACHINE_ARCH}"
+
+PR = "${INC_PR}.0"
+
+require recipes-ti/includes/ti-paths.inc
+
+DEPENDS = "ti-xdais ti-framework-components ti-cgt6x-native ti-sysbios ti-xdctools"
+
+export XDAIS_DIR = "${XDAIS_INSTALL_DIR}"
+export FC_DIR = "${FC_INSTALL_DIR}"
+export TI_CGT_INSTALL = "${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
+export BIOS_DIR = "${SYSBIOS_INSTALL_DIR}" 
+export XDC_DIR = "${XDC_INSTALL_DIR}"
+
+PLATFORM = ""
+PLATFORM_dra7xx = "DRA7xx_PLATFORM"
+
+EXTRA_OEMAKE = "PLATFORM=${PLATFORM}"
+
+PARALLEL_MAKE = ""
+
+S = "${WORKDIR}/git/gdbserver-c6x/src"
+
+do_install() {
+    install -d ${D}${datadir}/ti/gdbc6x/include
+    install -d ${D}${datadir}/ti/gdbc6x/lib
+    cp -f ../include/* ${D}${datadir}/ti/gdbc6x/include
+    cp -f ../lib/* ${D}${datadir}/ti/gdbc6x/lib
+}
+
+ALLOW_EMPTY_${PN} = "1"
+
+FILES_${PN}-dev += "\
+    ${datadir}/ti/gdbc6x \
+"
+
+do_configure[noexec] = "1"
+do_qa_configure[noexec] = "1"
-- 
1.7.9.5



^ permalink raw reply related	[flat|nested] 14+ messages in thread

* [PATCH 3/3] gdbc6x: Add version 1.1.0
  2015-05-18 17:32 [PATCH 0/3] gdbc6x 1.1.0 Jacob Stiffler
  2015-05-18 17:32 ` [PATCH 1/3] gdbserverproxy-module-drv: Add version 1.1.0 Jacob Stiffler
  2015-05-18 17:32 ` [PATCH 2/3] gdbserver-c6x: " Jacob Stiffler
@ 2015-05-18 17:32 ` Jacob Stiffler
  2015-05-18 17:54   ` Denys Dmytriyenko
  2 siblings, 1 reply; 14+ messages in thread
From: Jacob Stiffler @ 2015-05-18 17:32 UTC (permalink / raw)
  To: meta-ti

* The gdbc6x utility provides remote gdb capabilities for C6x DSP
  cores.

Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
---
 recipes-devtools/gdbc6x/gdbc6x/init   |   48 +++++++++++++++++++++++++++++++++
 recipes-devtools/gdbc6x/gdbc6x_git.bb |   48 +++++++++++++++++++++++++++++++++
 2 files changed, 96 insertions(+)
 create mode 100644 recipes-devtools/gdbc6x/gdbc6x/init
 create mode 100644 recipes-devtools/gdbc6x/gdbc6x_git.bb

diff --git a/recipes-devtools/gdbc6x/gdbc6x/init b/recipes-devtools/gdbc6x/gdbc6x/init
new file mode 100644
index 0000000..90210b8
--- /dev/null
+++ b/recipes-devtools/gdbc6x/gdbc6x/init
@@ -0,0 +1,48 @@
+#!/bin/bash
+
+GDB_DEVICE_NAME='/dev/gdbtty$i'
+DSP_FIRMWARE_NAME='/lib/firmware/dra7-dsp$[$i+1]-fw.xe66'
+
+DSP_L2_GLOBAL_OFFSET="40000000"
+DSP_L2_GLOBAL_SHIFT="00800000"
+
+case "$1" in
+  start)
+
+    dsp_core_l2_offset=$DSP_L2_GLOBAL_OFFSET
+
+    i=0
+    while [ -c "`eval echo $GDB_DEVICE_NAME`" ]
+    do
+      dsp_firmware=`eval echo $DSP_FIRMWARE_NAME`
+
+      if [ -f "`eval echo $DSP_FIRMWARE_NAME`" ]
+      then 
+        eval echo "Initializing $GDB_DEVICE_NAME based on $DSP_FIRMWARE_NAME ..."
+
+        gdb_data_local=`eval readelf -s "$DSP_FIRMWARE_NAME" | \
+                        grep 'gdb_globalData' | \
+                        awk '{print $2}'`
+
+        echo "gdb_globalData (local)  = $gdb_data_local"
+        printf "gdb_globalData (global) = %X\n" \
+          $[0x$gdb_data_local + 0x$dsp_core_l2_offset]
+
+
+        printf "%X\n" $[0x$gdb_data_local + 0x$dsp_core_l2_offset] \
+          >> `eval echo $GDB_DEVICE_NAME`
+      fi
+      i=$[$i + 1]
+      dsp_core_l2_offset=`printf "%X" $[0x$dsp_core_l2_offset + 0x$DSP_L2_GLOBAL_SHIFT]`
+    done
+  ;;
+  stop)
+    # Nothing to be done.
+  ;;
+  *)
+    echo "Usage: $0 {start|stop}"
+    exit 1
+  ;;
+esac
+
+
diff --git a/recipes-devtools/gdbc6x/gdbc6x_git.bb b/recipes-devtools/gdbc6x/gdbc6x_git.bb
new file mode 100644
index 0000000..9fc95fe
--- /dev/null
+++ b/recipes-devtools/gdbc6x/gdbc6x_git.bb
@@ -0,0 +1,48 @@
+DESCRIPTION="GNU debugger for TI C6X DSP."
+
+LICENSE = "GPLv3+"
+SECTION = "devel"
+DEPENDS = "expat ncurses readline"
+
+LIC_FILES_CHKSUM = "file://debian/copyright;md5=bf0fe2872eb3dfeebb2cbe38206fe81f"
+
+include gdbc6x.inc
+
+PR = "${INC_PR}.0"
+
+DEPENDS = "ncurses bison texinfo flex gettext"
+
+RDEPENDS_${PN}  = "gdbserver-c6x gdbserverproxy-module-drv"
+
+S = "${WORKDIR}/git/gdbc6x"
+
+SRC_URI_append = " \
+    file://init \
+"
+
+inherit update-rc.d
+
+INITSCRIPT_NAME = "gdbserverproxy"
+INITSCRIPT_PARAMS = "defaults 95"
+
+inherit gettext
+
+PARALLEL_MAKE = ""
+
+do_configure () {
+    cd ${S}
+    ./configure --program-suffix=c6x --target=tic6x-elf-tirtos --host=${HOST_SYS} --prefix=${S}/install_gdb
+}
+
+do_install () {
+    make install
+
+    # Custom install to prevent conflict with standard GDB.
+    install -d ${D}${bindir}
+    install -d ${D}${includedir}
+    install -m 755  ${S}/install_gdb/bin/gdbc6x ${D}${bindir} 
+    cp -rf ${S}/install_gdb/include/* ${D}${includedir}
+
+    install -d ${D}${sysconfdir}/init.d
+    install -m 0755 ${WORKDIR}/init ${D}${sysconfdir}/init.d/gdbserverproxy
+}
-- 
1.7.9.5



^ permalink raw reply related	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/3] gdbserverproxy-module-drv: Add version 1.1.0
  2015-05-18 17:32 ` [PATCH 1/3] gdbserverproxy-module-drv: Add version 1.1.0 Jacob Stiffler
@ 2015-05-18 17:40   ` Denys Dmytriyenko
  2015-05-18 17:54     ` Jacob Stiffler
  0 siblings, 1 reply; 14+ messages in thread
From: Denys Dmytriyenko @ 2015-05-18 17:40 UTC (permalink / raw)
  To: Jacob Stiffler; +Cc: meta-ti

On Mon, May 18, 2015 at 01:32:54PM -0400, Jacob Stiffler wrote:
> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> ---
>  recipes-devtools/gdbc6x/gdbc6x.inc                 |    9 ++++++
>  .../gdbc6x/gdbserverproxy-module-drv_git.bb        |   31 ++++++++++++++++++++
>  2 files changed, 40 insertions(+)
>  create mode 100644 recipes-devtools/gdbc6x/gdbc6x.inc
>  create mode 100644 recipes-devtools/gdbc6x/gdbserverproxy-module-drv_git.bb
> 
> diff --git a/recipes-devtools/gdbc6x/gdbc6x.inc b/recipes-devtools/gdbc6x/gdbc6x.inc
> new file mode 100644
> index 0000000..ecdc11e
> --- /dev/null
> +++ b/recipes-devtools/gdbc6x/gdbc6x.inc
> @@ -0,0 +1,9 @@
> +BRANCH = "master"
> +
> +PV = "1.1.0"
> +INC_PR = "r0"
> +
> +# This corresponds to version 1.1.0    
> +SRCREV = "1ab06f8bcfe7540754558bac331a0312c4b59142"
> +
> +SRC_URI = "git://git.ti.com/sdo-emu/gdbc6x.git;protocol=git;branch=${BRANCH}"
> diff --git a/recipes-devtools/gdbc6x/gdbserverproxy-module-drv_git.bb b/recipes-devtools/gdbc6x/gdbserverproxy-module-drv_git.bb
> new file mode 100644
> index 0000000..2b02236
> --- /dev/null
> +++ b/recipes-devtools/gdbc6x/gdbserverproxy-module-drv_git.bb
> @@ -0,0 +1,31 @@
> +DESCRIPTION="Interface for GDB to commincate witha TI C66X DSP"
> +
> +LICENSE = "GPLv2"
> +LIC_FILES_CHKSUM = "file://COPYING.txt;md5=75859989545e37968a99b631ef42722e"
> +
> +include gdbc6x.inc
> +
> +COMPATIBLE_MACHINE = "dra7xx"
> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> +
> +# This package builds a kernel module, use kernel PR as base and append a local
> +MACHINE_KERNEL_PR_append = "a+gitr${SRCPV}"
> +PR = "${MACHINE_KERNEL_PR}"
> +
> +PLATFORM = ""
> +PLATFORM_dra7xx = "DRA7xx_PLATFORM"
> +
> +EXTRA_OEMAKE = "PLATFORM=${PLATFORM}"
> +
> +S = "${WORKDIR}/git/kernel_module/gdbproxy-mod"
> +
> +module_autoload_gdbserverproxy = "gdbserverproxy"
> +
> +inherit module
> +
> +do_configure[noexec] = "1"
> +do_qa_configure[noexec] = "1"

What's the need for the above?


> +CREATE_SRCIPK = "1"
> +SRCIPK_INSTALL_DIR = "board-support/extra-drivers/${PN}-${PV}"
> +SRCIPK_PACKAGE_ARCH = "${MACHINE_ARCH}"

This is not the right place for these^^ either.

-- 
Denys


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 2/3] gdbserver-c6x: Add version 1.1.0
  2015-05-18 17:32 ` [PATCH 2/3] gdbserver-c6x: " Jacob Stiffler
@ 2015-05-18 17:41   ` Denys Dmytriyenko
  2015-05-18 19:00     ` Jacob Stiffler
  0 siblings, 1 reply; 14+ messages in thread
From: Denys Dmytriyenko @ 2015-05-18 17:41 UTC (permalink / raw)
  To: Jacob Stiffler; +Cc: meta-ti

On Mon, May 18, 2015 at 01:32:55PM -0400, Jacob Stiffler wrote:
> * c6x DSP development files required to support remote gdb debugging.
> 
> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> ---
>  recipes-devtools/gdbc6x/gdbserver-c6x_git.bb |   46 ++++++++++++++++++++++++++
>  1 file changed, 46 insertions(+)
>  create mode 100644 recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
> 
> diff --git a/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb b/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
> new file mode 100644
> index 0000000..d21f03f
> --- /dev/null
> +++ b/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
> @@ -0,0 +1,46 @@
> +DESCRIPTION="TI gdb-server to be used with c6xgdb."
> +
> +LICENSE = "BSD"
> +LIC_FILES_CHKSUM = "file://../debian/copyright;md5=82c616b6c8f9c11c46feaaf8f9a11495"
> +
> +include gdbc6x.inc
> +
> +COMPATIBLE_MACHINE = "dra7xx"
> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> +
> +PR = "${INC_PR}.0"
> +
> +require recipes-ti/includes/ti-paths.inc
> +
> +DEPENDS = "ti-xdais ti-framework-components ti-cgt6x-native ti-sysbios ti-xdctools"
> +
> +export XDAIS_DIR = "${XDAIS_INSTALL_DIR}"
> +export FC_DIR = "${FC_INSTALL_DIR}"
> +export TI_CGT_INSTALL = "${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> +export BIOS_DIR = "${SYSBIOS_INSTALL_DIR}" 
> +export XDC_DIR = "${XDC_INSTALL_DIR}"
> +
> +PLATFORM = ""
> +PLATFORM_dra7xx = "DRA7xx_PLATFORM"
> +
> +EXTRA_OEMAKE = "PLATFORM=${PLATFORM}"
> +
> +PARALLEL_MAKE = ""
> +
> +S = "${WORKDIR}/git/gdbserver-c6x/src"
> +
> +do_install() {
> +    install -d ${D}${datadir}/ti/gdbc6x/include
> +    install -d ${D}${datadir}/ti/gdbc6x/lib
> +    cp -f ../include/* ${D}${datadir}/ti/gdbc6x/include
> +    cp -f ../lib/* ${D}${datadir}/ti/gdbc6x/lib
> +}
> +
> +ALLOW_EMPTY_${PN} = "1"

Why?


> +FILES_${PN}-dev += "\
> +    ${datadir}/ti/gdbc6x \
> +"
> +
> +do_configure[noexec] = "1"
> +do_qa_configure[noexec] = "1"

Why?


> -- 
> 1.7.9.5
> 
> -- 
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 3/3] gdbc6x: Add version 1.1.0
  2015-05-18 17:32 ` [PATCH 3/3] gdbc6x: " Jacob Stiffler
@ 2015-05-18 17:54   ` Denys Dmytriyenko
  2015-05-18 18:59     ` Jacob Stiffler
  0 siblings, 1 reply; 14+ messages in thread
From: Denys Dmytriyenko @ 2015-05-18 17:54 UTC (permalink / raw)
  To: Jacob Stiffler; +Cc: meta-ti

On Mon, May 18, 2015 at 01:32:56PM -0400, Jacob Stiffler wrote:
> * The gdbc6x utility provides remote gdb capabilities for C6x DSP
>   cores.
> 
> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> ---
>  recipes-devtools/gdbc6x/gdbc6x/init   |   48 +++++++++++++++++++++++++++++++++
>  recipes-devtools/gdbc6x/gdbc6x_git.bb |   48 +++++++++++++++++++++++++++++++++
>  2 files changed, 96 insertions(+)
>  create mode 100644 recipes-devtools/gdbc6x/gdbc6x/init
>  create mode 100644 recipes-devtools/gdbc6x/gdbc6x_git.bb
> 
> diff --git a/recipes-devtools/gdbc6x/gdbc6x/init b/recipes-devtools/gdbc6x/gdbc6x/init
> new file mode 100644
> index 0000000..90210b8
> --- /dev/null
> +++ b/recipes-devtools/gdbc6x/gdbc6x/init
> @@ -0,0 +1,48 @@
> +#!/bin/bash
> +
> +GDB_DEVICE_NAME='/dev/gdbtty$i'
> +DSP_FIRMWARE_NAME='/lib/firmware/dra7-dsp$[$i+1]-fw.xe66'
> +
> +DSP_L2_GLOBAL_OFFSET="40000000"
> +DSP_L2_GLOBAL_SHIFT="00800000"
> +
> +case "$1" in
> +  start)
> +
> +    dsp_core_l2_offset=$DSP_L2_GLOBAL_OFFSET
> +
> +    i=0
> +    while [ -c "`eval echo $GDB_DEVICE_NAME`" ]
> +    do
> +      dsp_firmware=`eval echo $DSP_FIRMWARE_NAME`
> +
> +      if [ -f "`eval echo $DSP_FIRMWARE_NAME`" ]
> +      then 
> +        eval echo "Initializing $GDB_DEVICE_NAME based on $DSP_FIRMWARE_NAME ..."
> +
> +        gdb_data_local=`eval readelf -s "$DSP_FIRMWARE_NAME" | \
> +                        grep 'gdb_globalData' | \
> +                        awk '{print $2}'`
> +
> +        echo "gdb_globalData (local)  = $gdb_data_local"
> +        printf "gdb_globalData (global) = %X\n" \
> +          $[0x$gdb_data_local + 0x$dsp_core_l2_offset]
> +
> +
> +        printf "%X\n" $[0x$gdb_data_local + 0x$dsp_core_l2_offset] \
> +          >> `eval echo $GDB_DEVICE_NAME`
> +      fi
> +      i=$[$i + 1]
> +      dsp_core_l2_offset=`printf "%X" $[0x$dsp_core_l2_offset + 0x$DSP_L2_GLOBAL_SHIFT]`
> +    done
> +  ;;
> +  stop)
> +    # Nothing to be done.
> +  ;;
> +  *)
> +    echo "Usage: $0 {start|stop}"
> +    exit 1
> +  ;;
> +esac
> +
> +
> diff --git a/recipes-devtools/gdbc6x/gdbc6x_git.bb b/recipes-devtools/gdbc6x/gdbc6x_git.bb
> new file mode 100644
> index 0000000..9fc95fe
> --- /dev/null
> +++ b/recipes-devtools/gdbc6x/gdbc6x_git.bb
> @@ -0,0 +1,48 @@
> +DESCRIPTION="GNU debugger for TI C6X DSP."
> +
> +LICENSE = "GPLv3+"

Have you run this GPLv3 instance by OSRB yet?


> +SECTION = "devel"
> +DEPENDS = "expat ncurses readline"
> +
> +LIC_FILES_CHKSUM = "file://debian/copyright;md5=bf0fe2872eb3dfeebb2cbe38206fe81f"

A bit of consistency with variable ordering, indentation etc. would be much 
appreciated. I don't require patches to be passed through oe-stylize script 
from meta-oe (yet), but please keep it neat. Thanks.


> +include gdbc6x.inc
> +
> +PR = "${INC_PR}.0"
> +
> +DEPENDS = "ncurses bison texinfo flex gettext"
> +
> +RDEPENDS_${PN}  = "gdbserver-c6x gdbserverproxy-module-drv"
> +
> +S = "${WORKDIR}/git/gdbc6x"
> +
> +SRC_URI_append = " \
> +    file://init \
> +"
> +
> +inherit update-rc.d
> +
> +INITSCRIPT_NAME = "gdbserverproxy"
> +INITSCRIPT_PARAMS = "defaults 95"
> +
> +inherit gettext
> +
> +PARALLEL_MAKE = ""
> +
> +do_configure () {
> +    cd ${S}
> +    ./configure --program-suffix=c6x --target=tic6x-elf-tirtos --host=${HOST_SYS} --prefix=${S}/install_gdb
> +}
> +
> +do_install () {
> +    make install
> +
> +    # Custom install to prevent conflict with standard GDB.
> +    install -d ${D}${bindir}
> +    install -d ${D}${includedir}
> +    install -m 755  ${S}/install_gdb/bin/gdbc6x ${D}${bindir} 
> +    cp -rf ${S}/install_gdb/include/* ${D}${includedir}
> +
> +    install -d ${D}${sysconfdir}/init.d
> +    install -m 0755 ${WORKDIR}/init ${D}${sysconfdir}/init.d/gdbserverproxy
> +}
> -- 
> 1.7.9.5
> 
> -- 
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/3] gdbserverproxy-module-drv: Add version 1.1.0
  2015-05-18 17:40   ` Denys Dmytriyenko
@ 2015-05-18 17:54     ` Jacob Stiffler
  2015-05-18 18:23       ` Denys Dmytriyenko
  0 siblings, 1 reply; 14+ messages in thread
From: Jacob Stiffler @ 2015-05-18 17:54 UTC (permalink / raw)
  To: Denys Dmytriyenko; +Cc: meta-ti



On 5/18/2015 1:40 PM, Denys Dmytriyenko wrote:
> On Mon, May 18, 2015 at 01:32:54PM -0400, Jacob Stiffler wrote:
>> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
>> ---
>>   recipes-devtools/gdbc6x/gdbc6x.inc                 |    9 ++++++
>>   .../gdbc6x/gdbserverproxy-module-drv_git.bb        |   31 ++++++++++++++++++++
>>   2 files changed, 40 insertions(+)
>>   create mode 100644 recipes-devtools/gdbc6x/gdbc6x.inc
>>   create mode 100644 recipes-devtools/gdbc6x/gdbserverproxy-module-drv_git.bb
>>
>> diff --git a/recipes-devtools/gdbc6x/gdbc6x.inc b/recipes-devtools/gdbc6x/gdbc6x.inc
>> new file mode 100644
>> index 0000000..ecdc11e
>> --- /dev/null
>> +++ b/recipes-devtools/gdbc6x/gdbc6x.inc
>> @@ -0,0 +1,9 @@
>> +BRANCH = "master"
>> +
>> +PV = "1.1.0"
>> +INC_PR = "r0"
>> +
>> +# This corresponds to version 1.1.0
>> +SRCREV = "1ab06f8bcfe7540754558bac331a0312c4b59142"
>> +
>> +SRC_URI = "git://git.ti.com/sdo-emu/gdbc6x.git;protocol=git;branch=${BRANCH}"
>> diff --git a/recipes-devtools/gdbc6x/gdbserverproxy-module-drv_git.bb b/recipes-devtools/gdbc6x/gdbserverproxy-module-drv_git.bb
>> new file mode 100644
>> index 0000000..2b02236
>> --- /dev/null
>> +++ b/recipes-devtools/gdbc6x/gdbserverproxy-module-drv_git.bb
>> @@ -0,0 +1,31 @@
>> +DESCRIPTION="Interface for GDB to commincate witha TI C66X DSP"
>> +
>> +LICENSE = "GPLv2"
>> +LIC_FILES_CHKSUM = "file://COPYING.txt;md5=75859989545e37968a99b631ef42722e"
>> +
>> +include gdbc6x.inc
>> +
>> +COMPATIBLE_MACHINE = "dra7xx"
>> +PACKAGE_ARCH = "${MACHINE_ARCH}"
>> +
>> +# This package builds a kernel module, use kernel PR as base and append a local
>> +MACHINE_KERNEL_PR_append = "a+gitr${SRCPV}"
>> +PR = "${MACHINE_KERNEL_PR}"
>> +
>> +PLATFORM = ""
>> +PLATFORM_dra7xx = "DRA7xx_PLATFORM"
>> +
>> +EXTRA_OEMAKE = "PLATFORM=${PLATFORM}"
>> +
>> +S = "${WORKDIR}/git/kernel_module/gdbproxy-mod"
>> +
>> +module_autoload_gdbserverproxy = "gdbserverproxy"
>> +
>> +inherit module
>> +
>> +do_configure[noexec] = "1"
>> +do_qa_configure[noexec] = "1"
> What's the need for the above?
>

All three of these gdbc6x recipes use the same source. Without this 
there is a QA error due to the file "git/gdbc6x/intl/configure.ac", 
which is outside of the ${S} directory. The error is the following:

ERROR: virtual/gettext required but not in DEPENDS for file 
/media/hdd-1TB/jake/git/oe-layersetup_patches/build/arago-tmp-external-linaro-toolchain/work/am57xx_evm-linux-gnueabi/gdbserver-c6x/1.1.0-r0.1/git/gdbc6x/intl/configure.ac.
Missing inherit gettext?
ERROR: Function failed: do_qa_configure


Is there a better way to handle this?

>> +CREATE_SRCIPK = "1"
>> +SRCIPK_INSTALL_DIR = "board-support/extra-drivers/${PN}-${PV}"
>> +SRCIPK_PACKAGE_ARCH = "${MACHINE_ARCH}"
> This is not the right place for these^^ either.
>

This was an oversight, and I'll remove in the next version.



^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/3] gdbserverproxy-module-drv: Add version 1.1.0
  2015-05-18 17:54     ` Jacob Stiffler
@ 2015-05-18 18:23       ` Denys Dmytriyenko
  0 siblings, 0 replies; 14+ messages in thread
From: Denys Dmytriyenko @ 2015-05-18 18:23 UTC (permalink / raw)
  To: Jacob Stiffler; +Cc: meta-ti

On Mon, May 18, 2015 at 01:54:51PM -0400, Jacob Stiffler wrote:
> 
> 
> On 5/18/2015 1:40 PM, Denys Dmytriyenko wrote:
> >On Mon, May 18, 2015 at 01:32:54PM -0400, Jacob Stiffler wrote:
> >>Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> >>---
> >>  recipes-devtools/gdbc6x/gdbc6x.inc                 |    9 ++++++
> >>  .../gdbc6x/gdbserverproxy-module-drv_git.bb        |   31 ++++++++++++++++++++
> >>  2 files changed, 40 insertions(+)
> >>  create mode 100644 recipes-devtools/gdbc6x/gdbc6x.inc
> >>  create mode 100644 recipes-devtools/gdbc6x/gdbserverproxy-module-drv_git.bb
> >>
> >>diff --git a/recipes-devtools/gdbc6x/gdbc6x.inc b/recipes-devtools/gdbc6x/gdbc6x.inc
> >>new file mode 100644
> >>index 0000000..ecdc11e
> >>--- /dev/null
> >>+++ b/recipes-devtools/gdbc6x/gdbc6x.inc
> >>@@ -0,0 +1,9 @@
> >>+BRANCH = "master"
> >>+
> >>+PV = "1.1.0"
> >>+INC_PR = "r0"
> >>+
> >>+# This corresponds to version 1.1.0
> >>+SRCREV = "1ab06f8bcfe7540754558bac331a0312c4b59142"
> >>+
> >>+SRC_URI = "git://git.ti.com/sdo-emu/gdbc6x.git;protocol=git;branch=${BRANCH}"
> >>diff --git a/recipes-devtools/gdbc6x/gdbserverproxy-module-drv_git.bb b/recipes-devtools/gdbc6x/gdbserverproxy-module-drv_git.bb
> >>new file mode 100644
> >>index 0000000..2b02236
> >>--- /dev/null
> >>+++ b/recipes-devtools/gdbc6x/gdbserverproxy-module-drv_git.bb
> >>@@ -0,0 +1,31 @@
> >>+DESCRIPTION="Interface for GDB to commincate witha TI C66X DSP"
> >>+
> >>+LICENSE = "GPLv2"
> >>+LIC_FILES_CHKSUM = "file://COPYING.txt;md5=75859989545e37968a99b631ef42722e"
> >>+
> >>+include gdbc6x.inc
> >>+
> >>+COMPATIBLE_MACHINE = "dra7xx"
> >>+PACKAGE_ARCH = "${MACHINE_ARCH}"
> >>+
> >>+# This package builds a kernel module, use kernel PR as base and append a local
> >>+MACHINE_KERNEL_PR_append = "a+gitr${SRCPV}"
> >>+PR = "${MACHINE_KERNEL_PR}"
> >>+
> >>+PLATFORM = ""
> >>+PLATFORM_dra7xx = "DRA7xx_PLATFORM"
> >>+
> >>+EXTRA_OEMAKE = "PLATFORM=${PLATFORM}"
> >>+
> >>+S = "${WORKDIR}/git/kernel_module/gdbproxy-mod"
> >>+
> >>+module_autoload_gdbserverproxy = "gdbserverproxy"
> >>+
> >>+inherit module
> >>+
> >>+do_configure[noexec] = "1"
> >>+do_qa_configure[noexec] = "1"
> >What's the need for the above?
> >
> 
> All three of these gdbc6x recipes use the same source. Without this
> there is a QA error due to the file "git/gdbc6x/intl/configure.ac",
> which is outside of the ${S} directory. The error is the following:
> 
> ERROR: virtual/gettext required but not in DEPENDS for file /media/hdd-1TB/jake/git/oe-layersetup_patches/build/arago-tmp-external-linaro-toolchain/work/am57xx_evm-linux-gnueabi/gdbserver-c6x/1.1.0-r0.1/git/gdbc6x/intl/configure.ac.
> Missing inherit gettext?
> ERROR: Function failed: do_qa_configure
> 
> 
> Is there a better way to handle this?

Looks like there's no INSANE_SKIP for this check, but there's a way to bypass 
it w/o disabling all other QA checks:

EXTRA_OECONF = "--disable-nls"
do_configure() {
	:
}

Just add some comment above that block explaining why it's needed, something 
about erroneously tripping QA check for gettext due to unused configure.ac 
file...


> >>+CREATE_SRCIPK = "1"
> >>+SRCIPK_INSTALL_DIR = "board-support/extra-drivers/${PN}-${PV}"
> >>+SRCIPK_PACKAGE_ARCH = "${MACHINE_ARCH}"
> >This is not the right place for these^^ either.
> >
> 
> This was an oversight, and I'll remove in the next version.
> 


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 3/3] gdbc6x: Add version 1.1.0
  2015-05-18 17:54   ` Denys Dmytriyenko
@ 2015-05-18 18:59     ` Jacob Stiffler
  0 siblings, 0 replies; 14+ messages in thread
From: Jacob Stiffler @ 2015-05-18 18:59 UTC (permalink / raw)
  To: Denys Dmytriyenko; +Cc: meta-ti



On 5/18/2015 1:54 PM, Denys Dmytriyenko wrote:
> On Mon, May 18, 2015 at 01:32:56PM -0400, Jacob Stiffler wrote:
>> * The gdbc6x utility provides remote gdb capabilities for C6x DSP
>>    cores.
>>
>> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
>> ---
>>   recipes-devtools/gdbc6x/gdbc6x/init   |   48 +++++++++++++++++++++++++++++++++
>>   recipes-devtools/gdbc6x/gdbc6x_git.bb |   48 +++++++++++++++++++++++++++++++++
>>   2 files changed, 96 insertions(+)
>>   create mode 100644 recipes-devtools/gdbc6x/gdbc6x/init
>>   create mode 100644 recipes-devtools/gdbc6x/gdbc6x_git.bb
>>
>> diff --git a/recipes-devtools/gdbc6x/gdbc6x/init b/recipes-devtools/gdbc6x/gdbc6x/init
>> new file mode 100644
>> index 0000000..90210b8
>> --- /dev/null
>> +++ b/recipes-devtools/gdbc6x/gdbc6x/init
>> @@ -0,0 +1,48 @@
>> +#!/bin/bash
>> +
>> +GDB_DEVICE_NAME='/dev/gdbtty$i'
>> +DSP_FIRMWARE_NAME='/lib/firmware/dra7-dsp$[$i+1]-fw.xe66'
>> +
>> +DSP_L2_GLOBAL_OFFSET="40000000"
>> +DSP_L2_GLOBAL_SHIFT="00800000"
>> +
>> +case "$1" in
>> +  start)
>> +
>> +    dsp_core_l2_offset=$DSP_L2_GLOBAL_OFFSET
>> +
>> +    i=0
>> +    while [ -c "`eval echo $GDB_DEVICE_NAME`" ]
>> +    do
>> +      dsp_firmware=`eval echo $DSP_FIRMWARE_NAME`
>> +
>> +      if [ -f "`eval echo $DSP_FIRMWARE_NAME`" ]
>> +      then
>> +        eval echo "Initializing $GDB_DEVICE_NAME based on $DSP_FIRMWARE_NAME ..."
>> +
>> +        gdb_data_local=`eval readelf -s "$DSP_FIRMWARE_NAME" | \
>> +                        grep 'gdb_globalData' | \
>> +                        awk '{print $2}'`
>> +
>> +        echo "gdb_globalData (local)  = $gdb_data_local"
>> +        printf "gdb_globalData (global) = %X\n" \
>> +          $[0x$gdb_data_local + 0x$dsp_core_l2_offset]
>> +
>> +
>> +        printf "%X\n" $[0x$gdb_data_local + 0x$dsp_core_l2_offset] \
>> +          >> `eval echo $GDB_DEVICE_NAME`
>> +      fi
>> +      i=$[$i + 1]
>> +      dsp_core_l2_offset=`printf "%X" $[0x$dsp_core_l2_offset + 0x$DSP_L2_GLOBAL_SHIFT]`
>> +    done
>> +  ;;
>> +  stop)
>> +    # Nothing to be done.
>> +  ;;
>> +  *)
>> +    echo "Usage: $0 {start|stop}"
>> +    exit 1
>> +  ;;
>> +esac
>> +
>> +
>> diff --git a/recipes-devtools/gdbc6x/gdbc6x_git.bb b/recipes-devtools/gdbc6x/gdbc6x_git.bb
>> new file mode 100644
>> index 0000000..9fc95fe
>> --- /dev/null
>> +++ b/recipes-devtools/gdbc6x/gdbc6x_git.bb
>> @@ -0,0 +1,48 @@
>> +DESCRIPTION="GNU debugger for TI C6X DSP."
>> +
>> +LICENSE = "GPLv3+"
> Have you run this GPLv3 instance by OSRB yet?
>

This has already gone through the OSRB when this was released for keystone.

>> +SECTION = "devel"
>> +DEPENDS = "expat ncurses readline"
>> +
>> +LIC_FILES_CHKSUM = "file://debian/copyright;md5=bf0fe2872eb3dfeebb2cbe38206fe81f"
> A bit of consistency with variable ordering, indentation etc. would be much
> appreciated. I don't require patches to be passed through oe-stylize script
> from meta-oe (yet), but please keep it neat. Thanks.
>

I'll take a look at the script and make the updates.

>> +include gdbc6x.inc
>> +
>> +PR = "${INC_PR}.0"
>> +
>> +DEPENDS = "ncurses bison texinfo flex gettext"
>> +
>> +RDEPENDS_${PN}  = "gdbserver-c6x gdbserverproxy-module-drv"
>> +
>> +S = "${WORKDIR}/git/gdbc6x"
>> +
>> +SRC_URI_append = " \
>> +    file://init \
>> +"
>> +
>> +inherit update-rc.d
>> +
>> +INITSCRIPT_NAME = "gdbserverproxy"
>> +INITSCRIPT_PARAMS = "defaults 95"
>> +
>> +inherit gettext
>> +
>> +PARALLEL_MAKE = ""
>> +
>> +do_configure () {
>> +    cd ${S}
>> +    ./configure --program-suffix=c6x --target=tic6x-elf-tirtos --host=${HOST_SYS} --prefix=${S}/install_gdb
>> +}
>> +
>> +do_install () {
>> +    make install
>> +
>> +    # Custom install to prevent conflict with standard GDB.
>> +    install -d ${D}${bindir}
>> +    install -d ${D}${includedir}
>> +    install -m 755  ${S}/install_gdb/bin/gdbc6x ${D}${bindir}
>> +    cp -rf ${S}/install_gdb/include/* ${D}${includedir}
>> +
>> +    install -d ${D}${sysconfdir}/init.d
>> +    install -m 0755 ${WORKDIR}/init ${D}${sysconfdir}/init.d/gdbserverproxy
>> +}
>> -- 
>> 1.7.9.5
>>
>> -- 
>> _______________________________________________
>> meta-ti mailing list
>> meta-ti@yoctoproject.org
>> https://lists.yoctoproject.org/listinfo/meta-ti



^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 2/3] gdbserver-c6x: Add version 1.1.0
  2015-05-18 17:41   ` Denys Dmytriyenko
@ 2015-05-18 19:00     ` Jacob Stiffler
  2015-05-18 19:07       ` Denys Dmytriyenko
  0 siblings, 1 reply; 14+ messages in thread
From: Jacob Stiffler @ 2015-05-18 19:00 UTC (permalink / raw)
  To: Denys Dmytriyenko; +Cc: meta-ti



On 5/18/2015 1:41 PM, Denys Dmytriyenko wrote:
> On Mon, May 18, 2015 at 01:32:55PM -0400, Jacob Stiffler wrote:
>> * c6x DSP development files required to support remote gdb debugging.
>>
>> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
>> ---
>>   recipes-devtools/gdbc6x/gdbserver-c6x_git.bb |   46 ++++++++++++++++++++++++++
>>   1 file changed, 46 insertions(+)
>>   create mode 100644 recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
>>
>> diff --git a/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb b/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
>> new file mode 100644
>> index 0000000..d21f03f
>> --- /dev/null
>> +++ b/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
>> @@ -0,0 +1,46 @@
>> +DESCRIPTION="TI gdb-server to be used with c6xgdb."
>> +
>> +LICENSE = "BSD"
>> +LIC_FILES_CHKSUM = "file://../debian/copyright;md5=82c616b6c8f9c11c46feaaf8f9a11495"
>> +
>> +include gdbc6x.inc
>> +
>> +COMPATIBLE_MACHINE = "dra7xx"
>> +PACKAGE_ARCH = "${MACHINE_ARCH}"
>> +
>> +PR = "${INC_PR}.0"
>> +
>> +require recipes-ti/includes/ti-paths.inc
>> +
>> +DEPENDS = "ti-xdais ti-framework-components ti-cgt6x-native ti-sysbios ti-xdctools"
>> +
>> +export XDAIS_DIR = "${XDAIS_INSTALL_DIR}"
>> +export FC_DIR = "${FC_INSTALL_DIR}"
>> +export TI_CGT_INSTALL = "${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
>> +export BIOS_DIR = "${SYSBIOS_INSTALL_DIR}"
>> +export XDC_DIR = "${XDC_INSTALL_DIR}"
>> +
>> +PLATFORM = ""
>> +PLATFORM_dra7xx = "DRA7xx_PLATFORM"
>> +
>> +EXTRA_OEMAKE = "PLATFORM=${PLATFORM}"
>> +
>> +PARALLEL_MAKE = ""
>> +
>> +S = "${WORKDIR}/git/gdbserver-c6x/src"
>> +
>> +do_install() {
>> +    install -d ${D}${datadir}/ti/gdbc6x/include
>> +    install -d ${D}${datadir}/ti/gdbc6x/lib
>> +    cp -f ../include/* ${D}${datadir}/ti/gdbc6x/include
>> +    cp -f ../lib/* ${D}${datadir}/ti/gdbc6x/lib
>> +}
>> +
>> +ALLOW_EMPTY_${PN} = "1"
> Why?
>

This recipes only popultaes the -dev and -staticdev packages.

>> +FILES_${PN}-dev += "\
>> +    ${datadir}/ti/gdbc6x \
>> +"
>> +
>> +do_configure[noexec] = "1"
>> +do_qa_configure[noexec] = "1"
> Why?
>
>
>> -- 
>> 1.7.9.5
>>
>> -- 
>> _______________________________________________
>> meta-ti mailing list
>> meta-ti@yoctoproject.org
>> https://lists.yoctoproject.org/listinfo/meta-ti



^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 2/3] gdbserver-c6x: Add version 1.1.0
  2015-05-18 19:00     ` Jacob Stiffler
@ 2015-05-18 19:07       ` Denys Dmytriyenko
  2015-05-19 12:12         ` Jacob Stiffler
  0 siblings, 1 reply; 14+ messages in thread
From: Denys Dmytriyenko @ 2015-05-18 19:07 UTC (permalink / raw)
  To: Jacob Stiffler; +Cc: meta-ti

On Mon, May 18, 2015 at 03:00:37PM -0400, Jacob Stiffler wrote:
> 
> 
> On 5/18/2015 1:41 PM, Denys Dmytriyenko wrote:
> >On Mon, May 18, 2015 at 01:32:55PM -0400, Jacob Stiffler wrote:
> >>* c6x DSP development files required to support remote gdb debugging.
> >>
> >>Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> >>---
> >>  recipes-devtools/gdbc6x/gdbserver-c6x_git.bb |   46 ++++++++++++++++++++++++++
> >>  1 file changed, 46 insertions(+)
> >>  create mode 100644 recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
> >>
> >>diff --git a/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb b/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
> >>new file mode 100644
> >>index 0000000..d21f03f
> >>--- /dev/null
> >>+++ b/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
> >>@@ -0,0 +1,46 @@
> >>+DESCRIPTION="TI gdb-server to be used with c6xgdb."
> >>+
> >>+LICENSE = "BSD"
> >>+LIC_FILES_CHKSUM = "file://../debian/copyright;md5=82c616b6c8f9c11c46feaaf8f9a11495"
> >>+
> >>+include gdbc6x.inc
> >>+
> >>+COMPATIBLE_MACHINE = "dra7xx"
> >>+PACKAGE_ARCH = "${MACHINE_ARCH}"
> >>+
> >>+PR = "${INC_PR}.0"
> >>+
> >>+require recipes-ti/includes/ti-paths.inc
> >>+
> >>+DEPENDS = "ti-xdais ti-framework-components ti-cgt6x-native ti-sysbios ti-xdctools"
> >>+
> >>+export XDAIS_DIR = "${XDAIS_INSTALL_DIR}"
> >>+export FC_DIR = "${FC_INSTALL_DIR}"
> >>+export TI_CGT_INSTALL = "${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> >>+export BIOS_DIR = "${SYSBIOS_INSTALL_DIR}"
> >>+export XDC_DIR = "${XDC_INSTALL_DIR}"
> >>+
> >>+PLATFORM = ""
> >>+PLATFORM_dra7xx = "DRA7xx_PLATFORM"
> >>+
> >>+EXTRA_OEMAKE = "PLATFORM=${PLATFORM}"
> >>+
> >>+PARALLEL_MAKE = ""
> >>+
> >>+S = "${WORKDIR}/git/gdbserver-c6x/src"
> >>+
> >>+do_install() {
> >>+    install -d ${D}${datadir}/ti/gdbc6x/include
> >>+    install -d ${D}${datadir}/ti/gdbc6x/lib
> >>+    cp -f ../include/* ${D}${datadir}/ti/gdbc6x/include
> >>+    cp -f ../lib/* ${D}${datadir}/ti/gdbc6x/lib
> >>+}
> >>+
> >>+ALLOW_EMPTY_${PN} = "1"
> >Why?
> >
> 
> This recipes only popultaes the -dev and -staticdev packages.

So, why do you need an empty package then? If you only need -dev and 
-staticdev added as a dependency for the devkit, this shouldn't be required. 
Unless there's a need to depend on the main package someplace, in which case 
you would create an empty one...


> >>+FILES_${PN}-dev += "\
> >>+    ${datadir}/ti/gdbc6x \
> >>+"
> >>+
> >>+do_configure[noexec] = "1"
> >>+do_qa_configure[noexec] = "1"
> >Why?
> >
> >
> >>-- 
> >>1.7.9.5
> >>
> >>-- 
> >>_______________________________________________
> >>meta-ti mailing list
> >>meta-ti@yoctoproject.org
> >>https://lists.yoctoproject.org/listinfo/meta-ti
> 


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 2/3] gdbserver-c6x: Add version 1.1.0
  2015-05-18 19:07       ` Denys Dmytriyenko
@ 2015-05-19 12:12         ` Jacob Stiffler
  2015-05-19 13:30           ` Denys Dmytriyenko
  0 siblings, 1 reply; 14+ messages in thread
From: Jacob Stiffler @ 2015-05-19 12:12 UTC (permalink / raw)
  To: Denys Dmytriyenko; +Cc: meta-ti



On 5/18/2015 3:07 PM, Denys Dmytriyenko wrote:
> On Mon, May 18, 2015 at 03:00:37PM -0400, Jacob Stiffler wrote:
>>
>> On 5/18/2015 1:41 PM, Denys Dmytriyenko wrote:
>>> On Mon, May 18, 2015 at 01:32:55PM -0400, Jacob Stiffler wrote:
>>>> * c6x DSP development files required to support remote gdb debugging.
>>>>
>>>> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
>>>> ---
>>>>   recipes-devtools/gdbc6x/gdbserver-c6x_git.bb |   46 ++++++++++++++++++++++++++
>>>>   1 file changed, 46 insertions(+)
>>>>   create mode 100644 recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
>>>>
>>>> diff --git a/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb b/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
>>>> new file mode 100644
>>>> index 0000000..d21f03f
>>>> --- /dev/null
>>>> +++ b/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
>>>> @@ -0,0 +1,46 @@
>>>> +DESCRIPTION="TI gdb-server to be used with c6xgdb."
>>>> +
>>>> +LICENSE = "BSD"
>>>> +LIC_FILES_CHKSUM = "file://../debian/copyright;md5=82c616b6c8f9c11c46feaaf8f9a11495"
>>>> +
>>>> +include gdbc6x.inc
>>>> +
>>>> +COMPATIBLE_MACHINE = "dra7xx"
>>>> +PACKAGE_ARCH = "${MACHINE_ARCH}"
>>>> +
>>>> +PR = "${INC_PR}.0"
>>>> +
>>>> +require recipes-ti/includes/ti-paths.inc
>>>> +
>>>> +DEPENDS = "ti-xdais ti-framework-components ti-cgt6x-native ti-sysbios ti-xdctools"
>>>> +
>>>> +export XDAIS_DIR = "${XDAIS_INSTALL_DIR}"
>>>> +export FC_DIR = "${FC_INSTALL_DIR}"
>>>> +export TI_CGT_INSTALL = "${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
>>>> +export BIOS_DIR = "${SYSBIOS_INSTALL_DIR}"
>>>> +export XDC_DIR = "${XDC_INSTALL_DIR}"
>>>> +
>>>> +PLATFORM = ""
>>>> +PLATFORM_dra7xx = "DRA7xx_PLATFORM"
>>>> +
>>>> +EXTRA_OEMAKE = "PLATFORM=${PLATFORM}"
>>>> +
>>>> +PARALLEL_MAKE = ""
>>>> +
>>>> +S = "${WORKDIR}/git/gdbserver-c6x/src"
>>>> +
>>>> +do_install() {
>>>> +    install -d ${D}${datadir}/ti/gdbc6x/include
>>>> +    install -d ${D}${datadir}/ti/gdbc6x/lib
>>>> +    cp -f ../include/* ${D}${datadir}/ti/gdbc6x/include
>>>> +    cp -f ../lib/* ${D}${datadir}/ti/gdbc6x/lib
>>>> +}
>>>> +
>>>> +ALLOW_EMPTY_${PN} = "1"
>>> Why?
>>>
>> This recipes only popultaes the -dev and -staticdev packages.
> So, why do you need an empty package then? If you only need -dev and
> -staticdev added as a dependency for the devkit, this shouldn't be required.
> Unless there's a need to depend on the main package someplace, in which case
> you would create an empty one...
>

Without this, I am seeing an error when populating the devkit:

Collected errors:
  * satisfy_dependencies_for: Cannot satisfy the following dependencies 
for packagegroup-arago-toolchain-tisdk-target:
  *      libulm (= 1.2.0-r0.1) *         gdbserver-c6x (= 1.1.0-r0.1) *
  * opkg_install_cmd: Cannot install package 
packagegroup-arago-toolchain-tisdk-target.

>>>> +FILES_${PN}-dev += "\
>>>> +    ${datadir}/ti/gdbc6x \
>>>> +"
>>>> +
>>>> +do_configure[noexec] = "1"
>>>> +do_qa_configure[noexec] = "1"
>>> Why?
>>>
>>>
>>>> -- 
>>>> 1.7.9.5
>>>>
>>>> -- 
>>>> _______________________________________________
>>>> meta-ti mailing list
>>>> meta-ti@yoctoproject.org
>>>> https://lists.yoctoproject.org/listinfo/meta-ti



^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 2/3] gdbserver-c6x: Add version 1.1.0
  2015-05-19 12:12         ` Jacob Stiffler
@ 2015-05-19 13:30           ` Denys Dmytriyenko
  0 siblings, 0 replies; 14+ messages in thread
From: Denys Dmytriyenko @ 2015-05-19 13:30 UTC (permalink / raw)
  To: Jacob Stiffler; +Cc: meta-ti

On Tue, May 19, 2015 at 08:12:43AM -0400, Jacob Stiffler wrote:
> 
> 
> On 5/18/2015 3:07 PM, Denys Dmytriyenko wrote:
> >On Mon, May 18, 2015 at 03:00:37PM -0400, Jacob Stiffler wrote:
> >>
> >>On 5/18/2015 1:41 PM, Denys Dmytriyenko wrote:
> >>>On Mon, May 18, 2015 at 01:32:55PM -0400, Jacob Stiffler wrote:
> >>>>* c6x DSP development files required to support remote gdb debugging.
> >>>>
> >>>>Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> >>>>---
> >>>>  recipes-devtools/gdbc6x/gdbserver-c6x_git.bb |   46 ++++++++++++++++++++++++++
> >>>>  1 file changed, 46 insertions(+)
> >>>>  create mode 100644 recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
> >>>>
> >>>>diff --git a/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb b/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
> >>>>new file mode 100644
> >>>>index 0000000..d21f03f
> >>>>--- /dev/null
> >>>>+++ b/recipes-devtools/gdbc6x/gdbserver-c6x_git.bb
> >>>>@@ -0,0 +1,46 @@
> >>>>+DESCRIPTION="TI gdb-server to be used with c6xgdb."
> >>>>+
> >>>>+LICENSE = "BSD"
> >>>>+LIC_FILES_CHKSUM = "file://../debian/copyright;md5=82c616b6c8f9c11c46feaaf8f9a11495"
> >>>>+
> >>>>+include gdbc6x.inc
> >>>>+
> >>>>+COMPATIBLE_MACHINE = "dra7xx"
> >>>>+PACKAGE_ARCH = "${MACHINE_ARCH}"
> >>>>+
> >>>>+PR = "${INC_PR}.0"
> >>>>+
> >>>>+require recipes-ti/includes/ti-paths.inc
> >>>>+
> >>>>+DEPENDS = "ti-xdais ti-framework-components ti-cgt6x-native ti-sysbios ti-xdctools"
> >>>>+
> >>>>+export XDAIS_DIR = "${XDAIS_INSTALL_DIR}"
> >>>>+export FC_DIR = "${FC_INSTALL_DIR}"
> >>>>+export TI_CGT_INSTALL = "${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> >>>>+export BIOS_DIR = "${SYSBIOS_INSTALL_DIR}"
> >>>>+export XDC_DIR = "${XDC_INSTALL_DIR}"
> >>>>+
> >>>>+PLATFORM = ""
> >>>>+PLATFORM_dra7xx = "DRA7xx_PLATFORM"
> >>>>+
> >>>>+EXTRA_OEMAKE = "PLATFORM=${PLATFORM}"
> >>>>+
> >>>>+PARALLEL_MAKE = ""
> >>>>+
> >>>>+S = "${WORKDIR}/git/gdbserver-c6x/src"
> >>>>+
> >>>>+do_install() {
> >>>>+    install -d ${D}${datadir}/ti/gdbc6x/include
> >>>>+    install -d ${D}${datadir}/ti/gdbc6x/lib
> >>>>+    cp -f ../include/* ${D}${datadir}/ti/gdbc6x/include
> >>>>+    cp -f ../lib/* ${D}${datadir}/ti/gdbc6x/lib
> >>>>+}
> >>>>+
> >>>>+ALLOW_EMPTY_${PN} = "1"
> >>>Why?
> >>>
> >>This recipes only popultaes the -dev and -staticdev packages.
> >So, why do you need an empty package then? If you only need -dev and
> >-staticdev added as a dependency for the devkit, this shouldn't be required.
> >Unless there's a need to depend on the main package someplace, in which case
> >you would create an empty one...
> >
> 
> Without this, I am seeing an error when populating the devkit:
> 
> Collected errors:
>  * satisfy_dependencies_for: Cannot satisfy the following
> dependencies for packagegroup-arago-toolchain-tisdk-target:
>  *      libulm (= 1.2.0-r0.1) *         gdbserver-c6x (= 1.1.0-r0.1) *
>  * opkg_install_cmd: Cannot install package
> packagegroup-arago-toolchain-tisdk-target.

I guess we could break the default dependency between -dev and the main 
package, but an empty package would be fine in this case too.


> >>>>+FILES_${PN}-dev += "\
> >>>>+    ${datadir}/ti/gdbc6x \
> >>>>+"
> >>>>+
> >>>>+do_configure[noexec] = "1"
> >>>>+do_qa_configure[noexec] = "1"
> >>>Why?
> >>>
> >>>
> >>>>-- 
> >>>>1.7.9.5
> >>>>
> >>>>-- 
> >>>>_______________________________________________
> >>>>meta-ti mailing list
> >>>>meta-ti@yoctoproject.org
> >>>>https://lists.yoctoproject.org/listinfo/meta-ti
> 


^ permalink raw reply	[flat|nested] 14+ messages in thread

end of thread, other threads:[~2015-05-19 13:30 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-05-18 17:32 [PATCH 0/3] gdbc6x 1.1.0 Jacob Stiffler
2015-05-18 17:32 ` [PATCH 1/3] gdbserverproxy-module-drv: Add version 1.1.0 Jacob Stiffler
2015-05-18 17:40   ` Denys Dmytriyenko
2015-05-18 17:54     ` Jacob Stiffler
2015-05-18 18:23       ` Denys Dmytriyenko
2015-05-18 17:32 ` [PATCH 2/3] gdbserver-c6x: " Jacob Stiffler
2015-05-18 17:41   ` Denys Dmytriyenko
2015-05-18 19:00     ` Jacob Stiffler
2015-05-18 19:07       ` Denys Dmytriyenko
2015-05-19 12:12         ` Jacob Stiffler
2015-05-19 13:30           ` Denys Dmytriyenko
2015-05-18 17:32 ` [PATCH 3/3] gdbc6x: " Jacob Stiffler
2015-05-18 17:54   ` Denys Dmytriyenko
2015-05-18 18:59     ` Jacob Stiffler

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.