* Anderson Lizardo <anderson.lizardo@xxxxxxxxxxxxx> [2010-09-04 09:14:56 -0400]:
> On Sat, Sep 4, 2010 at 5:18 AM, Gustavo F. Padovan <gustavo@xxxxxxxxxxx> wrote:
> > diff --git a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c
> > index f2062aa..1f6d448 100644
> > --- a/net/bluetooth/l2cap_core.c
> > +++ b/net/bluetooth/l2cap_core.c
> > @@ -57,24 +57,21 @@
> >
> > #define VERSION "2.15"
> >
> > -static int disable_ertm = 0;
> > +int disable_ertm = 0;
> >
> > static u32 l2cap_feat_mask = L2CAP_FEAT_FIXED_CHAN;
> > static u8 l2cap_fixed_chan[8] = { 0x02, };
> >
> > -static const struct proto_ops l2cap_sock_ops;
> > -
> > static struct workqueue_struct *_busy_wq;
> >
> > -static struct bt_sock_list l2cap_sk_list = {
> > +struct bt_sock_list l2cap_sk_list = {
> > .lock = __RW_LOCK_UNLOCKED(l2cap_sk_list.lock)
> > };
> >
> > static void l2cap_busy_work(struct work_struct *work);
> >
> > -static void __l2cap_sock_close(struct sock *sk, int reason);
> > static void l2cap_sock_close(struct sock *sk);
> > -static void l2cap_sock_kill(struct sock *sk);
> > +void l2cap_sock_kill(struct sock *sk);
>
> If l2cap_core.c includes net/bluetooth/l2cap.h, you don't need the
> declaration above.
Yes, you are right. Thanks for the review.
--
Gustavo F. Padovan
ProFUSION embedded systems - http://profusion.mobi
--
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
