From: Mikel Astiz <mikel.astiz@xxxxxxxxxxxx> 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
