Hi Johan, Arun,
On 3/15/2011 10:08 AM, Johan Hedberg wrote:
Hi Brian,
On Tue, Mar 15, 2011, Brian Gix wrote:
This patch violates the AVDTP specification, as indicated in other
email. SEID may not be changed by RECONFIGURE.
Actually this patch doesn't seem to be about AVDTP_Reconfigure but about
AVDTP_Close + AVDTP_SetConfiguration + AVDTP_Open. So from that
perspective there's no violation of the spec as far as I can see.
However, as I mentioned in my other email this will break interop with
some car kits which expect the new stream to be configured to the same
SBC SEP in the case that the new codec is SBC. So some checks for
matching codec needs to be done and only if the new codec is different
than the old one can we clear setup->rsep.
Johan
Yes, Sorry I didn't look closer at the patch. I saw "Reconfigure" and
my internal alarm bells were going off.
My apologies.
--
Brian Gix
bgix@xxxxxxxxxxxxxx
Employee of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
