On Sat, 29 Aug 2026 08:44:44 -0700 Stephen Hemminger <[email protected]> wrote:
> Clang 23 adds -Wunused-but-set-global, which reports file scope > variables that are assigned but never read. It is enabled by -Wall and > does not appear with gcc or with clang 20 through 22. > > Three of these are dead variables and are simply removed. The fourth, > 'alive' in dpaa_of.c, is a real debug aid: it is only read by > DPAAX_HWWARN, which discards its condition when RTE_LIBRTE_DPAAX_DEBUG > is not defined. That one is fixed by making the empty expansion consume > its argument, which keeps the condition compiled in both builds and > covers every other DPAAX_HWWARN user as well. > > v2 - remove more unused code as suggested. > > Stephen Hemminger (4): > common/dpaax: fix unused variable warning in non-debug build > bus/dpaa: remove unused qman_clk variable > common/zsda: remove unused device counter > net/qede: remove unused app version variable > > drivers/bus/dpaa/base/qbman/qman_driver.c | 4 ---- > drivers/common/dpaax/dpaax_logs.h | 3 ++- > drivers/common/zsda/zsda_device.c | 5 ----- > drivers/net/qede/qede_debug.c | 20 -------------------- > 4 files changed, 2 insertions(+), 30 deletions(-) > Applied to next-net; so that clang 23 build tests can run across all drivers
