]> git.ipfire.org Git - thirdparty/u-boot.git/commitdiff
Merge patch series "drop volatile from global data"
authorTom Rini <trini@konsulko.com>
Fri, 13 Jun 2025 22:57:34 +0000 (16:57 -0600)
committerTom Rini <trini@konsulko.com>
Fri, 13 Jun 2025 22:57:34 +0000 (16:57 -0600)
Rasmus Villemoes <ravi@prevas.dk> says:

There's really no reason for the gd pointer to have the volatile
qualifier.

In fact, I claim that it's completely unnecessary and just pessimizes
code generation and forces ugly casts in lots of places. For example,
see the casts in drivers/core/tag.c where elements are added to
gd->dm_taglist, or a helper such as cyclic_get_list() that should not
be needed.

Also, it is what ends up standing in the way of an otherwise
innocent code cleanup of list.h:
https://lore.kernel.org/u-boot/20250522165656.GB2179216@bill-the-cat/

Note that riscv, x86 as well as arm64 with LTO enabled has not had
this volatile qualifier, so it's unlikely there's any generic code
that depends on it.

Link: https://lore.kernel.org/r/20250604195612.2312979-1-ravi@prevas.dk

Trivial merge