Hi Alex, On Fri, May 3, 2013 at 9:39 PM, Alex Deymo <deymo@xxxxxxxxxxxx> wrote: > Hi! > I'm far from an expert in the audio profile code (that's why I had to > include such amount of debug traces =] ). > Both patches are ok with me, I just find my approach a bit more robust since > the next inclusion an avrcp_disconnect() will also need to check if the > avctp is disconnected or not. > Anyway, I just tried yours on my system and it works for me. > Thanks for the patch! Patch is now upstream including some cleanups and other fixes to problems I discovered while testing, let us know if you find anything else has been broken. We will probably do some changes to profiles/audio/device.c, right now the only thing that it should be doing is auto connecting AVRCP if A2DP connects, so Im cleaning up it quite a bit. In future we might be able to completely remove it and perhaps have the core or a dedicated plugin handling profile dependencies such as AVRCP/A2DP using the service infrastructure, but service is not very stable right now so we first need to fix it first before converting the rest to use the service states. Btw, please respond inline, top posting is not recommended as stated http://www.bluez.org/development/lists/ -- Luiz Augusto von Dentz -- 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
