From: Mikel Astiz <[email protected]>

A second proposal integrating some ideas from Tanu and David:
- Hex numbers used for bit masks (patch 2/4).
- Link to the Bluetooth spec added in comment, to clarify magic numbers (patch 
2/4).
- Port names improved (patch 3/4).

There was no conclusion about how the port creation should be handled. This 
patchset doesn't modify the previous codebase in this sense and therefore both 
input and output ports are created always. This is not fantastic but it's 
mitigated by the port availability flag, which should clarify if the ports are 
usable.

>From original cover-letter:

This patchset considers the device's form factor (Bluetooth device class) in 
order to decide the name and description of the ports.

The approach assumes the form factor of a paired device will remain constant, 
which seems to be the case in any real-life scenario.

Patch 4/4 simplifies the core by removing dynamic port creation, recently added 
due to Bluetooth's interests. However, now that we have a slightly different 
interpretation of ports, this core extension should be reconsidered.

Mikel Astiz (4):
  bluetooth: Use enum to represent form factors
  bluetooth: Add specific form factor for phones
  bluetooth: Use form factor to set port name and description
  Revert "card: Support adding ports dynamically"

 src/modules/bluetooth/bluetooth-util.c          | 78 +++++++++++++++------
 src/modules/bluetooth/bluetooth-util.h          | 16 ++++-
 src/modules/bluetooth/module-bluetooth-device.c | 90 ++++++++++++++++++++++---
 src/modules/module-card-restore.c               | 43 ------------
 src/pulsecore/card.c                            | 19 ------
 src/pulsecore/card.h                            |  1 -
 src/pulsecore/core.h                            |  1 -
 7 files changed, 154 insertions(+), 94 deletions(-)

-- 
1.8.1.2

_______________________________________________
pulseaudio-discuss mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss

Reply via email to