From 84a071479253dd525099e403164313f36aec3d79 Mon Sep 17 00:00:00 2001 From: "Bjoern A. Zeeb" Date: Tue, 30 Nov 2021 14:23:18 +0000 Subject: [PATCH] fw_stub: fix -Wunused-but-set-variable for firmware files In case we are only embedding a single firmware image the variable "parent" gets set but never used. Add checks for the number of files for it and only print it out if we are exceeding the single file count. This fixes -Wunused-but-set-variable warnings for the majority of firmware files in the tree. Sponsored by: The FreeBSD Foundation (cherry picked from commit b394e16ef0b5ad0da5e029bcdd3a01e361724d8d) --- sys/tools/fw_stub.awk | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/sys/tools/fw_stub.awk b/sys/tools/fw_stub.awk index 807c51b6190..94975687f4e 100644 --- a/sys/tools/fw_stub.awk +++ b/sys/tools/fw_stub.awk @@ -151,8 +151,10 @@ for (file_i = 0; file_i < num_files; file_i++) { printc("\nstatic int\n"\ modname "_fw_modevent(module_t mod, int type, void *unused)\ {\ - const struct firmware *fp, *parent;\ - int error;\ + const struct firmware *fp;"); +if (num_files > 1) + printc("\tconst struct firmware *parent;"); +printc("\tint error;\ switch (type) {\ case MOD_LOAD:\n"); @@ -187,7 +189,7 @@ for (file_i = 0; file_i < num_files; file_i++) { printc("\t\tif (fp == NULL)"); printc("\t\t\tgoto fail_" file_i ";"); - if (file_i == 0) + if (file_i == 0 && num_files > 1) printc("\t\tparent = fp;"); }