[PATCH 01/18] linux-libc-headers: Add linux-libc-headers version 2.6.37.2

[PATCH 01/18] linux-libc-headers: Add linux-libc-headers version 2.6.37.2

[fcooperjr27]

From: Franklin S. Cooper Jr <fcooper@ti.com>

* Add linux-libc-headers version 2.6.37.2 which is needed by machines who use a
  2.6.37 kernel.

Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
---
 .../linux-libc-headers_2.6.37.2.bb                 |   10 ++++++++++
 1 files changed, 10 insertions(+), 0 deletions(-)
 create mode 100644 meta-arago-extras/recipes-kernel/linux-libc-headers/linux-libc-headers_2.6.37.2.bb

diff --git a/meta-arago-extras/recipes-kernel/linux-libc-headers/linux-libc-headers_2.6.37.2.bb b/meta-arago-extras/recipes-kernel/linux-libc-headers/linux-libc-headers_2.6.37.2.bb
new file mode 100644
index 0000000..5c00af5
--- /dev/null
+++ b/meta-arago-extras/recipes-kernel/linux-libc-headers/linux-libc-headers_2.6.37.2.bb
@@ -0,0 +1,10 @@
+require recipes-kernel/linux-libc-headers/linux-libc-headers.inc
+
+FILESEXTRAPATHS_append := "${THISDIR}/${PN}:${COREBASE}/meta/recipes-kernel/linux-libc-headers/${PN}"
+
+PR = "r1"
+
+SRC_URI += " file://connector-msg-size-fix.patch"
+SRC_URI[md5sum] = "89f681bc7c917a84aa7470da6eed5101"
+SRC_URI[sha256sum] = "2920c4cd3e87fe40ebee96d28779091548867e1c36f71c1fc3d07e6d5802161f"
+
-- 
1.7.0.4

[PATCH 02/18] ti-tisdk-makefile: Switch from ARMPKGARCH to PLATFORM_ARCH

[fcooperjr27]

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

[PATCH 03/18] arm-benchmarks_1.3: Tweak to allow am180x-evm to be built

[fcooperjr27]

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

[PATCH 04/18] sourceipk.bbclass: Provide the ability to change the source directory

[fcooperjr27]

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

[PATCH 05/18] ti-crypto-examples: Add ARM9 support

[fcooperjr27]

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

[PATCH 06/18] ti-crypto-examples: Tweak SRCIPK_SRC_DIR variable

[fcooperjr27]

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

[PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable

[fcooperjr27]

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

[PATCH 08/18] recipes: Only pull in SGX related tasks if SUPPORTS_SGX is set to YES

[fcooperjr27]

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

[PATCH 09/18] tisdk-image.bbclass: Create SECONDARY_BOOTLOADER_NAME variable

[fcooperjr27]

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

[PATCH 10/18] arago-amsdk-image: Set SECONDARY_BOOTLOADER_NAME to empty string for omapl138

[fcooperjr27]

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

[PATCH 11/18] tisdk-amsdk-sdk-host: Add flash utility to SDK

[fcooperjr27]

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

[PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources

[fcooperjr27]

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

[PATCH 13/18] board-port-labs: Inherit allarch class.

[fcooperjr27]

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

[PATCH 14/18] meta-toolchain-arago: Add cleanup_toolchain_packages

[fcooperjr27]

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

[PATCH 15/18] meta-toolchain-arago-tisdk: Set TOOLCHAIN_CLEANUP_PACKAGES

[fcooperjr27]

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

[PATCH 16/18] tisdk-image: Remove toolchain cleanup functionality

[fcooperjr27]

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

[PATCH 17/18] tisdk-image: Add support for new toolchain sdk structure

[fcooperjr27]

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

[PATCH 18/18] sdk_rootfs_ipk: Pull in rootfs_ipk.bbclass from oe-core denzil.

[fcooperjr27]

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

Re: [PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable

[Denys Dmytriyenko]

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
> 

Re: [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources

[Denys Dmytriyenko]

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
> 

Re: [PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable

[Cooper Jr., Franklin]

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
> >

Re: [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources

[Cooper Jr., Franklin]

> -----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

Re: [PATCH 17/18] tisdk-image: Add support for new toolchain sdk structure

[Denys Dmytriyenko]

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

Re: [PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable

[Paul Kovitz]

[-- 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 --]

Re: [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources

[Denys Dmytriyenko]

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

Re: [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources

[Cooper Jr., Franklin]

> -----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

Re: [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources

[Maupin, Chase]

+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

Re: [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources

[Denys Dmytriyenko]

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
> 

Re: [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources

[Maupin, Chase]

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
>> 

Re: [PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable

[Cooper Jr., Franklin]

[-- 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 --]

Re: [PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable

[Denys Dmytriyenko]

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

Re: [PATCH 07/18] arago.conf: Add SUPPORTS_SGX variable

[Cooper Jr., Franklin]

> -----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