On Wed, Feb 25, 2026 at 03:14:40PM +0000, Daniel Golle wrote: > On Wed, Feb 25, 2026 at 09:07:42AM -0600, Tom Rini wrote: > > On Wed, Feb 25, 2026 at 03:03:57PM +0000, Daniel Golle wrote: > > > > > tools/atmelimage.c: In function ‘atmel_find_pmecc_parameter_in_token’: > > > tools/atmelimage.c:64:31: error: assignment discards ‘const’ qualifier > > > from pointer target type [-Werror=discarded-qualifiers] > > > 64 | param = strstr(token, "="); > > > | ^ > > > cc1: all warnings being treated as errors > > > > > > Add 'const' qualifier to variable 'param' to prevent build failing > > > due to -Werror. > > > > > > Signed-off-by: Daniel Golle <[email protected]> > > > --- > > > tools/atmelimage.c | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/tools/atmelimage.c b/tools/atmelimage.c > > > index 6a2d9d8feab..770f7a0de4e 100644 > > > --- a/tools/atmelimage.c > > > +++ b/tools/atmelimage.c > > > @@ -55,7 +55,7 @@ static const char * const configs[] = { > > > static int atmel_find_pmecc_parameter_in_token(const char *token) > > > { > > > size_t pos; > > > - char *param; > > > + const char *param; > > > > > > debug("token: '%s'\n", token); > > > > > > > How / where are you triggering this problem? This is part of mkimage and > > built on CI every time (and on macOS and Windows, in Azure). Thanks! > > I'm using archlinux on my workstation which got a fairly > recent version of GCC which is used to build the host tools: > gcc (GCC) 15.2.1 20260209
Ah. We dont' support 15.x yet due to: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=121588 But a v2 where you note that newer compilers find this would be good and get us ahead of problems once a newer gcc comes out with the fix. > When trying to use the u-boot-size-test.sh scripts you had asked me for > it failed because of passing the '-E' parameter to buildman. > > Also 'dtc' fails building for the same reason and that can be fixed by > picking commit dfe057874b34c ("scripts/dtc: Update to upstream versionv > 1.7.2-62-ga26ef6400bd8") from Linux. > However, that has far-reaching consequences such as the > -Wno-graph_child_address parameter no longer existing, and also dtc > became more strict and issues new warnings for many of the dts files in > U-Boot, so then the build failed because of that... Good to know for the next time we sync up, which should be soon to avoid getting super out of date again. -- Tom
signature.asc
Description: PGP signature
