From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Thu, 25 Jun 2026 21:23:44 +0200 Received: from metis.whiteo.stw.pengutronix.de ([2a0a:edc0:2:b01:1d::104]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1wcpfs-009oiM-1g for lore@lore.pengutronix.de; Thu, 25 Jun 2026 21:23:44 +0200 Received: from [127.0.0.1] (helo=metis.whiteo.stw.pengutronix.de) by metis.whiteo.stw.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1wcpfr-0007z0-Av; Thu, 25 Jun 2026 21:23:43 +0200 Received: from mx1.white.stw.pengutronix.de ([185.203.200.13]) by metis.whiteo.stw.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1wcpfd-0007Jo-MM for ptxdist@pengutronix.de; Thu, 25 Jun 2026 21:23:30 +0200 Received: from drehscheibe.grey.stw.pengutronix.de (drehscheibe.grey.stw.pengutronix.de [IPv6:2a0a:edc0:0:c01:1d::a2]) (Authenticated sender: relay-from-drehscheibe.grey.stw.pengutronix.de) by mx1.white.stw.pengutronix.de (Postfix) with ESMTPSA id 661BD200954; Thu, 25 Jun 2026 21:23:29 +0200 (CEST) Received: from dude05.red.stw.pengutronix.de ([2a0a:edc0:0:1101:1d::54]) by drehscheibe.grey.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1wcpfd-004dYr-15; Thu, 25 Jun 2026 21:23:29 +0200 Received: from mol by dude05.red.stw.pengutronix.de with local (Exim 4.98.2) (envelope-from ) id 1wcpfd-00000003UVg-1Cje; Thu, 25 Jun 2026 21:23:29 +0200 From: Michael Olbrich To: ptxdist@pengutronix.de Date: Thu, 25 Jun 2026 21:23:29 +0200 Message-ID: <20260625192329.832257-1-m.olbrich@pengutronix.de> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260611164419.1530578-1-christian.melki@t2data.com> References: <20260611164419.1530578-1-christian.melki@t2data.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on metis.whiteo.stw.pengutronix.de X-Spam-Level: X-Spam-Status: No, score=-3.7 required=4.0 tests=ALL_TRUSTED,AWL,BAYES_00 autolearn=ham autolearn_force=no version=3.4.2 Subject: Re: [ptxdist] [APPLIED] dtc: Version bump. 1.7.2 -> 1.8.1 X-BeenThere: ptxdist@pengutronix.de X-Mailman-Version: 2.1.29 Precedence: list List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: ptxdist@pengutronix.de Cc: Christian Melki Sender: "ptxdist" X-SA-Exim-Connect-IP: 127.0.0.1 X-SA-Exim-Mail-From: ptxdist-bounces@pengutronix.de X-SA-Exim-Scanned: No (on metis.whiteo.stw.pengutronix.de); SAEximRunCond expanded to false Thanks, applied as 0e47397fe2872759dea82e1c8e441baf2aacc7e6. Michael [sent from post-receive hook] On Thu, 25 Jun 2026 21:23:29 +0200, Christian Melki wrote: > 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 > Message-Id: <20260611164419.1530578-1-christian.melki@t2data.com> > [mol: remove old patches/dtc-1.7.2/series] > Signed-off-by: Michael Olbrich > > 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 766dbccc2427..000000000000 > --- a/patches/dtc-1.7.2/0001-Fix-discarded-const-qualifiers.patch > +++ /dev/null > @@ -1,80 +0,0 @@ > -From: Stephen Gallagher > -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 > -Signed-off-by: David Gibson > ---- > - 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.7.2/series b/patches/dtc-1.7.2/series > deleted file mode 100644 > index d6c2782c41ab..000000000000 > --- a/patches/dtc-1.7.2/series > +++ /dev/null > @@ -1,4 +0,0 @@ > -# generated by git-ptx-patches > -#tag:base --start-number 1 > -0001-Fix-discarded-const-qualifiers.patch > -# 3a8b95763e3400e890427bab311c296a - git-ptx-patches magic > 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 000000000000..d14fdc1afe5a > --- /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?= > +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 > +--- > + 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 000000000000..fe54adf1c659 > --- /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 c9ad70681221..6e653a92c7e9 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 >