From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Fri, 29 Jun 2012 17:23:32 +0300 From: Andrei Emeltchenko To: Gustavo Padovan , linux-bluetooth@vger.kernel.org Subject: Re: [PATCHv2] Bluetooth: btmrvl: Do not send vendor events to bluetooth stack Message-ID: <20120629142330.GB8718@aemeltch-MOBL1> References: <477F20668A386D41ADCC57781B1F7043083A8A6E5F@SC-VEXCH1.marvell.com> <1339740654-5930-1-git-send-email-Andrei.Emeltchenko.news@gmail.com> <20120619032630.GE1679@joana> <20120628134803.GC14705@aemeltch-MOBL1> <20120629140635.GB29045@x220> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20120629140635.GB29045@x220> Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi Johan, On Fri, Jun 29, 2012 at 05:06:35PM +0300, Johan Hedberg wrote: > Hi Andrei, > > On Thu, Jun 28, 2012, Andrei Emeltchenko wrote: > > On Tue, Jun 19, 2012 at 12:26:30AM -0300, Gustavo Padovan wrote: > > > Hi Andrei, > > > > > > * Andrei Emeltchenko [2012-06-15 09:10:54 +0300]: > > > > > > > From: Andrei Emeltchenko > > > > > > > > Vendor-specific events shall be processed in driver and not sent > > > > to bluetooth stack where they screw up HCI command countings. > > > > > > > > Signed-off-by: Andrei Emeltchenko > > > > Tested-by: Bing Zhao > > > > --- > > > > drivers/bluetooth/btmrvl_drv.h | 2 +- > > > > drivers/bluetooth/btmrvl_main.c | 18 +++++++++++++++--- > > > > drivers/bluetooth/btmrvl_sdio.c | 9 ++++++--- > > > > 3 files changed, 22 insertions(+), 7 deletions(-) > > > > > > Patch has been applied to the bluetooth tree. Thanks. > > > > Which version have you applied? Old one? > > The patch that's upstream doesn't have the Tested-by tag so it seems > Gustavo might have applied the old version of the patch instead of v2. > Could you confirm? Yes, there is old version. Apart from missing Tested-by tag there is one style issue, debug issue and variable scope change which do not affect functionality though. If that patch would applied to bluetooth-next first I would notice that much earlier... Best regards Andrei Emeltchenko