Hi Andrei,
* Andrei Emeltchenko <Andrei.Emeltchenko.news@xxxxxxxxx> [2012-08-29 10:20:51 +0300]:
> From: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx>
>
> Return code is not needed in hci_chan_del
>
> Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx>
> ---
> include/net/bluetooth/hci_core.h | 2 +-
> net/bluetooth/hci_conn.c | 4 +---
> 2 files changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/include/net/bluetooth/hci_core.h b/include/net/bluetooth/hci_core.h
> index 0964954..36720f0 100644
> --- a/include/net/bluetooth/hci_core.h
> +++ b/include/net/bluetooth/hci_core.h
> @@ -573,7 +573,7 @@ void hci_conn_hash_flush(struct hci_dev *hdev);
> void hci_conn_check_pending(struct hci_dev *hdev);
>
> struct hci_chan *hci_chan_create(struct hci_conn *conn);
> -int hci_chan_del(struct hci_chan *chan);
> +void hci_chan_del(struct hci_chan *chan);
> void hci_chan_list_flush(struct hci_conn *conn);
> struct hci_chan *hci_chan_lookup_handle(struct hci_conn *hcon, __u16 handle);
> struct hci_chan *hci_chan_lookup_handle_all(struct hci_dev *hdev,
> diff --git a/net/bluetooth/hci_conn.c b/net/bluetooth/hci_conn.c
> index 1c79a76..4695880 100644
> --- a/net/bluetooth/hci_conn.c
> +++ b/net/bluetooth/hci_conn.c
> @@ -935,7 +935,7 @@ struct hci_chan *hci_chan_create(struct hci_conn *conn)
> return chan;
> }
>
> -int hci_chan_del(struct hci_chan *chan)
> +void hci_chan_del(struct hci_chan *chan)
> {
> struct hci_conn *conn = chan->conn;
> struct hci_dev *hdev = conn->hdev;
> @@ -948,8 +948,6 @@ int hci_chan_del(struct hci_chan *chan)
>
> skb_queue_purge(&chan->data_q);
> kfree(chan);
> -
> - return 0;
> }
>
> void hci_chan_list_flush(struct hci_conn *conn) > --
This patch doesn't apply on bluetooth-next anymore, please rebase.
Gustavo
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
