All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jacob Stiffler <j-stiffler@ti.com>
To: "Radhakrishnan, Mahesh" <m-radhakrishnan2@ti.com>,
	"Gou, Hongmei" <h-gou@ti.com>,
	"meta-arago@arago-project.org" <meta-arago@arago-project.org>
Subject: Re: [EXTERNAL] [meta-processor-sdk] [PATCH] icss-emac-lld*: update to version 01.00.00.08
Date: Wed, 20 Sep 2017 11:46:10 -0400	[thread overview]
Message-ID: <3e841243-7748-473a-a520-d4732c77944d@ti.com> (raw)
In-Reply-To: <5130E41A1BA9D145BA0F3B18D5CFB0A7437F67E5@DFLE12.ent.ti.com>



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.

>
> Regards
> Mahesh
>
> -----Original Message-----
> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-bounces@arago-project.org] On Behalf Of Stiffler, Jacob
> Sent: Wednesday, September 20, 2017 11:40 AM
> To: Gou, Hongmei; meta-arago@arago-project.org
> 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 <a0271529@ti.com>
>> ---
>>    .../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
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago



  reply	other threads:[~2017-09-20 15:46 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-09-20 14:39 [meta-processor-sdk] [PATCH] icss-emac-lld*: update to version 01.00.00.08 Hongmei Gou
2017-09-20 15:39 ` [EXTERNAL] " Jacob Stiffler
2017-09-20 15:42   ` Radhakrishnan, Mahesh
2017-09-20 15:46     ` Jacob Stiffler [this message]
2017-09-20 15:55       ` Gou, Hongmei

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3e841243-7748-473a-a520-d4732c77944d@ti.com \
    --to=j-stiffler@ti.com \
    --cc=h-gou@ti.com \
    --cc=m-radhakrishnan2@ti.com \
    --cc=meta-arago@arago-project.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
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.