[meta-ti] [PATCH] dra7xx-evm.conf: removing am57xx SOCS from PDK

[Stiffler, Jacob]

> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Thursday, January 25, 2018 2:15 PM
> To: Molfetta, Stephen
> Cc: Stiffler, Jacob; meta-ti at 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 at 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 <sjmolfetta at 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 at yoctoproject.org
> >    > https://lists.yoctoproject.org/listinfo/meta-ti