[RFC 1/2] bluetooth: Add MediaTransport volume management

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Fr?d?ric,

On Mon, Nov 12, 2012 at 4:19 PM, Fr?d?ric Danis
<frederic.danis at linux.intel.com> wrote:
> With BlueZ 5 volume updates for HFP will go through transport interface.
>
> Note that this is backward compatible.
> ---
>  src/modules/bluetooth/bluetooth-util.c          |   62 +++++++++++++++----
>  src/modules/bluetooth/bluetooth-util.h          |    4 ++
>  src/modules/bluetooth/module-bluetooth-device.c |   75 +++++++++++++++++++++++
>  3 files changed, 130 insertions(+), 11 deletions(-)
>

Without entering the patch details as already reviewed by Tanu, I have
a general question:

BlueZ 5 will rewrite the MediaTransport API to be following the
standard ObjectManager/Properties interfaces. This means the new API
will not be compatible with the current modules in PA. Don't you think
we should migrate to this new APIs first before adding new features to
PA?

I think you are already assuming that bluetooth-util will abstract the
specific BlueZ version (4.x vs 5.x) so that the modules don't have to
care much. I could agree with this assumption but I believe we should
implement this first.

Cheers,
Mikel


[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux