Hi Gustavo,
On Wed, Apr 25, 2012 at 04:58:25PM -0300, Gustavo Padovan wrote:
...
> > @@ -4554,10 +4555,14 @@ static inline int l2cap_data_channel(struct l2cap_conn *conn, u16 cid, struct sk
> >
> > chan = l2cap_get_chan_by_scid(conn, cid);
> > if (!chan) {
> > - BT_DBG("unknown cid 0x%4.4x", cid);
> > - /* Drop packet and return */
> > - kfree_skb(skb);
> > - return 0;
> > + if (cid == L2CAP_CID_A2MP) {
> > + chan = a2mp_channel_create(conn, skb);
>
> This code should be inside l2cap_recv_frame. take a look there.
No, it shouldn't. We do not want to reimplement ERTM logic inside A2MP.
Best regards
Andrei Emeltchenko
--
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
