diff options
| author | Rodrigo Gobbi <rodrigo.gobbi.7@gmail.com> | 2025-06-26 18:33:14 -0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2025-08-15 16:39:01 +0200 |
| commit | 0ba168d3867f0609e4a81fde93b8c88e92bc456c (patch) | |
| tree | 77aff8577b14c5c76b9f82193744631470ef57ac /drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c | |
| parent | 8a926e1ea733bee2138a9a3f4c7bf06502cdeb8b (diff) | |
soundwire: debugfs: move debug statement outside of error handling
[ Upstream commit 06f77ff9d852c9f2764659ea81489364d8a69a9c ]
The start_t and finish_t variables are not properly initialized
if errors happens over request_firmware actions.
This was also detected by smatch:
drivers/soundwire/debugfs.c:301 cmd_go() error: uninitialized symbol 'finish_t'.
drivers/soundwire/debugfs.c:301 cmd_go() error: uninitialized symbol 'start_t'.
Move the debug statement outside of firmware error handling.
Signed-off-by: Rodrigo Gobbi <rodrigo.gobbi.7@gmail.com>
Reported-by: Dan Carpenter <dan.carpenter@linaro.org>
Closes: https://lore.kernel.org/linux-sound/0db6d0bf-7bac-43a7-b624-a00d3d2bf829@stanley.mountain/
Fixes: bb5cb09eedce ("soundwire: debugfs: add interface for BPT/BRA transfers")
Link: https://lore.kernel.org/r/20250626213628.9575-1-rodrigo.gobbi.7@gmail.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c')
0 files changed, 0 insertions, 0 deletions