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

[Andrei Emeltchenko <Andrei.Emeltchenko.news@gmail.com>]

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...

Best regards 
Andrei Emeltchenko