Hi Mat, On Thu, Feb 23, 2012 at 6:37 PM, Mat Martineau <mathewm@xxxxxxxxxxxxxx> wrote: > This change affects data structures storing ERTM state and control > fields, and adds new definitions for states and events. An > l2cap_seq_list structure is added for tracking ERTM sequence numbers > without repeated memory allocations. Control fields are carried in > the bt_skb_cb struct rather than constantly doing shift and mask > operations. > > Signed-off-by: Mat Martineau <mathewm@xxxxxxxxxxxxxx> > --- > include/net/bluetooth/bluetooth.h | 14 ++- > include/net/bluetooth/l2cap.h | 260 +++++++++---------------------------- > 2 files changed, 73 insertions(+), 201 deletions(-) <snip> > diff --git a/include/net/bluetooth/l2cap.h b/include/net/bluetooth/l2cap.h > index d6d8ec8..a499b60 100644 > --- a/include/net/bluetooth/l2cap.h > +++ b/include/net/bluetooth/l2cap.h <snip> > @@ -645,200 +664,43 @@ static inline bool l2cap_clear_timer(struct l2cap_chan *chan, > > #define __set_chan_timer(c, t) l2cap_set_timer(c, &c->chan_timer, (t)) > #define __clear_chan_timer(c) l2cap_clear_timer(c, &c->chan_timer) Are these two still needed? I saw you moved others to l2cap_core.c which is fine but what about these? Regards, -- Ulisses Furquim ProFUSION embedded systems http://profusion.mobi Mobile: +55 19 9250 0942 Skype: ulissesffs -- 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
