From: Herve Codina <herve.codina@bootlin.com>
To: David Gibson <david@gibson.dropbear.id.au>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk@kernel.org>,
Conor Dooley <conor+dt@kernel.org>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
David Lechner <dlechner@baylibre.com>,
Ayush Singh <ayush@beagleboard.org>,
Geert Uytterhoeven <geert@linux-m68k.org>,
devicetree-compiler@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, devicetree-spec@vger.kernel.org,
Hui Pu <hui.pu@gehealthcare.com>,
Ian Ray <ian.ray@gehealthcare.com>,
Luca Ceresoli <luca.ceresoli@bootlin.com>,
Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
Herve Codina <herve.codina@bootlin.com>
Subject: [PATCH v3 13/15] tests: Add wip_func utility
Date: Wed, 26 Aug 2026 10:31:44 +0200 [thread overview]
Message-ID: <20260826083146.304291-14-herve.codina@bootlin.com> (raw)
In-Reply-To: <20260826083146.304291-1-herve.codina@bootlin.com>
wip_func is a test utility allowing to wrap the write inplace functions.
It allows to create tests with custom wip write sequence.
Also the modified dtb is saved allowing to perform later checks on this
modified dtb.
Signed-off-by: Herve Codina <herve.codina@bootlin.com>
---
tests/Makefile.tests | 3 +-
tests/meson.build | 1 +
tests/wip_func.c | 127 +++++++++++++++++++++++++++++++++++++++++++
3 files changed, 130 insertions(+), 1 deletion(-)
create mode 100644 tests/wip_func.c
diff --git a/tests/Makefile.tests b/tests/Makefile.tests
index 0d4d399f..a71c4cbf 100644
--- a/tests/Makefile.tests
+++ b/tests/Makefile.tests
@@ -31,7 +31,8 @@ LIB_TESTS_L = get_mem_rsv \
check_path check_header check_full \
fs_tree1 \
truncated_property truncated_string \
- truncated_memrsv unterminated_memrsv
+ truncated_memrsv unterminated_memrsv \
+ wip_func
LIB_TESTS = $(LIB_TESTS_L:%=$(TESTS_PREFIX)%)
ifneq ($(STATIC_BUILD),1)
diff --git a/tests/meson.build b/tests/meson.build
index dae6a941..e7a88936 100644
--- a/tests/meson.build
+++ b/tests/meson.build
@@ -96,6 +96,7 @@ tests = [
'truncated_string',
'unterminated_memrsv',
'utilfdt_test',
+ 'wip_func',
]
test_deps = [testutil_dep, util_dep, libfdt_dep]
diff --git a/tests/wip_func.c b/tests/wip_func.c
new file mode 100644
index 00000000..3becc4f1
--- /dev/null
+++ b/tests/wip_func.c
@@ -0,0 +1,127 @@
+// SPDX-License-Identifier: LGPL-2.1-or-later
+/*
+ * libfdt - Flat Device Tree manipulation
+ * Wrapper wip functions family
+ * Copyright (C) 2026 Herve Codina, Bootlin.
+ */
+
+#include <inttypes.h>
+#include <stdlib.h>
+#include <stdio.h>
+#include <string.h>
+#include <ctype.h>
+#include <stdint.h>
+#include <errno.h>
+
+#include <libfdt.h>
+
+#include "tests.h"
+
+struct args {
+ char *dtb_path;
+ enum {
+ NOP_NODE,
+ NOP_PROP,
+ SET_PROP,
+ } op;
+ char *node_path;
+ char *prop_name;
+ int index;
+ char *value;
+};
+
+const char *usage_txt = "Usage:\n"
+ " - wip_func <dtb> nop_node <node_path>\n"
+ " - wip_func <dtb> nop_prop <node_path> <prop_name>\n"
+ " - wip_func <dtb> set_prop <node_path> <prop_name> <index> <str>\n"
+ ;
+
+static void parse_args(struct args *args, int argc, char *argv[])
+{
+ char *endptr;
+
+ memset(args, 0, sizeof(*args));
+
+ if (argc < 3)
+ goto fail;
+
+ args->dtb_path = argv[1];
+
+ if (!strcmp(argv[2], "nop_node")) {
+ args->op = NOP_NODE;
+ if (argc != 4)
+ goto fail;
+ args->node_path = argv[3];
+ } else if (!strcmp(argv[2], "nop_prop")) {
+ args->op = NOP_PROP;
+ if (argc != 5)
+ goto fail;
+ args->node_path = argv[3];
+ args->prop_name = argv[4];
+ } else if (!strcmp(argv[2], "set_prop")) {
+ args->op = SET_PROP;
+ if (argc != 7)
+ goto fail;
+ args->node_path = argv[3];
+ args->prop_name = argv[4];
+ errno = 0;
+ args->index = strtol(argv[5], &endptr, 0);
+ if (*endptr != '\0' || errno)
+ goto fail;
+ args->value = argv[6];
+ } else {
+ goto fail;
+ }
+
+ return;
+
+fail:
+ CONFIG("%s\n", usage_txt);
+}
+
+int main(int argc, char *argv[])
+{
+ struct args args;
+ void *fdt;
+ int node;
+ int err;
+
+ test_init(argc, argv);
+
+ parse_args(&args, argc, argv);
+
+ fdt = load_blob(args.dtb_path);
+
+ node = fdt_path_offset(fdt, args.node_path);
+ if (node < 0)
+ FAIL("Fail to get '%s': %s", args.node_path, fdt_strerror(node));
+
+ switch (args.op) {
+ case NOP_NODE:
+ err = fdt_nop_node(fdt, node);
+ if (err)
+ FAIL("fdt_nop_node('%s'): %s", args.node_path, fdt_strerror(err));
+ break;
+
+ case NOP_PROP:
+ err = fdt_nop_property(fdt, node, args.prop_name);
+ if (err)
+ FAIL("fdt_nop_node('%s', '%s'): %s", args.node_path,
+ args.prop_name, fdt_strerror(err));
+ break;
+
+ case SET_PROP:
+ err = fdt_setprop_inplace_namelen_partial(fdt, node,
+ args.prop_name, strlen(args.prop_name), args.index,
+ args.value, strlen(args.value) + 1);
+ if (err)
+ FAIL("fdt_setprop_inplace_namelen_partial('%s', '%s', %d, \"%s\"): %s",
+ args.node_path, args.prop_name, args.index,
+ args.value, fdt_strerror(err));
+ break;
+ }
+
+ save_blob(args.dtb_path, fdt);
+
+ PASS();
+}
--
2.55.0
next prev parent reply other threads:[~2026-08-26 8:33 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-26 8:31 [PATCH v3 00/15] Add support for structured tags and v18 dtb version Herve Codina
2026-08-26 8:31 ` [PATCH v3 01/15] fdtget: Use libfdt iterators instead of open coded loops Herve Codina
2026-08-27 3:55 ` David Gibson
2026-08-26 8:31 ` [PATCH v3 02/15] libfdt: Don't assume the root node is available at offset 0 Herve Codina
2026-08-26 8:31 ` [PATCH v3 03/15] tests: " Herve Codina
2026-08-26 8:31 ` [PATCH v3 04/15] tests/nopulate: Add a FDT_NOP before the root node Herve Codina
2026-08-26 8:31 ` [PATCH v3 05/15] tests: treegen: Introduce emit_fdt_header_vers() Herve Codina
2026-08-26 8:31 ` [PATCH v3 06/15] Introduce structured tag value definition Herve Codina
2026-08-26 8:31 ` [PATCH v3 07/15] fdtdump: Handle unknown tags Herve Codina
2026-08-26 8:31 ` [PATCH v3 08/15] flattree: " Herve Codina
2026-08-26 8:31 ` [PATCH v3 09/15] libfdt: Handle unknown tags in fdt_next_tag() Herve Codina
2026-08-26 8:31 ` [PATCH v3 10/15] libfdt: Introduce fdt_ptr_offset_() Herve Codina
2026-08-26 8:31 ` [PATCH v3 11/15] libfdt: Introduce fdt_getprop_by_offset_w() Herve Codina
2026-08-26 8:31 ` [PATCH v3 12/15] libfdt: Introduce fdt_getprop_offset_namelen() Herve Codina
2026-08-26 8:31 ` Herve Codina [this message]
2026-08-26 8:31 ` [PATCH v3 14/15] libfdt: Handle unknown tags on dtb modifications Herve Codina
2026-08-26 8:31 ` [PATCH v3 15/15] Introduce v18 dtb version Herve Codina
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=20260826083146.304291-14-herve.codina@bootlin.com \
--to=herve.codina@bootlin.com \
--cc=ayush@beagleboard.org \
--cc=conor+dt@kernel.org \
--cc=david@gibson.dropbear.id.au \
--cc=devicetree-compiler@vger.kernel.org \
--cc=devicetree-spec@vger.kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dlechner@baylibre.com \
--cc=geert@linux-m68k.org \
--cc=hui.pu@gehealthcare.com \
--cc=ian.ray@gehealthcare.com \
--cc=krzk@kernel.org \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-kernel@vger.kernel.org \
--cc=luca.ceresoli@bootlin.com \
--cc=robh@kernel.org \
--cc=thomas.petazzoni@bootlin.com \
/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