git: b394e16ef0b5 - main - fw_stub: fix -Wunused-but-set-variable for firmware files
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Tue, 30 Nov 2021 14:31:32 UTC
The branch main has been updated by bz:
URL: https://cgit.FreeBSD.org/src/commit/?id=b394e16ef0b5ad0da5e029bcdd3a01e361724d8d
commit b394e16ef0b5ad0da5e029bcdd3a01e361724d8d
Author: Bjoern A. Zeeb <bz@FreeBSD.org>
AuthorDate: 2021-11-30 14:23:18 +0000
Commit: Bjoern A. Zeeb <bz@FreeBSD.org>
CommitDate: 2021-11-30 14:23:18 +0000
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
MFC after: 2 weeks
---
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 807c51b6190b..94975687f4e2 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;");
}