All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alexey Brodkin <Alexey.Brodkin@synopsys.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] u-boot: 2015.07 - fix creation of .config
Date: Mon, 31 Aug 2015 16:43:20 +0000	[thread overview]
Message-ID: <1441039400.2935.8.camel@synopsys.com> (raw)
In-Reply-To: <87io7wts85.fsf@dell.be.48ers.dk>

Hi Peter,

On Sun, 2015-08-30 at 22:51 +0200, Peter Korsgaard wrote:
> > > > > > "Alexey" == Alexey Brodkin <Alexey.Brodkin@synopsys.com> writes:
> 
>  > Due to recent changes in U-Boot, see
>  > http://git.denx.de/?p=u-boot.git;a=commit;h=a26cd04920dc069fd6e91abb785426cf6c29f45f
>  > re-creation of .config from defconfig by "make oldconfig" happened
>  > incorrectly.
> 
>  > Default prompt for target selection was set as No that lead to missing platform
>  > selection by automated scripts like that:
>  ------------------> 8------------------
>  > "yes "" | make oldconfig" on defconfig
>  ------------------> 8------------------
> 
>  > And that lead to build failure:
>  ------------------> 8------------------
>  > $ make
>  > scripts/kconfig/conf --silentoldconfig Kconfig
>  >   CHK     include/config.h
>  >   UPD     include/config.h
>  >   GEN     include/autoconf.mk
>  > In file included from ./include/common.h:18:0:
>  > include/config.h:5:22: fatal error: configs/.h: No such file or directory
>  >  #include <configs/.h>
>  >                       ^
>  > compilation terminated.
>  > scripts/Makefile.autoconf:72: recipe for target 'include/autoconf.mk' failed
>  > make[1]: *** [include/autoconf.mk] Error 1
>  ------------------> 8------------------
> 
>  > This patch reverts mentioned change making possible to build U-Boot
>  > 2015.07 in Buildroot.
> 
>  > Note there's a probability that this particular patch won't make its way
>  > in upstream U-Boot and better solution will be found there. But for now
>  > we need to fix U-Boot building in Buildroot anyways.
> 
>  > Hopefully for the next release this patch won't be necessary.
> 
>  > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
>  > Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> 
> Committed, thanks.
> 
> While doing a test build I noticed that u-boot nowadays needs dtc:
> 
>   CHK     include/generated/timestamp_autogenerated.h
>   UPD     include/generated/timestamp_autogenerated.h
> ./scripts/dtc-version.sh: line 17: dtc: command not found
> ./scripts/dtc-version.sh: line 18: dtc: command not found
> *** Your dtc is too old, please upgrade to dtc 1.4 or newer
> Makefile:1297: recipe for target 'checkdtc' failed
> make[1]: *** [checkdtc] Error 1
> make[1]: *** Waiting for unfinished jobs....
>   UPD     include/config/uboot.release
> make[1]: *** wait: No child processes.  Stop.
> package/pkg-generic.mk:156: recipe for target '/home/peko/source/buildroot/output/build/uboot-2015.07/.stamp_built'
> failed
> 
> So perhaps we should consider adding host support to our dtc package and add
> host-dtc to the uboot dependencies?

Indeed that will help on older machines.
I haven't seen that failure just because I'm on latest Fedora so DTC is pretty recent.

I'll take a look at how to enable host DTC.

-Alexey

      reply	other threads:[~2015-08-31 16:43 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-31 14:04 [Buildroot] [PATCH] u-boot: 2015.07 - fix creation of .config Alexey Brodkin
2015-07-31 14:18 ` Thomas Petazzoni
2015-07-31 15:14   ` Alexey Brodkin
2015-08-03 11:16     ` Alexey Brodkin
2015-08-11 12:20       ` Alexey Brodkin
2015-08-12  8:23         ` Thomas Petazzoni
2015-08-16 21:43           ` Jörg Krause
2015-08-17  6:37             ` Alexey Brodkin
2015-08-17  9:31               ` Yann E. MORIN
2015-08-17  9:58                 ` Alexey Brodkin
2015-08-22 22:57           ` Arnout Vandecappelle
2015-08-28  9:36             ` Alexey Brodkin
2015-08-30 20:41             ` Peter Korsgaard
2015-08-30 20:51 ` Peter Korsgaard
2015-08-31 16:43   ` Alexey Brodkin [this message]

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=1441039400.2935.8.camel@synopsys.com \
    --to=alexey.brodkin@synopsys.com \
    --cc=buildroot@busybox.net \
    /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.