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: [email protected] 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 <[email protected]> > --- > 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 > [email protected] > https://lists.yoctoproject.org/listinfo/meta-ti -- _______________________________________________ meta-ti mailing list [email protected] https://lists.yoctoproject.org/listinfo/meta-ti
