From: "Uwe Kleine-König" <u.kleine-koenig-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
To: David Gibson <david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+@public.gmane.org>
Cc: devicetree-compiler-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: [PATCH] dtc: When compiling to dts interpret /__symbols__ and /__local_fixups__
Date: Wed, 26 Apr 2023 20:25:58 +0200 [thread overview]
Message-ID: <20230426182558.573161-1-u.kleine-koenig@pengutronix.de> (raw)
The data contained in these nodes can be used to restore labels for nodes
(from __symbols__ if -@ is given) and to identify which data values are
references (from __local_fixups__ if -L is given).
The resulting dts doesn't necessarily compiles back into the bitwise same
dtb, but the result is equivalent and easier to modify without breaking
references.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
---
dtc.c | 16 +++++++---
dtc.h | 2 ++
livetree.c | 90 ++++++++++++++++++++++++++++++++++++++++++++++++++++
treesource.c | 37 +++++++++++++++------
4 files changed, 132 insertions(+), 13 deletions(-)
diff --git a/dtc.c b/dtc.c
index d2e4e2b55b5c..e36f0dfad313 100644
--- a/dtc.c
+++ b/dtc.c
@@ -335,12 +335,20 @@ int main(int argc, char *argv[])
if (auto_label_aliases)
generate_label_tree(dti, "aliases", false);
- if (generate_symbols)
- generate_label_tree(dti, "__symbols__", true);
+ if (generate_symbols) {
+ if (streq(inform, "fs") || streq(inform, "dtb"))
+ generate_labels_from_tree(dti, "__symbols__");
+ else
+ generate_label_tree(dti, "__symbols__", true);
+ }
if (generate_fixups) {
- generate_fixups_tree(dti, "__fixups__");
- generate_local_fixups_tree(dti, "__local_fixups__");
+ if (streq(inform, "fs") || streq(inform, "dtb")) {
+ fixup_local_phandles(dti, "__local_fixups__");
+ } else {
+ generate_fixups_tree(dti, "__fixups__");
+ generate_local_fixups_tree(dti, "__local_fixups__");
+ }
}
if (sort)
diff --git a/dtc.h b/dtc.h
index 4c4aaca1fc41..4c278ed30b26 100644
--- a/dtc.h
+++ b/dtc.h
@@ -337,8 +337,10 @@ struct dt_info *build_dt_info(unsigned int dtsflags,
struct node *tree, uint32_t boot_cpuid_phys);
void sort_tree(struct dt_info *dti);
void generate_label_tree(struct dt_info *dti, const char *name, bool allocph);
+void generate_labels_from_tree(struct dt_info *dti, const char *name);
void generate_fixups_tree(struct dt_info *dti, const char *name);
void generate_local_fixups_tree(struct dt_info *dti, const char *name);
+void fixup_local_phandles(struct dt_info *dti, const char *name);
/* Checks */
diff --git a/livetree.c b/livetree.c
index 0ec47ed41e2b..a4fa5278718e 100644
--- a/livetree.c
+++ b/livetree.c
@@ -1056,6 +1056,25 @@ void generate_label_tree(struct dt_info *dti, const char *name, bool allocph)
dti->dt, allocph);
}
+void generate_labels_from_tree(struct dt_info *dti, const char *name)
+{
+ struct node *an;
+ struct property *p;
+
+ an = get_subnode(dti->dt, name);
+ if (!an)
+ return;
+
+ for_each_property(an, p) {
+ struct node *labeled_node;
+
+ labeled_node = get_node_by_path(dti->dt, p->val.val);
+ add_label(&labeled_node->labels, p->name);
+ }
+
+ delete_node(an);
+}
+
void generate_fixups_tree(struct dt_info *dti, const char *name)
{
if (!any_fixup_tree(dti, dti->dt))
@@ -1071,3 +1090,74 @@ void generate_local_fixups_tree(struct dt_info *dti, const char *name)
generate_local_fixups_tree_internal(dti, build_root_node(dti->dt, name),
dti->dt);
}
+
+static void fixup_local_phandles_node(struct node *lf, struct node *n)
+{
+ struct property *lfp;
+ struct node *lfsubnode;
+
+ for_each_property(lf, lfp) {
+ struct property *p = get_property(n, lfp->name);
+ size_t i;
+
+ if (!p || p->val.len % sizeof(fdt32_t))
+ die("invalid length of property %s in node %s\n", lfp->name, lf->fullpath);
+
+ for (i = 0; i < lfp->val.len; i += sizeof(fdt32_t)) {
+ struct marker *m, **mi;
+
+ m = xmalloc(sizeof(*m));
+ m->offset = fdt32_to_cpu(*(fdt32_t *)(lfp->val.val + i));
+ m->type = REF_PHANDLE;
+ m->ref = NULL;
+
+ /* keep the list sorted in ascending ->offset order */
+ for (mi = &p->val.markers; *mi && (*mi)->offset < m->offset;)
+ mi = &(*mi)->next;
+
+ m->next = *mi;
+ *mi = m;
+ }
+ }
+
+ for_each_child(lf, lfsubnode) {
+ struct node *subnode = get_subnode(n, lfsubnode->name);
+
+ if (!subnode)
+ die("fixup for non-existent node in %s\n", lfsubnode->fullpath);
+
+ fixup_local_phandles_node(lfsubnode, subnode);
+ }
+}
+
+static void drop_phandles(struct node *n)
+{
+ struct property *p;
+ struct node *sn;
+
+ p = get_property(n, "phandle");
+ if (p)
+ delete_property(p);
+
+ p = get_property(n, "linux,phandle");
+ if (p)
+ delete_property(p);
+
+ for_each_child(n, sn)
+ drop_phandles(sn);
+}
+
+void fixup_local_phandles(struct dt_info *dti, const char *name)
+{
+ struct node *an;
+
+ an = get_subnode(dti->dt, name);
+ if (!an)
+ return;
+
+ fixup_local_phandles_node(an, dti->dt);
+
+ drop_phandles(dti->dt);
+
+ delete_node(an);
+}
diff --git a/treesource.c b/treesource.c
index de30188189fb..32f3171c14c3 100644
--- a/treesource.c
+++ b/treesource.c
@@ -172,7 +172,7 @@ static enum markertype guess_value_type(struct property *prop)
return TYPE_UINT8;
}
-static void write_propval(FILE *f, struct property *prop)
+static void write_propval(FILE *f, struct node *root, struct property *prop)
{
size_t len = prop->val.len;
struct marker *m = prop->val.markers;
@@ -219,6 +219,9 @@ static void write_propval(FILE *f, struct property *prop)
if (emit_type == TYPE_NONE || chunk_len == 0)
continue;
+ if (m->offset != 0)
+ fputc(' ', f);
+
switch(emit_type) {
case TYPE_UINT16:
write_propval_int(f, p, chunk_len, 2);
@@ -230,10 +233,26 @@ static void write_propval(FILE *f, struct property *prop)
break;
if (m_phandle) {
- if (m_phandle->ref[0] == '/')
- fprintf(f, "&{%s}", m_phandle->ref);
- else
- fprintf(f, "&%s", m_phandle->ref);
+ if (m_phandle->ref) {
+ if (m_phandle->ref[0] == '/')
+ fprintf(f, "&{%s}", m_phandle->ref);
+ else
+ fprintf(f, "&%s", m_phandle->ref);
+ } else {
+ cell_t phandle = fdt32_to_cpu(*(const fdt32_t *)p);
+ struct node *n = get_node_by_phandle(root, phandle);
+
+ if (!n) {
+ fprintf(f, "&??");
+ } else {
+ if (n->labels) {
+ fprintf(f, "&%s", n->labels->label);
+ } else {
+ fprintf(f, "&{%s}", n->fullpath);
+ }
+ }
+
+ }
if (chunk_len > 4) {
fputc(' ', f);
write_propval_int(f, p + 4, chunk_len - 4, 4);
@@ -270,7 +289,7 @@ static void write_propval(FILE *f, struct property *prop)
fprintf(f, "\n");
}
-static void write_tree_source_node(FILE *f, struct node *tree, int level)
+static void write_tree_source_node(FILE *f, struct node *root, struct node *tree, int level)
{
struct property *prop;
struct node *child;
@@ -299,11 +318,11 @@ static void write_tree_source_node(FILE *f, struct node *tree, int level)
for_each_label(prop->labels, l)
fprintf(f, "%s: ", l->label);
fprintf(f, "%s", prop->name);
- write_propval(f, prop);
+ write_propval(f, root, prop);
}
for_each_child(tree, child) {
fprintf(f, "\n");
- write_tree_source_node(f, child, level+1);
+ write_tree_source_node(f, root, child, level+1);
}
write_prefix(f, level);
fprintf(f, "};");
@@ -333,5 +352,5 @@ void dt_to_source(FILE *f, struct dt_info *dti)
(unsigned long long)re->size);
}
- write_tree_source_node(f, dti->dt, 0);
+ write_tree_source_node(f, dti->dt, dti->dt, 0);
}
--
2.39.2
next reply other threads:[~2023-04-26 18:25 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-04-26 18:25 Uwe Kleine-König [this message]
[not found] ` <20230426182558.573161-1-u.kleine-koenig-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2023-04-28 6:11 ` [PATCH] dtc: When compiling to dts interpret /__symbols__ and /__local_fixups__ David Gibson
2023-04-28 19:11 ` Uwe Kleine-König
[not found] ` <20230428191130.nuibhzf3zjoqwpwm-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2023-05-03 14:06 ` David Gibson
2023-05-03 21:05 ` Uwe Kleine-König
[not found] ` <20230503210535.denueqhecvuedppv-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2023-05-14 6:30 ` David Gibson
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=20230426182558.573161-1-u.kleine-koenig@pengutronix.de \
--to=u.kleine-koenig-bicnvbalz9megne8c9+irq@public.gmane.org \
--cc=david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+@public.gmane.org \
--cc=devicetree-compiler-u79uwXL29TY76Z2rM5mHXA@public.gmane.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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).