From: Max Filippov <jcmvbkbc@gmail.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 16/23] arch/xtensa: get rid of an intermediate blind kconfig option
Date: Tue, 14 Mar 2017 11:30:45 -0700 [thread overview]
Message-ID: <1489516252-3803-17-git-send-email-jcmvbkbc@gmail.com> (raw)
In-Reply-To: <1489516252-3803-1-git-send-email-jcmvbkbc@gmail.com>
From: "Yann E. MORIN" <yann.morin.1998@free.fr>
It is not needed to have an intermediate blind option, we can just
hide the prompt behind the same dependency as the non-blind symbol.
Update our qemu-xtensa defconfig acordingly (note: it was using
different values for both options, which is not possible; the blind
option was just set to the non-blind one in the .config).
Also remove an unneeded empty default for the BR2_XTENSA_OVERLAY_DIR
string option (strings are empty by default).
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
Signed-off-by: Max Filippov <jcmvbkbc@gmail.com>
---
Config.in.legacy | 11 +++++++++++
arch/Config.in.xtensa | 16 ++++++----------
configs/qemu_xtensa_lx60_defconfig | 3 +--
configs/qemu_xtensa_lx60_nommu_defconfig | 3 +--
4 files changed, 19 insertions(+), 14 deletions(-)
diff --git a/Config.in.legacy b/Config.in.legacy
index 090e60a..2261e0c 100644
--- a/Config.in.legacy
+++ b/Config.in.legacy
@@ -194,6 +194,17 @@ config BR2_GLIBC_VERSION_2_22
Support for glibc version 2.22 has been removed. The current
default version has been selected instead.
+config BR2_XTENSA_CUSTOM_NAME
+ string "The BR2_XTENSA_CUSTOM_NAME option has been renamed"
+ help
+ The BR2_XTENSA_CUSTOM_NAME option has been renamed
+ to BR2_XTENSA_CORE_NAME.
+
+config BR2_XTENSA_CUSTOM_NAME_WRAP
+ bool
+ default y if BR2_XTENSA_CUSTOM_NAME != ""
+ select BR2_LEGACY
+
###############################################################################
comment "Legacy options removed in 2017.02"
diff --git a/arch/Config.in.xtensa b/arch/Config.in.xtensa
index fcb3dc9..263af80 100644
--- a/arch/Config.in.xtensa
+++ b/arch/Config.in.xtensa
@@ -2,30 +2,26 @@ choice
prompt "Target Architecture Variant"
depends on BR2_xtensa
default BR2_xtensa_fsf
+
config BR2_XTENSA_CUSTOM
select BR2_ARCH_HAS_MMU_OPTIONAL
bool "Custom Xtensa processor configuration"
+
config BR2_xtensa_fsf
select BR2_ARCH_HAS_MMU_MANDATORY
bool "fsf - Default configuration"
+
endchoice
-config BR2_XTENSA_CUSTOM_NAME
- string "Custom Xtensa processor configuration name"
- depends on BR2_XTENSA_CUSTOM
- default ""
+config BR2_XTENSA_CORE_NAME
+ string "Custom Xtensa processor configuration name" if BR2_XTENSA_CUSTOM
+ default BR2_XTENSA_CUSTOM_NAME if BR2_XTENSA_CUSTOM_NAME != "" # legacy
help
Name given to a custom Xtensa processor configuration.
-config BR2_XTENSA_CORE_NAME
- string
- default BR2_XTENSA_CUSTOM_NAME if BR2_XTENSA_CUSTOM
- default "" if BR2_xtensa_fsf
-
config BR2_XTENSA_OVERLAY_DIR
string "Overlay directory for custom configuration"
depends on BR2_XTENSA_CUSTOM
- default ""
help
Provide the directory path that contains the overlay file
for a custom processor configuration. The path is relative
diff --git a/configs/qemu_xtensa_lx60_defconfig b/configs/qemu_xtensa_lx60_defconfig
index 54d844d..5c0cf8c 100644
--- a/configs/qemu_xtensa_lx60_defconfig
+++ b/configs/qemu_xtensa_lx60_defconfig
@@ -1,8 +1,7 @@
# Architecture
BR2_xtensa=y
BR2_XTENSA_CUSTOM=y
-BR2_XTENSA_CUSTOM_NAME="dc233c"
-BR2_XTENSA_CORE_NAME="lx60"
+BR2_XTENSA_CORE_NAME="dc233c"
BR2_XTENSA_OVERLAY_DIR="board/qemu/xtensa-lx60"
# Patches
diff --git a/configs/qemu_xtensa_lx60_nommu_defconfig b/configs/qemu_xtensa_lx60_nommu_defconfig
index 256cbee..97e5c98 100644
--- a/configs/qemu_xtensa_lx60_nommu_defconfig
+++ b/configs/qemu_xtensa_lx60_nommu_defconfig
@@ -1,8 +1,7 @@
# Architecture
BR2_xtensa=y
BR2_XTENSA_CUSTOM=y
-BR2_XTENSA_CUSTOM_NAME="dc233c"
-BR2_XTENSA_CORE_NAME="lx60"
+BR2_XTENSA_CORE_NAME="dc233c"
BR2_XTENSA_OVERLAY_DIR="board/qemu/xtensa-lx60"
# Toolchain
--
2.1.4
next prev parent reply other threads:[~2017-03-14 18:30 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-14 18:30 [Buildroot] [PATCH 00/23] xtensa overlay application rework Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 01/23] core: include arch-specific definitions Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 02/23] arch/xtensa: add definitions for the Xtensa overlay Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 03/23] package/binutils: use the Xtensa variables Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 04/23] package/gdb: " Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 05/23] package/gcc: " Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 06/23] arch/xtensa: add macro to extract the overlay Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 07/23] package/binutils: use macro to extract Xtensa overlay Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 08/23] package/gdb: " Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 09/23] package/gcc: " Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 10/23] package/binutils: extract Xtensa overlay as post-extract hook Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 11/23] package/gdb: " Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 12/23] arch/xtensa: add macros to generate hooks Max Filippov
2017-03-26 19:44 ` Thomas Petazzoni
2017-07-05 20:03 ` Arnout Vandecappelle
2017-07-06 8:27 ` Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 13/23] package/binutils: use the Xtensa macro generators Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 14/23] package/gdb: " Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 15/23] package/gcc: " Max Filippov
2017-03-14 18:30 ` Max Filippov [this message]
2017-03-14 18:30 ` [Buildroot] [PATCH 17/23] arch/xtensa: allow specifying path to tarball file Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 18/23] arch/xtensa: accept the overlay to be compressed Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 19/23] arch/xtensa: accept the overlay to be an URL Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 20/23] configs/qemu-xtensa: use overlay from the github repository Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 21/23] linux: apply xtensa overlay Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 22/23] uboot: " Max Filippov
2017-03-14 18:30 ` [Buildroot] [PATCH 23/23] linux: enable uBoot image type for xtensa kernel Max Filippov
2017-03-14 20:55 ` Thomas Petazzoni
2017-03-14 20:20 ` [Buildroot] [PATCH 00/23] xtensa overlay application rework Thomas Petazzoni
2017-03-14 23:07 ` Max Filippov
2017-03-26 19:41 ` Thomas Petazzoni
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=1489516252-3803-17-git-send-email-jcmvbkbc@gmail.com \
--to=jcmvbkbc@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox