Return-Path: Date: Thu, 15 Jul 2010 11:43:08 -0700 From: "Luis R. Rodriguez" To: "Gustavo F. Padovan" CC: Dan Tian , "linux-bluetooth@vger.kernel.org" , Haijun Liu , Luis Rodriguez Subject: Re: [PATCH 1/3] Add BT3 AMP device support, by Atheros Linux BT3 team. Message-ID: <20100715184308.GF1166@tux> References: <20100715150921.GA3188@vigoh> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" In-Reply-To: <20100715150921.GA3188@vigoh> Sender: linux-bluetooth-owner@vger.kernel.org List-ID: On Thu, Jul 15, 2010 at 08:09:21AM -0700, Gustavo F. Padovan wrote: > Hi Dan, > > * Dan Tian [2010-07-15 15:37:32 +0800]: > > > From 40cbfbf5c1bc9109364970956d9b6f74f5846c70 Mon Sep 17 00:00:00 2001 > > From: Haijun.Liu > > Date: Wed, 14 Jul 2010 22:50:56 +0800 > > Subject: [PATCH 1/3] Add BT3 AMP device support, by Atheros Linux BT3 team. > > > > > > Signed-off-by: Haijun.Liu > > --- > > include/net/bluetooth/hci.h | 342 +++++++++++++++++++- > > include/net/bluetooth/hci_core.h | 264 ++++++++++++++- > > net/bluetooth/cmtp/core.c | 1 + > > net/bluetooth/hci_conn.c | 201 +++++++++++- > > net/bluetooth/hci_core.c | 349 ++++++++++++++++++-- > > net/bluetooth/hci_event.c | 698 +++++++++++++++++++++++++++++++++++++- > > 6 files changed, 1822 insertions(+), 33 deletions(-) > > > Could you split all your patches in little chunks? +1882 is very hard to > review. And please be more verbose on the commit message, it can help us > figure out what's happening. This used to be 1 large patch, it was recently split up to 3, I havent't had a chance yet to review further but my understanding is this this was split up as much as possible in the last try. If you can provide suggestions how to split it up more that might be useful, but agreed, if possible more splitup would help. Dan, can you also please drop the "by Atheros Linux BT3 team." on the subject of the patches. This is already implied by the From and the Signed-off-by. Luis