diff mbox series

Bluetooth: hci_uart: Remove redundant assignment to fw_ptr

Message ID YMpUvvOUKGOKZWuH@fedora
State Accepted
Commit 1f0536139cb8e8175ca034e12706b86f77f9061e
Headers show
Series Bluetooth: hci_uart: Remove redundant assignment to fw_ptr | expand

Commit Message

Nigel Christian June 16, 2021, 7:45 p.m. UTC
The variable fw_ptr is assigned a value that is not read and the same value
is assigned in the patch goto. The assignment is redundant and can be
removed.

Addresses-Coverity: ("Unused value")
Signed-off-by: Nigel Christian <nigel.l.christian@gmail.com>
---
 drivers/bluetooth/hci_ag6xx.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Marcel Holtmann June 26, 2021, 5:53 a.m. UTC | #1
Hi Nigel,

> The variable fw_ptr is assigned a value that is not read and the same value

> is assigned in the patch goto. The assignment is redundant and can be

> removed.

> 

> Addresses-Coverity: ("Unused value")

> Signed-off-by: Nigel Christian <nigel.l.christian@gmail.com>

> ---

> drivers/bluetooth/hci_ag6xx.c | 1 -

> 1 file changed, 1 deletion(-)


patch has been applied to bluetooth-next tree.

Regards

Marcel
diff mbox series

Patch

diff --git a/drivers/bluetooth/hci_ag6xx.c b/drivers/bluetooth/hci_ag6xx.c
index 1f55df93e4ce..2d40302409ff 100644
--- a/drivers/bluetooth/hci_ag6xx.c
+++ b/drivers/bluetooth/hci_ag6xx.c
@@ -199,7 +199,6 @@  static int ag6xx_setup(struct hci_uart *hu)
 			   fwname, err);
 		goto patch;
 	}
-	fw_ptr = fw->data;
 
 	bt_dev_info(hdev, "Applying bddata (%s)", fwname);