> -----Original Message----- > From: openembedded-core-bounces at lists.openembedded.org <openembedded- > core-bounces at lists.openembedded.org> On Behalf Of Otavio Salvador > Sent: den 19 november 2018 20:22 > To: Tom Rini <trini at konsulko.com> > Cc: Marek Vasut <marex at denx.de>; linux-snps-arc at lists.infradead.org; > Alexey Brodkin <alexey.brodkin at synopsys.com>; Otavio Salvador > <otavio at ossystems.com.br>; Patches and discussions about the oe-core > layer <openembedded-core at lists.openembedded.org> > Subject: Re: [OE-core] [PATCH v3] u-boot: Add mkenvimage tool > > On Mon, Nov 19, 2018 at 5:08 PM Tom Rini <trini at konsulko.com> wrote: > > > > On Mon, Nov 19, 2018 at 01:51:41PM +0100, Marek Vasut wrote: > > > On 11/19/2018 11:04 AM, Alexey Brodkin wrote: > > > > This utility is used for creation of images containing > > > > usable in run-time U-Boot environment. > > > > > > > > As of today this utility is added per-board like here [1] > > > > for Intel Edison board. > > > > > > > > [1] http://git.yoctoproject.org/cgit/cgit.cgi/meta-intel- > edison/tree/meta-intel-edison-bsp/recipes-bsp/u-boot/u-boot- > tools_2014.04.bb > > > > > > > > Given there're quite some U-Boot tools that we may want to add > later > > > > this recipe name switch from "u-boot-mkimage" to generic "u-boot- > tools" > > > > still for compatibility we provide "u-boot-mkimage" with help > > > > of PROVIDES as well as proposed "u-boot-mkenvimage". > > > > > > > > Signed-off-by: Alexey Brodkin <abrodkin at synopsys.com> > > > > Cc: Richard Purdie <richard.purdie at linuxfoundation.org> > > > > Cc: Otavio Salvador <otavio at ossystems.com.br> > > > > Cc: Martin Jansa <martin.jansa at gmail.com> > > > > Cc: Ross Burton <ross.burton at intel.com> > > > > Cc: Marek Vasut <marex at denx.de> > > > > > > Do we want to generate one package per tool or not ? > > > > I think so, yes. Not all places will want all tools. > > And a bonus to add a u-boot-tools-meta which rdepends on all. You can > copy code to do that from gstreamer-plugins packages. Actually, why mess with an unnecessary meta package. Just use the empty ${PN} package for that. That works for the GStreamer packages as well these days (since commit 8e1009a3), even though they do it by depending on the meta packages for backwards compatibility. //Peter > -- > Otavio Salvador O.S. Systems > http://www.ossystems.com.br http://code.ossystems.com.br > Mobile: +55 (53) 9 9981-7854 Mobile: +1 (347) 903-9750
