From: Ian Campbell <ijc@hellion.org.uk>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH for-next 0/3] sunxi: Kconfig consolidation and cleanup
Date: Sat, 04 Oct 2014 09:47:46 +0100 [thread overview]
Message-ID: <1412412466.17796.65.camel@hellion.org.uk> (raw)
A few simple patches which consolidate some Kconfig options and make
SPL_FEL a first class .config member instead of putting it in
CONFIG_SYS_EXTRA_OPTIONS.
This means it is possible to do a FEL build for any board with:
make <board>_config
echo CONFIG_SPL_FEL=y >> .config
echo CONFIG_SPL_FEL=y >> spl/.config
or better
make <board>_config
sed -i -e 's/\# CONFIG_SPL_FEL is not set/CONFIG_SPL_FEL=y/g' .config spl/.config
which is still one more step than I would like but still preferable IMHO
to:
sed -i -e 's/^CONFIG_SYS_EXTRA_OPTIONS="[^"]*/&,SPL_FEL/g' .config spl/.config
I've been using a little hack (below) to MAKEALL to help build test
this.
There's still plenty of stuff in sunxi CONFIG_SYS_EXTRA_OPTIONS which
could be moved out, but most of it is dependent on drivers/*/Kconfig
switching first.
Probably the board and power controller selection could be moved out
without any dependencies, although the board one in particular will be
quite a big patch I think it would be worth it.
Ian.
My MAKEALL hack, surely not upstreamable ;-)
diff --git a/MAKEALL b/MAKEALL
index 7c16319..3a8d1d0 100755
--- a/MAKEALL
+++ b/MAKEALL
@@ -660,6 +660,11 @@ build_target() {
echo "Building ${target} board..."
${MAKE} -s ${target}_defconfig >/dev/null
+ if [ -n "$SUNXI_FORCE_FEL" ] ; then
+ sed -i -e 's/\# CONFIG_SPL_FEL is not set/CONFIG_SPL_FEL=y/g' \
+ .config spl/.config
+ fi
+
${MAKE} ${JOBS} ${CHECK} all \
>${LOG_DIR}/$target.MAKELOG 2> ${LOG_DIR}/$target.ERR
next reply other threads:[~2014-10-04 8:47 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-04 8:47 Ian Campbell [this message]
2014-10-04 8:48 ` [U-Boot] [PATCH for-next 1/3] sunxi: Kconfig: Consolidate SYS_CONFIG_NAME settings Ian Campbell
2014-10-06 7:48 ` Hans de Goede
2014-10-06 8:23 ` Ian Campbell
2014-10-04 8:48 ` [U-Boot] [PATCH for-next 2/3] sunxi: kconfig: Add top-level TARGET_SUNXI Ian Campbell
2014-10-05 2:37 ` Chen-Yu Tsai
2014-10-06 1:39 ` Masahiro Yamada
2014-10-06 8:27 ` Ian Campbell
2014-10-06 10:54 ` Masahiro Yamada
2014-10-22 19:14 ` Ian Campbell
2014-10-24 11:46 ` Masahiro Yamada
2014-10-24 13:22 ` Ian Campbell
2014-10-24 14:04 ` Hans de Goede
2014-10-26 16:55 ` Masahiro YAMADA
2014-10-06 7:55 ` Hans de Goede
2014-10-04 8:48 ` [U-Boot] [PATCH for-next 3/3] sunxi: Kconfig: Make SPL_FEL a toplevel Kconfig option Ian Campbell
2014-10-06 7:58 ` Hans de Goede
2014-10-06 8:28 ` Ian Campbell
2014-10-06 8:43 ` Hans de Goede
2014-10-04 9:32 ` [U-Boot] [PATCH for-next 0/3] sunxi: Kconfig consolidation and cleanup Ian Campbell
2014-10-06 2:16 ` Masahiro Yamada
2014-10-06 8:22 ` Ian Campbell
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=1412412466.17796.65.camel@hellion.org.uk \
--to=ijc@hellion.org.uk \
--cc=u-boot@lists.denx.de \
/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.