mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
* [ptxdist] [PATCH v2] dtc: Version bump. 1.7.2 -> 1.8.1
@ 2026-06-11 16:42 Christian Melki
  0 siblings, 0 replies; only message in thread
From: Christian Melki @ 2026-06-11 16:42 UTC (permalink / raw)
  To: ptxdist

Looks like mostly minor things.
https://github.com/dgibson/dtc/compare/v1.7.2...v1.8.1

* License hash changed. Typo fixes.

* Remove old patch, already in upstream.

* Add patch for ARM (32-bit) build failure.
See https://github.com/dgibson/dtc/issues/186

Signed-off-by: Christian Melki <christian.melki@t2data.com>
---
 .../0001-Fix-discarded-const-qualifiers.patch | 80 -------------------
 ...omparison-of-integers-with-different.patch | 37 +++++++++
 patches/dtc-1.8.1/series                      |  4 +
 rules/dtc.make                                |  6 +-
 4 files changed, 44 insertions(+), 83 deletions(-)
 delete mode 100644 patches/dtc-1.7.2/0001-Fix-discarded-const-qualifiers.patch
 create mode 100644 patches/dtc-1.8.1/0001-livetree-Fix-a-comparison-of-integers-with-different.patch
 create mode 100644 patches/dtc-1.8.1/series

