Bluetooth: btnxpuart: remove useless assignment

dolcini · Vudentz · commit 3c83800a6c5b · 2023-12-22T12:58:57.000-05:00
Remove useless assignment of rx_skb to NULL in case the skb is in error,
this is already done in h4_recv_buf() that is executed a few lines
before.

Signed-off-by: Francesco Dolcini &lt;francesco.dolcini@toradex.com&gt;
Signed-off-by: Luiz Augusto von Dentz &lt;luiz.von.dentz@intel.com&gt;
diff --git a/drivers/bluetooth/btnxpuart.c b/drivers/bluetooth/btnxpuart.c
@@ -1278,7 +1278,6 @@ static int btnxpuart_receive_buf(struct serdev_device *serdev, const u8 *data,
 		/* Safe to ignore out-of-sync bootloader signatures */
 		if (!is_fw_downloading(nxpdev))
 			bt_dev_err(nxpdev->hdev, "Frame reassembly failed (%d)", err);
-		nxpdev->rx_skb = NULL;
 		return count;
 	}
 	if (!is_fw_downloading(nxpdev))

Original file line number	Diff line number	Diff line change
`@@ -1278,7 +1278,6 @@ static int btnxpuart_receive_buf(struct serdev_device serdev, const u8 data,`
`1278`	`1278`	`/* Safe to ignore out-of-sync bootloader signatures */`
`1279`	`1279`	`if (!is_fw_downloading(nxpdev))`
`1280`	`1280`	`bt_dev_err(nxpdev->hdev, "Frame reassembly failed (%d)", err);`
`1281`		`- nxpdev->rx_skb = NULL;`
`1282`	`1281`	`return count;`
`1283`	`1282`	`}`
`1284`	`1283`	`if (!is_fw_downloading(nxpdev))`