Return-Path: Date: Wed, 4 May 2011 09:03:57 +0300 From: Johan Hedberg To: Claudio Takahasi Cc: linux-bluetooth@vger.kernel.org Subject: Re: [PATCH v2 02/13] Add Bluetooth address type definition Message-ID: <20110504060357.GA3711@jh-x301> References: <1304030229-6672-3-git-send-email-claudio.takahasi@openbossa.org> <1304458357-21571-1-git-send-email-claudio.takahasi@openbossa.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1304458357-21571-1-git-send-email-claudio.takahasi@openbossa.org> Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi Claudio, On Tue, May 03, 2011, Claudio Takahasi wrote: > Values defined to LE(public and random) are defined in the Bluetooth > Core Specification. For basic rate, there isn't address type concept. > The constants introduced by this commit will be used to identify the > remote address type, basically to distinguish LE/BR devices before > to request the L2CAP connection. > --- > lib/bluetooth.h | 4 ++++ > 1 files changed, 4 insertions(+), 0 deletions(-) > > diff --git a/lib/bluetooth.h b/lib/bluetooth.h > index 738e07a..98b8f1c 100644 > --- a/lib/bluetooth.h > +++ b/lib/bluetooth.h > @@ -130,6 +130,10 @@ typedef struct { > #define BDADDR_ALL (&(bdaddr_t) {{0xff, 0xff, 0xff, 0xff, 0xff, 0xff}}) > #define BDADDR_LOCAL (&(bdaddr_t) {{0, 0, 0, 0xff, 0xff, 0xff}}) > > +#define BDADDR_TYPE_LE_PUBLIC 0x00 > +#define BRADDR_TYPE_LE_RANDOM 0x01 > +#define BDADDR_TYPE_BR 0xff Firstly this still needs Marcel's blessing (though I don't really see any other solution than an address type with three possible values). Secondly, if this is ever going to be added to the L2CAP socket address you'd have to have 0x00 as BDADDR_TYPE_BR for backwards compatibility. Johan