Re: [PATCH 2/2] icss-emac-lld: update to version 1.0.0.8

[Denys Dmytriyenko <denys@ti.com>]

On Mon, Nov 06, 2017 at 03:57:19PM -0500, Jacob Stiffler wrote:
> 
> 
> On 11/6/2017 2:42 PM, Denys Dmytriyenko wrote:
> >On Thu, Nov 02, 2017 at 03:19:30PM -0400, Jacob Stiffler wrote:
> >>* Extend compatibility to k2g-evm
> >>* Simplify compile and install tasks as "TARGET" is no longer needed.
> >>
> >>Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> >>---
> >>  .../icss-emac-lld/icss-emac-lld-test_git.bb        | 47 +++++++---------------
> >>  recipes-bsp/icss-emac-lld/icss-emac-lld.inc        |  6 +--
> >>  recipes-bsp/icss-emac-lld/icss-emac-lld_git.bb     |  3 +-
> >>  3 files changed, 19 insertions(+), 37 deletions(-)
> >>
> >>diff --git a/recipes-bsp/icss-emac-lld/icss-emac-lld-test_git.bb b/recipes-bsp/icss-emac-lld/icss-emac-lld-test_git.bb
> >>index f62bf33..5093d7c 100644
> >>--- a/recipes-bsp/icss-emac-lld/icss-emac-lld-test_git.bb
> >>+++ b/recipes-bsp/icss-emac-lld/icss-emac-lld-test_git.bb
> >>@@ -1,6 +1,6 @@
> >>  DESCRIPTION = "TI ICSS-EMAC low level driver tests"
> >>-DEPENDS="common-csl-ip pruss-lld icss-emac-lld osal"
> >>+DEPENDS="common-csl-ip pruss-lld icss-emac-lld osal ti-pdk-build-rtos icss-emac-lld-rtos"
> >Why is rtos specific packages being pulled into linux build?
> 
> This requires the RTOS format for the firmware image, so it is using
> the RTOS package to get that firmware image.

How much of RTOS dependencies are getting into Linux build? I see starterware, 
what else? Is it getting too heavy just for a firmware image?


