media: wl128x: fix a clang warning
authorMauro Carvalho Chehab <mchehab@kernel.org>
Wed, 28 Jun 2023 19:48:53 +0000 (21:48 +0200)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Thu, 29 Jun 2023 07:56:13 +0000 (09:56 +0200)
Clang-16 produces this warning, which is fatal with CONFIG_WERROR:

../drivers/media/radio/wl128x/fmdrv_common.c:1237:19: error: variable 'cmd_cnt' set but not used [-Werror,-Wunused-but-set-variable]
        int ret, fw_len, cmd_cnt;
                         ^
1 error generated.

What happens is that cmd_cnt tracks the amount of firmware data packets
were transfered, which is printed only when debug is used.

Switch to use the firmware count, as the message is all about reporting
a partial firmware transfer.

Link: https://lore.kernel.org/linux-media/6badd27ebfa718d5737f517f18b29a3e0f6e43f8.1687981726.git.mchehab@kernel.org
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/media/radio/wl128x/fmdrv_common.c

index cbd49dff6d74c9fe2d7323472b39d5927a5025e0..b31b7ed60bbebae8f929d4c4772e2b49bbb98683 100644 (file)
@@ -1234,9 +1234,8 @@ static int fm_download_firmware(struct fmdev *fmdev, const u8 *fw_name)
        struct bts_action *action;
        struct bts_action_delay *delay;
        u8 *fw_data;
-       int ret, fw_len, cmd_cnt;
+       int ret, fw_len;
 
-       cmd_cnt = 0;
        set_bit(FM_FW_DW_INPROGRESS, &fmdev->flag);
 
        ret = request_firmware(&fw_entry, fw_name,
@@ -1272,7 +1271,6 @@ static int fm_download_firmware(struct fmdev *fmdev, const u8 *fw_name)
                        if (ret)
                                goto rel_fw;
 
-                       cmd_cnt++;
                        break;
 
                case ACTION_DELAY:      /* Delay */
@@ -1284,7 +1282,7 @@ static int fm_download_firmware(struct fmdev *fmdev, const u8 *fw_name)
                fw_data += (sizeof(struct bts_action) + (action->size));
                fw_len -= (sizeof(struct bts_action) + (action->size));
        }
-       fmdbg("Firmware commands(%d) loaded to chip\n", cmd_cnt);
+       fmdbg("Transfered only %d of %d bytes of the firmware to chip\n", fw_entry->size - fw_len, fw_entry->size);
 rel_fw:
        release_firmware(fw_entry);
        clear_bit(FM_FW_DW_INPROGRESS, &fmdev->flag);