* [PATCH 1/3] SOC_FAMILY: add to MACHINE_OVERRIDES locally
@ 2012-03-03 10:53 Koen Kooi
2012-03-03 10:53 ` [PATCH 2/3] ti-hw-bringup-image: change require to include to make meta-angstrom a soft dependency Koen Kooi
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Koen Kooi @ 2012-03-03 10:53 UTC (permalink / raw)
To: meta-ti
This breaks the hard dependency on Angstrom for getting a sane set of recipes.
The variable name is kept the same and the actual code moved from angstrom.inc to soc-family.inc to make it clear what it is doing and how.
Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
---
conf/machine/include/omap3.inc | 1 +
conf/machine/include/omap4.inc | 1 +
conf/machine/include/omapl138.inc | 1 +
conf/machine/include/soc-family.inc | 3 +++
conf/machine/include/ti33x.inc | 1 +
conf/machine/include/ti814x.inc | 1 +
conf/machine/include/ti816x.inc | 1 +
7 files changed, 9 insertions(+), 0 deletions(-)
create mode 100644 conf/machine/include/soc-family.inc
diff --git a/conf/machine/include/omap3.inc b/conf/machine/include/omap3.inc
index 8fc0903..9cafe85 100644
--- a/conf/machine/include/omap3.inc
+++ b/conf/machine/include/omap3.inc
@@ -1,4 +1,5 @@
SOC_FAMILY = "omap3"
+require conf/machine/include/soc-family.inc
require conf/machine/include/tune-cortexa8.inc
PREFERRED_PROVIDER_virtual/kernel = "linux-omap"
diff --git a/conf/machine/include/omap4.inc b/conf/machine/include/omap4.inc
index 84b7742..a295352 100644
--- a/conf/machine/include/omap4.inc
+++ b/conf/machine/include/omap4.inc
@@ -1,4 +1,5 @@
SOC_FAMILY = "omap4"
+require conf/machine/include/soc-family.inc
require conf/machine/include/tune-cortexa8.inc
diff --git a/conf/machine/include/omapl138.inc b/conf/machine/include/omapl138.inc
index 4c22275..1949642 100644
--- a/conf/machine/include/omapl138.inc
+++ b/conf/machine/include/omapl138.inc
@@ -1 +1,2 @@
SOC_FAMILY = "omapl138"
+require conf/machine/include/soc-family.inc
diff --git a/conf/machine/include/soc-family.inc b/conf/machine/include/soc-family.inc
new file mode 100644
index 0000000..39ca338
--- /dev/null
+++ b/conf/machine/include/soc-family.inc
@@ -0,0 +1,3 @@
+# Add SOC_FAMILY to machine overrides so we get access to e.g. 'omap3' and 'ti335x'
+MACHINEOVERRIDES .= "${@['', ':${SOC_FAMILY}']['${SOC_FAMILY}' != '']}"
+
diff --git a/conf/machine/include/ti33x.inc b/conf/machine/include/ti33x.inc
index fb8bad3..f4954f5 100644
--- a/conf/machine/include/ti33x.inc
+++ b/conf/machine/include/ti33x.inc
@@ -1,4 +1,5 @@
SOC_FAMILY = "ti33x"
+require conf/machine/include/soc-family.inc
require conf/machine/include/tune-cortexa8.inc
PREFERRED_PROVIDER_virtual/kernel = "linux-ti33x-psp"
diff --git a/conf/machine/include/ti814x.inc b/conf/machine/include/ti814x.inc
index 23b6a1d..c53557f 100644
--- a/conf/machine/include/ti814x.inc
+++ b/conf/machine/include/ti814x.inc
@@ -1,4 +1,5 @@
SOC_FAMILY = "ti814x"
+require conf/machine/include/soc-family.inc
require conf/machine/include/tune-cortexa8.inc
diff --git a/conf/machine/include/ti816x.inc b/conf/machine/include/ti816x.inc
index 06cb7c0..2fe4006 100644
--- a/conf/machine/include/ti816x.inc
+++ b/conf/machine/include/ti816x.inc
@@ -1,4 +1,5 @@
SOC_FAMILY = "ti816x"
+require conf/machine/include/soc-family.inc
require conf/machine/include/tune-cortexa8.inc
--
1.7.2.5
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH 2/3] ti-hw-bringup-image: change require to include to make meta-angstrom a soft dependency
2012-03-03 10:53 [PATCH 1/3] SOC_FAMILY: add to MACHINE_OVERRIDES locally Koen Kooi
@ 2012-03-03 10:53 ` Koen Kooi
2012-03-03 10:53 ` [PATCH 3/3] README: move meta-angstrom to optional layers Koen Kooi
2012-03-03 13:55 ` [PATCH 1/3] SOC_FAMILY: add to MACHINE_OVERRIDES locally Denys Dmytriyenko
2 siblings, 0 replies; 8+ messages in thread
From: Koen Kooi @ 2012-03-03 10:53 UTC (permalink / raw)
To: meta-ti
The image will be non-functional, but parsing completes with only oe-core + meta-ti
Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
---
recipes-misc/images/ti-hw-bringup-image.bb | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/recipes-misc/images/ti-hw-bringup-image.bb b/recipes-misc/images/ti-hw-bringup-image.bb
index 5f7afce..2be8baf 100644
--- a/recipes-misc/images/ti-hw-bringup-image.bb
+++ b/recipes-misc/images/ti-hw-bringup-image.bb
@@ -1,6 +1,6 @@
# Image for assisting in hardware bringup
-require recipes-images/angstrom/systemd-image.bb
+include recipes-images/angstrom/systemd-image.bb
EXTRA_MACHINE_IMAGE_INSTALL ?= ""
--
1.7.2.5
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 3/3] README: move meta-angstrom to optional layers
2012-03-03 10:53 [PATCH 1/3] SOC_FAMILY: add to MACHINE_OVERRIDES locally Koen Kooi
2012-03-03 10:53 ` [PATCH 2/3] ti-hw-bringup-image: change require to include to make meta-angstrom a soft dependency Koen Kooi
@ 2012-03-03 10:53 ` Koen Kooi
2012-03-03 11:24 ` William Mills
2012-03-03 13:55 ` [PATCH 1/3] SOC_FAMILY: add to MACHINE_OVERRIDES locally Denys Dmytriyenko
2 siblings, 1 reply; 8+ messages in thread
From: Koen Kooi @ 2012-03-03 10:53 UTC (permalink / raw)
To: meta-ti
Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
---
README | 6 ++++--
1 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/README b/README
index 9bf6154..918c0f1 100644
--- a/README
+++ b/README
@@ -1,4 +1,4 @@
-This layer depends on:
+This layer hard depends on:
URI: git://git.openembedded.org/openembedded-core
branch: master
@@ -8,6 +8,8 @@ URI: git://git.openembedded.org/meta-openembedded
branch: master
revision: HEAD
+And optionally depends on:
+
URI: git://git.angstrom-distribution.org/meta-angstrom
branch: master
revision: HEAD
@@ -24,7 +26,7 @@ Arago.
Due to the above, it is now recommended to follow the instructions at
http://www.angstrom-distribution.org/building-angstrom
-This will set it up for the OpenEmbedded-core layout instead of the
+This will set it up for the OpenEmbedded-core ('yocto') layout instead of the
old "Classic" OpenEmbedded-dev layout. You can optionally tweak
sources/layers.txt and conf/bblayers.conf to (de)select BSP layers.
--
1.7.2.5
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH 1/3] SOC_FAMILY: add to MACHINE_OVERRIDES locally
2012-03-03 10:53 [PATCH 1/3] SOC_FAMILY: add to MACHINE_OVERRIDES locally Koen Kooi
2012-03-03 10:53 ` [PATCH 2/3] ti-hw-bringup-image: change require to include to make meta-angstrom a soft dependency Koen Kooi
2012-03-03 10:53 ` [PATCH 3/3] README: move meta-angstrom to optional layers Koen Kooi
@ 2012-03-03 13:55 ` Denys Dmytriyenko
2012-03-03 19:01 ` Koen Kooi
2 siblings, 1 reply; 8+ messages in thread
From: Denys Dmytriyenko @ 2012-03-03 13:55 UTC (permalink / raw)
To: Koen Kooi; +Cc: meta-ti
On Sat, Mar 03, 2012 at 11:53:06AM +0100, Koen Kooi wrote:
> This breaks the hard dependency on Angstrom for getting a sane set of
> recipes.
>
> The variable name is kept the same and the actual code moved from
> angstrom.inc to soc-family.inc to make it clear what it is doing and how.
Ah, nice workaround until/if it gets accepted upstream to OE-Core! Thanks,
Koen.
--
Denys
> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> ---
> conf/machine/include/omap3.inc | 1 +
> conf/machine/include/omap4.inc | 1 +
> conf/machine/include/omapl138.inc | 1 +
> conf/machine/include/soc-family.inc | 3 +++
> conf/machine/include/ti33x.inc | 1 +
> conf/machine/include/ti814x.inc | 1 +
> conf/machine/include/ti816x.inc | 1 +
> 7 files changed, 9 insertions(+), 0 deletions(-)
> create mode 100644 conf/machine/include/soc-family.inc
>
> diff --git a/conf/machine/include/omap3.inc b/conf/machine/include/omap3.inc
> index 8fc0903..9cafe85 100644
> --- a/conf/machine/include/omap3.inc
> +++ b/conf/machine/include/omap3.inc
> @@ -1,4 +1,5 @@
> SOC_FAMILY = "omap3"
> +require conf/machine/include/soc-family.inc
>
> require conf/machine/include/tune-cortexa8.inc
> PREFERRED_PROVIDER_virtual/kernel = "linux-omap"
> diff --git a/conf/machine/include/omap4.inc b/conf/machine/include/omap4.inc
> index 84b7742..a295352 100644
> --- a/conf/machine/include/omap4.inc
> +++ b/conf/machine/include/omap4.inc
> @@ -1,4 +1,5 @@
> SOC_FAMILY = "omap4"
> +require conf/machine/include/soc-family.inc
>
> require conf/machine/include/tune-cortexa8.inc
>
> diff --git a/conf/machine/include/omapl138.inc b/conf/machine/include/omapl138.inc
> index 4c22275..1949642 100644
> --- a/conf/machine/include/omapl138.inc
> +++ b/conf/machine/include/omapl138.inc
> @@ -1 +1,2 @@
> SOC_FAMILY = "omapl138"
> +require conf/machine/include/soc-family.inc
> diff --git a/conf/machine/include/soc-family.inc b/conf/machine/include/soc-family.inc
> new file mode 100644
> index 0000000..39ca338
> --- /dev/null
> +++ b/conf/machine/include/soc-family.inc
> @@ -0,0 +1,3 @@
> +# Add SOC_FAMILY to machine overrides so we get access to e.g. 'omap3' and 'ti335x'
> +MACHINEOVERRIDES .= "${@['', ':${SOC_FAMILY}']['${SOC_FAMILY}' != '']}"
> +
> diff --git a/conf/machine/include/ti33x.inc b/conf/machine/include/ti33x.inc
> index fb8bad3..f4954f5 100644
> --- a/conf/machine/include/ti33x.inc
> +++ b/conf/machine/include/ti33x.inc
> @@ -1,4 +1,5 @@
> SOC_FAMILY = "ti33x"
> +require conf/machine/include/soc-family.inc
>
> require conf/machine/include/tune-cortexa8.inc
> PREFERRED_PROVIDER_virtual/kernel = "linux-ti33x-psp"
> diff --git a/conf/machine/include/ti814x.inc b/conf/machine/include/ti814x.inc
> index 23b6a1d..c53557f 100644
> --- a/conf/machine/include/ti814x.inc
> +++ b/conf/machine/include/ti814x.inc
> @@ -1,4 +1,5 @@
> SOC_FAMILY = "ti814x"
> +require conf/machine/include/soc-family.inc
>
> require conf/machine/include/tune-cortexa8.inc
>
> diff --git a/conf/machine/include/ti816x.inc b/conf/machine/include/ti816x.inc
> index 06cb7c0..2fe4006 100644
> --- a/conf/machine/include/ti816x.inc
> +++ b/conf/machine/include/ti816x.inc
> @@ -1,4 +1,5 @@
> SOC_FAMILY = "ti816x"
> +require conf/machine/include/soc-family.inc
>
> require conf/machine/include/tune-cortexa8.inc
>
> --
> 1.7.2.5
>
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti
>
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH 1/3] SOC_FAMILY: add to MACHINE_OVERRIDES locally
2012-03-03 13:55 ` [PATCH 1/3] SOC_FAMILY: add to MACHINE_OVERRIDES locally Denys Dmytriyenko
@ 2012-03-03 19:01 ` Koen Kooi
2012-03-04 4:10 ` Richard Purdie
0 siblings, 1 reply; 8+ messages in thread
From: Koen Kooi @ 2012-03-03 19:01 UTC (permalink / raw)
To: Denys Dmytriyenko; +Cc: meta-ti
Op 3 mrt. 2012, om 14:55 heeft Denys Dmytriyenko het volgende geschreven:
> On Sat, Mar 03, 2012 at 11:53:06AM +0100, Koen Kooi wrote:
>> This breaks the hard dependency on Angstrom for getting a sane set of
>> recipes.
>>
>> The variable name is kept the same and the actual code moved from
>> angstrom.inc to soc-family.inc to make it clear what it is doing and how.
>
> Ah, nice workaround until/if it gets accepted upstream to OE-Core! Thanks,
> Koen.
The consensus at ELC was that these kind of tweaks should be done at the machine level now that we have MACHINE_OVERRIDES seperated out. If a lot of BSPs are going to use SOC_FAMILY we can try to push it into OE-core again. But currently we wouldn't stand a chance.
regards,
Koen
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/3] SOC_FAMILY: add to MACHINE_OVERRIDES locally
2012-03-03 19:01 ` Koen Kooi
@ 2012-03-04 4:10 ` Richard Purdie
2012-03-04 5:50 ` Denys Dmytriyenko
0 siblings, 1 reply; 8+ messages in thread
From: Richard Purdie @ 2012-03-04 4:10 UTC (permalink / raw)
To: Koen Kooi; +Cc: meta-ti
On Sat, 2012-03-03 at 20:01 +0100, Koen Kooi wrote:
> Op 3 mrt. 2012, om 14:55 heeft Denys Dmytriyenko het volgende geschreven:
>
> > On Sat, Mar 03, 2012 at 11:53:06AM +0100, Koen Kooi wrote:
> >> This breaks the hard dependency on Angstrom for getting a sane set of
> >> recipes.
> >>
> >> The variable name is kept the same and the actual code moved from
> >> angstrom.inc to soc-family.inc to make it clear what it is doing and how.
> >
> > Ah, nice workaround until/if it gets accepted upstream to OE-Core! Thanks,
> > Koen.
>
> The consensus at ELC was that these kind of tweaks should be done at
> the machine level now that we have MACHINE_OVERRIDES seperated out. If
> a lot of BSPs are going to use SOC_FAMILY we can try to push it into
> OE-core again. But currently we wouldn't stand a chance.
MACHINE_OVERRIDES was added to give people a generic way of extending
overrides from a machine context. This works for various situations such
as the "x86" override that made more sense than the previous 386|486|
586|xxx madness as well as providing a mechanism for SOC_FAMILY and
other uses.
I guess the .inc file could make it into OE-Core but I don't see much
value in adding SOC_FAMILY directly as a standard variable at this
point.
Cheers,
Richard
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/3] SOC_FAMILY: add to MACHINE_OVERRIDES locally
2012-03-04 4:10 ` Richard Purdie
@ 2012-03-04 5:50 ` Denys Dmytriyenko
0 siblings, 0 replies; 8+ messages in thread
From: Denys Dmytriyenko @ 2012-03-04 5:50 UTC (permalink / raw)
To: Richard Purdie; +Cc: meta-ti
On Sat, Mar 03, 2012 at 08:10:20PM -0800, Richard Purdie wrote:
> On Sat, 2012-03-03 at 20:01 +0100, Koen Kooi wrote:
> > Op 3 mrt. 2012, om 14:55 heeft Denys Dmytriyenko het volgende geschreven:
> >
> > > On Sat, Mar 03, 2012 at 11:53:06AM +0100, Koen Kooi wrote:
> > >> This breaks the hard dependency on Angstrom for getting a sane set of
> > >> recipes.
> > >>
> > >> The variable name is kept the same and the actual code moved from
> > >> angstrom.inc to soc-family.inc to make it clear what it is doing and how.
> > >
> > > Ah, nice workaround until/if it gets accepted upstream to OE-Core! Thanks,
> > > Koen.
> >
> > The consensus at ELC was that these kind of tweaks should be done at
> > the machine level now that we have MACHINE_OVERRIDES seperated out. If
> > a lot of BSPs are going to use SOC_FAMILY we can try to push it into
> > OE-core again. But currently we wouldn't stand a chance.
>
> MACHINE_OVERRIDES was added to give people a generic way of extending
> overrides from a machine context. This works for various situations such
> as the "x86" override that made more sense than the previous 386|486|
> 586|xxx madness as well as providing a mechanism for SOC_FAMILY and
> other uses.
>
> I guess the .inc file could make it into OE-Core but I don't see much
> value in adding SOC_FAMILY directly as a standard variable at this
> point.
Thanks, Richard! That's a suitable compromise. I'll send a patch to OE-Core.
--
Denys
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2012-03-04 5:50 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-03-03 10:53 [PATCH 1/3] SOC_FAMILY: add to MACHINE_OVERRIDES locally Koen Kooi
2012-03-03 10:53 ` [PATCH 2/3] ti-hw-bringup-image: change require to include to make meta-angstrom a soft dependency Koen Kooi
2012-03-03 10:53 ` [PATCH 3/3] README: move meta-angstrom to optional layers Koen Kooi
2012-03-03 11:24 ` William Mills
2012-03-03 13:55 ` [PATCH 1/3] SOC_FAMILY: add to MACHINE_OVERRIDES locally Denys Dmytriyenko
2012-03-03 19:01 ` Koen Kooi
2012-03-04 4:10 ` Richard Purdie
2012-03-04 5:50 ` Denys Dmytriyenko
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.