Re: [PATCHv2] Bluetooth: btmrvl: Do not send vendor events to bluetooth stack

[Gustavo Padovan <gustavo@padovan.org>]

Hi Andrei,

* Andrei Emeltchenko <Andrei.Emeltchenko.news@gmail.com> [2012-06-29 17:23:32 +0300]:

> 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 <Andrei.Emeltchenko.news@gmail.com> [2012-06-15 09:10:54 +0300]:
> > > > 
> > > > > From: Andrei Emeltchenko <andrei.emeltchenko@intel.com>
> > > > > 
> > > > > 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 <andrei.emeltchenko@intel.com>
> > > > > Tested-by: Bing Zhao <bzhao@marvell.com>
> > > > > ---
> > > > >  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...

This patch already went to wireless, so I can't change it anymore, can you
send a delta patch with the fixes that are in v2?

	Gustavo