Re: [rocko][PATCH] ti-pdk: Add tool chain dependencies for am65xx platform

[Denys Dmytriyenko <denys@ti.com>]

On Thu, Aug 23, 2018 at 03:23:57PM -0400, Nelson, Sam wrote:
> 
> 
> > -----Original Message-----
> > From: Dmytriyenko, Denys
> > Sent: Thursday, August 23, 2018 12:04 PM
> > To: Nelson, Sam
> > Cc: meta-ti@yoctoproject.org; Zhang, Hao
> > Subject: Re: [meta-ti] [rocko][PATCH] ti-pdk: Add tool chain dependencies for
> > am65xx platform
> > 
> > On Wed, Aug 22, 2018 at 07:14:16PM -0400, Nelson, Sam wrote:
> > >
> > >
> > > > -----Original Message-----
> > > > From: Dmytriyenko, Denys
> > > > Sent: Wednesday, August 22, 2018 6:50 PM
> > > > To: Nelson, Sam
> > > > Cc: meta-ti@yoctoproject.org; Zhang, Hao
> > > > Subject: Re: [meta-ti] [rocko][PATCH] ti-pdk: Add tool chain dependencies
> > for
> > > > am65xx platform
> > > >
> > > > On Wed, Aug 22, 2018 at 05:18:50PM -0400, Sam Nelson wrote:
> > > > > Adds the dependency on toolchain needed for A53 and R5 cores
> > > > >
> > > > > Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> > > > > ---
> > > > >  classes/ti-pdk.bbclass | 4 ++++
> > > > >  1 file changed, 4 insertions(+)
> > > > >
> > > > > diff --git a/classes/ti-pdk.bbclass b/classes/ti-pdk.bbclass
> > > > > index 7ad6205..f0d14a7 100644
> > > > > --- a/classes/ti-pdk.bbclass
> > > > > +++ b/classes/ti-pdk.bbclass
> > > > > @@ -9,6 +9,8 @@ DEPENDS_append_omap-a15 = " ti-cgt-arm-native"
> > > > >  DEPENDS_remove_ti33x = "ti-cgt6x-native"
> > > > >  DEPENDS_remove_ti43x = "ti-cgt6x-native"
> > > > >  DEPENDS_append_omapl1 = " ti-cgt-arm-native"
> > > > > +DEPENDS_remove_am65xx-evm = "gcc-arm-none-eabi-native ti-cgt6x-
> > > > native"
> > > >
> > > > Will gcc-arm-none-eabi-native recipe be removed?
> > >
> > > My understanding is gcc-arm-none-eabi-native will still be used for A15.
> > 
> > That's strange, because both armv7 and armv8 versions of baremetal Linaro
> > gcc
> > were added by Hao recently...
> 
> Checked with Hao to confirm.
> Looks like we cannot use baremetal gcc linaro for a15 currently.  
> (we may move to use baremetal gcc linaro later)
> So this patch has to stay in the current form for now.

Ok, sounds good, thanks. Please don't forget to clean it up later.


> > > > > +DEPENDS_append_am65xx-evm = " ti-cgt-arm-native gcc-linaro-
> > baremetal-
> > > > aarch64-native"
> > > > >
> > > > >  S = "${WORKDIR}/git"
> > > > >  B = "${WORKDIR}/build"
> > > > > @@ -42,6 +44,8 @@ export TOOLCHAIN_PATH_M4 =
> > > > "${M4_TOOLCHAIN_INSTALL_DIR}"
> > > > >  export TOOLCHAIN_PATH_Arm9 = "${M4_TOOLCHAIN_INSTALL_DIR}"
> > > > >  export C6X_GEN_INSTALL_PATH =
> > > > "${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> > > > >  export CL_PRU_INSTALL_PATH = "${TI_CGT_PRU_INSTALL_DIR}"
> > > > > +export TOOLCHAIN_PATH_GCC_ARCH64 =
> > > > "${GCC_LINARO_BAREMETAL_AARCH64_TOOLCHAIN}"
> > > > > +export TOOLCHAIN_PATH_R5 = "${M4_TOOLCHAIN_INSTALL_DIR}"
> > > > >
> > > > >  export ROOTDIR = "${B}"
> > > > >  export BIOS_INSTALL_PATH = "${SYSBIOS_INSTALL_DIR}"
> > > > > --
> > > > > 1.9.1
> > > > >
> > > > > --
> > > > > _______________________________________________
> > > > > meta-ti mailing list
> > > > > meta-ti@yoctoproject.org
> > > > > https://lists.yoctoproject.org/listinfo/meta-ti