> -----Original Message-----
> From: Stiffler, Jacob
> Sent: Wednesday, September 20, 2017 11:46 AM
> To: Radhakrishnan, Mahesh; 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 11:42 AM, Radhakrishnan, Mahesh wrote:
> > 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/
> 
> We will still need to bump the version to 1.0.0.8.
> 

I will separate the original patch into two patches, one for RTOS which 
uses http://patchwork.dal.design.ti.com/patch/32325/, and another one 
which includes version bump and the update for linux user space.

> >
> > Regards
> > Mahesh
> >
> > -----Original Message-----
> > From: [email protected] [mailto:meta-arago-
> [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_DESTSUFFI
> X};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

Reply via email to