From: Gustavo Padovan <gustavo@padovan.org>
To: "John W. Linville" <linville@tuxdriver.com>
Cc: linux-wireless@vger.kernel.org, linux-bluetooth@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: pull request: bluetooth 2012-09-08
Date: Wed, 12 Sep 2012 15:30:44 -0300 [thread overview]
Message-ID: <20120912183044.GA28232@joana> (raw)
In-Reply-To: <20120912181309.GB28823@tuxdriver.com>
Hi John,
* John W. Linville <linville@tuxdriver.com> [2012-09-12 14:13:10 -0400]:
> On Sat, Sep 08, 2012 at 06:59:56PM -0300, Gustavo Padovan wrote:
> > Hi John,
> >
> > A few more fixes to 3.6, here we have four important fix to the MGMT interface,
> > two from Johan Hedberg and Andrzej Kaczmarek. Andrei fixed a free on
> > uninitialized memory and I added support for Broadcom/Foxconn devices.
>
> "Bluetooth: mgmt: Implement support for passkey notification" doesn't
> look like a fix at all.
It is a fix in the sense that we were failing to work with some keyboards out
there that only support Bluetooth SSP KeyboardOnly IO capability and there are
few of those in the market now. But yes, we are not fix any crash, etc in the
kernel, not critical though. It is just improving compatibility of the
Bluetooth stack (which is important too). It's up to you merge this or not.
>
> "Bluetooth: Update management interface revision" is suspect. It kinda
> makes sense, but if it is so important than it should have been merged
> much earlier, no?
Not really, but we like to bump version when we do significative changes to
MGMT like the above.
>
> "Bluetooth: Add USB_VENDOR_AND_INTERFACE_INFO() for Broadcom/Foxconn"
> might normally be OK, but it is getting late enough in the cycle
> that I would prefer to delay any device ID patches until the next
> merge window.
Ok.
>
> The others look fine. Do you want me to merge them directly?
> Or would you rather respin your pull request?
You can pull them directly, I'll redo my tree once they get pulled.
Thanks, John.
Gustavo
next prev parent reply other threads:[~2012-09-12 18:30 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-09-08 21:59 pull request: bluetooth 2012-09-08 Gustavo Padovan
2012-09-12 18:13 ` John W. Linville
2012-09-12 18:30 ` Gustavo Padovan [this message]
2012-09-12 18:31 ` Johan Hedberg
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20120912183044.GA28232@joana \
--to=gustavo@padovan.org \
--cc=linux-bluetooth@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).