Re: [PATCH v2 0/8] Adopt btd_profile for A2DP

Hi Mikel,

On Thu, Nov 15, 2012 at 9:27 AM, Mikel Astiz [off-list ref] wrote:

From: Mikel Astiz <redacted>

The proposal is to split A2DP roles into different btd_profile instances, in order to expose .connect and .disconnect.

The changes in v2 include:

- Renaming of functions as suggested by Luiz (avoid using local_xxx)
- Merge of source_shutdown and source_disconnect: they were quite similar and having a generalized function seems a better idea (to be discussed if we can get rid of the gboolean shutdown parameter)
- New patches have been added to address the sink part as well (v1 focused on the source role)

Mikel Astiz (8):
  audio: Trivial function renames
  sink: Generalize disconnection function
  source: Add missing code in source_disconnect()
  source: Expose internal connection API
  sink: Expose internal connection API
  audio: Split A2DP into three btd_profile
  source: Add profile .connect and .disconnect
  sink: Add profile .connect and .disconnect

 profiles/audio/a2dp.c    |  88 ++++++++---------
 profiles/audio/a2dp.h    |   4 +-
 profiles/audio/device.c  |   4 +-
 profiles/audio/device.h  |   3 +
 profiles/audio/manager.c | 242 +++++++++++++++++++++++++++++++++++++++++++----
 profiles/audio/sink.c    | 179 ++++++++++++++++++-----------------
 profiles/audio/sink.h    |   4 +-
 profiles/audio/source.c  | 189 ++++++++++++++++++++----------------
 profiles/audio/source.h  |   4 +-
 9 files changed, 480 insertions(+), 237 deletions(-)

--
1.7.11.7

First 5 patches are now pushed, I had to changed sink_disconnect and
source_disconnect since they were crashing. Please rebase the rest and
reasend.


--
Luiz Augusto von Dentz