> -----Original Message----- > From: Dmytriyenko, Denys > Sent: Thursday, January 25, 2018 2:15 PM > To: Molfetta, Stephen > Cc: Stiffler, Jacob; meta-ti@yoctoproject.org > Subject: Re: [meta-ti] [PATCH] dra7xx-evm.conf: removing am57xx SOCS > from PDK > > Jake? >
I would like to run a test build to make sure nothing else fails. I will try to provide an update tomorrow. > > On Wed, Jan 24, 2018 at 08:16:51AM -0500, Molfetta, Stephen wrote: > > Yes, please merge into morty. > > > > Agreed on where PDK SOCs ought to be set, but I'm not sure whether this > > will break am57xx builds right now and will let Jake comment further. > > > > Thanks, > > Stephen > > > > -----Original Message----- > > From: Dmytriyenko, Denys > > Sent: Tuesday, January 23, 2018 4:16 PM > > To: Molfetta, Stephen; Stiffler, Jacob > > Cc: meta-ti@yoctoproject.org > > Subject: Re: [meta-ti] [PATCH] dra7xx-evm.conf: removing am57xx SOCS > from > > PDK > > > > I assume, you want this merged into morty too, correct? > > > > This change is fine with me, but I'd go even further and move setting > > initial TI_PDK_LIMIT_SOCS from omap-a15.inc into am57xx-evm.conf > > > > Jake, > > > > Do you have any comments? > > > > On Mon, Jan 22, 2018 at 02:15:48PM -0600, Stephen Molfetta wrote: > > > opencl-monitor does not distinguish between dra7xx and am57xx > devices, > > > treating both as the same and attempting to build for am57xx > regardless. > > > However, this is dependent on common-csl-ip-rtos, a component of > PDK, > > > which in general does handle these devices differently. The definition > > > of am57xx SOCs in the PDK list from omap-15.conf gets inherited by > > > dra7xx-evm.conf, thereby creating a rule that all dra7xx PDK recipes > > > must build am57xx libraries. This results in a dra7xx image which is > > > much larger than necessary, and more severely, creates an artifical > > > dependency link between these two devices and introduces build > failures > > > for various PDK recipes. > > > > > > Instead, we now override the PDK SOC definition in dra7xx-evm.conf, > and > > > only in the CSL RTOS recipe do we append the am57xx SOCS for dra7xx- > evm > > > machine build. This allows for opencl-monitor to build successfully > > for > > > dra7xx while preventing any other PDK recipes from attempting to build > > > libraries for am57xx devices. > > > > > > This is a workaround that must stay in place until opencl build can > > > properly comprehend am57xx and dra7xx independently, and/or omap- > a15 > > > is not implied to be synonymous with am57xx in various recipes. > > > > > > Signed-off-by: Stephen Molfetta <sjmolfe...@ti.com> > > > --- > > > conf/machine/dra7xx-evm.conf | 2 +- > > > recipes-bsp/common-csl-ip/common-csl-ip-rtos_git.bb | 5 ++++- > > > 2 files changed, 5 insertions(+), 2 deletions(-) > > > > > > diff --git a/conf/machine/dra7xx-evm.conf b/conf/machine/dra7xx- > evm.conf > > > index 91afd85..7db9ff6 100644 > > > --- a/conf/machine/dra7xx-evm.conf > > > +++ b/conf/machine/dra7xx-evm.conf > > > @@ -40,4 +40,4 @@ UBINIZE_ARGS = "-m 2048 -p 128KiB -s 512 -O 2048" > > > > > > # Set the list of device targets for ti-pdk class recipes > > > TI_PDK_LIMIT_BOARDS = "evmDRA72x evmDRA75x evmDRA78x" > > > -TI_PDK_LIMIT_SOCS_append = " dra72x dra75x dra78x" > > > +TI_PDK_LIMIT_SOCS = "dra72x dra75x dra78x" > > > diff --git a/recipes-bsp/common-csl-ip/common-csl-ip-rtos_git.bb > > b/recipes-bsp/common-csl-ip/common-csl-ip-rtos_git.bb > > > index f4496b9..100d2f4 100644 > > > --- a/recipes-bsp/common-csl-ip/common-csl-ip-rtos_git.bb > > > +++ b/recipes-bsp/common-csl-ip/common-csl-ip-rtos_git.bb > > > @@ -1,5 +1,5 @@ > > > require common-csl-ip.inc > > > -PR = "${INC_PR}.5" > > > +PR = "${INC_PR}.6" > > > > > > PACKAGE_ARCH = "${MACHINE_ARCH}" > > > COMPATIBLE_MACHINE_append = "|c66x" > > > @@ -16,3 +16,6 @@ export DEST_ROOT="${S}" > > > > > > # HTML doc link params > > > PDK_COMP_LINK_TEXT = "CSL-Chip Support Library" > > > + > > > +# Workaround: dra7xx build requires am57xx CSL libraries for > > opencl-monitor > > > +TI_PDK_LIMIT_SOCS_append_dra7xx = " am571x am572x" > > > -- > > > 2.7.4 > > > > > > -- > > > _______________________________________________ > > > meta-ti mailing list > > > meta-ti@yoctoproject.org > > > https://lists.yoctoproject.org/listinfo/meta-ti -- _______________________________________________ meta-ti mailing list meta-ti@yoctoproject.org https://lists.yoctoproject.org/listinfo/meta-ti