diff options
author | Johan Hedberg <johan.hedberg@intel.com> | 2014-07-04 09:17:23 -0400 |
---|---|---|
committer | Marcel Holtmann <marcel@holtmann.org> | 2014-07-04 09:54:29 -0400 |
commit | d1dbf12e3be0befcd3fd1f978202c5f72d2cc67b (patch) | |
tree | 93321567f8e229a45b495086871e54107f4f48a8 /net/bluetooth/mgmt.c | |
parent | 99a6768e0e55d19a47934ccd653ff0f9b3236401 (diff) |
Bluetooth: Use list_del when freeing the list entry
It's wasteful to use list_del_init (which re-initializes the list_head)
if we're just about to free the element and never use it again.
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Diffstat (limited to 'net/bluetooth/mgmt.c')
-rw-r--r-- | net/bluetooth/mgmt.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c index f1672b15c0f3..90eabcae3ed2 100644 --- a/net/bluetooth/mgmt.c +++ b/net/bluetooth/mgmt.c | |||
@@ -5225,7 +5225,7 @@ static int remove_device(struct sock *sk, struct hci_dev *hdev, | |||
5225 | goto unlock; | 5225 | goto unlock; |
5226 | } | 5226 | } |
5227 | 5227 | ||
5228 | list_del_init(¶ms->action); | 5228 | list_del(¶ms->action); |
5229 | list_del(¶ms->list); | 5229 | list_del(¶ms->list); |
5230 | kfree(params); | 5230 | kfree(params); |
5231 | hci_update_background_scan(hdev); | 5231 | hci_update_background_scan(hdev); |