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); |
