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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux