All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2] Bluetooth: trivial: Remove empty line
@ 2012-04-16 13:32 Andrei Emeltchenko
  2012-04-16 13:32 ` [PATCH 2/2] Bluetooth: Fix debug printing unallocated name Andrei Emeltchenko
  2012-04-16 13:39 ` [PATCH 1/2] Bluetooth: trivial: Remove empty line Marcel Holtmann
  0 siblings, 2 replies; 4+ messages in thread
From: Andrei Emeltchenko @ 2012-04-16 13:32 UTC (permalink / raw)
  To: linux-bluetooth

From: Andrei Emeltchenko <andrei.emeltchenko@intel.com>


Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@intel.com>
---
 include/net/bluetooth/l2cap.h |    1 -
 1 file changed, 1 deletion(-)

diff --git a/include/net/bluetooth/l2cap.h b/include/net/bluetooth/l2cap.h
index e0e5549..0fac788 100644
--- a/include/net/bluetooth/l2cap.h
+++ b/include/net/bluetooth/l2cap.h
@@ -532,7 +532,6 @@ struct l2cap_ops {
 	void			(*state_change) (void *data, int state);
 	struct sk_buff		*(*alloc_skb) (struct l2cap_chan *chan,
 					unsigned long len, int nb);
-
 };
 
 struct l2cap_conn {
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH 2/2] Bluetooth: Fix debug printing unallocated name
  2012-04-16 13:32 [PATCH 1/2] Bluetooth: trivial: Remove empty line Andrei Emeltchenko
@ 2012-04-16 13:32 ` Andrei Emeltchenko
  2012-04-16 13:41   ` Marcel Holtmann
  2012-04-16 13:39 ` [PATCH 1/2] Bluetooth: trivial: Remove empty line Marcel Holtmann
  1 sibling, 1 reply; 4+ messages in thread
From: Andrei Emeltchenko @ 2012-04-16 13:32 UTC (permalink / raw)
  To: linux-bluetooth

From: Andrei Emeltchenko <andrei.emeltchenko@intel.com>

It does make sense to print hdev name after allocation.

Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@intel.com>
---
 net/bluetooth/hci_core.c |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/net/bluetooth/hci_core.c b/net/bluetooth/hci_core.c
index 26a6efc..c4dc263 100644
--- a/net/bluetooth/hci_core.c
+++ b/net/bluetooth/hci_core.c
@@ -1742,8 +1742,6 @@ int hci_register_dev(struct hci_dev *hdev)
 	struct list_head *head = &hci_dev_list, *p;
 	int i, id, error;
 
-	BT_DBG("%p name %s bus %d", hdev, hdev->name, hdev->bus);
-
 	if (!hdev->open || !hdev->close)
 		return -EINVAL;
 
@@ -1763,6 +1761,9 @@ int hci_register_dev(struct hci_dev *hdev)
 
 	sprintf(hdev->name, "hci%d", id);
 	hdev->id = id;
+
+	BT_DBG("%p name %s bus %d", hdev, hdev->name, hdev->bus);
+
 	list_add_tail(&hdev->list, head);
 
 	mutex_init(&hdev->lock);
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/2] Bluetooth: trivial: Remove empty line
  2012-04-16 13:32 [PATCH 1/2] Bluetooth: trivial: Remove empty line Andrei Emeltchenko
  2012-04-16 13:32 ` [PATCH 2/2] Bluetooth: Fix debug printing unallocated name Andrei Emeltchenko
@ 2012-04-16 13:39 ` Marcel Holtmann
  1 sibling, 0 replies; 4+ messages in thread
From: Marcel Holtmann @ 2012-04-16 13:39 UTC (permalink / raw)
  To: Andrei Emeltchenko; +Cc: linux-bluetooth

Hi Andrei,

> Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@intel.com>
> ---
>  include/net/bluetooth/l2cap.h |    1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/include/net/bluetooth/l2cap.h b/include/net/bluetooth/l2cap.h
> index e0e5549..0fac788 100644
> --- a/include/net/bluetooth/l2cap.h
> +++ b/include/net/bluetooth/l2cap.h
> @@ -532,7 +532,6 @@ struct l2cap_ops {
>  	void			(*state_change) (void *data, int state);
>  	struct sk_buff		*(*alloc_skb) (struct l2cap_chan *chan,
>  					unsigned long len, int nb);
> -
>  };
>  
>  struct l2cap_conn {

patch has been applied to bluetooth-next tree.

Regards

Marcel



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH 2/2] Bluetooth: Fix debug printing unallocated name
  2012-04-16 13:32 ` [PATCH 2/2] Bluetooth: Fix debug printing unallocated name Andrei Emeltchenko
@ 2012-04-16 13:41   ` Marcel Holtmann
  0 siblings, 0 replies; 4+ messages in thread
From: Marcel Holtmann @ 2012-04-16 13:41 UTC (permalink / raw)
  To: Andrei Emeltchenko; +Cc: linux-bluetooth

Hi Andrei,

> It does make sense to print hdev name after allocation.
> 
> Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@intel.com>
> ---
>  net/bluetooth/hci_core.c |    5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)

patch has been applied to bluetooth-next tree.

Regards

Marcel



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2012-04-16 13:41 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-04-16 13:32 [PATCH 1/2] Bluetooth: trivial: Remove empty line Andrei Emeltchenko
2012-04-16 13:32 ` [PATCH 2/2] Bluetooth: Fix debug printing unallocated name Andrei Emeltchenko
2012-04-16 13:41   ` Marcel Holtmann
2012-04-16 13:39 ` [PATCH 1/2] Bluetooth: trivial: Remove empty line Marcel Holtmann

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.