Hi Andrei, * Andrei Emeltchenko <andrei.emeltchenko.news@xxxxxxxxx> [2012-05-24 12:24:26 +0300]: > Hi Gustavo, > > On Thu, May 24, 2012 at 03:02:53AM -0300, Gustavo Padovan wrote: > > From: Gustavo Padovan <gustavo.padovan@xxxxxxxxxxxxxxx> > > > > bt_accept_enqueue() can be easily placed at the end of > > l2cap_sock_new_connection_cb(). > > > > Signed-off-by: Gustavo Padovan <gustavo.padovan@xxxxxxxxxxxxxxx> > > --- > > net/bluetooth/l2cap_core.c | 4 ---- > > net/bluetooth/l2cap_sock.c | 2 ++ > > 2 files changed, 2 insertions(+), 4 deletions(-) > > > > diff --git a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c > > index c9de4f5..5ff294f9 100644 > > --- a/net/bluetooth/l2cap_core.c > > +++ b/net/bluetooth/l2cap_core.c > > @@ -1165,8 +1165,6 @@ static void l2cap_le_conn_ready(struct l2cap_conn *conn) > > bacpy(&bt_sk(sk)->src, conn->src); > > bacpy(&bt_sk(sk)->dst, conn->dst); > > > > - bt_accept_enqueue(parent, sk); > > - > > Shall we also move to l2cap_sock_new_connection_cb bacpys above? I don't think so, bacpy and bacmp are used everywhere, they need to fixed differently. 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
