Jake, For the RTOS recipe, can we apply the below meta-ti patch (under review) which has addressed the review comments ? http://patchwork.dal.design.ti.com/patch/32325/
Regards Mahesh -----Original Message----- From: [email protected] [mailto:[email protected]] On Behalf Of Stiffler, Jacob Sent: Wednesday, September 20, 2017 11:40 AM To: Gou, Hongmei; [email protected] Subject: Re: [meta-arago] [EXTERNAL] [meta-processor-sdk] [PATCH] icss-emac-lld*: update to version 01.00.00.08 On 9/20/2017 10:39 AM, Hongmei Gou wrote: > * Add icss-emac-lld-rtos to build firmware binaries > * Add k2g support for linux user space > * Cleanup to remove TARGET from oe_runmake > > Signed-off-by: Hongmei Gou <[email protected]> > --- > .../icss-emac-lld/icss-emac-lld-01.00.00.inc | 6 ++-- > .../icss-emac-lld/icss-emac-lld-rtos_git.bb | 28 ++++++++++++++++++ > .../icss-emac-lld/icss-emac-lld-test_git.bb | 34 > +++++++++++++--------- > recipes-bsp/icss-emac-lld/icss-emac-lld_git.bb | 5 ++-- > 4 files changed, 54 insertions(+), 19 deletions(-) > create mode 100644 > recipes-bsp/icss-emac-lld/icss-emac-lld-rtos_git.bb > > diff --git a/recipes-bsp/icss-emac-lld/icss-emac-lld-01.00.00.inc > b/recipes-bsp/icss-emac-lld/icss-emac-lld-01.00.00.inc > index 8e2ffd4..621219b 100644 > --- a/recipes-bsp/icss-emac-lld/icss-emac-lld-01.00.00.inc > +++ b/recipes-bsp/icss-emac-lld/icss-emac-lld-01.00.00.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-rtos_git.bb > b/recipes-bsp/icss-emac-lld/icss-emac-lld-rtos_git.bb > new file mode 100644 > index 0000000..06ee279 > --- /dev/null > +++ b/recipes-bsp/icss-emac-lld/icss-emac-lld-rtos_git.bb > @@ -0,0 +1,28 @@ > +inherit ti-pdk > +require icss-emac-lld-01.00.00.inc > + > +PR = "${INC_PR}.0" > + > +DEPENDS_append = " osal-rtos \ > + pruss-lld-rtos \ > +" > +DEPENDS_append_ti33x = " starterware-rtos \ " > +DEPENDS_append_ti43x = " starterware-rtos \ " > + > +# Build with make instead of XDC > +TI_PDK_XDCMAKE = "0" > + > +LIMSOCS_ti33x = "am335x" > +LIMSOCS_ti43x = "am437x" > +LIMSOCS_am57xx-evm = "am571x am572x" > +LIMSOCS_k2g = "k2g" The LIMIT_SOCS are not used by the ti-pdk.bbclass anymore. See http://arago-project.org/git/?p=meta-ti.git;a=commitdiff;h=b00044c43e7785decb1d6ac35349ca4b2711676e The correct variables are set in the machine configuration, so they can be removed from there. > + > +export PDK_ICSS_EMAC_ROOT_PATH ="${WORKDIR}/build" > +export DEST_ROOT="${S}" > + > +# HTML doc link params > +PDK_COMP_LINK_TEXT = "ICSS-EMAC LLD" > + > +INSANE_SKIP_${PN} = "arch" > 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 6beafbf..375bfbc 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,47 +1,55 @@ > 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" > > DEPENDS_append_ti33x = " starterware" > DEPENDS_append_ti43x = " starterware" > > include icss-emac-lld-01.00.00.inc > > -PR = "${INC_PR}.2" > +PR = "${INC_PR}.0" > > -COMPATIBLE_MACHINE = "am57xx-evm|ti33x|ti43x" > - > -EXTRA_OEMAKE += "-f makefile_armv7 PDK_INSTALL_PATH=${STAGING_INCDIR} > ICSS_EMAC_SRC_DIR=${S}" > +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}" > > do_compile_am57xx-evm () { > # Build the tests > - oe_runmake tests DEVICE="am571x" TARGET="SOC_AM571x" > + oe_runmake tests DEVICE="am571x" > oe_runmake clean > - oe_runmake tests DEVICE="am572x" TARGET="SOC_AM572x" > + oe_runmake tests DEVICE="am572x" > } > > do_compile_ti33x () { > # Build the tests > - oe_runmake tests DEVICE="am335x" TARGET="SOC_AM335x" > + oe_runmake tests DEVICE="am335x" > } > > do_compile_ti43x () { > # Build the tests > - oe_runmake tests DEVICE="am437x" TARGET="SOC_AM437x" > + oe_runmake tests DEVICE="am437x" > +} > + > +do_compile_k2g-evm () { > +# Build the tests > + oe_runmake tests DEVICE="k2g" > } > > 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" > + oe_runmake installbin INSTALL_BIN_BASE_DIR=${D}${bindir} DEVICE="am571x" > + oe_runmake installbin INSTALL_BIN_BASE_DIR=${D}${bindir} DEVICE="am572x" > } > > do_install_ti33x () { > # Install the binary > - oe_runmake installbin INSTALL_BIN_BASE_DIR=${D}${bindir} DEVICE="am335x" > TARGET="SOC_AM335x" > + oe_runmake installbin INSTALL_BIN_BASE_DIR=${D}${bindir} DEVICE="am335x" > } > > do_install_ti43x () { > # Install the binary > - oe_runmake installbin INSTALL_BIN_BASE_DIR=${D}${bindir} DEVICE="am437x" > TARGET="SOC_AM437x" > + oe_runmake installbin INSTALL_BIN_BASE_DIR=${D}${bindir} DEVICE="am437x" > +} > + > +do_install_k2g-evm () { > +# Install the binary > + oe_runmake installbin INSTALL_BIN_BASE_DIR=${D}${bindir} DEVICE="k2g" > } It appears that this "DEVICE" is an element out of the TI_PDK_LIMIT_SOCS list. If this is true, then these compile and install tasks can be greatly simplified by looping over the TI_PDK_LIMIT_SOCS list. > 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 b3be89b..bfa655d 100644 > --- a/recipes-bsp/icss-emac-lld/icss-emac-lld_git.bb > +++ b/recipes-bsp/icss-emac-lld/icss-emac-lld_git.bb > @@ -4,13 +4,12 @@ DEPENDS="common-csl-ip pruss-lld osal" > > include icss-emac-lld-01.00.00.inc > > -PR = "${INC_PR}.2" > - > -COMPATIBLE_MACHINE = "am57xx-evm|ti33x|ti43x" > +PR = "${INC_PR}.0" > > 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}" > _______________________________________________ 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
