On Thu, 27 Aug 2026 at 02:00, Stephen Hemminger <[email protected]> wrote: > > Fix compiler warning on clang-23. > > dpaa_of.c:14:12: warning: variable 'alive' set but not used > [-Wunused-but-set-global] > > The 'alive' flag is only read by DPAAX_HWWARN, which expands to an empty > statement and discards its condition unless RTE_LIBRTE_DPAAX_DEBUG is > defined. Make the empty expansion consume its argument so the condition > is still checked in a non-debug build. > > Fixes: 8c83f28cc8a4 ("common/dpaax: move OF library from DPAA bus")
Not that important, the issue probably predates this commit. And looking at this change, it seems DPAA_BUS_HWWARN is dead code since then. > Cc: [email protected] > > Signed-off-by: Stephen Hemminger <[email protected]> Otherwise lgtm. -- David Marchand
