* [PATCH 02/18] ti-tisdk-makefile: Switch from ARMPKGARCH to PLATFORM_ARCH
2013-02-12 0:46 [PATCH 01/18] linux-libc-headers: Add linux-libc-headers version 2.6.37.2 fcooperjr27
@ 2013-02-12 0:46 ` fcooperjr27
2013-02-12 0:46 ` [PATCH 03/18] arm-benchmarks_1.3: Tweak to allow am180x-evm to be built fcooperjr27
` (15 subsequent siblings)
16 siblings, 0 replies; 32+ messages in thread
From: fcooperjr27 @ 2013-02-12 0:46 UTC (permalink / raw)
To: meta-arago; +Cc: Franklin S. Cooper Jr
From: Franklin S. Cooper Jr <fcooper@ti.com>
* Use PLATFORM_ARCH variable to allow omapl138 devices to pass armv5te which is
expected by other software that uses Rules.make
Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
---
.../ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb | 8 ++++++--
1 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb
index c6cea95..99e0484 100644
--- a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb
+++ b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb
@@ -30,7 +30,7 @@ SRC_URI = "\
file://Makefile_wireless \
"
-PR = "r4"
+PR = "r5"
MAKEFILES_COMMON = "linux \
matrix-gui \
@@ -61,6 +61,10 @@ MAKEFILES_am180x-evm += "pru \
wireless \
"
+PLATFORM_ARCH = "${ARMPKGARCH}"
+# Use ARCH format expected by the makefile
+PLATFORM_ARCH_omapl138 = "armv5te"
+
# This step will stitch together the final Makefile based on the supported
# make targets.
do_install () {
@@ -104,7 +108,7 @@ do_install () {
# fixup Rules.make values
sed -i -e "s/__PLATFORM__/${MACHINE}/" ${D}/Rules.make
- sed -i -e "s/__ARCH__/${ARMPKGARCH}/" ${D}/Rules.make
+ sed -i -e "s/__ARCH__/${PLATFORM_ARCH}/" ${D}/Rules.make
sed -i -e "s/__UBOOT_MACHINE__/${UBOOT_MACHINE}/" ${D}/Rules.make
}
--
1.7.0.4
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 03/18] arm-benchmarks_1.3: Tweak to allow am180x-evm to be built
2013-02-12 0:46 [PATCH 01/18] linux-libc-headers: Add linux-libc-headers version 2.6.37.2 fcooperjr27
2013-02-12 0:46 ` [PATCH 02/18] ti-tisdk-makefile: Switch from ARMPKGARCH to PLATFORM_ARCH fcooperjr27
@ 2013-02-12 0:46 ` fcooperjr27
2013-02-12 0:46 ` [PATCH 04/18] sourceipk.bbclass: Provide the ability to change the source directory fcooperjr27
` (14 subsequent siblings)
16 siblings, 0 replies; 32+ messages in thread
From: fcooperjr27 @ 2013-02-12 0:46 UTC (permalink / raw)
To: meta-arago; +Cc: Franklin S. Cooper Jr
From: Franklin S. Cooper Jr <fcooper@ti.com>
* Makefile expects armv5te for ARM9 devices which was done in oe-classic.
* In oe-core ARMPKGARCH only specifies armv5 for am180x-evm.
* Pass in ARCH=armv5te to make for omapl138 devices
Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
---
.../arm-benchmarks/arm-benchmarks_1.3.bb | 12 +++++++-----
1 files changed, 7 insertions(+), 5 deletions(-)
diff --git a/meta-arago-extras/recipes-benchmark/arm-benchmarks/arm-benchmarks_1.3.bb b/meta-arago-extras/recipes-benchmark/arm-benchmarks/arm-benchmarks_1.3.bb
index 9fda6b0..b19c1ed 100644
--- a/meta-arago-extras/recipes-benchmark/arm-benchmarks/arm-benchmarks_1.3.bb
+++ b/meta-arago-extras/recipes-benchmark/arm-benchmarks/arm-benchmarks_1.3.bb
@@ -4,7 +4,7 @@ LICENSE = "BSD"
LIC_FILES_CHKSUM = "file://COPYING;md5=7aefb5e1cffc7b6a3ef18b803f957922"
SECTION = "system"
-PR = "r2"
+PR = "r3"
BRANCH ?= "master"
SRCREV = "e9fbf7990e93d97e7471e509626969d244cca214"
@@ -13,14 +13,16 @@ SRC_URI = "git://gitorious.org/arm_benchmarks/arm_benchmarks.git;protocol=git;br
S = "${WORKDIR}/git"
+PLATFORM_ARCH = "${ARMPKGARCH}"
+# Use ARCH format expected by the makefile
+PLATFORM_ARCH_omapl138 = "armv5te"
+
do_compile() {
export CROSS_COMPILE=${TARGET_PREFIX}
- export ARCH=${ARMPKGARCH}
# build the release version
- make release
+ make ARCH=${PLATFORM_ARCH} release
}
do_install() {
- export ARCH=${ARMPKGARCH}
- make DESTDIR=${D} install
+ make ARCH=${PLATFORM_ARCH} DESTDIR=${D} install
}
--
1.7.0.4
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 04/18] sourceipk.bbclass: Provide the ability to change the source directory
2013-02-12 0:46 [PATCH 01/18] linux-libc-headers: Add linux-libc-headers version 2.6.37.2 fcooperjr27
2013-02-12 0:46 ` [PATCH 02/18] ti-tisdk-makefile: Switch from ARMPKGARCH to PLATFORM_ARCH fcooperjr27
2013-02-12 0:46 ` [PATCH 03/18] arm-benchmarks_1.3: Tweak to allow am180x-evm to be built fcooperjr27
@ 2013-02-12 0:46 ` fcooperjr27
2013-02-12 0:46 ` [PATCH 05/18] ti-crypto-examples: Add ARM9 support fcooperjr27
` (13 subsequent siblings)
16 siblings, 0 replies; 32+ messages in thread
From: fcooperjr27 @ 2013-02-12 0:46 UTC (permalink / raw)
To: meta-arago; +Cc: Franklin S. Cooper Jr
From: Franklin S. Cooper Jr <fcooper@ti.com>
* Allow the directory where the sources are grabbed to be changed.
Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
---
meta-arago-distro/classes/sourceipk.bbclass | 14 ++++++++------
1 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/meta-arago-distro/classes/sourceipk.bbclass b/meta-arago-distro/classes/sourceipk.bbclass
index ecc92c2..1844ff3 100644
--- a/meta-arago-distro/classes/sourceipk.bbclass
+++ b/meta-arago-distro/classes/sourceipk.bbclass
@@ -47,6 +47,9 @@ CREATE_SRCIPK ?= "0"
# Default installation prefix
SRCIPK_INSTALL_DIR ?= "/usr/src/${PN}-src"
+# Specify the directory of the sources
+SRCIPK_SRC_DIR ?= "${S}"
+
# Default PACKAGE_ARCH for sources is "all"
SRCIPK_PACKAGE_ARCH ?= "all"
@@ -56,7 +59,6 @@ SRCIPK_SECTION ?= "${SECTION}"
# Default SRCIPK_INCLUDE_EXTRAFILES is to include the extra files
SRCIPK_INCLUDE_EXTRAFILES ?= "1"
-
# Remove git repositories before packaging up the sources
clear_git() {
@@ -126,11 +128,11 @@ sourceipk_do_create_srcipk() {
# Copy sources for packaging
mkdir -p $tmp_dir/${SRCIPK_INSTALL_DIR}
- if [ -e ${S} ]; then
- if [ "${S}" = "${WORKDIR}" ]; then
- excludes='--exclude ./temp --exclude ./sourceipk-tmp --exclude ./sourceipk-data'
- fi
- tar -C ${S} -cO $excludes . | tar -C $tmp_dir/${SRCIPK_INSTALL_DIR} -xpf -
+ if [ -e ${SRCIPK_SRC_DIR} ]; then
+ if [ "${SRCIPK_SRC_DIR}" = "${WORKDIR}" ]; then
+ excludes='--exclude ./temp --exclude ./sourceipk-tmp --exclude ./sourceipk-data'
+ fi
+ tar -C ${SRCIPK_SRC_DIR} -cO $excludes . | tar -C $tmp_dir/${SRCIPK_INSTALL_DIR} -xpf -
fi
if [ ${SRCIPK_INCLUDE_EXTRAFILES} != "0" ]
--
1.7.0.4
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 05/18] ti-crypto-examples: Add ARM9 support
2013-02-12 0:46 [PATCH 01/18] linux-libc-headers: Add linux-libc-headers version 2.6.37.2 fcooperjr27
` (2 preceding siblings ...)
2013-02-12 0:46 ` [PATCH 04/18] sourceipk.bbclass: Provide the ability to change the source directory fcooperjr27
@ 2013-02-12 0:46 ` fcooperjr27
2013-02-12 0:46 ` [PATCH 06/18] ti-crypto-examples: Tweak SRCIPK_SRC_DIR variable fcooperjr27
` (12 subsequent siblings)
16 siblings, 0 replies; 32+ messages in thread
From: fcooperjr27 @ 2013-02-12 0:46 UTC (permalink / raw)
To: meta-arago; +Cc: Franklin S. Cooper Jr
From: Franklin S. Cooper Jr <fcooper@ti.com>
* Previously crypto examples were fixed to cortex A8
* Git repository were updated to add ARM9 support by placing ARM9 and Cortex
A8 versions into separate directories.
Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
---
.../ti-crypto-examples/ti-crypto-examples_git.bb | 19 +++++++++++++++----
1 files changed, 15 insertions(+), 4 deletions(-)
diff --git a/meta-arago-extras/recipes-core/ti-crypto-examples/ti-crypto-examples_git.bb b/meta-arago-extras/recipes-core/ti-crypto-examples/ti-crypto-examples_git.bb
index 43810e2..648d7dd 100644
--- a/meta-arago-extras/recipes-core/ti-crypto-examples/ti-crypto-examples_git.bb
+++ b/meta-arago-extras/recipes-core/ti-crypto-examples/ti-crypto-examples_git.bb
@@ -1,24 +1,35 @@
DESCRIPTION = "TI Cryptography Example Applications"
HOMEPAGE = "http://arago-project.org/git/projects/?p=crypto-example-apps.git;a=summary"
LICENSE = "BSD"
-LIC_FILES_CHKSUM = "file://AES/aes_256.c;beginline=9;endline=35;md5=8edbb4dee965d2f2eb5ca2822addcae5"
+LIC_FILES_CHKSUM = "file://armv5te/AES/aes_256.c;beginline=9;endline=35;md5=8edbb4dee965d2f2eb5ca2822addcae5 \
+ file://armv7a/AES/aes_256.c;beginline=9;endline=35;md5=8edbb4dee965d2f2eb5ca2822addcae5"
SECTION = "console"
DEPENDS += "openssl"
-PR = "r2"
+PR = "r3"
BRANCH ?= "master"
-SRCREV = "37aad72a4d49d0a6dbe934495e5c3bd973a72f1a"
+SRCREV = "95007ae6a4284841e624680afff8e0859f28d429"
SRC_URI = "git://arago-project.org/git/projects/crypto-example-apps.git;protocol=git;branch=${BRANCH}"
-S = "${WORKDIR}/git"
+S = "${WORKDIR}/git/"
+
+# This software contains two directories depending on ARM architecture
+SOURCE_DIR = "${S}/${ARMPKGARCH}"
+
+# Specify armv5te for ARM9 devices
+SOURCE_DIR_omapl138 = "${S}/armv5te"
do_compile() {
+ cd ${SOURCE_DIR}
+
# build the release version
oe_runmake release
}
do_install() {
+ cd ${SOURCE_DIR}
+
oe_runmake DESTDIR=${D} install
}
--
1.7.0.4
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 06/18] ti-crypto-examples: Tweak SRCIPK_SRC_DIR variable
2013-02-12 0:46 [PATCH 01/18] linux-libc-headers: Add linux-libc-headers version 2.6.37.2 fcooperjr27
` (3 preceding siblings ...)
2013-02-12 0:46 ` [PATCH 05/18] ti-crypto-examples: Add ARM9 support fcooperjr27
@ 2013-02-12 0:46 ` fcooperjr27
2013-02-12 0:46 ` [PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable fcooperjr27
` (11 subsequent siblings)
16 siblings, 0 replies; 32+ messages in thread
From: fcooperjr27 @ 2013-02-12 0:46 UTC (permalink / raw)
To: meta-arago; +Cc: Franklin S. Cooper Jr
From: Franklin S. Cooper Jr <fcooper@ti.com>
* Tweak the directory that the sources are grabbed from. This insures that the
correct source is grabbed depending on the ARM architecture.
Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
---
.../ti-crypto-examples_git.bbappend | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
create mode 100644 meta-arago-distro/recipes-core/ti-crypto-examples/ti-crypto-examples_git.bbappend
diff --git a/meta-arago-distro/recipes-core/ti-crypto-examples/ti-crypto-examples_git.bbappend b/meta-arago-distro/recipes-core/ti-crypto-examples/ti-crypto-examples_git.bbappend
new file mode 100644
index 0000000..3afd154
--- /dev/null
+++ b/meta-arago-distro/recipes-core/ti-crypto-examples/ti-crypto-examples_git.bbappend
@@ -0,0 +1,3 @@
+PR_append = "-arago0"
+
+SRCIPK_SRC_DIR = "${SOURCE_DIR}"
--
1.7.0.4
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable
2013-02-12 0:46 [PATCH 01/18] linux-libc-headers: Add linux-libc-headers version 2.6.37.2 fcooperjr27
` (4 preceding siblings ...)
2013-02-12 0:46 ` [PATCH 06/18] ti-crypto-examples: Tweak SRCIPK_SRC_DIR variable fcooperjr27
@ 2013-02-12 0:46 ` fcooperjr27
2013-02-15 3:39 ` Denys Dmytriyenko
2013-02-16 0:26 ` Paul Kovitz
2013-02-12 0:46 ` [PATCH 08/18] recipes: Only pull in SGX related tasks if SUPPORTS_SGX is set to YES fcooperjr27
` (10 subsequent siblings)
16 siblings, 2 replies; 32+ messages in thread
From: fcooperjr27 @ 2013-02-12 0:46 UTC (permalink / raw)
To: meta-arago; +Cc: Franklin S. Cooper Jr
From: Franklin S. Cooper Jr <fcooper@ti.com>
* Add SUPPORTS_SGX variable to arago.conf.
* Create a variable that specifies if a board or device has SGX support.
* This will determine which tasks are pulled in to various filesystem and SDK
images.
Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
---
meta-arago-distro/conf/distro/arago.conf | 9 ++++++---
1 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/meta-arago-distro/conf/distro/arago.conf b/meta-arago-distro/conf/distro/arago.conf
index 8330115..691c5b8 100644
--- a/meta-arago-distro/conf/distro/arago.conf
+++ b/meta-arago-distro/conf/distro/arago.conf
@@ -45,9 +45,12 @@ PREFERRED_VERSION_hostap-daemon = "2.0-devel-ti"
PREFERRED_VERSION_omap3-sgx-modules = "4.08.00.01"
PREFERRED_VERSION_libgles-omap3 = "4.08.00.01"
-ARAGO_QT_PROVIDER = "qt4-embedded"
-# Enable Qt+GLES for ARM v7a devices
-ARAGO_QT_PROVIDER_armv7a = "qt4-embedded-gles"
+SUPPORTS_SGX = "YES"
+
+# No omapl138 variant includes SGX
+SUPPORTS_SGX_omapl138 = "NO"
+
+ARAGO_QT_PROVIDER = "${@base_conditional('SUPPORTS_SGX', 'YES', 'qt4-embedded-gles', 'qt4-embedded', d)}"
PREFERRED_PROVIDER_qt4-embedded = "${ARAGO_QT_PROVIDER}"
--
1.7.0.4
^ permalink raw reply related [flat|nested] 32+ messages in thread
* Re: [PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable
2013-02-12 0:46 ` [PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable fcooperjr27
@ 2013-02-15 3:39 ` Denys Dmytriyenko
2013-02-15 3:46 ` Cooper Jr., Franklin
2013-02-16 0:26 ` Paul Kovitz
1 sibling, 1 reply; 32+ messages in thread
From: Denys Dmytriyenko @ 2013-02-15 3:39 UTC (permalink / raw)
To: fcooperjr27; +Cc: meta-arago, Franklin S. Cooper Jr
On Mon, Feb 11, 2013 at 06:46:08PM -0600, fcooperjr27@gmail.com wrote:
> From: Franklin S. Cooper Jr <fcooper@ti.com>
>
> * Add SUPPORTS_SGX variable to arago.conf.
I was thinking about this lately and I think we shold add a new "sgx" flag in
MACHINE_FEATURES to some of our machine configs (those that support it) and
then base the logic around it - it's easily checked with base_contains().
--
Denys
> * Create a variable that specifies if a board or device has SGX support.
> * This will determine which tasks are pulled in to various filesystem and SDK
> images.
>
> Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
> ---
> meta-arago-distro/conf/distro/arago.conf | 9 ++++++---
> 1 files changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/meta-arago-distro/conf/distro/arago.conf b/meta-arago-distro/conf/distro/arago.conf
> index 8330115..691c5b8 100644
> --- a/meta-arago-distro/conf/distro/arago.conf
> +++ b/meta-arago-distro/conf/distro/arago.conf
> @@ -45,9 +45,12 @@ PREFERRED_VERSION_hostap-daemon = "2.0-devel-ti"
> PREFERRED_VERSION_omap3-sgx-modules = "4.08.00.01"
> PREFERRED_VERSION_libgles-omap3 = "4.08.00.01"
>
> -ARAGO_QT_PROVIDER = "qt4-embedded"
> -# Enable Qt+GLES for ARM v7a devices
> -ARAGO_QT_PROVIDER_armv7a = "qt4-embedded-gles"
> +SUPPORTS_SGX = "YES"
> +
> +# No omapl138 variant includes SGX
> +SUPPORTS_SGX_omapl138 = "NO"
> +
> +ARAGO_QT_PROVIDER = "${@base_conditional('SUPPORTS_SGX', 'YES', 'qt4-embedded-gles', 'qt4-embedded', d)}"
>
> PREFERRED_PROVIDER_qt4-embedded = "${ARAGO_QT_PROVIDER}"
>
> --
> 1.7.0.4
>
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable
2013-02-15 3:39 ` Denys Dmytriyenko
@ 2013-02-15 3:46 ` Cooper Jr., Franklin
0 siblings, 0 replies; 32+ messages in thread
From: Cooper Jr., Franklin @ 2013-02-15 3:46 UTC (permalink / raw)
To: Dmytriyenko, Denys, fcooperjr27; +Cc: meta-arago
I definitely agree :)
> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Thursday, February 14, 2013 9:39 PM
> To: fcooperjr27@gmail.com
> Cc: meta-arago@arago-project.org; Cooper Jr., Franklin
> Subject: Re: [meta-arago] [PATCH 07/18] arago.conf: Add SUPPORTS_SGX
> variable
>
> On Mon, Feb 11, 2013 at 06:46:08PM -0600, fcooperjr27@gmail.com wrote:
> > From: Franklin S. Cooper Jr <fcooper@ti.com>
> >
> > * Add SUPPORTS_SGX variable to arago.conf.
>
> I was thinking about this lately and I think we shold add a new "sgx" flag in
> MACHINE_FEATURES to some of our machine configs (those that support it)
> and then base the logic around it - it's easily checked with base_contains().
>
> --
> Denys
>
>
> > * Create a variable that specifies if a board or device has SGX support.
> > * This will determine which tasks are pulled in to various filesystem and SDK
> > images.
> >
> > Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
> > ---
> > meta-arago-distro/conf/distro/arago.conf | 9 ++++++---
> > 1 files changed, 6 insertions(+), 3 deletions(-)
> >
> > diff --git a/meta-arago-distro/conf/distro/arago.conf b/meta-arago-
> distro/conf/distro/arago.conf
> > index 8330115..691c5b8 100644
> > --- a/meta-arago-distro/conf/distro/arago.conf
> > +++ b/meta-arago-distro/conf/distro/arago.conf
> > @@ -45,9 +45,12 @@ PREFERRED_VERSION_hostap-daemon = "2.0-devel-ti"
> > PREFERRED_VERSION_omap3-sgx-modules = "4.08.00.01"
> > PREFERRED_VERSION_libgles-omap3 = "4.08.00.01"
> >
> > -ARAGO_QT_PROVIDER = "qt4-embedded"
> > -# Enable Qt+GLES for ARM v7a devices
> > -ARAGO_QT_PROVIDER_armv7a = "qt4-embedded-gles"
> > +SUPPORTS_SGX = "YES"
> > +
> > +# No omapl138 variant includes SGX
> > +SUPPORTS_SGX_omapl138 = "NO"
> > +
> > +ARAGO_QT_PROVIDER = "${@base_conditional('SUPPORTS_SGX', 'YES',
> 'qt4-embedded-gles', 'qt4-embedded', d)}"
> >
> > PREFERRED_PROVIDER_qt4-embedded = "${ARAGO_QT_PROVIDER}"
> >
> > --
> > 1.7.0.4
> >
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> >
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable
2013-02-12 0:46 ` [PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable fcooperjr27
2013-02-15 3:39 ` Denys Dmytriyenko
@ 2013-02-16 0:26 ` Paul Kovitz
2013-02-18 15:14 ` Cooper Jr., Franklin
1 sibling, 1 reply; 32+ messages in thread
From: Paul Kovitz @ 2013-02-16 0:26 UTC (permalink / raw)
To: fcooperjr27; +Cc: meta-arago, Franklin S. Cooper Jr
[-- Attachment #1: Type: text/plain, Size: 513 bytes --]
Please comment on coding SUPPORTS_SGX ?= "YES" instead of SUPPORTS_SGX =
"YES" For example, if you want to simulate an AM3352 device on an AM335x
eval board, you could write SUPPORTS_SGX = "NO" in local.conf. This change
requires a base_conditional checks for SUPPORTS_SGX in
task-arago-tisdk-matrix.bb and task-arago-tisdk-graphics-sdk-target.bb to
avoid other dependencies on OpenGL for the AM3352. I have a similar patch
in the code I am currently using for my own development.
Thanks for your help.
[-- Attachment #2: Type: text/html, Size: 829 bytes --]
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable
2013-02-16 0:26 ` Paul Kovitz
@ 2013-02-18 15:14 ` Cooper Jr., Franklin
2013-02-18 17:43 ` Denys Dmytriyenko
0 siblings, 1 reply; 32+ messages in thread
From: Cooper Jr., Franklin @ 2013-02-18 15:14 UTC (permalink / raw)
To: Paul Kovitz, fcooperjr27; +Cc: meta-arago
[-- Attachment #1: Type: text/plain, Size: 1168 bytes --]
Hi Paul,
In this case ?= and = would be equivalent. If your setting SUPPORTS_SGX in your local.conf then it will override the value in the arago.conf file.
I do see your point for the other two recipes. Currently it isn't friendly for people wanting to support am3352. I'll send updated patches. Thanks!
From: Paul Kovitz [mailto:paul.kovitz@welchallyn.com]
Sent: Friday, February 15, 2013 6:26 PM
To: fcooperjr27@gmail.com
Cc: meta-arago@arago-project.org; Cooper Jr., Franklin
Subject: Re: [meta-arago] [PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable
Please comment on coding SUPPORTS_SGX ?= "YES" instead of SUPPORTS_SGX = "YES" For example, if you want to simulate an AM3352 device on an AM335x eval board, you could write SUPPORTS_SGX = "NO" in local.conf. This change requires a base_conditional checks for SUPPORTS_SGX in task-arago-tisdk-matrix.bb<http://task-arago-tisdk-matrix.bb> and task-arago-tisdk-graphics-sdk-target.bb<http://task-arago-tisdk-graphics-sdk-target.bb> to avoid other dependencies on OpenGL for the AM3352. I have a similar patch in the code I am currently using for my own development.
Thanks for your help.
[-- Attachment #2: Type: text/html, Size: 4622 bytes --]
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable
2013-02-18 15:14 ` Cooper Jr., Franklin
@ 2013-02-18 17:43 ` Denys Dmytriyenko
2013-02-18 18:34 ` Cooper Jr., Franklin
0 siblings, 1 reply; 32+ messages in thread
From: Denys Dmytriyenko @ 2013-02-18 17:43 UTC (permalink / raw)
To: Cooper Jr., Franklin; +Cc: meta-arago
On Mon, Feb 18, 2013 at 03:14:17PM +0000, Cooper Jr., Franklin wrote:
> Hi Paul,
>
> In this case ?= and = would be equivalent. If your setting SUPPORTS_SGX in
> your local.conf then it will override the value in the arago.conf file.
Not really - distro conf always overrides local.conf. You'd need to use
_forcevariable override in local.conf to make it work:
SUPPORT_SGX_forcevariable = "blah"
But, anyway, I'd rather work on extending MACHINE_FEATURES with "sgx" instead.
--
Denys
> I do see your point for the other two recipes. Currently it isn't friendly
> for people wanting to support am3352. I'll send updated patches. Thanks!
>
> From: Paul Kovitz [mailto:paul.kovitz@welchallyn.com]
> Sent: Friday, February 15, 2013 6:26 PM
> To: fcooperjr27@gmail.com
> Cc: meta-arago@arago-project.org; Cooper Jr., Franklin
> Subject: Re: [meta-arago] [PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable
>
> Please comment on coding SUPPORTS_SGX ?= "YES" instead of SUPPORTS_SGX = "YES" For example, if you want to simulate an AM3352 device on an AM335x eval board, you could write SUPPORTS_SGX = "NO" in local.conf. This change requires a base_conditional checks for SUPPORTS_SGX in task-arago-tisdk-matrix.bb<http://task-arago-tisdk-matrix.bb> and task-arago-tisdk-graphics-sdk-target.bb<http://task-arago-tisdk-graphics-sdk-target.bb> to avoid other dependencies on OpenGL for the AM3352. I have a similar patch in the code I am currently using for my own development.
>
> Thanks for your help.
>
>
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable
2013-02-18 17:43 ` Denys Dmytriyenko
@ 2013-02-18 18:34 ` Cooper Jr., Franklin
0 siblings, 0 replies; 32+ messages in thread
From: Cooper Jr., Franklin @ 2013-02-18 18:34 UTC (permalink / raw)
To: Dmytriyenko, Denys; +Cc: meta-arago
> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Monday, February 18, 2013 11:43 AM
> To: Cooper Jr., Franklin
> Cc: Paul Kovitz; fcooperjr27@gmail.com; meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH 07/18] arago.conf: Add SUPPORTS_SGX
> variable
>
> On Mon, Feb 18, 2013 at 03:14:17PM +0000, Cooper Jr., Franklin wrote:
> > Hi Paul,
> >
> > In this case ?= and = would be equivalent. If your setting
> > SUPPORTS_SGX in your local.conf then it will override the value in the
> arago.conf file.
>
> Not really - distro conf always overrides local.conf. You'd need to use
> _forcevariable override in local.conf to make it work:
>
> SUPPORT_SGX_forcevariable = "blah"
>
> But, anyway, I'd rather work on extending MACHINE_FEATURES with "sgx"
> instead.
>
I guess I was wrong. I thought it was the other way around.
Adding sgx to MACHINE_FEATURES won't solve his problem since he isn't creating a new machine he is just tweaking the current am335x-evm machine.
So he can either use forcevariable or use something like oe_filter_out to remove sgx from the am335x-evm MACHINE_FEATURES.
> --
> Denys
>
>
> > I do see your point for the other two recipes. Currently it isn't friendly
> > for people wanting to support am3352. I'll send updated patches. Thanks!
> >
> > From: Paul Kovitz [mailto:paul.kovitz@welchallyn.com]
> > Sent: Friday, February 15, 2013 6:26 PM
> > To: fcooperjr27@gmail.com
> > Cc: meta-arago@arago-project.org; Cooper Jr., Franklin
> > Subject: Re: [meta-arago] [PATCH 07/18] arago.conf: Add SUPPORTS_SGX
> variable
> >
> > Please comment on coding SUPPORTS_SGX ?= "YES" instead of
> SUPPORTS_SGX = "YES" For example, if you want to simulate an AM3352 device
> on an AM335x eval board, you could write SUPPORTS_SGX = "NO" in local.conf.
> This change requires a base_conditional checks for SUPPORTS_SGX in task-
> arago-tisdk-matrix.bb<http://task-arago-tisdk-matrix.bb> and task-arago-tisdk-
> graphics-sdk-target.bb<http://task-arago-tisdk-graphics-sdk-target.bb> to avoid
> other dependencies on OpenGL for the AM3352. I have a similar patch in the
> code I am currently using for my own development.
> >
> > Thanks for your help.
> >
> >
>
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
^ permalink raw reply [flat|nested] 32+ messages in thread
* [PATCH 08/18] recipes: Only pull in SGX related tasks if SUPPORTS_SGX is set to YES
2013-02-12 0:46 [PATCH 01/18] linux-libc-headers: Add linux-libc-headers version 2.6.37.2 fcooperjr27
` (5 preceding siblings ...)
2013-02-12 0:46 ` [PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable fcooperjr27
@ 2013-02-12 0:46 ` fcooperjr27
2013-02-12 0:46 ` [PATCH 09/18] tisdk-image.bbclass: Create SECONDARY_BOOTLOADER_NAME variable fcooperjr27
` (9 subsequent siblings)
16 siblings, 0 replies; 32+ messages in thread
From: fcooperjr27 @ 2013-02-12 0:46 UTC (permalink / raw)
To: meta-arago; +Cc: Franklin S. Cooper Jr
From: Franklin S. Cooper Jr <fcooper@ti.com>
Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
---
.../recipes-core/images/tisdk-rootfs-image.bb | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/meta-arago-distro/recipes-core/images/tisdk-rootfs-image.bb b/meta-arago-distro/recipes-core/images/tisdk-rootfs-image.bb
index f5215d1..41254cd 100644
--- a/meta-arago-distro/recipes-core/images/tisdk-rootfs-image.bb
+++ b/meta-arago-distro/recipes-core/images/tisdk-rootfs-image.bb
@@ -5,6 +5,7 @@ require arago-base-tisdk-image.bb
IMAGE_INSTALL += "\
packagegroup-arago-test \
+ ${@base_conditional('SUPPORTS_SGX', 'YES','packagegroup-arago-tisdk-graphics', '',d)} \
packagegroup-arago-tisdk-graphics \
packagegroup-arago-tisdk-qte \
packagegroup-arago-tisdk-addons \
--
1.7.0.4
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 09/18] tisdk-image.bbclass: Create SECONDARY_BOOTLOADER_NAME variable
2013-02-12 0:46 [PATCH 01/18] linux-libc-headers: Add linux-libc-headers version 2.6.37.2 fcooperjr27
` (6 preceding siblings ...)
2013-02-12 0:46 ` [PATCH 08/18] recipes: Only pull in SGX related tasks if SUPPORTS_SGX is set to YES fcooperjr27
@ 2013-02-12 0:46 ` fcooperjr27
2013-02-12 0:46 ` [PATCH 10/18] arago-amsdk-image: Set SECONDARY_BOOTLOADER_NAME to empty string for omapl138 fcooperjr27
` (8 subsequent siblings)
16 siblings, 0 replies; 32+ messages in thread
From: fcooperjr27 @ 2013-02-12 0:46 UTC (permalink / raw)
To: meta-arago; +Cc: Franklin S. Cooper Jr
From: Franklin S. Cooper Jr <fcooper@ti.com>
* Am180x-evm secondary bootloader isn't built as part of OE.
* Tisdk-image.bbclass fails since no MLO is copied as part of the am180x-evm
SDK.
* Use a user defined variable to specify the secondary bootloader and allow the
copy step to be skipped if secondary bootloader variable is an empty string.
Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
---
meta-arago-distro/classes/tisdk-image.bbclass | 18 ++++++++++++------
1 files changed, 12 insertions(+), 6 deletions(-)
diff --git a/meta-arago-distro/classes/tisdk-image.bbclass b/meta-arago-distro/classes/tisdk-image.bbclass
index 4434155..adc6359 100644
--- a/meta-arago-distro/classes/tisdk-image.bbclass
+++ b/meta-arago-distro/classes/tisdk-image.bbclass
@@ -46,6 +46,9 @@ TARGET_IMAGE_TYPES ?= "tar.bz2 tar.gz ubi"
# the variable is in the BB_ENV_EXTRAWHITE list.
EXTRA_TISDK_FILES ?= ""
+# Variable to specify the name of the secondary bootloader
+SECONDARY_BOOTLOADER_NAME ?= "MLO-${MACHINE}"
+
# Manifest file location which will be created as part of the image build
# process.
# This manifest follows the TI manifest format requirements which is why
@@ -552,13 +555,16 @@ do_sdk_image () {
return 1
fi
- # Copy the MLO image if it exists
- if [ -e ${DEPLOY_DIR_IMAGE}/MLO-${MACHINE} ]
+ if [ "${SECONDARY_BOOTLOADER_NAME}" != "" ]
then
- cp ${DEPLOY_DIR_IMAGE}/MLO-${MACHINE} ${prebuilt_dir}/
- else
- echo "Could not find the MLO image"
- return 1
+ # Copy the secondary bootloader image if it exists
+ if [ -e ${DEPLOY_DIR_IMAGE}/${SECONDARY_BOOTLOADER_NAME} ]
+ then
+ cp ${DEPLOY_DIR_IMAGE}/${SECONDARY_BOOTLOADER_NAME} ${prebuilt_dir}/
+ else
+ echo "Could not find the secondary bootloader image"
+ return 1
+ fi
fi
# Add the EXTRA_TISDK_FILES contents if they exist
--
1.7.0.4
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 10/18] arago-amsdk-image: Set SECONDARY_BOOTLOADER_NAME to empty string for omapl138
2013-02-12 0:46 [PATCH 01/18] linux-libc-headers: Add linux-libc-headers version 2.6.37.2 fcooperjr27
` (7 preceding siblings ...)
2013-02-12 0:46 ` [PATCH 09/18] tisdk-image.bbclass: Create SECONDARY_BOOTLOADER_NAME variable fcooperjr27
@ 2013-02-12 0:46 ` fcooperjr27
2013-02-12 0:46 ` [PATCH 11/18] tisdk-amsdk-sdk-host: Add flash utility to SDK fcooperjr27
` (7 subsequent siblings)
16 siblings, 0 replies; 32+ messages in thread
From: fcooperjr27 @ 2013-02-12 0:46 UTC (permalink / raw)
To: meta-arago; +Cc: Franklin S. Cooper Jr
From: Franklin S. Cooper Jr <fcooper@ti.com>
* The secondary bootloader for omapl138 devices aren't built via oe.
* Set to an empty string so SDK image doesn't fail during the secondary
bootloader copy stage.
Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
---
.../recipes-core/images/arago-amsdk-image.bb | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/meta-arago-distro/recipes-core/images/arago-amsdk-image.bb b/meta-arago-distro/recipes-core/images/arago-amsdk-image.bb
index 4e4345d..08b460e 100644
--- a/meta-arago-distro/recipes-core/images/arago-amsdk-image.bb
+++ b/meta-arago-distro/recipes-core/images/arago-amsdk-image.bb
@@ -12,6 +12,8 @@ COMPATIBLE_MACHINE = "omap3|omapl138|ti33x"
HOST_CLEANUP_PACKAGES = ""
TOOLCHAIN_CLEANUP_PACKAGES = "libgnutls-dev libgnutls-extra26 libgnutls-openssl27 libtasn1-dev"
+SECONDARY_BOOTLOADER_NAME_omapl138 = ""
+
# List of target side images to build for the SDK
TARGET_IMAGES = "arago-base-tisdk-image tisdk-rootfs-image"
--
1.7.0.4
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 11/18] tisdk-amsdk-sdk-host: Add flash utility to SDK
2013-02-12 0:46 [PATCH 01/18] linux-libc-headers: Add linux-libc-headers version 2.6.37.2 fcooperjr27
` (8 preceding siblings ...)
2013-02-12 0:46 ` [PATCH 10/18] arago-amsdk-image: Set SECONDARY_BOOTLOADER_NAME to empty string for omapl138 fcooperjr27
@ 2013-02-12 0:46 ` fcooperjr27
2013-02-12 0:46 ` [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources fcooperjr27
` (6 subsequent siblings)
16 siblings, 0 replies; 32+ messages in thread
From: fcooperjr27 @ 2013-02-12 0:46 UTC (permalink / raw)
To: meta-arago; +Cc: Franklin S. Cooper Jr
From: Franklin S. Cooper Jr <fcooper@ti.com>
Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
---
.../packagegroup-arago-tisdk-amsdk-sdk-host.bb | 8 ++++++--
1 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb
index e216e8e..14626c8 100644
--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb
+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb
@@ -1,6 +1,6 @@
DESCRIPTION = "Task to install additional scripts and applications into the SDK"
LICENSE = "MIT"
-PR = "r2"
+PR = "r3"
inherit packagegroup
@@ -13,8 +13,12 @@ KERNEL_SRC = ""
U-BOOT_SRC_am335x-evm = "u-boot-src"
KERNEL_SRC_am335x-evm = "linux-am335x-src"
+TOOLS = "pinmux-utility"
+TOOLS_am37x-evm_append = " flash-utility"
+TOOLS_am3517-evm_append = " flash-utility"
+
RDEPENDS_${PN} = "\
- pinmux-utility \
+ ${TOOLS} \
ti-tisdk-setup \
board-port-labs-src \
ti-tisdk-makefile \
--
1.7.0.4
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources
2013-02-12 0:46 [PATCH 01/18] linux-libc-headers: Add linux-libc-headers version 2.6.37.2 fcooperjr27
` (9 preceding siblings ...)
2013-02-12 0:46 ` [PATCH 11/18] tisdk-amsdk-sdk-host: Add flash utility to SDK fcooperjr27
@ 2013-02-12 0:46 ` fcooperjr27
2013-02-15 3:42 ` Denys Dmytriyenko
2013-02-12 0:46 ` [PATCH 13/18] board-port-labs: Inherit allarch class fcooperjr27
` (5 subsequent siblings)
16 siblings, 1 reply; 32+ messages in thread
From: fcooperjr27 @ 2013-02-12 0:46 UTC (permalink / raw)
To: meta-arago; +Cc: Franklin S. Cooper Jr
From: Franklin S. Cooper Jr <fcooper@ti.com>
* Use generic name for kernel sources which will allow for any machine to be
used without modification.
Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
---
.../packagegroup-arago-tisdk-amsdk-sdk-host.bb | 8 +++-----
1 files changed, 3 insertions(+), 5 deletions(-)
diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb
index 14626c8..b990235 100644
--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb
+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb
@@ -1,17 +1,15 @@
DESCRIPTION = "Task to install additional scripts and applications into the SDK"
LICENSE = "MIT"
-PR = "r3"
+PR = "r4"
inherit packagegroup
PACKAGE_ARCH = "${MACHINE_ARCH}"
# Choose the kernel and u-boot recipe sources to use
-U-BOOT_SRC = ""
-KERNEL_SRC = ""
-U-BOOT_SRC_am335x-evm = "u-boot-src"
-KERNEL_SRC_am335x-evm = "linux-am335x-src"
+U-BOOT_SRC = "u-boot-src"
+KERNEL_SRC := "${PREFERRED_PROVIDER_virtual/kernel}-src"
TOOLS = "pinmux-utility"
TOOLS_am37x-evm_append = " flash-utility"
--
1.7.0.4
^ permalink raw reply related [flat|nested] 32+ messages in thread
* Re: [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources
2013-02-12 0:46 ` [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources fcooperjr27
@ 2013-02-15 3:42 ` Denys Dmytriyenko
2013-02-15 3:54 ` Cooper Jr., Franklin
0 siblings, 1 reply; 32+ messages in thread
From: Denys Dmytriyenko @ 2013-02-15 3:42 UTC (permalink / raw)
To: fcooperjr27; +Cc: meta-arago, Franklin S. Cooper Jr
On Mon, Feb 11, 2013 at 06:46:13PM -0600, fcooperjr27@gmail.com wrote:
> From: Franklin S. Cooper Jr <fcooper@ti.com>
>
> * Use generic name for kernel sources which will allow for any machine to be
> used without modification.
>
> Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
> ---
> .../packagegroup-arago-tisdk-amsdk-sdk-host.bb | 8 +++-----
> 1 files changed, 3 insertions(+), 5 deletions(-)
>
> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb
> index 14626c8..b990235 100644
> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb
> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb
> @@ -1,17 +1,15 @@
> DESCRIPTION = "Task to install additional scripts and applications into the SDK"
> LICENSE = "MIT"
> -PR = "r3"
> +PR = "r4"
>
> inherit packagegroup
>
> PACKAGE_ARCH = "${MACHINE_ARCH}"
>
> # Choose the kernel and u-boot recipe sources to use
> -U-BOOT_SRC = ""
> -KERNEL_SRC = ""
>
> -U-BOOT_SRC_am335x-evm = "u-boot-src"
> -KERNEL_SRC_am335x-evm = "linux-am335x-src"
> +U-BOOT_SRC = "u-boot-src"
> +KERNEL_SRC := "${PREFERRED_PROVIDER_virtual/kernel}-src"
1. Do you need to use := assignment here?
2. Should we also consider using existing PREFERRED_PROVIDER_virtual/bootloader
for u-boot sources here as well?
--
Denys
> TOOLS = "pinmux-utility"
> TOOLS_am37x-evm_append = " flash-utility"
> --
> 1.7.0.4
>
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources
2013-02-15 3:42 ` Denys Dmytriyenko
@ 2013-02-15 3:54 ` Cooper Jr., Franklin
2013-02-17 20:35 ` Denys Dmytriyenko
0 siblings, 1 reply; 32+ messages in thread
From: Cooper Jr., Franklin @ 2013-02-15 3:54 UTC (permalink / raw)
To: Dmytriyenko, Denys, fcooperjr27; +Cc: meta-arago
> -----Original Message-----
> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> bounces@arago-project.org] On Behalf Of Dmytriyenko, Denys
> Sent: Thursday, February 14, 2013 9:42 PM
> To: fcooperjr27@gmail.com
> Cc: meta-arago@arago-project.org; Cooper Jr., Franklin
> Subject: Re: [meta-arago] [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use
> generic name for kernel sources
>
> On Mon, Feb 11, 2013 at 06:46:13PM -0600, fcooperjr27@gmail.com wrote:
> > From: Franklin S. Cooper Jr <fcooper@ti.com>
> >
> > * Use generic name for kernel sources which will allow for any machine to be
> > used without modification.
> >
> > Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
> > ---
> > .../packagegroup-arago-tisdk-amsdk-sdk-host.bb | 8 +++-----
> > 1 files changed, 3 insertions(+), 5 deletions(-)
> >
> > diff --git
> > a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisd
> > k-amsdk-sdk-host.bb
> > b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisd
> > k-amsdk-sdk-host.bb
> > index 14626c8..b990235 100644
> > ---
> > a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisd
> > k-amsdk-sdk-host.bb
> > +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
> > +++ tisdk-amsdk-sdk-host.bb
> > @@ -1,17 +1,15 @@
> > DESCRIPTION = "Task to install additional scripts and applications into the
> SDK"
> > LICENSE = "MIT"
> > -PR = "r3"
> > +PR = "r4"
> >
> > inherit packagegroup
> >
> > PACKAGE_ARCH = "${MACHINE_ARCH}"
> >
> > # Choose the kernel and u-boot recipe sources to use -U-BOOT_SRC = ""
> > -KERNEL_SRC = ""
> >
> > -U-BOOT_SRC_am335x-evm = "u-boot-src"
> > -KERNEL_SRC_am335x-evm = "linux-am335x-src"
> > +U-BOOT_SRC = "u-boot-src"
> > +KERNEL_SRC := "${PREFERRED_PROVIDER_virtual/kernel}-src"
>
> 1. Do you need to use := assignment here?
No. I'll fix that.
> 2. Should we also consider using existing
> PREFERRED_PROVIDER_virtual/bootloader
> for u-boot sources here as well?
PREFERRED_PROVIDER_virtual/bootloader is a big annoyance that I struggled with. The problem is in meta-arago we define PREFERRED_PROVIDER_virtual/kernel for all platforms so it works well for the kernel.
Some platforms don't define that variable so it will fail for those platforms. All of our platforms use u-boot as the ${PN} so it was safe to make the assumption.
>
> --
> Denys
>
> > TOOLS = "pinmux-utility"
> > TOOLS_am37x-evm_append = " flash-utility"
> > --
> > 1.7.0.4
> >
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> >
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources
2013-02-15 3:54 ` Cooper Jr., Franklin
@ 2013-02-17 20:35 ` Denys Dmytriyenko
2013-02-17 21:07 ` Cooper Jr., Franklin
0 siblings, 1 reply; 32+ messages in thread
From: Denys Dmytriyenko @ 2013-02-17 20:35 UTC (permalink / raw)
To: Cooper Jr., Franklin; +Cc: meta-arago
On Fri, Feb 15, 2013 at 03:54:29AM +0000, Cooper Jr., Franklin wrote:
>
>
> > -----Original Message-----
> > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > bounces@arago-project.org] On Behalf Of Dmytriyenko, Denys
> > Sent: Thursday, February 14, 2013 9:42 PM
> > To: fcooperjr27@gmail.com
> > Cc: meta-arago@arago-project.org; Cooper Jr., Franklin
> > Subject: Re: [meta-arago] [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use
> > generic name for kernel sources
> >
> > On Mon, Feb 11, 2013 at 06:46:13PM -0600, fcooperjr27@gmail.com wrote:
> > > From: Franklin S. Cooper Jr <fcooper@ti.com>
> > >
> > > * Use generic name for kernel sources which will allow for any machine to be
> > > used without modification.
> > >
> > > Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
> > > ---
> > > .../packagegroup-arago-tisdk-amsdk-sdk-host.bb | 8 +++-----
> > > 1 files changed, 3 insertions(+), 5 deletions(-)
> > >
> > > diff --git
> > > a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisd
> > > k-amsdk-sdk-host.bb
> > > b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisd
> > > k-amsdk-sdk-host.bb
> > > index 14626c8..b990235 100644
> > > ---
> > > a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisd
> > > k-amsdk-sdk-host.bb
> > > +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
> > > +++ tisdk-amsdk-sdk-host.bb
> > > @@ -1,17 +1,15 @@
> > > DESCRIPTION = "Task to install additional scripts and applications into the
> > SDK"
> > > LICENSE = "MIT"
> > > -PR = "r3"
> > > +PR = "r4"
> > >
> > > inherit packagegroup
> > >
> > > PACKAGE_ARCH = "${MACHINE_ARCH}"
> > >
> > > # Choose the kernel and u-boot recipe sources to use -U-BOOT_SRC = ""
> > > -KERNEL_SRC = ""
> > >
> > > -U-BOOT_SRC_am335x-evm = "u-boot-src"
> > > -KERNEL_SRC_am335x-evm = "linux-am335x-src"
> > > +U-BOOT_SRC = "u-boot-src"
> > > +KERNEL_SRC := "${PREFERRED_PROVIDER_virtual/kernel}-src"
> >
> > 1. Do you need to use := assignment here?
> No. I'll fix that.
> > 2. Should we also consider using existing
> > PREFERRED_PROVIDER_virtual/bootloader
> > for u-boot sources here as well?
> PREFERRED_PROVIDER_virtual/bootloader is a big annoyance that I struggled
> with. The problem is in meta-arago we define
> PREFERRED_PROVIDER_virtual/kernel for all platforms so it works well for the
> kernel. Some platforms don't define that variable so it will fail for those
> platforms. All of our platforms use u-boot as the ${PN} so it was safe to
> make the assumption.
I thought all our platforms in meta-ti had virtual/bootloader set - which ones
are missing?
On the other hand, I guess we could just drop it altogether...
--
Denys
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources
2013-02-17 20:35 ` Denys Dmytriyenko
@ 2013-02-17 21:07 ` Cooper Jr., Franklin
2013-02-17 21:29 ` Maupin, Chase
0 siblings, 1 reply; 32+ messages in thread
From: Cooper Jr., Franklin @ 2013-02-17 21:07 UTC (permalink / raw)
To: Dmytriyenko, Denys; +Cc: meta-arago
> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Sunday, February 17, 2013 2:35 PM
> To: Cooper Jr., Franklin
> Cc: Dmytriyenko, Denys; fcooperjr27@gmail.com; meta-arago@arago-
> project.org
> Subject: Re: [meta-arago] [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use
> generic name for kernel sources
>
> On Fri, Feb 15, 2013 at 03:54:29AM +0000, Cooper Jr., Franklin wrote:
> >
> >
> > > -----Original Message-----
> > > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > > bounces@arago-project.org] On Behalf Of Dmytriyenko, Denys
> > > Sent: Thursday, February 14, 2013 9:42 PM
> > > To: fcooperjr27@gmail.com
> > > Cc: meta-arago@arago-project.org; Cooper Jr., Franklin
> > > Subject: Re: [meta-arago] [PATCH 12/18] arago-tisdk-amsdk-sdk-host:
> > > Use generic name for kernel sources
> > >
> > > On Mon, Feb 11, 2013 at 06:46:13PM -0600, fcooperjr27@gmail.com
> wrote:
> > > > From: Franklin S. Cooper Jr <fcooper@ti.com>
> > > >
> > > > * Use generic name for kernel sources which will allow for any machine
> to be
> > > > used without modification.
> > > >
> > > > Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
> > > > ---
> > > > .../packagegroup-arago-tisdk-amsdk-sdk-host.bb | 8 +++-----
> > > > 1 files changed, 3 insertions(+), 5 deletions(-)
> > > >
> > > > diff --git
> > > > a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
> > > > tisd
> > > > k-amsdk-sdk-host.bb
> > > > b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
> > > > tisd
> > > > k-amsdk-sdk-host.bb
> > > > index 14626c8..b990235 100644
> > > > ---
> > > > a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
> > > > tisd
> > > > k-amsdk-sdk-host.bb
> > > > +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-ar
> > > > +++ ago-
> > > > +++ tisdk-amsdk-sdk-host.bb
> > > > @@ -1,17 +1,15 @@
> > > > DESCRIPTION = "Task to install additional scripts and
> > > > applications into the
> > > SDK"
> > > > LICENSE = "MIT"
> > > > -PR = "r3"
> > > > +PR = "r4"
> > > >
> > > > inherit packagegroup
> > > >
> > > > PACKAGE_ARCH = "${MACHINE_ARCH}"
> > > >
> > > > # Choose the kernel and u-boot recipe sources to use -U-BOOT_SRC = ""
> > > > -KERNEL_SRC = ""
> > > >
> > > > -U-BOOT_SRC_am335x-evm = "u-boot-src"
> > > > -KERNEL_SRC_am335x-evm = "linux-am335x-src"
> > > > +U-BOOT_SRC = "u-boot-src"
> > > > +KERNEL_SRC := "${PREFERRED_PROVIDER_virtual/kernel}-src"
> > >
> > > 1. Do you need to use := assignment here?
> > No. I'll fix that.
> > > 2. Should we also consider using existing
> > > PREFERRED_PROVIDER_virtual/bootloader
> > > for u-boot sources here as well?
> > PREFERRED_PROVIDER_virtual/bootloader is a big annoyance that I
> > struggled with. The problem is in meta-arago we define
> > PREFERRED_PROVIDER_virtual/kernel for all platforms so it works well
> > for the kernel. Some platforms don't define that variable so it will
> > fail for those platforms. All of our platforms use u-boot as the ${PN}
> > so it was safe to make the assumption.
>
> I thought all our platforms in meta-ti had virtual/bootloader set - which ones
> are missing?
> On the other hand, I guess we could just drop it altogether...
Machines missing virtual/bootloader being set in machine config file:
Beagleboard <-- the one that caused me problems.
Beaglebone
Hawkboard
Keystone
Omap3-touchbook
I can add virtual/bootloader to all the missing machines in meta-ti and update this recipe to use virtual/bootloader or we can use this recipe as is minus the := change.
I would vote for adding virtual/bootloader to all machines in meta-ti since it will future proof this recipe a bit more if someone later decides to use a bootloader recipe whose ${PN} isn't u-boot.
>
> --
> Denys
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources
2013-02-17 21:07 ` Cooper Jr., Franklin
@ 2013-02-17 21:29 ` Maupin, Chase
2013-02-17 21:35 ` Denys Dmytriyenko
0 siblings, 1 reply; 32+ messages in thread
From: Maupin, Chase @ 2013-02-17 21:29 UTC (permalink / raw)
To: Cooper Jr., Franklin; +Cc: meta-arago
+1
Sincerely,
Chase Maupin
On Feb 17, 2013, at 3:07 PM, "Cooper Jr., Franklin" <fcooper@ti.com> wrote:
>
>
>> -----Original Message-----
>> From: Dmytriyenko, Denys
>> Sent: Sunday, February 17, 2013 2:35 PM
>> To: Cooper Jr., Franklin
>> Cc: Dmytriyenko, Denys; fcooperjr27@gmail.com; meta-arago@arago-
>> project.org
>> Subject: Re: [meta-arago] [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use
>> generic name for kernel sources
>>
>> On Fri, Feb 15, 2013 at 03:54:29AM +0000, Cooper Jr., Franklin wrote:
>>>
>>>
>>>> -----Original Message-----
>>>> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
>>>> bounces@arago-project.org] On Behalf Of Dmytriyenko, Denys
>>>> Sent: Thursday, February 14, 2013 9:42 PM
>>>> To: fcooperjr27@gmail.com
>>>> Cc: meta-arago@arago-project.org; Cooper Jr., Franklin
>>>> Subject: Re: [meta-arago] [PATCH 12/18] arago-tisdk-amsdk-sdk-host:
>>>> Use generic name for kernel sources
>>>>
>>>> On Mon, Feb 11, 2013 at 06:46:13PM -0600, fcooperjr27@gmail.com
>> wrote:
>>>>> From: Franklin S. Cooper Jr <fcooper@ti.com>
>>>>>
>>>>> * Use generic name for kernel sources which will allow for any machine
>> to be
>>>>> used without modification.
>>>>>
>>>>> Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
>>>>> ---
>>>>> .../packagegroup-arago-tisdk-amsdk-sdk-host.bb | 8 +++-----
>>>>> 1 files changed, 3 insertions(+), 5 deletions(-)
>>>>>
>>>>> diff --git
>>>>> a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
>>>>> tisd
>>>>> k-amsdk-sdk-host.bb
>>>>> b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
>>>>> tisd
>>>>> k-amsdk-sdk-host.bb
>>>>> index 14626c8..b990235 100644
>>>>> ---
>>>>> a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
>>>>> tisd
>>>>> k-amsdk-sdk-host.bb
>>>>> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-ar
>>>>> +++ ago-
>>>>> +++ tisdk-amsdk-sdk-host.bb
>>>>> @@ -1,17 +1,15 @@
>>>>> DESCRIPTION = "Task to install additional scripts and
>>>>> applications into the
>>>> SDK"
>>>>> LICENSE = "MIT"
>>>>> -PR = "r3"
>>>>> +PR = "r4"
>>>>>
>>>>> inherit packagegroup
>>>>>
>>>>> PACKAGE_ARCH = "${MACHINE_ARCH}"
>>>>>
>>>>> # Choose the kernel and u-boot recipe sources to use -U-BOOT_SRC = ""
>>>>> -KERNEL_SRC = ""
>>>>>
>>>>> -U-BOOT_SRC_am335x-evm = "u-boot-src"
>>>>> -KERNEL_SRC_am335x-evm = "linux-am335x-src"
>>>>> +U-BOOT_SRC = "u-boot-src"
>>>>> +KERNEL_SRC := "${PREFERRED_PROVIDER_virtual/kernel}-src"
>>>>
>>>> 1. Do you need to use := assignment here?
>>> No. I'll fix that.
>>>> 2. Should we also consider using existing
>>>> PREFERRED_PROVIDER_virtual/bootloader
>>>> for u-boot sources here as well?
>>> PREFERRED_PROVIDER_virtual/bootloader is a big annoyance that I
>>> struggled with. The problem is in meta-arago we define
>>> PREFERRED_PROVIDER_virtual/kernel for all platforms so it works well
>>> for the kernel. Some platforms don't define that variable so it will
>>> fail for those platforms. All of our platforms use u-boot as the ${PN}
>>> so it was safe to make the assumption.
>>
>> I thought all our platforms in meta-ti had virtual/bootloader set - which ones
>> are missing?
>> On the other hand, I guess we could just drop it altogether...
>
> Machines missing virtual/bootloader being set in machine config file:
> Beagleboard <-- the one that caused me problems.
> Beaglebone
> Hawkboard
> Keystone
> Omap3-touchbook
>
> I can add virtual/bootloader to all the missing machines in meta-ti and update this recipe to use virtual/bootloader or we can use this recipe as is minus the := change.
>
> I would vote for adding virtual/bootloader to all machines in meta-ti since it will future proof this recipe a bit more if someone later decides to use a bootloader recipe whose ${PN} isn't u-boot.
>>
>> --
>> Denys
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources
2013-02-17 21:29 ` Maupin, Chase
@ 2013-02-17 21:35 ` Denys Dmytriyenko
2013-02-17 21:46 ` Maupin, Chase
0 siblings, 1 reply; 32+ messages in thread
From: Denys Dmytriyenko @ 2013-02-17 21:35 UTC (permalink / raw)
To: Maupin, Chase; +Cc: meta-arago, Cooper Jr., Franklin
On Sun, Feb 17, 2013 at 09:29:55PM +0000, Maupin, Chase wrote:
> +1
Is that +1 for using this patch as is with := fixed? :)
I'm going to fix the remaining machines for now, since I'm already working on
meta-ti today. BTW, some of those can be removed soon, like hawkboard and
omap3-touchbook, as we don't support them at all.
--
Denys
> On Feb 17, 2013, at 3:07 PM, "Cooper Jr., Franklin" <fcooper@ti.com> wrote:
>
> >
> >
> >> -----Original Message-----
> >> From: Dmytriyenko, Denys
> >> Sent: Sunday, February 17, 2013 2:35 PM
> >> To: Cooper Jr., Franklin
> >> Cc: Dmytriyenko, Denys; fcooperjr27@gmail.com; meta-arago@arago-
> >> project.org
> >> Subject: Re: [meta-arago] [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use
> >> generic name for kernel sources
> >>
> >> On Fri, Feb 15, 2013 at 03:54:29AM +0000, Cooper Jr., Franklin wrote:
> >>>
> >>>
> >>>> -----Original Message-----
> >>>> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> >>>> bounces@arago-project.org] On Behalf Of Dmytriyenko, Denys
> >>>> Sent: Thursday, February 14, 2013 9:42 PM
> >>>> To: fcooperjr27@gmail.com
> >>>> Cc: meta-arago@arago-project.org; Cooper Jr., Franklin
> >>>> Subject: Re: [meta-arago] [PATCH 12/18] arago-tisdk-amsdk-sdk-host:
> >>>> Use generic name for kernel sources
> >>>>
> >>>> On Mon, Feb 11, 2013 at 06:46:13PM -0600, fcooperjr27@gmail.com
> >> wrote:
> >>>>> From: Franklin S. Cooper Jr <fcooper@ti.com>
> >>>>>
> >>>>> * Use generic name for kernel sources which will allow for any machine
> >> to be
> >>>>> used without modification.
> >>>>>
> >>>>> Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
> >>>>> ---
> >>>>> .../packagegroup-arago-tisdk-amsdk-sdk-host.bb | 8 +++-----
> >>>>> 1 files changed, 3 insertions(+), 5 deletions(-)
> >>>>>
> >>>>> diff --git
> >>>>> a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
> >>>>> tisd
> >>>>> k-amsdk-sdk-host.bb
> >>>>> b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
> >>>>> tisd
> >>>>> k-amsdk-sdk-host.bb
> >>>>> index 14626c8..b990235 100644
> >>>>> ---
> >>>>> a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
> >>>>> tisd
> >>>>> k-amsdk-sdk-host.bb
> >>>>> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-ar
> >>>>> +++ ago-
> >>>>> +++ tisdk-amsdk-sdk-host.bb
> >>>>> @@ -1,17 +1,15 @@
> >>>>> DESCRIPTION = "Task to install additional scripts and
> >>>>> applications into the
> >>>> SDK"
> >>>>> LICENSE = "MIT"
> >>>>> -PR = "r3"
> >>>>> +PR = "r4"
> >>>>>
> >>>>> inherit packagegroup
> >>>>>
> >>>>> PACKAGE_ARCH = "${MACHINE_ARCH}"
> >>>>>
> >>>>> # Choose the kernel and u-boot recipe sources to use -U-BOOT_SRC = ""
> >>>>> -KERNEL_SRC = ""
> >>>>>
> >>>>> -U-BOOT_SRC_am335x-evm = "u-boot-src"
> >>>>> -KERNEL_SRC_am335x-evm = "linux-am335x-src"
> >>>>> +U-BOOT_SRC = "u-boot-src"
> >>>>> +KERNEL_SRC := "${PREFERRED_PROVIDER_virtual/kernel}-src"
> >>>>
> >>>> 1. Do you need to use := assignment here?
> >>> No. I'll fix that.
> >>>> 2. Should we also consider using existing
> >>>> PREFERRED_PROVIDER_virtual/bootloader
> >>>> for u-boot sources here as well?
> >>> PREFERRED_PROVIDER_virtual/bootloader is a big annoyance that I
> >>> struggled with. The problem is in meta-arago we define
> >>> PREFERRED_PROVIDER_virtual/kernel for all platforms so it works well
> >>> for the kernel. Some platforms don't define that variable so it will
> >>> fail for those platforms. All of our platforms use u-boot as the ${PN}
> >>> so it was safe to make the assumption.
> >>
> >> I thought all our platforms in meta-ti had virtual/bootloader set - which ones
> >> are missing?
> >> On the other hand, I guess we could just drop it altogether...
> >
> > Machines missing virtual/bootloader being set in machine config file:
> > Beagleboard <-- the one that caused me problems.
> > Beaglebone
> > Hawkboard
> > Keystone
> > Omap3-touchbook
> >
> > I can add virtual/bootloader to all the missing machines in meta-ti and update this recipe to use virtual/bootloader or we can use this recipe as is minus the := change.
> >
> > I would vote for adding virtual/bootloader to all machines in meta-ti since it will future proof this recipe a bit more if someone later decides to use a bootloader recipe whose ${PN} isn't u-boot.
> >>
> >> --
> >> Denys
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources
2013-02-17 21:35 ` Denys Dmytriyenko
@ 2013-02-17 21:46 ` Maupin, Chase
0 siblings, 0 replies; 32+ messages in thread
From: Maupin, Chase @ 2013-02-17 21:46 UTC (permalink / raw)
To: Dmytriyenko, Denys; +Cc: meta-arago, Cooper Jr., Franklin
I meant to his comment of fixing the machines and making the solution more robust for the future :)
Sincerely,
Chase Maupin
On Feb 17, 2013, at 3:36 PM, "Dmytriyenko, Denys" <denys@ti.com> wrote:
> On Sun, Feb 17, 2013 at 09:29:55PM +0000, Maupin, Chase wrote:
>> +1
>
> Is that +1 for using this patch as is with := fixed? :)
>
> I'm going to fix the remaining machines for now, since I'm already working on
> meta-ti today. BTW, some of those can be removed soon, like hawkboard and
> omap3-touchbook, as we don't support them at all.
>
> --
> Denys
>
>
>> On Feb 17, 2013, at 3:07 PM, "Cooper Jr., Franklin" <fcooper@ti.com> wrote:
>>
>>>
>>>
>>>> -----Original Message-----
>>>> From: Dmytriyenko, Denys
>>>> Sent: Sunday, February 17, 2013 2:35 PM
>>>> To: Cooper Jr., Franklin
>>>> Cc: Dmytriyenko, Denys; fcooperjr27@gmail.com; meta-arago@arago-
>>>> project.org
>>>> Subject: Re: [meta-arago] [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use
>>>> generic name for kernel sources
>>>>
>>>> On Fri, Feb 15, 2013 at 03:54:29AM +0000, Cooper Jr., Franklin wrote:
>>>>>
>>>>>
>>>>>> -----Original Message-----
>>>>>> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
>>>>>> bounces@arago-project.org] On Behalf Of Dmytriyenko, Denys
>>>>>> Sent: Thursday, February 14, 2013 9:42 PM
>>>>>> To: fcooperjr27@gmail.com
>>>>>> Cc: meta-arago@arago-project.org; Cooper Jr., Franklin
>>>>>> Subject: Re: [meta-arago] [PATCH 12/18] arago-tisdk-amsdk-sdk-host:
>>>>>> Use generic name for kernel sources
>>>>>>
>>>>>> On Mon, Feb 11, 2013 at 06:46:13PM -0600, fcooperjr27@gmail.com
>>>> wrote:
>>>>>>> From: Franklin S. Cooper Jr <fcooper@ti.com>
>>>>>>>
>>>>>>> * Use generic name for kernel sources which will allow for any machine
>>>> to be
>>>>>>> used without modification.
>>>>>>>
>>>>>>> Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
>>>>>>> ---
>>>>>>> .../packagegroup-arago-tisdk-amsdk-sdk-host.bb | 8 +++-----
>>>>>>> 1 files changed, 3 insertions(+), 5 deletions(-)
>>>>>>>
>>>>>>> diff --git
>>>>>>> a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
>>>>>>> tisd
>>>>>>> k-amsdk-sdk-host.bb
>>>>>>> b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
>>>>>>> tisd
>>>>>>> k-amsdk-sdk-host.bb
>>>>>>> index 14626c8..b990235 100644
>>>>>>> ---
>>>>>>> a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-
>>>>>>> tisd
>>>>>>> k-amsdk-sdk-host.bb
>>>>>>> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-ar
>>>>>>> +++ ago-
>>>>>>> +++ tisdk-amsdk-sdk-host.bb
>>>>>>> @@ -1,17 +1,15 @@
>>>>>>> DESCRIPTION = "Task to install additional scripts and
>>>>>>> applications into the
>>>>>> SDK"
>>>>>>> LICENSE = "MIT"
>>>>>>> -PR = "r3"
>>>>>>> +PR = "r4"
>>>>>>>
>>>>>>> inherit packagegroup
>>>>>>>
>>>>>>> PACKAGE_ARCH = "${MACHINE_ARCH}"
>>>>>>>
>>>>>>> # Choose the kernel and u-boot recipe sources to use -U-BOOT_SRC = ""
>>>>>>> -KERNEL_SRC = ""
>>>>>>>
>>>>>>> -U-BOOT_SRC_am335x-evm = "u-boot-src"
>>>>>>> -KERNEL_SRC_am335x-evm = "linux-am335x-src"
>>>>>>> +U-BOOT_SRC = "u-boot-src"
>>>>>>> +KERNEL_SRC := "${PREFERRED_PROVIDER_virtual/kernel}-src"
>>>>>>
>>>>>> 1. Do you need to use := assignment here?
>>>>> No. I'll fix that.
>>>>>> 2. Should we also consider using existing
>>>>>> PREFERRED_PROVIDER_virtual/bootloader
>>>>>> for u-boot sources here as well?
>>>>> PREFERRED_PROVIDER_virtual/bootloader is a big annoyance that I
>>>>> struggled with. The problem is in meta-arago we define
>>>>> PREFERRED_PROVIDER_virtual/kernel for all platforms so it works well
>>>>> for the kernel. Some platforms don't define that variable so it will
>>>>> fail for those platforms. All of our platforms use u-boot as the ${PN}
>>>>> so it was safe to make the assumption.
>>>>
>>>> I thought all our platforms in meta-ti had virtual/bootloader set - which ones
>>>> are missing?
>>>> On the other hand, I guess we could just drop it altogether...
>>>
>>> Machines missing virtual/bootloader being set in machine config file:
>>> Beagleboard <-- the one that caused me problems.
>>> Beaglebone
>>> Hawkboard
>>> Keystone
>>> Omap3-touchbook
>>>
>>> I can add virtual/bootloader to all the missing machines in meta-ti and update this recipe to use virtual/bootloader or we can use this recipe as is minus the := change.
>>>
>>> I would vote for adding virtual/bootloader to all machines in meta-ti since it will future proof this recipe a bit more if someone later decides to use a bootloader recipe whose ${PN} isn't u-boot.
>>>>
>>>> --
>>>> Denys
>>> _______________________________________________
>>> meta-arago mailing list
>>> meta-arago@arago-project.org
>>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>> _______________________________________________
>> meta-arago mailing list
>> meta-arago@arago-project.org
>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>>
^ permalink raw reply [flat|nested] 32+ messages in thread
* [PATCH 13/18] board-port-labs: Inherit allarch class.
2013-02-12 0:46 [PATCH 01/18] linux-libc-headers: Add linux-libc-headers version 2.6.37.2 fcooperjr27
` (10 preceding siblings ...)
2013-02-12 0:46 ` [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources fcooperjr27
@ 2013-02-12 0:46 ` fcooperjr27
2013-02-12 0:46 ` [PATCH 14/18] meta-toolchain-arago: Add cleanup_toolchain_packages fcooperjr27
` (4 subsequent siblings)
16 siblings, 0 replies; 32+ messages in thread
From: fcooperjr27 @ 2013-02-12 0:46 UTC (permalink / raw)
To: meta-arago; +Cc: Franklin S. Cooper Jr
From: Franklin S. Cooper Jr <fcooper@ti.com>
* Inherit allarch class which sets PACKAGE_ARCHS = "all" and some other things.
* This fixes a problem where board-port-labs was being rebuilt for every machine
Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
---
.../board-port-labs/board-port-labs_git.bb | 5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/meta-arago-distro/recipes-tisdk/board-port-labs/board-port-labs_git.bb b/meta-arago-distro/recipes-tisdk/board-port-labs/board-port-labs_git.bb
index 2c75c65..87a6735 100644
--- a/meta-arago-distro/recipes-tisdk/board-port-labs/board-port-labs_git.bb
+++ b/meta-arago-distro/recipes-tisdk/board-port-labs/board-port-labs_git.bb
@@ -6,11 +6,12 @@ DEPENDS += "git-native"
SRC_URI = "file://LICENSE"
LIC_FILES_CHKSUM = "file://LICENSE;md5=d9fb2a547cd8e6bb75c0ecf427a139a9"
-PACKAGE_ARCH = "all"
+inherit allarch
+
CREATE_SRCIPK = "1"
SRCIPK_INSTALL_DIR = "board-support/board-port-labs"
-PR = "r2"
+PR = "r3"
REPOS ?= "git://gitorious.org/sitara-board-port/sitara-board-port-linux.git \
git://gitorious.org/sitara-board-port/sitara-board-port-uboot.git \
--
1.7.0.4
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 14/18] meta-toolchain-arago: Add cleanup_toolchain_packages
2013-02-12 0:46 [PATCH 01/18] linux-libc-headers: Add linux-libc-headers version 2.6.37.2 fcooperjr27
` (11 preceding siblings ...)
2013-02-12 0:46 ` [PATCH 13/18] board-port-labs: Inherit allarch class fcooperjr27
@ 2013-02-12 0:46 ` fcooperjr27
2013-02-12 0:46 ` [PATCH 15/18] meta-toolchain-arago-tisdk: Set TOOLCHAIN_CLEANUP_PACKAGES fcooperjr27
` (3 subsequent siblings)
16 siblings, 0 replies; 32+ messages in thread
From: fcooperjr27 @ 2013-02-12 0:46 UTC (permalink / raw)
To: meta-arago; +Cc: Franklin S. Cooper Jr
From: Franklin S. Cooper Jr <fcooper@ti.com>
* Move cleanup_toolchain_packages from tisdk-image.bbclass and add it to
meta-toolchain-arago
Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
---
.../recipes-core/meta/meta-toolchain-arago.bb | 26 +++++++++++++++++++-
1 files changed, 25 insertions(+), 1 deletions(-)
diff --git a/meta-arago-extras/recipes-core/meta/meta-toolchain-arago.bb b/meta-arago-extras/recipes-core/meta/meta-toolchain-arago.bb
index f32da6a..b75583b 100644
--- a/meta-arago-extras/recipes-core/meta/meta-toolchain-arago.bb
+++ b/meta-arago-extras/recipes-core/meta/meta-toolchain-arago.bb
@@ -1,10 +1,11 @@
TOOLCHAIN_HOST_TASK ?= "nativesdk-packagegroup-arago-sdk-host packagegroup-arago-cross-canadian-${TRANSLATED_TARGET_ARCH}"
TOOLCHAIN_TARGET_TASK ?= "packagegroup-arago-standalone-sdk-target"
TOOLCHAIN_OUTPUTNAME ?= "${SDK_NAME}-${ARMPKGARCH}-${TARGET_OS}-sdk-${SDK_ARCH}"
+TOOLCHAIN_CLEANUP_PACKAGES ?= ""
require recipes-core/meta/meta-toolchain.bb
-PR = "r18"
+PR = "r19"
SDKTARGETSYSROOT = "${SDKPATH}/${ARAGO_TARGET_SYS}"
@@ -61,6 +62,9 @@ populate_sdk_ipk_append () {
for i in `find ${SDK_OUTPUT}/${SDKPATH} -name \*.la`; do
rm -f $i
done
+
+ cleanup_toolchain_packages
+
mkdir -p "${SDK_OUTPUT}/${SDKPATHNATIVE}${prefix_nativesdk}/lib/${TUNE_PKGARCH}${TARGET_VENDOR}-${TARGET_OS}"
}
@@ -81,6 +85,26 @@ fakeroot create_sdk_files() {
sed -i -e "s:##DEFAULT_INSTALL_DIR##:$escaped_sdkpath:" ${SDK_OUTPUT}/${SDKPATH}/relocate_sdk.py
}
+# Remove undesired packages that may be pulled into the toolchain by -dev
+# package dependencies. This is usually GPLv3 components.
+cleanup_toolchain_packages() {
+ if [ "${TOOLCHAIN_CLEANUP_PACKAGES}" != "" ]
+ then
+ # Clean up the native side of the toolchain
+ opkg_dir="${SDK_OUTPUT}/${SDKPATH}/"
+ opkg_conf="${opkg_dir}/etc/opkg-sdk.conf"
+ opkg-cl -o $opkg_dir -f $opkg_conf --force-depends remove ${TOOLCHAIN_CLEANUP_PACKAGES}
+
+ # Clean up the target side of the toolchain
+ opkg_dir="${SDK_OUTPUT}/${SDKTARGETSYSROOT}"
+ opkg_conf="${opkg_dir}/etc/opkg.conf"
+ opkg-cl -o $opkg_dir -f $opkg_conf --force-depends remove ${TOOLCHAIN_CLEANUP_PACKAGES}
+
+ # Clean up empty directories from cleaned up packages
+ find ${SDK_OUTPUT} -depth -type d -empty -print0 | xargs -r0 /bin/rmdir
+ fi
+}
+
fakeroot tar_sdk() {
# Package it up
mkdir -p ${SDK_DEPLOY}
--
1.7.0.4
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 15/18] meta-toolchain-arago-tisdk: Set TOOLCHAIN_CLEANUP_PACKAGES
2013-02-12 0:46 [PATCH 01/18] linux-libc-headers: Add linux-libc-headers version 2.6.37.2 fcooperjr27
` (12 preceding siblings ...)
2013-02-12 0:46 ` [PATCH 14/18] meta-toolchain-arago: Add cleanup_toolchain_packages fcooperjr27
@ 2013-02-12 0:46 ` fcooperjr27
2013-02-12 0:46 ` [PATCH 16/18] tisdk-image: Remove toolchain cleanup functionality fcooperjr27
` (2 subsequent siblings)
16 siblings, 0 replies; 32+ messages in thread
From: fcooperjr27 @ 2013-02-12 0:46 UTC (permalink / raw)
To: meta-arago; +Cc: Franklin S. Cooper Jr
From: Franklin S. Cooper Jr <fcooper@ti.com>
* Add TOOLCHAIN_CLEANUP_PACKAGES which list packages to remove from the
toolchain sdk.
---
.../meta/meta-toolchain-arago-tisdk.bb | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/meta-arago-distro/recipes-core/meta/meta-toolchain-arago-tisdk.bb b/meta-arago-distro/recipes-core/meta/meta-toolchain-arago-tisdk.bb
index 8666625..b7f3607 100644
--- a/meta-arago-distro/recipes-core/meta/meta-toolchain-arago-tisdk.bb
+++ b/meta-arago-distro/recipes-core/meta/meta-toolchain-arago-tisdk.bb
@@ -2,8 +2,9 @@ require recipes-core/meta/meta-toolchain-arago-qte.bb
TOOLCHAIN_TARGET_TASK = "packagegroup-arago-toolchain-tisdk-target"
TOOLCHAIN_OUTPUTNAME = "${SDK_NAME}-${ARMPKGARCH}-${TARGET_OS}-tisdk-${SDK_ARCH}"
+TOOLCHAIN_CLEANUP_PACKAGES = "libgnutls-dev libgnutls-extra26 libgnutls-openssl27 libtasn1-dev"
-PR = "r0"
+PR = "r1"
toolchain_create_sdk_env_script_append() {
echo -e 'export PS1="\[\\e[32;1m\][linux-devkit]\[\\e[0m\]:\w> "' >> $script
--
1.7.0.4
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 16/18] tisdk-image: Remove toolchain cleanup functionality
2013-02-12 0:46 [PATCH 01/18] linux-libc-headers: Add linux-libc-headers version 2.6.37.2 fcooperjr27
` (13 preceding siblings ...)
2013-02-12 0:46 ` [PATCH 15/18] meta-toolchain-arago-tisdk: Set TOOLCHAIN_CLEANUP_PACKAGES fcooperjr27
@ 2013-02-12 0:46 ` fcooperjr27
2013-02-12 0:46 ` [PATCH 17/18] tisdk-image: Add support for new toolchain sdk structure fcooperjr27
2013-02-12 0:46 ` [PATCH 18/18] sdk_rootfs_ipk: Pull in rootfs_ipk.bbclass from oe-core denzil fcooperjr27
16 siblings, 0 replies; 32+ messages in thread
From: fcooperjr27 @ 2013-02-12 0:46 UTC (permalink / raw)
To: meta-arago; +Cc: Franklin S. Cooper Jr
From: Franklin S. Cooper Jr <fcooper@ti.com>
* Removing packages from the toolchain sdk is now handled by
meta-toolchain-arago.
* Removing that functionality from this class.
Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
---
meta-arago-distro/classes/tisdk-image.bbclass | 34 +------------------------
1 files changed, 1 insertions(+), 33 deletions(-)
diff --git a/meta-arago-distro/classes/tisdk-image.bbclass b/meta-arago-distro/classes/tisdk-image.bbclass
index adc6359..c2f93e2 100644
--- a/meta-arago-distro/classes/tisdk-image.bbclass
+++ b/meta-arago-distro/classes/tisdk-image.bbclass
@@ -75,14 +75,11 @@ do_sdk_image[cleandirs] += "${S}"
# Call the cleanup_host_packes to remove packages that should be removed from
# the host for various reasons. This may include licensing issues as well.
-OPKG_POSTPROCESS_COMMANDS = "cleanup_host_packages; cleanup_toolchain_packages; "
+OPKG_POSTPROCESS_COMMANDS = "cleanup_host_packages; "
# List of packages to remove from the SDK host package set
HOST_CLEANUP_PACKAGES ?= ""
-# List of packages to remove from the SDK toolchain package set
-TOOLCHAIN_CLEANUP_PACKAGES ?= ""
-
# Remove any packages that may have been pulled in by other package installs
# that are not desired. This should be used with caution.
cleanup_host_packages() {
@@ -92,30 +89,6 @@ cleanup_host_packages() {
fi
}
-# Remove undesired packages that may be pulled into the toolchain by -dev
-# package dependencies. This is usually GPLv3 components.
-cleanup_toolchain_packages() {
- if [ "${TOOLCHAIN_CLEANUP_PACKAGES}" != "" ]
- then
- # Make sure the TISDK_TOOLCHAIN_PATH is valid
- if [ ! -d ${IMAGE_ROOTFS}/${TISDK_TOOLCHAIN_PATH} ]
- then
- echo "Could not find the TISDK_TOOLCHAIN_PATH (${TISDK_TOOLCHAIN_PATH})"
- return 1
- fi
-
- # Clean up the native side of the toolchain
- opkg_dir="${IMAGE_ROOTFS}/${TISDK_TOOLCHAIN_PATH}"
- opkg_conf="${opkg_dir}/etc/opkg-sdk.conf"
- opkg-cl -o $opkg_dir -f $opkg_conf --force-depends remove ${TOOLCHAIN_CLEANUP_PACKAGES}
-
- # Clean up the target side of the toolchain
- opkg_dir="${IMAGE_ROOTFS}/${TISDK_TOOLCHAIN_PATH}/arm-arago-linux-gnueabi"
- opkg_conf="${opkg_dir}/etc/opkg.conf"
- opkg-cl -o $opkg_dir -f $opkg_conf --force-depends remove ${TOOLCHAIN_CLEANUP_PACKAGES}
- fi
-}
-
# Copy log_check from image.bbclass since the rootfs_ipk_do_rootfs function
# uses it, but we are not inheriting the full image class.
log_check() {
@@ -505,11 +478,6 @@ do_sdk_image () {
mkdir -p ${IMAGE_ROOTFS}/etc
- # Extract the toolchain SDK here so that it will be available when the
- # OPKG_POSTPROCESS_COMMANDS are run
- tar xjf ${DEPLOY_DIR}/sdk/${SDK_NAME}-${ARMPKGARCH}-${TARGET_OS}-tisdk-${SDK_ARCH}* -C ${IMAGE_ROOTFS}/
- mv ${IMAGE_ROOTFS}/${SDK_NAME} ${IMAGE_ROOTFS}/${TISDK_TOOLCHAIN_PATH}
-
# Creat the base SDK image
rootfs_${IMAGE_PKGTYPE}_do_rootfs
--
1.7.0.4
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 17/18] tisdk-image: Add support for new toolchain sdk structure
2013-02-12 0:46 [PATCH 01/18] linux-libc-headers: Add linux-libc-headers version 2.6.37.2 fcooperjr27
` (14 preceding siblings ...)
2013-02-12 0:46 ` [PATCH 16/18] tisdk-image: Remove toolchain cleanup functionality fcooperjr27
@ 2013-02-12 0:46 ` fcooperjr27
2013-02-15 22:27 ` Denys Dmytriyenko
2013-02-12 0:46 ` [PATCH 18/18] sdk_rootfs_ipk: Pull in rootfs_ipk.bbclass from oe-core denzil fcooperjr27
16 siblings, 1 reply; 32+ messages in thread
From: fcooperjr27 @ 2013-02-12 0:46 UTC (permalink / raw)
To: meta-arago; +Cc: Franklin S. Cooper Jr
From: Franklin S. Cooper Jr <fcooper@ti.com>
* Previous the toolchain sdk was delivered as a simple tar.gz file that
could be extracted.
* Now the toolchain sdk needs to be installed via a self contained script
installer that provides alot more flexibility.
* Need to tweak this class to reflect this change.
Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
---
meta-arago-distro/classes/tisdk-image.bbclass | 18 ++++++++++++++++++
1 files changed, 18 insertions(+), 0 deletions(-)
diff --git a/meta-arago-distro/classes/tisdk-image.bbclass b/meta-arago-distro/classes/tisdk-image.bbclass
index c2f93e2..b08f93b 100644
--- a/meta-arago-distro/classes/tisdk-image.bbclass
+++ b/meta-arago-distro/classes/tisdk-image.bbclass
@@ -478,6 +478,11 @@ do_sdk_image () {
mkdir -p ${IMAGE_ROOTFS}/etc
+ chmod 755 ${DEPLOY_DIR}/sdk/${SDK_NAME}-${ARMPKGARCH}-${TARGET_OS}-tisdk-${SDK_ARCH}*
+
+ # Temporarily extract the toolchain sdk so we can read license information from it.
+ echo "${IMAGE_ROOTFS}/${TISDK_TOOLCHAIN_PATH}" | ${DEPLOY_DIR}/sdk/${SDK_NAME}-${ARMPKGARCH}-${TARGET_OS}-tisdk-${SDK_ARCH}*
+
# Creat the base SDK image
rootfs_${IMAGE_PKGTYPE}_do_rootfs
@@ -562,6 +567,14 @@ do_sdk_image () {
# Create the TI software manifest
generate_sw_manifest
+ # Delete installed toolchain sdk since we need the toolchain sdk installer
+ # not the extracted version
+ rm -rf ${IMAGE_ROOTFS}/${TISDK_TOOLCHAIN_PATH}
+
+ # Copy over the toolchain sdk installer an give it a simple name which
+ # matches the traditional name within the SDK.
+ cp ${DEPLOY_DIR}/sdk/${SDK_NAME}-${ARMPKGARCH}-${TARGET_OS}-tisdk-${SDK_ARCH}* ${IMAGE_ROOTFS}/linux-devkit.sh
+
# Copy the opkg.conf used by the image to allow for future updates
cp ${WORKDIR}/opkg.conf ${IMAGE_ROOTFS}/etc/
@@ -574,6 +587,11 @@ do_sdk_image () {
${@get_imagecmds(d)}
}
+license_create_manifest() {
+ # Override license_create_manifest and do nothing
+ echo ""
+}
+
EXPORT_FUNCTIONS do_sdk_image
addtask sdk_image before do_build after do_install
--
1.7.0.4
^ permalink raw reply related [flat|nested] 32+ messages in thread
* Re: [PATCH 17/18] tisdk-image: Add support for new toolchain sdk structure
2013-02-12 0:46 ` [PATCH 17/18] tisdk-image: Add support for new toolchain sdk structure fcooperjr27
@ 2013-02-15 22:27 ` Denys Dmytriyenko
0 siblings, 0 replies; 32+ messages in thread
From: Denys Dmytriyenko @ 2013-02-15 22:27 UTC (permalink / raw)
To: fcooperjr27; +Cc: meta-arago, Franklin S. Cooper Jr
On Mon, Feb 11, 2013 at 06:46:18PM -0600, fcooperjr27@gmail.com wrote:
> From: Franklin S. Cooper Jr <fcooper@ti.com>
>
> * Previous the toolchain sdk was delivered as a simple tar.gz file that
> could be extracted.
> * Now the toolchain sdk needs to be installed via a self contained script
> installer that provides alot more flexibility.
> * Need to tweak this class to reflect this change.
>
> Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
> ---
> meta-arago-distro/classes/tisdk-image.bbclass | 18 ++++++++++++++++++
> 1 files changed, 18 insertions(+), 0 deletions(-)
>
> diff --git a/meta-arago-distro/classes/tisdk-image.bbclass b/meta-arago-distro/classes/tisdk-image.bbclass
> index c2f93e2..b08f93b 100644
> --- a/meta-arago-distro/classes/tisdk-image.bbclass
> +++ b/meta-arago-distro/classes/tisdk-image.bbclass
> @@ -478,6 +478,11 @@ do_sdk_image () {
>
> mkdir -p ${IMAGE_ROOTFS}/etc
>
> + chmod 755 ${DEPLOY_DIR}/sdk/${SDK_NAME}-${ARMPKGARCH}-${TARGET_OS}-tisdk-${SDK_ARCH}*
> +
> + # Temporarily extract the toolchain sdk so we can read license information from it.
> + echo "${IMAGE_ROOTFS}/${TISDK_TOOLCHAIN_PATH}" | ${DEPLOY_DIR}/sdk/${SDK_NAME}-${ARMPKGARCH}-${TARGET_OS}-tisdk-${SDK_ARCH}*
Why not use -y and -d path?
${DEPLOY_DIR}/sdk/${SDK_NAME}-${ARMPKGARCH}-${TARGET_OS}-tisdk-${SDK_ARCH}* -y -d "${IMAGE_ROOTFS}/${TISDK_TOOLCHAIN_PATH}"
> # Creat the base SDK image
> rootfs_${IMAGE_PKGTYPE}_do_rootfs
>
> @@ -562,6 +567,14 @@ do_sdk_image () {
> # Create the TI software manifest
> generate_sw_manifest
>
> + # Delete installed toolchain sdk since we need the toolchain sdk installer
> + # not the extracted version
> + rm -rf ${IMAGE_ROOTFS}/${TISDK_TOOLCHAIN_PATH}
> +
> + # Copy over the toolchain sdk installer an give it a simple name which
> + # matches the traditional name within the SDK.
> + cp ${DEPLOY_DIR}/sdk/${SDK_NAME}-${ARMPKGARCH}-${TARGET_OS}-tisdk-${SDK_ARCH}* ${IMAGE_ROOTFS}/linux-devkit.sh
> +
> # Copy the opkg.conf used by the image to allow for future updates
> cp ${WORKDIR}/opkg.conf ${IMAGE_ROOTFS}/etc/
>
> @@ -574,6 +587,11 @@ do_sdk_image () {
> ${@get_imagecmds(d)}
> }
>
> +license_create_manifest() {
> + # Override license_create_manifest and do nothing
> + echo ""
Usually you'd use colon ":" for noop command:
blah() {
:
}
> +}
> +
> EXPORT_FUNCTIONS do_sdk_image
> addtask sdk_image before do_build after do_install
>
> --
> 1.7.0.4
>
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
^ permalink raw reply [flat|nested] 32+ messages in thread
* [PATCH 18/18] sdk_rootfs_ipk: Pull in rootfs_ipk.bbclass from oe-core denzil.
2013-02-12 0:46 [PATCH 01/18] linux-libc-headers: Add linux-libc-headers version 2.6.37.2 fcooperjr27
` (15 preceding siblings ...)
2013-02-12 0:46 ` [PATCH 17/18] tisdk-image: Add support for new toolchain sdk structure fcooperjr27
@ 2013-02-12 0:46 ` fcooperjr27
16 siblings, 0 replies; 32+ messages in thread
From: fcooperjr27 @ 2013-02-12 0:46 UTC (permalink / raw)
To: meta-arago; +Cc: Franklin S. Cooper Jr
From: Franklin S. Cooper Jr <fcooper@ti.com>
* Tisdk-image inherits rootfs_ipk.bbclass to generate SDK image.
* Changes have been made to rootfs_ipk.bbclass that breaks SDK images.
* For now pull in this class and rename it to sdk_rootfs_ipk.
* Need to rework tisdk-image to play nice with possible future changes of
rootfs_ipk.bbclass or remove the dependency and make tisdk-image self
contained.
---
meta-arago-distro/classes/sdk_rootfs_ipk.bbclass | 203 ++++++++++++++++++++++
meta-arago-distro/classes/tisdk-image.bbclass | 2 +-
2 files changed, 204 insertions(+), 1 deletions(-)
create mode 100644 meta-arago-distro/classes/sdk_rootfs_ipk.bbclass
diff --git a/meta-arago-distro/classes/sdk_rootfs_ipk.bbclass b/meta-arago-distro/classes/sdk_rootfs_ipk.bbclass
new file mode 100644
index 0000000..1580086
--- /dev/null
+++ b/meta-arago-distro/classes/sdk_rootfs_ipk.bbclass
@@ -0,0 +1,203 @@
+#
+# Creates a root filesystem out of IPKs
+#
+# This rootfs can be mounted via root-nfs or it can be put into an cramfs/jffs etc.
+# See image.bbclass for a usage of this.
+#
+
+EXTRAOPKGCONFIG ?= ""
+ROOTFS_PKGMANAGE = "opkg opkg-collateral ${EXTRAOPKGCONFIG}"
+ROOTFS_PKGMANAGE_BOOTSTRAP = "run-postinsts"
+
+do_rootfs[depends] += "opkg-native:do_populate_sysroot opkg-utils-native:do_populate_sysroot"
+do_rootfs[recrdeptask] += "do_package_write_ipk"
+
+IPKG_ARGS = "-f ${IPKGCONF_TARGET} -o ${IMAGE_ROOTFS} --force-overwrite"
+
+OPKG_PREPROCESS_COMMANDS = "package_update_index_ipk; package_generate_ipkg_conf"
+
+OPKG_POSTPROCESS_COMMANDS = "ipk_insert_feed_uris; rootfs_install_all_locales; "
+
+opkglibdir = "${localstatedir}/lib/opkg"
+
+# Which packages to not install on the basis of a recommendation
+BAD_RECOMMENDATIONS ?= ""
+MULTILIBRE_ALLOW_REP = "${opkglibdir}"
+
+fakeroot rootfs_ipk_do_rootfs () {
+ set -x
+
+ rm -f ${IPKGCONF_TARGET}
+ touch ${IPKGCONF_TARGET}
+
+ ${OPKG_PREPROCESS_COMMANDS}
+
+ mkdir -p ${T}/
+
+ STATUS=${IMAGE_ROOTFS}${opkglibdir}/status
+ mkdir -p ${IMAGE_ROOTFS}${opkglibdir}
+
+ opkg-cl ${IPKG_ARGS} update
+
+ # prime the status file with bits that we don't want
+ for i in ${BAD_RECOMMENDATIONS}; do
+ pkginfo="`opkg-cl ${IPKG_ARGS} info $i`"
+ if [ ! -z "$pkginfo" ]; then
+ echo "$pkginfo" | grep -e '^Package:' -e '^Architecture:' -e '^Version:' >> $STATUS
+ echo "Status: deinstall hold not-installed" >> $STATUS
+ echo >> $STATUS
+ else
+ echo "Requested ignored recommendation $i is not a package"
+ fi
+ done
+
+ #install
+ export INSTALL_PACKAGES_ATTEMPTONLY_IPK="${PACKAGE_INSTALL_ATTEMPTONLY}"
+ export INSTALL_PACKAGES_LINGUAS_IPK="${LINGUAS_INSTALL}"
+ export INSTALL_TASK_IPK="rootfs"
+
+ export INSTALL_ROOTFS_IPK="${IMAGE_ROOTFS}"
+ export INSTALL_CONF_IPK="${IPKGCONF_TARGET}"
+ export INSTALL_PACKAGES_IPK="${PACKAGE_INSTALL}"
+
+ #post install
+ export D=${IMAGE_ROOTFS}
+ export OFFLINE_ROOT=${IMAGE_ROOTFS}
+ export IPKG_OFFLINE_ROOT=${IMAGE_ROOTFS}
+ export OPKG_OFFLINE_ROOT=${IPKG_OFFLINE_ROOT}
+
+ package_install_internal_ipk
+
+ # Distro specific packages should create this
+ #mkdir -p ${IMAGE_ROOTFS}/etc/opkg/
+ #grep "^arch" ${IPKGCONF_TARGET} >${IMAGE_ROOTFS}/etc/opkg/arch.conf
+
+ ${OPKG_POSTPROCESS_COMMANDS}
+ ${ROOTFS_POSTINSTALL_COMMAND}
+
+ if ${@base_contains("IMAGE_FEATURES", "read-only-rootfs", "true", "false" ,d)}; then
+ if grep Status:.install.ok.unpacked ${STATUS}; then
+ echo "Some packages could not be configured offline and rootfs is read-only."
+ exit 1
+ fi
+ fi
+
+ install -d ${IMAGE_ROOTFS}/${sysconfdir}
+ echo ${BUILDNAME} > ${IMAGE_ROOTFS}/${sysconfdir}/version
+
+ ${ROOTFS_POSTPROCESS_COMMAND}
+
+ rm -f ${IMAGE_ROOTFS}${opkglibdir}/lists/*
+ if ${@base_contains("IMAGE_FEATURES", "package-management", "false", "true", d)}; then
+ if ! grep Status:.install.ok.unpacked ${STATUS}; then
+ # All packages were successfully configured.
+ # update-rc.d, base-passwd are no further use, remove them now
+ opkg-cl ${IPKG_ARGS} --force-depends remove update-rc.d base-passwd || true
+
+ # Also delete the status files
+ rm -rf ${IMAGE_ROOTFS}${opkglibdir}
+ fi
+ fi
+ set +x
+ log_check rootfs
+}
+
+rootfs_ipk_write_manifest() {
+ manifest=${DEPLOY_DIR_IMAGE}/${IMAGE_NAME}.rootfs.manifest
+ cp ${IMAGE_ROOTFS}${opkglibdir}/status $manifest
+
+ sed '/Depends/d' -i $manifest
+ sed '/Status/d' -i $manifest
+ sed '/Architecture/d' -i $manifest
+ sed '/Installed-Time/d' -i $manifest
+ sed '/Auto-Installed/d' -i $manifest
+ sed '/Recommends/d' -i $manifest
+ sed '/Provides/d' -i $manifest
+ sed '/Conflicts/d' -i $manifest
+}
+
+remove_packaging_data_files() {
+ rm -rf ${IMAGE_ROOTFS}${opkglibdir}
+ # We need the directory for the package manager lock
+ mkdir ${IMAGE_ROOTFS}${opkglibdir}
+}
+
+list_installed_packages() {
+ grep ^Package: ${IMAGE_ROOTFS}${opkglibdir}/status | sed "s/^Package: //"
+}
+
+get_package_filename() {
+ set +x
+ info=`opkg-cl ${IPKG_ARGS} info $1 | grep -B 7 -A 7 "^Status.* \(\(installed\)\|\(unpacked\)\)" || true`
+ name=`echo "${info}" | awk '/^Package/ {printf $2"_"}'`
+ name=$name`echo "${info}" | awk -F: '/^Version/ {printf $NF"_"}' | sed 's/^\s*//g'`
+ name=$name`echo "${info}" | awk '/^Archi/ {print $2".ipk"}'`
+ set -x
+
+ fullname=`find ${DEPLOY_DIR_IPK} -name "$name" || true`
+ if [ "$fullname" = "" ] ; then
+ echo $name
+ else
+ echo $fullname
+ fi
+}
+
+list_package_depends() {
+ opkg-cl ${IPKG_ARGS} info $1 | grep ^Depends | sed -e 's/^Depends: //' -e 's/,//g' -e 's:([=<>]* [^ )]*)::g'
+}
+
+list_package_recommends() {
+ opkg-cl ${IPKG_ARGS} info $1 | grep ^Recommends | sed -e 's/^Recommends: //' -e 's/,//g' -e 's:([=<>]* [^ )]*)::g'
+}
+
+rootfs_check_package_exists() {
+ if [ `opkg-cl ${IPKG_ARGS} info $1 | wc -l` -gt 2 ]; then
+ echo $1
+ fi
+}
+
+rootfs_install_packages() {
+ opkg-cl ${IPKG_ARGS} install $PACKAGES_TO_INSTALL
+}
+
+ipk_insert_feed_uris () {
+
+ echo "Building from feeds activated!"
+
+ for line in ${IPK_FEED_URIS}
+ do
+ # strip leading and trailing spaces/tabs, then split into name and uri
+ line_clean="`echo "$line"|sed 's/^[ \t]*//;s/[ \t]*$//'`"
+ feed_name="`echo "$line_clean" | sed -n 's/\(.*\)##\(.*\)/\1/p'`"
+ feed_uri="`echo "$line_clean" | sed -n 's/\(.*\)##\(.*\)/\2/p'`"
+
+ echo "Added $feed_name feed with URL $feed_uri"
+
+ # insert new feed-sources
+ echo "src/gz $feed_name $feed_uri" >> ${IPKGCONF_TARGET}
+ done
+
+ # Allow to use package deploy directory contents as quick devel-testing
+ # feed. This creates individual feed configs for each arch subdir of those
+ # specified as compatible for the current machine.
+ # NOTE: Development-helper feature, NOT a full-fledged feed.
+ if [ -n "${FEED_DEPLOYDIR_BASE_URI}" ]; then
+ for arch in ${PACKAGE_ARCHS}
+ do
+ echo "src/gz local-$arch ${FEED_DEPLOYDIR_BASE_URI}/$arch" >> ${IMAGE_ROOTFS}/etc/opkg/local-$arch-feed.conf
+ done
+ fi
+}
+
+python () {
+
+ if d.getVar('BUILD_IMAGES_FROM_FEEDS', True):
+ flags = d.getVarFlag('do_rootfs', 'recrdeptask')
+ flags = flags.replace("do_package_write_ipk", "")
+ flags = flags.replace("do_deploy", "")
+ flags = flags.replace("do_populate_sysroot", "")
+ d.setVarFlag('do_rootfs', 'recrdeptask', flags)
+ d.setVar('OPKG_PREPROCESS_COMMANDS', "package_generate_archlist\nipk_insert_feed_uris")
+ d.setVar('OPKG_POSTPROCESS_COMMANDS', '')
+}
+
diff --git a/meta-arago-distro/classes/tisdk-image.bbclass b/meta-arago-distro/classes/tisdk-image.bbclass
index b08f93b..5d83287 100644
--- a/meta-arago-distro/classes/tisdk-image.bbclass
+++ b/meta-arago-distro/classes/tisdk-image.bbclass
@@ -1,4 +1,4 @@
-inherit rootfs_ipk
+inherit sdk_rootfs_ipk
inherit image_types
# This defines the list of features that we want to include in the SDK
--
1.7.0.4
^ permalink raw reply related [flat|nested] 32+ messages in thread