[OE-core] [PATCH v7] u-boot: Add mkenvimage tool

[Alexey Brodkin]

Hi Tom,

On Mon, 2018-11-26 at 19:51 -0500, Tom Rini wrote:
> On Mon, Nov 26, 2018 at 09:07:37PM -0200, Otavio Salvador wrote:
> > Hello Alexey,
> > 
> > On Mon, Nov 26, 2018 at 7:30 PM Alexey Brodkin
> > <alexey.brodkin at synopsys.com> wrote:
> > > On Sat, 2018-11-24 at 06:57 -0800, Khem Raj wrote:
> > > > http://errors.yoctoproject.org/Errors/Details/202185/
> > > 
> > > Looking at the target name "u-boot-tools-1_2018.11-r0 do_compile"
> > > I think it might have something to do with bump of U-Boot to v2018.11
> > > and the error message in question was added in v2018.11-rc1,
> > > see http://git.denx.de/?p=u-boot.git;a=commitdiff;h=a4958a71017fb142542f977c843c5fce769fc6ea
> > > 
> > > The problem is we use "sandbox_defconfig" as a dummy defconfig
> > > because otherwise (not configured) U-Boot's build-system won't
> > > allow us to proceed and in its turn mentioned patch adds a check
> > > for a target platform like "if defined(__x86_64__)" which
> > > we essentially don't satisfy and so we fail.
> > > 
> > > There're the following solutions:
> > >  1. Revert mentioned patch
> > >  2. Disable "CONFIG_EFI_LOADER"
> > > 
> > > Reverting is not nice and not future-proof.
> > > Disabling of "CONFIG_EFI_LOADER" might be done in a couple of ways
> > > like:
> > >  1. Patching sandbox_defconfig
> > >  2. Filtering "CONFIG_EFI_LOADER" from "sandbox_defconfig" right before
> > >     execution of "make sandbox_defconfig"
> > > 
> > > I think latter option is the simplest and cleanest.
> > > 
> > > Should I send a patch for that?
> > 
> > Let's add Tom (U-Boot upstream main maintainer) to this so he can comment.
> 
> Hey, so you've hit the problem Fedora also hit, but we haven't gotten a
> real solution to.  Yes, the best "fix" here for just building tools
> would be to turn off EFI_LOADER in sandbox, on all arches.

But why don't we simply use our own really dummy defconfig like that?
------------------->8--------------------
echo "CONFIG_SYS_TEXT_BASE=0" > .config
make olddefconfig
make tools
------------------->8--------------------

This is much cleaner approach and much simpler to implement.

-Alexey