diff --git a/patches/dtc-1.7.2/0001-Fix-discarded-const-qualifiers.patch b/patches/dtc-1.7.2/0001-Fix-discarded-const-qualifiers.patch
deleted file mode 100644
index 766dbccc2..000000000
--- a/patches/dtc-1.7.2/0001-Fix-discarded-const-qualifiers.patch
+++ /dev/null
@@ -1,80 +0,0 @@
-From: Stephen Gallagher <sgallagh@redhat.com>
-Date: Tue, 6 Jan 2026 14:19:30 -0500
-Subject: [PATCH] Fix discarded const qualifiers
-
-It's unsafe to implicitly discard the const qualifier on a pointer. In
-overlay_fixup_phandle(), this was probably just an oversight, and making
-the "sep" variable a const char * is sufficient to fix it.
-
-In create_node(), however, the "p" variable is directly modifying the
-buffer pointed to by "const char* node_name". To fix this, we need to
-actually make a duplicate of the buffer and operate on that instead.
-
-This introduces a malloc()/free()  and an unbounded strdup() into the
-operation, but fdtput isn't a long-running service and the node_name
-argument comes directly from argv, so this shouldn't introduce a
-significant performance impact.
-
-Signed-off-by: Stephen Gallagher <sgallagh@redhat.com>
-Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
----
- fdtput.c             | 8 +++++---
- libfdt/fdt_overlay.c | 3 ++-
- meson.build          | 1 +
- 3 files changed, 8 insertions(+), 4 deletions(-)
-
-diff --git a/fdtput.c b/fdtput.c
-index c2fecf45a977..8deec7eebcf5 100644
---- a/fdtput.c
-+++ b/fdtput.c
-@@ -230,19 +230,21 @@ static int create_paths(char **blob, const char *in_path)
- static int create_node(char **blob, const char *node_name)
- {
- 	int node = 0;
--	char *p;
-+	const char *p;
-+	char *path = NULL;
- 
- 	p = strrchr(node_name, '/');
- 	if (!p) {
- 		report_error(node_name, -1, -FDT_ERR_BADPATH);
- 		return -1;
- 	}
--	*p = '\0';
- 
- 	*blob = realloc_node(*blob, p + 1);
- 
- 	if (p > node_name) {
--		node = fdt_path_offset(*blob, node_name);
-+		path = xstrndup(node_name, (size_t)(p - node_name));
-+		node = fdt_path_offset(*blob, path);
-+		free(path);
- 		if (node < 0) {
- 			report_error(node_name, -1, node);
- 			return -1;
-diff --git a/libfdt/fdt_overlay.c b/libfdt/fdt_overlay.c
-index 28b667ffc490..5f8aa62b2686 100644
---- a/libfdt/fdt_overlay.c
-+++ b/libfdt/fdt_overlay.c
-@@ -409,7 +409,8 @@ static int overlay_fixup_phandle(void *fdt, void *fdto, int symbols_off,
- 		const char *fixup_str = value;
- 		uint32_t path_len, name_len;
- 		uint32_t fixup_len;
--		char *sep, *endptr;
-+		const char *sep;
-+		char *endptr;
- 		int poffset, ret;
- 
- 		fixup_end = memchr(value, '\0', len);
-diff --git a/meson.build b/meson.build
-index 310699f0c874..2966b0614a69 100644
---- a/meson.build
-+++ b/meson.build
-@@ -18,6 +18,7 @@ add_project_arguments(
-     '-Wshadow',
-     '-Wsuggest-attribute=format',
-     '-Wwrite-strings',
-+    '-Wdiscarded-qualifiers',
-   ]),
-   language: 'c'
- )
diff --git a/patches/dtc-1.8.1/0001-livetree-Fix-a-comparison-of-integers-with-different.patch b/patches/dtc-1.8.1/0001-livetree-Fix-a-comparison-of-integers-with-different.patch
new file mode 100644
index 000000000..d14fdc1af
--- /dev/null
+++ b/patches/dtc-1.8.1/0001-livetree-Fix-a-comparison-of-integers-with-different.patch
@@ -0,0 +1,37 @@
+From: =?UTF-8?q?Uwe=20Kleine-K=C3=B6nig?= <u.kleine-koenig@baylibre.com>
+Date: Mon, 8 Jun 2026 18:33:05 +0200
+Subject: [PATCH] livetree: Fix a comparison of integers with different
+ signedness
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+On some architectures we have:
+
+	livetree.c: In function 'fixup_phandles':
+	livetree.c:1237:54: error: comparison of integer expressions of different signedness: 'long int' and 'unsigned int' [-Werror=sign-compare]
+	 1237 |                         if (offset < 0 || offset + 4 > p->val.len) {
+	      |                                                      ^
+	cc1: all warnings being treated as errors
+	make: *** [Makefile:307: livetree.o] Error 1
+
+Fixes: a26ef6400bd8 ("Restore phandle references from __fixups__ node")
+Closes: https://github.com/dgibson/dtc/issues/186
+Signed-off-by: Uwe Kleine-König <u.kleine-koenig@baylibre.com>
+---
+ livetree.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/livetree.c b/livetree.c
+index 5d72abceb526..8baed1fca8c5 100644
+--- a/livetree.c
++++ b/livetree.c
+@@ -1234,7 +1234,7 @@ void fixup_phandles(struct dt_info *dti, const char *name)
+ 				continue;
+ 
+ 			offset = strtol(soffset, NULL, 0);
+-			if (offset < 0 || offset + 4 > p->val.len) {
++			if (offset < 0 || (unsigned int)offset + 4 > p->val.len) {
+ 				if (quiet < 1)
+ 					fprintf(stderr,
+ 						"Warning: Label %s contains invalid offset for property %s in node %s\n",
diff --git a/patches/dtc-1.8.1/series b/patches/dtc-1.8.1/series
new file mode 100644
index 000000000..fe54adf1c
--- /dev/null
+++ b/patches/dtc-1.8.1/series
@@ -0,0 +1,4 @@
+# generated by git-ptx-patches
+#tag:base --start-number 1
+0001-livetree-Fix-a-comparison-of-integers-with-different.patch
+# 75a849fa7e78e62b80e96c1499e71331  - git-ptx-patches magic
diff --git a/rules/dtc.make b/rules/dtc.make
index c9ad70681..6e653a92c 100644
--- a/rules/dtc.make
+++ b/rules/dtc.make
@@ -15,8 +15,8 @@ PACKAGES-$(PTXCONF_DTC) += dtc
 #
 # Paths and names
 #
-DTC_VERSION	:= 1.7.2
-DTC_SHA256	:= 8f1486962f093f28a2f79f01c1fd82f144ef640ceabe555536d43362212ceb7c
+DTC_VERSION	:= 1.8.1
+DTC_SHA256	:= 74b50bb19134f6562490afea53e59953dd6c4afb17e5ccb60be32221262d3390
 DTC		:= dtc-$(DTC_VERSION)
 DTC_SUFFIX	:= tar.gz
 DTC_URL		:= https://git.kernel.org/pub/scm/utils/dtc/dtc.git/snapshot/$(DTC).$(DTC_SUFFIX)
@@ -24,7 +24,7 @@ DTC_SOURCE	:= $(SRCDIR)/$(DTC).$(DTC_SUFFIX)
 DTC_DIR		:= $(BUILDDIR)/$(DTC)
 DTC_LICENSE	:= GPL-2.0-or-later OR BSD-2-Clause
 DTC_LICENSE_FILES := \
-	file://README.license;md5=a1eb22e37f09df5b5511b8a278992d0e \
+	file://README.license;md5=a5696bd07fcc7285cbbacc42c2132248 \
 	file://GPL;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
 	file://BSD-2-Clause;md5=5d6306d1b08f8df623178dfd81880927
 
-- 
2.43.0




^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2026-06-11 16:44 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-06-11 16:42 [ptxdist] [PATCH v2] dtc: Version bump. 1.7.2 -> 1.8.1 Christian Melki

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox