From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3B2971643A for ; Fri, 30 Jun 2023 15:54:29 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 54E29C433C0; Fri, 30 Jun 2023 15:54:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1688140469; bh=3UxnCn0uKRWtw0gFGbXkslK1m3C/e7mRctYkrXYRER8=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=nEOT2RDrMyVotMws4YABB+hxaLTELOoPVGVFqSiq41H6IesDtPs9UZC4zDuYg0Up2 Zg8PXkbR9PFVqlQFtCJfssEp4HZn+zk7axTiOO9n3ygma+ddLKYrTgl/25YkAiE5/f 0h/0KkksDmvOVZnVhbwvXuJMA5L9GwEppi025BMB5dDcsEgFeQc/OMgw3ZuKgtFwPm udXrOkNqKPkjxumsMI0uKSXPPULvFjUQjqT16FeMgxCpqV9b0OV9OldFpZP2ghh6NC cwK6+LM3KIomgurPZMzIBYcJTYao2gw57JCI6PJB4LAmyLFvzVlsbi/8PXlmZBU1xC z473u/Jpk3YXQ== Date: Fri, 30 Jun 2023 08:54:27 -0700 From: Nathan Chancellor To: Mauro Carvalho Chehab Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, Hans Verkuil , Nick Desaulniers , Tom Rix , Zhou jie , llvm@lists.linux.dev Subject: Re: [PATCH] media: wl128x: fix a clang warning Message-ID: <20230630155427.GA2889176@dev-arch.thelio-3990X> References: <6badd27ebfa718d5737f517f18b29a3e0f6e43f8.1687981726.git.mchehab@kernel.org> Precedence: bulk X-Mailing-List: llvm@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <6badd27ebfa718d5737f517f18b29a3e0f6e43f8.1687981726.git.mchehab@kernel.org> On Wed, Jun 28, 2023 at 09:48:53PM +0200, Mauro Carvalho Chehab wrote: > 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. Alternatively, fmdbg() could be defined with no_printk(), so that the variable appears used in all configurations, but this does not seem unreasonable either. > Switch to use the firmware count, as the message is all about reporting > a partial firmware transfer. > > Signed-off-by: Mauro Carvalho Chehab > --- > drivers/media/radio/wl128x/fmdrv_common.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/drivers/media/radio/wl128x/fmdrv_common.c b/drivers/media/radio/wl128x/fmdrv_common.c > index cbd49dff6d74..b31b7ed60bbe 100644 > --- a/drivers/media/radio/wl128x/fmdrv_common.c > +++ b/drivers/media/radio/wl128x/fmdrv_common.c > @@ -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); > -- > 2.41.0 >