Hi Denys, The patch for starterware has been submitted. Please review.
Thanks, Hongmei > -----Original Message----- > From: [email protected] [mailto:meta-ti- > [email protected]] On Behalf Of Gou, Hongmei > Sent: Wednesday, August 30, 2017 10:18 AM > To: Dmytriyenko, Denys > Cc: [email protected] > Subject: [EXTERNAL] Re: [meta-ti] [PATCH] icss-emac-lld*: upstream version > 1.0.0.7C with recipes for linux user space > > > -----Original Message----- > > From: Dmytriyenko, Denys > > Sent: Tuesday, August 29, 2017 5:21 PM > > To: Gou, Hongmei > > Cc: [email protected] > > Subject: Re: [meta-ti] [PATCH] icss-emac-lld*: upstream version > > 1.0.0.7C with recipes for linux user space > > > > On Tue, Aug 29, 2017 at 02:43:41PM -0400, Gou, Hongmei wrote: > > > Please see below inline. > > > > Ok, makes sense. > > > > > > > Besides, icss-emac-lld-test has dependencies on starterware for > > > AM3/AM4, I will upstream the recipes for starterware soon. > > > > Why is there a different starterware vs. starterware-rtos? Isn't it > > RTOS- specific anyway and has nothing to do with Linux? > > This is similar to common-csl-ip vs. common-csl-ip-rtos. > starterware and starterware-rtos use the same source git repo. starterware > used for Linux will only install the header files without compiling the > libraries. > > > > > > > -----Original Message----- > > > > From: Dmytriyenko, Denys > > > > Sent: Thursday, August 24, 2017 5:48 PM > > > > To: Gou, Hongmei > > > > Cc: [email protected] > > > > Subject: Re: [meta-ti] [PATCH] icss-emac-lld*: upstream version > > > > 1.0.0.7C > > with > > > > recipes for linux user space > > > > > > > > On Tue, Jul 11, 2017 at 03:09:28PM -0400, Hongmei Gou wrote: > > > > > Signed-off-by: Hongmei Gou <[email protected]> > > > > > --- > > > > > .../icss-emac-lld/icss-emac-lld-test_git.bb | 47 > > > > ++++++++++++++++++++++ > > > > > recipes-bsp/icss-emac-lld/icss-emac-lld.inc | 22 ++++++++++ > > > > > recipes-bsp/icss-emac-lld/icss-emac-lld_git.bb | 26 ++++++++++++ > > > > > 3 files changed, 95 insertions(+) create mode 100644 > > > > > recipes-bsp/icss-emac-lld/icss-emac-lld- > > test_git.bb > > > > > create mode 100644 recipes-bsp/icss-emac-lld/icss-emac-lld.inc > > > > > create mode 100644 > > > > > recipes-bsp/icss-emac-lld/icss-emac-lld_git.bb > > > > > > > > > > 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 > > > > > new file mode 100644 > > > > > index 0000000..f62bf33 > > > > > --- /dev/null > > > > > +++ b/recipes-bsp/icss-emac-lld/icss-emac-lld-test_git.bb > > > > > @@ -0,0 +1,47 @@ > > > > > +DESCRIPTION = "TI ICSS-EMAC low level driver tests" > > > > > + > > > > > +DEPENDS="common-csl-ip pruss-lld icss-emac-lld osal" > > > > > + > > > > > +DEPENDS_append_ti33x = " starterware" > > > > > +DEPENDS_append_ti43x = " starterware" > > > > > + > > > > > +include icss-emac-lld.inc > > > > > + > > > > > +PR = "${INC_PR}.0" > > > > > + > > > > > +COMPATIBLE_MACHINE = "am57xx-evm|ti33x|ti43x" > > > > > > > > What about am57xx-hs-evm? Or dra7xx-evm? > > > > > > ICSS-EMAC LLD will not be supported for am57xx-hs-evm and dra7xx- > evm. > > > > > > > > > > > > + > > > > > +EXTRA_OEMAKE += "-f makefile_armv7 > > > > PDK_INSTALL_PATH=${STAGING_INCDIR} ICSS_EMAC_SRC_DIR=${S}" > > > > > + > > > > > +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" > > > > > > > > Wouldn't this remove the am571x binaries before building am572x > ones? > > > > > > The make clean operation in the current makefile is not removing the > > binaries, but only the library files. > > > > > > > > > > > > +} > > > > > + > > > > > +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" > > > > > +} > > > > > diff --git a/recipes-bsp/icss-emac-lld/icss-emac-lld.inc > > > > > b/recipes- > > bsp/icss- > > > > emac-lld/icss-emac-lld.inc > > > > > new file mode 100644 > > > > > index 0000000..8e2ffd4 > > > > > --- /dev/null > > > > > +++ b/recipes-bsp/icss-emac-lld/icss-emac-lld.inc > > > > > @@ -0,0 +1,22 @@ > > > > > +LICENSE = "BSD-3-Clause" > > > > > +LIC_FILES_CHKSUM = > > > > > > > "file://icss_emacDrv.h;beginline=1;endline=40;md5=88d270788c1d85165113 > > 0 > > > > 4f11a28586d" > > > > > + > > > > > +COMPATIBLE_MACHINE = "ti33x|ti43x|am57xx-evm|k2g-evm" > > > > > > > > AM5 HS and K2G HS platforms will be missed. > > > > > > Yes, this is intended as ICSS-EMAC LLD will not be supported for > > > AM5HS > > and K2G HS. > > > > > > > > > > > > +PACKAGE_ARCH = "${MACHINE_ARCH}" > > > > > + > > > > > +ICSS_EMAC_LLD_GIT_URI = "git://git.ti.com/keystone-rtos/icss- > > emac.git" > > > > > +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" > > > > > + > > > > > +BRANCH = "${ICSS_EMAC_LLD_GIT_BRANCH}" > > > > > +SRC_URI = > > > > > > > "${ICSS_EMAC_LLD_GIT_URI};destsuffix=${ICSS_EMAC_LLD_GIT_DESTSUFFI > > > > X};protocol=${ICSS_EMAC_LLD_GIT_PROTOCOL};branch=${BRANCH}" > > > > > + > > > > > +SRCREV = "${ICSS_EMAC_LLD_SRCREV}" > > > > > +PV = "01.00.00.07C" > > > > > +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 > > > > > new file mode 100644 > > > > > index 0000000..272529b > > > > > --- /dev/null > > > > > +++ b/recipes-bsp/icss-emac-lld/icss-emac-lld_git.bb > > > > > @@ -0,0 +1,26 @@ > > > > > +DESCRIPTION = "TI ICSS-EMAC low level driver library" > > > > > + > > > > > +DEPENDS="common-csl-ip pruss-lld osal" > > > > > + > > > > > +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" > > > > > + > > > > > +EXTRA_OEMAKE += "-f makefile_armv7 > > > > PDK_INSTALL_PATH=${STAGING_INCDIR} TARGET=${TARGET_NAME}" > > > > > + > > > > > +do_compile () { > > > > > +# Build the lib > > > > > + oe_runmake clean > > > > > + oe_runmake lib > > > > > +} > > > > > + > > > > > +do_install () { > > > > > +# Install the lib > > > > > + oe_runmake install INSTALL_INC_BASE_DIR=${D}${includedir} > > > > INSTALL_LIB_BASE_DIR=${D}${libdir} > > > > > +} > > > > > -- > > > > > 1.9.1 > > > > > > > > > > -- > > > > > _______________________________________________ > > > > > meta-ti mailing list > > > > > [email protected] > > > > > https://lists.yoctoproject.org/listinfo/meta-ti > -- > _______________________________________________ > meta-ti mailing list > [email protected] > https://lists.yoctoproject.org/listinfo/meta-ti -- _______________________________________________ meta-ti mailing list [email protected] https://lists.yoctoproject.org/listinfo/meta-ti
