Re: [rocko][PATCH v2] ti-ipc: Update to new version 3.50

[Denys Dmytriyenko <denys@ti.com>]

On Thu, Aug 16, 2018 at 02:03:08PM -0400, Nelson, Sam wrote:
> 
> 
> > -----Original Message-----
> > From: Anna, Suman
> > Sent: Thursday, August 16, 2018 12:40 PM
> > To: Dmytriyenko, Denys; Nelson, Sam
> > Cc: meta-ti@yoctoproject.org
> > Subject: Re: [meta-ti] [rocko][PATCH v2] ti-ipc: Update to new version 3.50
> > 
> > Hi Sam,
> > 
> > On 08/16/2018 10:47 AM, Denys Dmytriyenko wrote:
> > > +Suman,
> > >
> > > On Tue, Aug 14, 2018 at 02:59:28PM -0400, Sam Nelson wrote:
> > >> Add AM65XX platform support
> > >>
> > >> Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> > >>
> > >> ---
> > >> Change from previous version of patch
> > >> - Update gcc toolchain to gcc-linaro-baremetal-aarch64
> > >> NOTE: This patch depends on gcc-linaro-baremetal-aarch64-native patch to
> > be checked in
> > >> ---
> > >> ---
> > >>  recipes-ti/ipc/ti-ipc-common.inc   |  3 ++-
> > >>  recipes-ti/ipc/ti-ipc-examples.inc |  2 +-
> > >>  recipes-ti/ipc/ti-ipc-rtos.inc     | 16 ++++++++++++----
> > >>  recipes-ti/ipc/ti-ipc-rtos_git.bb  |  8 ++++++++
> > >>  recipes-ti/ipc/ti-ipc.inc          |  6 +++---
> > >>  5 files changed, 26 insertions(+), 9 deletions(-)
> > >>
> > >> diff --git a/recipes-ti/ipc/ti-ipc-common.inc b/recipes-ti/ipc/ti-ipc-
> > common.inc
> > >> index 7f93842..6aea8e6 100644
> > >> --- a/recipes-ti/ipc/ti-ipc-common.inc
> > >> +++ b/recipes-ti/ipc/ti-ipc-common.inc
> > >> @@ -1,4 +1,4 @@
> > >> -PV = "3.47.02.00"
> > >> +PV = "3.50.01.00"
> > >>  INC_PR = "r0"
> > >>
> > >>  PACKAGE_ARCH = "${MACHINE_ARCH}"
> > >> @@ -13,6 +13,7 @@ PLATFORM_k2g = "66AK2G"
> > >>  PLATFORM_omapl138 = "OMAPL138"
> > >>  PLATFORM_c667x-evm = "C6678"
> > >>  PLATFORM_c665x-evm = "C6678"
> > >> +PLATFORM_am65xx-evm = "AM65XX"
> > >>
> > >>  ALT_PLATFORM_k2hk = "TCI6636"
> > >>  ALT_PLATFORM_am57xx-evm = "AM572X AM571X"
> > >> diff --git a/recipes-ti/ipc/ti-ipc-examples.inc b/recipes-ti/ipc/ti-ipc-
> > examples.inc
> > >> index cd2240c..022b13f 100644
> > >> --- a/recipes-ti/ipc/ti-ipc-examples.inc
> > >> +++ b/recipes-ti/ipc/ti-ipc-examples.inc
> > >> @@ -3,7 +3,7 @@ require ti-ipc-common.inc
> > >>  require ti-ipc-rtos.inc
> > >>
> > >>  LICENSE = "BSD"
> > >> -LIC_FILES_CHKSUM = "file://${S_ipc-
> > examples}/src/makefile;beginline=1;endline=30;md5=fb83580b16bce88e8ed
> > 568a6005c8f02"
> > >> +LIC_FILES_CHKSUM = "file://${S_ipc-
> > examples}/src/makefile;beginline=1;endline=30;md5=a52324bd5033bb49ea0
> > 7bade1244ac9a"
> > >>
> > >>  INSANE_SKIP_${PN} += "arch"
> > >>
> > >> diff --git a/recipes-ti/ipc/ti-ipc-rtos.inc b/recipes-ti/ipc/ti-ipc-rtos.inc
> > >> index 868743e..13b1914 100644
> > >> --- a/recipes-ti/ipc/ti-ipc-rtos.inc
> > >> +++ b/recipes-ti/ipc/ti-ipc-rtos.inc
> > >> @@ -24,11 +24,11 @@ protocol=${TI_IPC_METADATA_GIT_PROTOCOL};\
> > >>  branch=${TI_IPC_METADATA_GIT_BRANCH};\
> > >>  name=${TI_IPC_METADATA_NAME}"
> > >>
> > >> -# Corresponds to tag: 3.47.02.00
> > >> -SRCREV_ipc-metadata =
> > "e8aff83111b8a0d64d7a77413b6f66d3106084d8"
> > >> +# Corresponds to tag: 3.50.01.00
> > >> +SRCREV_ipc-metadata =
> > "19a8e70cf6ca4fe0984987e25d937c61a62c7658"
> > >>
> > >> -# Corresponds to tag: 3.47.02.00
> > >> -SRCREV_ipc-examples =
> > "351e9a6d94c68f7ff64b2c002ee6750242ece6d9"
> > >> +# Corresponds to tag: 3.50.01.00
> > >> +SRCREV_ipc-examples =
> > "1975687076426b587be8d2530df56c24425ca53a"
> > >>
> > >>  S_ipc-examples = "${WORKDIR}/git/ipc-examples"
> > >>  S_ipc-metadata = "${WORKDIR}/git/ipc-metadata"
> > >> @@ -50,6 +50,9 @@ DEPENDS_append_omapl138 = " ti-cgt6x-native \
> > >>  DEPENDS_append_c66x = " ti-cgt6x-native \
> > >>  "
> > >>
> > >> +DEPENDS_append_am65xx-evm = " ti-cgt-arm-native \
> > >> +                              gcc-linaro-baremetal-aarch64-native"
> > >> +
> > >>  IPC_TARGETS = ""
> > >>
> > >>  IPC_TARGETS_omap-a15 = "\
> > >> @@ -73,6 +76,11 @@ IPC_TARGETS_c66x = " \
> > >>
> > ti.targets.elf.C66_big_endian="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> > \
> > >>  "
> > >>
> > >> +IPC_TARGETS_am65xx-evm = "\
> > >> +
> > gnu.targets.arm.A53F="${GCC_LINARO_BAREMETAL_AARCH64_TOOLCHAIN}
> > " \
> > >> +    ti.targets.arm.elf.R5F="${M4_TOOLCHAIN_INSTALL_DIR}" \
> > >> +"
> > >> +
> > >>  EXTRA_OEMAKE = "\
> > >>      PLATFORM=${PLATFORM} \
> > >>      XDC_INSTALL_DIR="${XDC_INSTALL_DIR}" \
> > >> diff --git a/recipes-ti/ipc/ti-ipc-rtos_git.bb b/recipes-ti/ipc/ti-ipc-
> > rtos_git.bb
> > >> index b292d67..b70935a 100644
> > >> --- a/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > >> +++ b/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > >> @@ -122,3 +122,11 @@ pkg_postinst_${PN}-fw_omapl138 () {
> > >>  pkg_postrm_${PN}-fw_omapl138 () {
> > >>    update-alternatives --remove rproc-dsp-fw
> > ipc/ti_platforms_evmOMAPL138_DSP/messageq_single.xe674
> > >>  }
> > >> +
> > >> +pkg_postinst_${PN}-fw_am65xx-evm () {
> > >> +  update-alternatives --install /lib/firmware/rproc-r5f-fw rproc-r5f-fw
> > ipc/ti_platforms_cortexR_AM65X/messageq_single.xer5f
> > ${ALTERNATIVE_PRIORITY}
> > 
> > Not sure where you got the rproc-r5f-fw name from, but this is
> > definitely not the name that I am using in the driver. This is what I
> > will be using for the two R5F cores: am65x-mcu-r5f0-fw and
> > am65x-mcu-r5f1-fw. And the Core0 image name is the one we will use in
> > LockStep mode as well.
> 
> I am fine with this. I can update with new patch.

Thanks, Sam. I had to drop all BIOS/XDC/IPC updates from "rocko-next" due to 
IPUMM breakage anyway.

Denys


> > I am open to suggestions if we want to rename it something else, but I
> > have been following the convention that I have been using so far on
> > other SoCs.
> > 
> > regards
> > Suman
> > 
> > >> +}
> > >> +
> > >> +pkg_postrm_${PN}-fw_am65xx-evm () {
> > >> +  update-alternatives --remove rproc-r5f-fw
> > ipc/ti_platforms_cortexR_AM65X/messageq_single.xe674
> > >> +}
> > >> diff --git a/recipes-ti/ipc/ti-ipc.inc b/recipes-ti/ipc/ti-ipc.inc
> > >> index e8038a5..07321cb 100644
> > >> --- a/recipes-ti/ipc/ti-ipc.inc
> > >> +++ b/recipes-ti/ipc/ti-ipc.inc
> > >> @@ -6,10 +6,10 @@ LIC_FILES_CHKSUM = "file://${S}/ipc-
> > linux.mak;beginline=1;endline=30;md5=7b327f9
> > >>
> > >>  TI_IPC_GIT_URI = "git://git.ti.com/ipc/ipcdev.git"
> > >>  TI_IPC_GIT_PROTOCOL = "git"
> > >> -TI_IPC_GIT_BRANCH = "3.47"
> > >> +TI_IPC_GIT_BRANCH = "3.50"
> > >>
> > >> -#Corresponds to 3.47.02.00
> > >> -TI_IPC_SRCREV = "24c36385d096613af123d1e1696fb2fee652ef36"
> > >> +#Corresponds to 3.50.01.00
> > >> +TI_IPC_SRCREV = "f0d0edaf3ce7ed8d81d8969eaaa98cec8b662424"
> > >>
> > >>  BRANCH = "${TI_IPC_GIT_BRANCH}"
> > >>  SRC_URI =
> > "${TI_IPC_GIT_URI};protocol=${TI_IPC_GIT_PROTOCOL};branch=${BRANCH};n
> > ame=ipcdev"
> > >> --
> > >> 1.9.1
> > >>
> > >> --
> > >> _______________________________________________
> > >> meta-ti mailing list
> > >> meta-ti@yoctoproject.org
> > >> https://lists.yoctoproject.org/listinfo/meta-ti
>