From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Sender: "Gustavo F. Padovan" Date: Fri, 2 Dec 2011 09:02:47 +0900 From: Gustavo Padovan To: Marcel Holtmann Cc: johan.hedberg@gmail.com, linux-bluetooth@vger.kernel.org Subject: Re: [PATCH 2/2] Bluetooth: Ignore HCI_Read_Local_Commands failures Message-ID: <20111202000247.GB2688@joana> References: <1321898484-17952-1-git-send-email-johan.hedberg@gmail.com> <1321898484-17952-2-git-send-email-johan.hedberg@gmail.com> <20111201124039.GB2894@joana> <1322744973.26198.31.camel@aeonflux> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1322744973.26198.31.camel@aeonflux> List-ID: Hi Marcel, * Marcel Holtmann [2011-12-01 14:09:33 +0100]: > Hi Gustavo, > > > > Some 1.2 controllers will fail the HCI_Read_Local_Commands HCI command > > > even though (according to the specification) they should support it. > > > Since this HCI command is part of the controller init sequence > > > controllers broken in this manner would be unusable. Since not having > > > the list of supported commands is not critical for these controllers it > > > is better to just ignore failures in this case (additionally, this is > > > also how older kernels used to behave). > > > > > > Signed-off-by: Johan Hedberg > > > --- > > > net/bluetooth/hci_event.c | 16 ++++++++++++++++ > > > 1 files changed, 16 insertions(+), 0 deletions(-) > > > > Both patches have been applied, thanks. > > I would have preferred you didn't. Since now I have to get this all > fixed up as well. Is this for both of them? Gustavo