From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.free-electrons.com (top.free-electrons.com [176.31.233.9]) by yocto-www.yoctoproject.org (Postfix) with ESMTP id 254EDE00406 for ; Thu, 22 Aug 2013 16:06:16 -0700 (PDT) Received: by mail.free-electrons.com (Postfix, from userid 106) id 0CB31820; Fri, 23 Aug 2013 01:06:18 +0200 (CEST) X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on mail.free-electrons.com X-Spam-Level: X-Spam-Status: No, score=-1.0 required=5.0 tests=ALL_TRUSTED,SHORTCIRCUIT, URIBL_BLOCKED shortcircuit=ham autolearn=disabled version=3.3.2 Received: from [192.168.1.53] (128-79-216-144.hfc.dyn.abo.bbox.fr [128.79.216.144]) by mail.free-electrons.com (Postfix) with ESMTPSA id 93A5981D; Fri, 23 Aug 2013 01:06:17 +0200 (CEST) Message-ID: <52169966.5020607@free-electrons.com> Date: Fri, 23 Aug 2013 01:06:14 +0200 From: Alexandre Belloni Organization: Free Electrons User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130803 Thunderbird/17.0.8 MIME-Version: 1.0 To: Otavio Salvador References: <1377183077-5270-1-git-send-email-alexandre.belloni@free-electrons.com> <1377183077-5270-2-git-send-email-alexandre.belloni@free-electrons.com> <20130822191421.53fd6f95@e6520eb> In-Reply-To: X-Enigmail-Version: 1.4.6 Cc: "meta-freescale@yoctoproject.org" , brian@crystalfontz.com, Maxime Ripard , jimwall@q.com Subject: Re: [meta-fsl-arm-extra][PATCH 1/4] barebox: add a recipe to support mainline 2013.08.0 X-BeenThere: meta-freescale@yoctoproject.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Usage and development list for the meta-fsl-* layers List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 22 Aug 2013 23:06:18 -0000 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8bit On 22/08/2013 20:00, Otavio Salvador wrote: > On Thu, Aug 22, 2013 at 2:14 PM, Eric Bénard wrote: >> Hi Alexandre, >> >> Le Thu, 22 Aug 2013 16:51:14 +0200, >> Alexandre Belloni a écrit : >> >>> Signed-off-by: Alexandre Belloni >>> --- >>> recipes-bsp/barebox/barebox_2013.08.0.bb | 26 ++++++++++++++++++++++++++ >>> 1 file changed, 26 insertions(+) >>> create mode 100644 recipes-bsp/barebox/barebox_2013.08.0.bb >>> >>> diff --git a/recipes-bsp/barebox/barebox_2013.08.0.bb b/recipes-bsp/barebox/barebox_2013.08.0.bb >>> new file mode 100644 >>> index 0000000..2f6118b >>> --- /dev/null >>> +++ b/recipes-bsp/barebox/barebox_2013.08.0.bb >>> @@ -0,0 +1,26 @@ >>> +require recipes-bsp/barebox/barebox.inc >>> + >>> +PR = "r1" >>> + >>> +SRC_URI = "http://barebox.org/download/${PN}-${PV}.tar.bz2" >>> + >>> +SRC_URI[md5sum] = "a61b97a2fedebc808d4b182c2ca18a0c" >>> +SRC_URI[sha256sum] = "d9e46dd3c68b04ab4cbc37135dc34907dbebef393da4b3e54abb203c295c1620" >>> + >>> +python () { >>> + if not d.getVar("BAREBOX_MACHINE", True): >>> + PN = d.getVar("PN", True) >>> + FILE = os.path.basename(d.getVar("FILE", True)) >>> + bb.debug(1, "To build %s, see %s for instructions on \ >>> + setting up your machine config" % (PN, FILE)) >>> + raise bb.parse.SkipPackage("BAREBOX_MACHINE is not set in the %s machine configuration." % d.getVar("MACHINE", True)) >>> +} >>> + >> if you plan to create a generic barebox recipe and as barebox is using >> Kconfig like Linux does, I would recommend to include the defconfig for >> your board here as done on the kernel side instead of using the >> defconfig which may exist into barebox's mainline sources (that makes >> customization easier especially when using SOM). >> >> If you don't plan do create the generic barebox recipe, that's still on >> my todo list but with a very low priority :-) > Agreed. > I'm actually wondering about that. The defconfig included in the barebox mainline is perfectly fine. And people wanting to change it will have to write a bbappend anyway or am I missing something ? Anyway, I don't have a strong opinion on that one. -- Alexandre Belloni, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com