* [PATCH] dra7xx-evm.conf: removing am57xx SOCS from PDK
@ 2018-01-22 20:15 Stephen Molfetta
2018-01-23 22:16 ` Denys Dmytriyenko
0 siblings, 1 reply; 5+ messages in thread
From: Stephen Molfetta @ 2018-01-22 20:15 UTC (permalink / raw)
To: meta-ti; +Cc: sjmolfetta
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@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
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] dra7xx-evm.conf: removing am57xx SOCS from PDK
2018-01-22 20:15 [PATCH] dra7xx-evm.conf: removing am57xx SOCS from PDK Stephen Molfetta
@ 2018-01-23 22:16 ` Denys Dmytriyenko
2018-01-24 13:16 ` Molfetta, Stephen
0 siblings, 1 reply; 5+ messages in thread
From: Denys Dmytriyenko @ 2018-01-23 22:16 UTC (permalink / raw)
To: Stephen Molfetta, Stiffler, Jacob; +Cc: meta-ti
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@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
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dra7xx-evm.conf: removing am57xx SOCS from PDK
2018-01-23 22:16 ` Denys Dmytriyenko
@ 2018-01-24 13:16 ` Molfetta, Stephen
2018-01-25 19:15 ` Denys Dmytriyenko
0 siblings, 1 reply; 5+ messages in thread
From: Molfetta, Stephen @ 2018-01-24 13:16 UTC (permalink / raw)
To: Dmytriyenko, Denys, Stiffler, Jacob; +Cc: meta-ti@yoctoproject.org
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 <sjmolfetta@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
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dra7xx-evm.conf: removing am57xx SOCS from PDK
2018-01-24 13:16 ` Molfetta, Stephen
@ 2018-01-25 19:15 ` Denys Dmytriyenko
2018-01-25 21:32 ` Stiffler, Jacob
0 siblings, 1 reply; 5+ messages in thread
From: Denys Dmytriyenko @ 2018-01-25 19:15 UTC (permalink / raw)
To: Molfetta, Stephen; +Cc: meta-ti@yoctoproject.org
Jake?
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 <sjmolfetta@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
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dra7xx-evm.conf: removing am57xx SOCS from PDK
2018-01-25 19:15 ` Denys Dmytriyenko
@ 2018-01-25 21:32 ` Stiffler, Jacob
0 siblings, 0 replies; 5+ messages in thread
From: Stiffler, Jacob @ 2018-01-25 21:32 UTC (permalink / raw)
To: Dmytriyenko, Denys, Molfetta, Stephen; +Cc: meta-ti@yoctoproject.org
> -----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 <sjmolfetta@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
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-01-25 21:32 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-01-22 20:15 [PATCH] dra7xx-evm.conf: removing am57xx SOCS from PDK Stephen Molfetta
2018-01-23 22:16 ` Denys Dmytriyenko
2018-01-24 13:16 ` Molfetta, Stephen
2018-01-25 19:15 ` Denys Dmytriyenko
2018-01-25 21:32 ` Stiffler, Jacob
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.