Return-Path: MIME-Version: 1.0 In-Reply-To: <1349374107.27233.36.camel@aeonflux> References: <1349356447-8369-1-git-send-email-chanyeol.park@samsung.com> <1349374107.27233.36.camel@aeonflux> Date: Thu, 4 Oct 2012 23:22:50 +0300 Message-ID: Subject: Re: [PATCH 4/4] audio: Add check for non-a2dp codec From: Luiz Augusto von Dentz To: Marcel Holtmann Cc: chanyeol.park@samsung.com, linux-bluetooth@vger.kernel.org Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi Marcel, On Thu, Oct 4, 2012 at 9:08 PM, Marcel Holtmann wrote: > Hi Luiz, > >> > This patch adds checks(vendor ID, vendor specific codec ID) to make sure of >> > non-a2dp codec selection. >> > --- >> > audio/a2dp-codecs.h | 6 +++++ >> > audio/a2dp.c | 68 ++++++++++++++++++++++++++++++++++++++++++++++++++- >> > 2 files changed, 73 insertions(+), 1 deletion(-) >> > >> > diff --git a/audio/a2dp-codecs.h b/audio/a2dp-codecs.h >> > index 51c796a..e3d2cba 100644 >> > --- a/audio/a2dp-codecs.h >> > +++ b/audio/a2dp-codecs.h >> > @@ -26,6 +26,7 @@ >> > #define A2DP_CODEC_MPEG12 0x01 >> > #define A2DP_CODEC_MPEG24 0x02 >> > #define A2DP_CODEC_ATRAC 0x03 >> > +#define A2DP_CODEC_NON_A2DP 0xFF >> >> I prefer A2DP_CODEC_VENDOR > > what is the specification calling these? And yes, I would prefer VENDOR > as well. Apparently it comes from the assigned numbers http://www.bluetooth.org/Technical/AssignedNumbers/audio-video.htm: 1 1 1 1 1 1 1 1 non-A2DP n/a A2DP But the spec refer to it as 4.7 Vendor Specific A2DP Codec, IMO the term non-A2DP can be confused with not defined in A2DP spec when in fact it is defined: "4.2.3 Vendor Specific A2DP Codecs The device may support other codecs as Vendor Specific A2DP codecs. A user of a Vendor Specific A2DP codec (hereafter the Vendor) will need to define parameters and other information necessary for use of the codec in A2DP." There is no such a thing as non-A2DP in the spec, so again to avoid confusion I would stick to the terminology vendor or vendor specific. -- Luiz Augusto von Dentz