From: Sascha Hauer <s.hauer@pengutronix.de>
To: Barebox List <barebox@lists.infradead.org>
Subject: state fixes/clarifications
Date: Thu, 4 May 2017 13:02:15 +0200 [thread overview]
Message-ID: <20170504110219.17929-1-s.hauer@pengutronix.de> (raw)
There are some common pitfalls when using the barebox state framework.
This series has some fixes clarifications to hopefully make usage easier.
First of all it doesn't seem to be clear that the state node in the device
tree the Kernel is started with is created by barebox; a previously existing
node will be replaced. For this reason we now issue a warning when a device
tree contains a state node which does not have any effect.
The userspace barebox-state utility needs a alias which for the state node
(it may work with a full path to the node, like /state, but this has some
additional surprises). We therefore make the alias mandatory so that we
can issue a clear error message when no alias exists and we can make
sure barebox-state will find an alias.
----------------------------------------------------------------
Sascha Hauer (4):
state: Binding: remove @0 from node name
state: warn when a state node will be overwritten
state: Make an alias mandatory
state: Create alias in of_state_fixup()
.../devicetree/bindings/barebox/barebox,state.rst | 8 ++++++-
common/state/state.c | 28 +++++++++++++++++++---
2 files changed, 32 insertions(+), 4 deletions(-)
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next reply other threads:[~2017-05-04 11:02 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-04 11:02 Sascha Hauer [this message]
2017-05-04 11:02 ` [PATCH 1/4] state: Binding: remove @0 from node name Sascha Hauer
2017-05-04 11:02 ` [PATCH 2/4] state: warn when a state node will be overwritten Sascha Hauer
2017-05-08 6:15 ` Uwe Kleine-König
2017-05-08 11:58 ` Sascha Hauer
2017-05-04 11:02 ` [PATCH 3/4] state: Make an alias mandatory Sascha Hauer
2017-05-04 11:02 ` [PATCH 4/4] state: Create alias in of_state_fixup() 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=20170504110219.17929-1-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