> >>  DEPENDS_append_ti33x = " starterware"
> >>  DEPENDS_append_ti43x = " starterware"
> >>@@ -9,39 +9,22 @@ include icss-emac-lld.inc
> >>  PR = "${INC_PR}.0"
> >>-COMPATIBLE_MACHINE = "am57xx-evm|ti33x|ti43x"
> >>+EXTRA_OEMAKE += "-f makefile_armv7 PDK_INSTALL_PATH=${STAGING_INCDIR} PDK_PKG_PATH=${STAGING_DATADIR}/ti/ti-pdk-tree/packages ICSS_EMAC_SRC_DIR=${S}"
> >>-EXTRA_OEMAKE += "-f makefile_armv7 PDK_INSTALL_PATH=${STAGING_INCDIR} ICSS_EMAC_SRC_DIR=${S}"
> >>+DEVICE_LIST ?= "${TI_PDK_LIMIT_SOCS}"
> >>-do_compile_am57xx-evm () {
> >>-#   Build the tests
> >>-    oe_runmake tests DEVICE="am571x" TARGET="SOC_AM571x"
> >>-    oe_runmake clean
> >>-    oe_runmake tests DEVICE="am572x" TARGET="SOC_AM572x"
> >>+do_compile () {
> >>+    # Build the tests
> >>+    for device in ${DEVICE_LIST}
> >>+    do
> >>+        oe_runmake tests DEVICE="$device"
> >>+    done
> >>  }
> >>-do_compile_ti33x () {
> >>-#   Build the tests
> >>-    oe_runmake tests DEVICE="am335x" TARGET="SOC_AM335x"
> >>-}
> >>-
> >>-do_compile_ti43x () {
> >>-#   Build the tests
> >>-    oe_runmake tests DEVICE="am437x" TARGET="SOC_AM437x"
> >>-}
> >>-
> >>-do_install_am57xx-evm () {
> >>-#   Install the binary
> >>-    oe_runmake installbin INSTALL_BIN_BASE_DIR=${D}${bindir} DEVICE="am571x" TARGET="SOC_AM571x"
> >>-    oe_runmake installbin INSTALL_BIN_BASE_DIR=${D}${bindir} DEVICE="am572x" TARGET="SOC_AM572x"
> >>-}
> >>-
> >>-do_install_ti33x () {
> >>-#   Install the binary
> >>-    oe_runmake installbin INSTALL_BIN_BASE_DIR=${D}${bindir} DEVICE="am335x" TARGET="SOC_AM335x"
> >>-}
> >>-
> >>-do_install_ti43x () {
> >>-#   Install the binary
> >>-    oe_runmake installbin INSTALL_BIN_BASE_DIR=${D}${bindir} DEVICE="am437x" TARGET="SOC_AM437x"
> >>+do_install() {
> >>+    # Install the binary
> >>+    for device in ${DEVICE_LIST}
> >>+    do
> >>+        oe_runmake installbin INSTALL_BIN_BASE_DIR=${D}${bindir} DEVICE="$device"
> >>+    done
> >>  }
> >>diff --git a/recipes-bsp/icss-emac-lld/icss-emac-lld.inc b/recipes-bsp/icss-emac-lld/icss-emac-lld.inc
> >>index 8e2ffd4..621219b 100644
> >>--- a/recipes-bsp/icss-emac-lld/icss-emac-lld.inc
> >>+++ b/recipes-bsp/icss-emac-lld/icss-emac-lld.inc
> >>@@ -9,14 +9,14 @@ ICSS_EMAC_LLD_GIT_PROTOCOL = "git"
> >>  ICSS_EMAC_LLD_GIT_BRANCH = "master"
> >>  ICSS_EMAC_LLD_GIT_DESTSUFFIX = "git/ti/drv/icss_emac"
> >>-# Below commit ID corresponds to "DEV.ICSS_EMAC_LLD.01.00.00.07C"
> >>-ICSS_EMAC_LLD_SRCREV = "08cb0540eeab7721fe82b603cb6285393f314117"
> >>+# Below commit ID corresponds to "DEV.ICSS_EMAC_LLD.01.00.00.08"
> >>+ICSS_EMAC_LLD_SRCREV = "e689df18417236110590f7b290525a610d63f574"
> >>  BRANCH = "${ICSS_EMAC_LLD_GIT_BRANCH}"
> >>  SRC_URI = "${ICSS_EMAC_LLD_GIT_URI};destsuffix=${ICSS_EMAC_LLD_GIT_DESTSUFFIX};protocol=${ICSS_EMAC_LLD_GIT_PROTOCOL};branch=${BRANCH}"
> >>  SRCREV = "${ICSS_EMAC_LLD_SRCREV}"
> >>-PV = "01.00.00.07C"
> >>+PV = "01.00.00.08"
> >>  INC_PR = "r0"
> >>  S = "${WORKDIR}/${ICSS_EMAC_LLD_GIT_DESTSUFFIX}"
> >>diff --git a/recipes-bsp/icss-emac-lld/icss-emac-lld_git.bb b/recipes-bsp/icss-emac-lld/icss-emac-lld_git.bb
> >>index 272529b..81a6f5e 100644
> >>--- a/recipes-bsp/icss-emac-lld/icss-emac-lld_git.bb
> >>+++ b/recipes-bsp/icss-emac-lld/icss-emac-lld_git.bb
> >>@@ -6,11 +6,10 @@ include icss-emac-lld.inc
> >>  PR = "${INC_PR}.0"
> >>-COMPATIBLE_MACHINE = "am57xx-evm|ti33x|ti43x"
> >>-
> >>  TARGET_NAME_am57xx-evm = "SOC_AM572x"
> >>  TARGET_NAME_ti33x = "SOC_AM335x"
> >>  TARGET_NAME_ti43x = "SOC_AM437x"
> >>+TARGET_NAME_k2g-evm = "SOC_K2G"
> >>  EXTRA_OEMAKE += "-f makefile_armv7 PDK_INSTALL_PATH=${STAGING_INCDIR} TARGET=${TARGET_NAME}"
> >>-- 
> >>2.7.4
> >>
> >>-- 
> >>_______________________________________________
> >>meta-ti mailing list
> >>meta-ti@yoctoproject.org
> >>https://lists.yoctoproject.org/listinfo/meta-ti
>