From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Mon, 23 Jan 2012 11:24:38 +0200 From: Johan Hedberg To: Emeltchenko Andrei , linux-bluetooth@vger.kernel.org Subject: Re: [RFC] Bluetooth: Add alloc_skb chan operator Message-ID: <20120123092438.GC9471@x220> References: <1326978832-9888-1-git-send-email-Andrei.Emeltchenko.news@gmail.com> <20120123090046.GB8800@x220> <20120123090930.GC8800@x220> <20120123091329.GB6214@aemeltch-MOBL1> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20120123091329.GB6214@aemeltch-MOBL1> Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi Andrei, On Mon, Jan 23, 2012, Emeltchenko Andrei wrote: > > On Mon, Jan 23, 2012, Johan Hedberg wrote: > > > On Thu, Jan 19, 2012, Emeltchenko Andrei wrote: > > > > From: Andrei Emeltchenko > > > > > > > > Add channel-specific skb allocation method > > > > > > > > Signed-off-by: Andrei Emeltchenko > > > > --- > > > > include/net/bluetooth/l2cap.h | 3 +++ > > > > net/bluetooth/l2cap_core.c | 22 +++++++++++++++------- > > > > net/bluetooth/l2cap_sock.c | 9 +++++++++ > > > > 3 files changed, 27 insertions(+), 7 deletions(-) > > > > > > Applied. Thanks. > > > > Actually I had to remove this commit (hopefully nobody pulled from my > > tree meanwhile). It introduces the following compiler warning: > > > > net/bluetooth/l2cap_core.c: In function 'l2cap_skbuff_fromiovec': > > net/bluetooth/l2cap_core.c:1523:15: warning: unused variable 'sk' [-Wunused-variable] > > > > Please fix and resend (I think you can keep the ack from Marcel). > > Sorry, I've noticed it also and sent last week v2. Yep, found the v2 now and applied it. If you want to avoid this kind of confusion in the future please also reply to the original thread in addition to sending the new version (especially if the original thread has no indication that there's a new patch on the way). Johan