Re: [PATCH 1/2] Bluetooth: btintel: remove useless code in btintel_set_dsm_reset_method

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Dear Su,


Thank you for your patch. Some minor comments.


Am 20.05.24 um 04:16 schrieb Su Hui:
Clang static checker(scan-build) warning:

Please add a space before (. Noting the version of scan build would also be nice.

drivers/bluetooth/btintel.c:2537:14:
Value stored to 'handle' during its initialization is never read.

No need to repeatedly assign values to 'handle'. Remove this useless
code to save some space.

The plural “values” is misleading to me. Maybe just remove the sentence, and say:

Remove this unused assignment.

For the summary, “useless code” could also be more specific:

Bluetooth: btintel: Remove unused assignement in btintel_set_dsm_reset_method()

Maybe also add a Fixes: tag.

Signed-off-by: Su Hui <suhui@xxxxxxxxxxxx>


Kind regards,

Paul


---
  drivers/bluetooth/btintel.c | 2 --
  1 file changed, 2 deletions(-)

diff --git a/drivers/bluetooth/btintel.c b/drivers/bluetooth/btintel.c
index 0c855c3ee1c1..f1c101dc0c28 100644
--- a/drivers/bluetooth/btintel.c
+++ b/drivers/bluetooth/btintel.c
@@ -2542,8 +2542,6 @@ static void btintel_set_dsm_reset_method(struct hci_dev *hdev,
  		RESET_TYPE_VSEC
  	};
- handle = ACPI_HANDLE(GET_HCIDEV_DEV(hdev));
-
  	if (!handle) {
  		bt_dev_dbg(hdev, "No support for bluetooth device in ACPI firmware");
  		return;




[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux