From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Tue, 5 Mar 2013 20:15:24 -0300 From: Gustavo Padovan To: Marcel Holtmann Cc: Johan Hedberg , linux-bluetooth@vger.kernel.org Subject: Re: [PATCH v3 00/11] Bluetooth: Add asynchronous request support Message-ID: <20130305231524.GA9738@joana> References: <1362508671-6460-1-git-send-email-johan.hedberg@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: List-ID: Hi Johan, * Marcel Holtmann [2013-03-05 11:31:38 -0800]: > Hi Johan, > > > Here's a v3 of the patch set with Marcel's feedback taken into account. > > > > Johan > > > > ---------------------------------------------------------------- > > Johan Hedberg (11): > > Bluetooth: Rename hci_request to hci_req_sync > > Bluetooth: Fix __hci_request() handling of empty requests > > Bluetooth: Split HCI init sequence into three stages > > Bluetooth: Add initial skeleton for asynchronous HCI requests > > Bluetooth: Refactor HCI command skb creation > > Bluetooth: Introduce new hci_req_add function > > Bluetooth: Fix stand-alone HCI command handling > > Bluetooth: Add request cmd_complete and cmd_status functions > > Bluetooth: Use async requests internally in hci_req_sync > > Bluetooth: Remove unused hdev->init_last_cmd > > Bluetooth: Remove empty HCI event handlers > > > > include/net/bluetooth/bluetooth.h | 10 + > > include/net/bluetooth/hci_core.h | 15 +- > > net/bluetooth/hci_core.c | 668 +++++++++++++++++++++++++++++++------ > > net/bluetooth/hci_event.c | 504 +--------------------------- > > net/bluetooth/hci_sock.c | 5 + > > 5 files changed, 603 insertions(+), 599 deletions(-) > > I am fine with this patch set now. > > Acked-by: Marcel Holtmann > > It would be still good if someone else does a sanity read through and extra testing with this patch set before we push it into bluetooth-next. It looked pretty good to me and since we got positive testing feedback from Andre I've gone ahead and pushed all patches to bluetooth-next. Thanks. Gustavo