diff options
| author | Johan Hedberg <johan.hedberg@intel.com> | 2014-02-18 17:14:32 +0200 |
|---|---|---|
| committer | Marcel Holtmann <marcel@holtmann.org> | 2014-02-18 08:58:20 -0800 |
| commit | 35f7498a87794ca531335f7c782e5b9495fec6d6 (patch) | |
| tree | 8c328c325df22b858281dd0259fd3d17624039ab /net/bluetooth/mgmt.c | |
| parent | e0b2b27e622da0ba8a3d253b985d3d8f174b4313 (diff) | |
| download | lwn-35f7498a87794ca531335f7c782e5b9495fec6d6.tar.gz lwn-35f7498a87794ca531335f7c782e5b9495fec6d6.zip | |
Bluetooth: Remove return values from functions that don't need them
There are many functions that never fail but still declare an integer
return value for no reason. This patch converts these functions to use a
void return value to avoid any confusion of whether they can fail or not.
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 473f8687b28b..fbb76a0de580 100644 --- a/net/bluetooth/mgmt.c +++ b/net/bluetooth/mgmt.c @@ -2073,7 +2073,7 @@ static int remove_uuid(struct sock *sk, struct hci_dev *hdev, void *data, } if (memcmp(cp->uuid, bt_uuid_any, 16) == 0) { - err = hci_uuids_clear(hdev); + hci_uuids_clear(hdev); if (enable_service_cache(hdev)) { err = cmd_complete(sk, hdev->id, MGMT_OP_REMOVE_UUID, |