From: Sascha Hauer <s.hauer@pengutronix.de>
To: Barebox List <barebox@lists.infradead.org>
Subject:
Date: Fri, 22 Jul 2016 14:44:14 +0200 [thread overview]
Message-ID: <1469191472-14491-1-git-send-email-s.hauer@pengutronix.de> (raw)
From Sascha Hauer <s.hauer@pengutronix.de> # This line is ignored.
From: Sascha Hauer <s.hauer@pengutronix.de>
Subject: rework boot/blspec code
In-Reply-To:
The boot command code is for converting a list of strings into boot
entries and iterating over the boot entries starting one after the
other until one succeeds. boot entries can be boot scripts or bootloader
spec entries, nevertheless the boot code uses a "struct blspec" for
both. This series separated this properly, puts the bootloader spec
specific data into its own struct leaving the generic data for the boot
code.
As a central component the boot code should also have a C API, which it
currently does not have as the it's implemented in the boot command
code, so this series move the bulk of the code to a place under common/
where it can be called by C code aswell.
Also several other cleanups are done like (hopefully) more consistent
message printing.
Sascha
----------------------------------------------------------------
Sascha Hauer (18):
blspec: remove unused blspec_boot_devicename
blspec: Remove once/default handling
blspec: remove unused function prototype
boot: Call blspec_scan_directory() only on strings containing an absolute path
include: Move bulk of boot.h to bootm.h
blpec: rename struct lspec -> bootentries
blspec: factor out a struct bootentry
bootentries: Add title/description
blspec: separate bootentries from blspec entries
blspec: Make blspec_boot static
bootentries: Move menu display string allocation to bootentries_alloc()
bootentries: Move struct bootentries to include/boot.h
boot: Use struct bootentries to pass around data
boot: Move code to common/
boot: add single quotes when printing boot target names
boot command: Explicitly complain when boot target list is empty
blspec: Turn message back to debug level
boot: Print a message when a boot target string does not lead to a boot target
arch/arm/boards/archosg9/archos_features.c | 1 -
arch/arm/lib/bootm.c | 1 +
commands/Kconfig | 1 +
commands/boot.c | 415 ++---------------------------
commands/bootm.c | 2 +-
common/Kconfig | 3 +
common/Makefile | 1 +
common/blspec.c | 343 ++++++++++--------------
common/boot.c | 339 +++++++++++++++++++++++
common/bootm.c | 3 +-
common/image-fit.c | 2 +-
drivers/usb/gadget/f_fastboot.c | 2 +-
include/blspec.h | 91 +------
include/boot.h | 139 ++--------
include/bootm.h | 125 +++++++++
include/image-fit.h | 2 +-
16 files changed, 676 insertions(+), 794 deletions(-)
create mode 100644 common/boot.c
create mode 100644 include/bootm.h
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next reply other threads:[~2016-07-22 12:44 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-07-22 12:44 Sascha Hauer [this message]
2016-07-22 12:44 ` [PATCH 01/18] blspec: remove unused blspec_boot_devicename Sascha Hauer
2016-07-22 12:44 ` [PATCH 02/18] blspec: Remove once/default handling Sascha Hauer
2016-07-22 12:44 ` [PATCH 03/18] blspec: remove unused function prototype Sascha Hauer
2016-07-22 12:44 ` [PATCH 04/18] boot: Call blspec_scan_directory() only on strings containing an absolute path Sascha Hauer
2016-07-22 12:44 ` [PATCH 05/18] include: Move bulk of boot.h to bootm.h Sascha Hauer
2016-07-22 12:44 ` [PATCH 06/18] blpec: rename struct lspec -> bootentries Sascha Hauer
2016-07-22 12:44 ` [PATCH 07/18] blspec: factor out a struct bootentry Sascha Hauer
2016-07-22 12:44 ` [PATCH 08/18] bootentries: Add title/description Sascha Hauer
2016-07-22 12:44 ` [PATCH 09/18] blspec: separate bootentries from blspec entries Sascha Hauer
2016-07-22 12:44 ` [PATCH 10/18] blspec: Make blspec_boot static Sascha Hauer
2016-07-22 12:44 ` [PATCH 11/18] bootentries: Move menu display string allocation to bootentries_alloc() Sascha Hauer
2016-07-22 12:44 ` [PATCH 12/18] bootentries: Move struct bootentries to include/boot.h Sascha Hauer
2016-07-22 12:44 ` [PATCH 13/18] boot: Use struct bootentries to pass around data Sascha Hauer
2016-07-22 12:44 ` [PATCH 14/18] boot: Move code to common/ Sascha Hauer
2016-07-22 12:44 ` [PATCH 15/18] boot: add single quotes when printing boot target names Sascha Hauer
2016-07-22 12:44 ` [PATCH 16/18] boot command: Explicitly complain when boot target list is empty Sascha Hauer
2016-07-22 12:44 ` [PATCH 17/18] blspec: Turn message back to debug level Sascha Hauer
2016-07-22 12:44 ` [PATCH 18/18] boot: Print a message when a boot target string does not lead to a boot target Sascha Hauer
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=1469191472-14491-1-git-send-email-s.hauer@pengutronix.de \
--to=s.hauer@pengutronix.de \
--cc=barebox@lists.infradead.org \
/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