All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Cooper Jr., Franklin" <fcooper@ti.com>
To: "Dmytriyenko, Denys" <denys@ti.com>,
	"fcooperjr27@gmail.com" <fcooperjr27@gmail.com>
Cc: "meta-arago@arago-project.org" <meta-arago@arago-project.org>
Subject: Re: [PATCH 12/18] arago-tisdk-amsdk-sdk-host: Use generic name for kernel sources
Date: Fri, 15 Feb 2013 03:54:29 +0000	[thread overview]
Message-ID: <8F29D6B095ED194EA1980491A5E029710C3918EB@DFLE09.ent.ti.com> (raw)
In-Reply-To: <20130215034205.GF9154@denix.org>



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


  reply	other threads:[~2013-02-15  3:54 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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 ` [PATCH 04/18] sourceipk.bbclass: Provide the ability to change the source directory fcooperjr27
2013-02-12  0:46 ` [PATCH 05/18] ti-crypto-examples: Add ARM9 support fcooperjr27
2013-02-12  0:46 ` [PATCH 06/18] ti-crypto-examples: Tweak SRCIPK_SRC_DIR variable fcooperjr27
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
2013-02-18 15:14     ` Cooper Jr., Franklin
2013-02-18 17:43       ` Denys Dmytriyenko
2013-02-18 18:34         ` Cooper Jr., Franklin
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 ` [PATCH 09/18] tisdk-image.bbclass: Create SECONDARY_BOOTLOADER_NAME variable fcooperjr27
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 ` [PATCH 11/18] tisdk-amsdk-sdk-host: Add flash utility to SDK fcooperjr27
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 [this message]
2013-02-17 20:35       ` Denys Dmytriyenko
2013-02-17 21:07         ` Cooper Jr., Franklin
2013-02-17 21:29           ` Maupin, Chase
2013-02-17 21:35             ` Denys Dmytriyenko
2013-02-17 21:46               ` Maupin, Chase
2013-02-12  0:46 ` [PATCH 13/18] board-port-labs: Inherit allarch class fcooperjr27
2013-02-12  0:46 ` [PATCH 14/18] meta-toolchain-arago: Add cleanup_toolchain_packages fcooperjr27
2013-02-12  0:46 ` [PATCH 15/18] meta-toolchain-arago-tisdk: Set TOOLCHAIN_CLEANUP_PACKAGES fcooperjr27
2013-02-12  0:46 ` [PATCH 16/18] tisdk-image: Remove toolchain cleanup functionality fcooperjr27
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
2013-02-12  0:46 ` [PATCH 18/18] sdk_rootfs_ipk: Pull in rootfs_ipk.bbclass from oe-core denzil fcooperjr27

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8F29D6B095ED194EA1980491A5E029710C3918EB@DFLE09.ent.ti.com \
    --to=fcooper@ti.com \
    --cc=denys@ti.com \
    --cc=fcooperjr27@gmail.com \
    --cc=meta-arago@arago-project.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.