Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni via buildroot <buildroot@buildroot.org>
To: Julien Olivain <ju.o@free.fr>
Cc: buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH 1/1] support/config-fragments/autobuild: fix bootlin-xtensa-uclibc.config
Date: Mon, 31 Oct 2022 22:07:13 +0100	[thread overview]
Message-ID: <20221031220713.5c81cc21@windsurf> (raw)
In-Reply-To: <20221010030937.6432-1-ju.o@free.fr>

Hello Julien,

On Sun,  9 Oct 2022 23:09:37 -0400
Julien Olivain <ju.o@free.fr> wrote:

> Commit 4cbf7336914f25478aea943456ba7dc3c892c21a
> "arch/xtensa: custom configuration requires an overlay"
> added an error message if BR2_XTENSA_CUSTOM is set and
> BR_ARCH_XTENSA_OVERLAY_FILE is unset.
> 
> When testing a package with the command:
> 
>     ./utils/test-pkg --all --package somepackage
> 
> bootlin-xtensa-uclibc toolchain is always failing with error in the logfile:
> 
>     arch/arch.mk.xtensa:4: *** No xtensa overlay file provided. Check your BR2_XTENSA_OVERLAY_FILE setting.  Stop.
> 
> This commit fixes the toolchain config fragment by adding the same overlay
> file found in qemu_xtensa_lx60_defconfig.
> 
> Signed-off-by: Julien Olivain <ju.o@free.fr>
> ---
>  support/config-fragments/autobuild/bootlin-xtensa-uclibc.config | 1 +
>  1 file changed, 1 insertion(+)

Thanks, but adding an overlay here is not the right solution. The
problem is that 4cbf7336914f25478aea943456ba7dc3c892c21a is incorrect.
I already sent a series to fix that on Sep 25, 2022:

  https://lore.kernel.org/buildroot/20220925090920.1585682-1-thomas.petazzoni@bootlin.com/

but it did receive some comments, and therefore was not applied. But
the solution is more along the lines of changing how Xtensa is handled
rather than working around the problem in bootlin-xtensa-uclibc.config.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

      reply	other threads:[~2022-10-31 21:07 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-10-10  3:09 [Buildroot] [PATCH 1/1] support/config-fragments/autobuild: fix bootlin-xtensa-uclibc.config Julien Olivain
2022-10-31 21:07 ` Thomas Petazzoni via buildroot [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=20221031220713.5c81cc21@windsurf \
    --to=buildroot@buildroot.org \
    --cc=ju.o@free.fr \
    --cc=thomas.petazzoni@bootlin.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox