Return-Path: Date: Tue, 12 Oct 2010 15:50:52 +0300 From: Ville Tervo To: "ext Gustavo F. Padovan" Cc: Marcel Holtmann , "linux-bluetooth@vger.kernel.org" Subject: Re: [PATCH 2/7] Bluetooth: Add LE connect support Message-ID: <20101012125052.GB642@null> References: <1286390535-27462-1-git-send-email-ville.tervo@nokia.com> <1286390535-27462-3-git-send-email-ville.tervo@nokia.com> <1286449089.6145.91.camel@aeonflux> <20101007163619.GB4893@vigoh> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20101007163619.GB4893@vigoh> List-ID: On Thu, Oct 07, 2010 at 06:36:20PM +0200, ext Gustavo F. Padovan wrote: > Hi Ville, > > * Marcel Holtmann [2010-10-07 12:58:09 +0200]: > > > Hi Ville, > > > > > Add logic to create LE connections. > > > Could you be more verbose on the commit message, that way people can > understand better what you are doing in the patch. ;) Sure. > > > > > > > Signed-off-by: Ville Tervo > > > --- > > > include/net/bluetooth/hci.h | 1 + > > > include/net/bluetooth/hci_core.h | 6 ++- > > > net/bluetooth/hci_conn.c | 38 ++++++++++++++- > > > net/bluetooth/hci_event.c | 100 +++++++++++++++++++++++++++++++++++++- > > > 4 files changed, 141 insertions(+), 4 deletions(-) > > > > > > diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h > > > index b86aed5..b326240 100644 > > > --- a/include/net/bluetooth/hci.h > > > +++ b/include/net/bluetooth/hci.h > > > @@ -162,6 +162,7 @@ enum { > > > #define SCO_LINK 0x00 > > > #define ACL_LINK 0x01 > > > #define ESCO_LINK 0x02 > > > +#define LE_LINK 0x03 > > > > this is not a value defined by the specification, while the others are. > > And some functions match these to HCI event. So if wanna do it like > > this, then using something like 0x80 is better. > > A comment in the code saying that is also a good ideia. Done -- VIlle