Re: [pulseaudio-discuss] [PATCH v7 04/13] bluetooth: Modular API for A2DP codecs

2019-04-06 Thread Pali Rohár
Tanu, I think I addressed all review comments in V8 version. Please look at V8. -- Pali Rohár pali.ro...@gmail.com signature.asc Description: PGP signature ___ pulseaudio-discuss mailing list pulseaudio-discuss@lists.freedesktop.org

[pulseaudio-discuss] [PATCH v8 07/12] bluetooth: Add A2DP aptX and aptX HD codecs support

2019-04-06 Thread Pali Rohár
This patch provides support for aptX and aptX HD codecs in bluetooth A2DP profile. It uses open source LGPLv2.1+ licensed libopenaptx library which can be found at https://github.com/pali/libopenaptx. aptX for s24 stereo samples provides fixed 6:1 compression ratio and bitrate 352.8 kbit/s, aptX

[pulseaudio-discuss] [PATCH v8 09/12] bluetooth: Add more variants of SBC codec

2019-04-06 Thread Pali Rohár
Specify configuration for Low, Middle, High and Ultra High Quality of SBC codec. SBC codec in Ultra High Quality has higher quality than aptX. Automatic Quality mode matches configuration of SBC codec which was used before this change. Which means that it accept configuration between Low and High

[pulseaudio-discuss] [PATCH v8 10/12] bluetooth: policy: Reflect a2dp profile names

2019-04-06 Thread Pali Rohár
In next patches, codec name is appended end the end of a2dp profile names. --- src/modules/bluetooth/module-bluetooth-policy.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/modules/bluetooth/module-bluetooth-policy.c

[pulseaudio-discuss] [PATCH v8 06/12] bluetooth: Set correct endianity of audio samples for SBC codec

2019-04-06 Thread Pali Rohár
Pulseaudio SBC codec defines that audio samples are in PA_SAMPLE_S16LE format which is little endian. But libsbc library expects audio samples by default in host endianity which is big endian on big endian system. So SBC support on big endian system is broken. To fix this problem tell libsbc

[pulseaudio-discuss] [PATCH v8 12/12] bluetooth: policy: Treat bi-directional A2DP profiles as suitable for VOIP

2019-04-06 Thread Pali Rohár
Previously module-bluetooth-policy was switching from A2DP to HSP profile when VOIP application started recording of source. Now it switch to profile with the highest priority which has both sink and source. In most cases it is HSP profile, but it can be also bi-directional A2DP profile (e.g.

[pulseaudio-discuss] [PATCH v8 05/12] bluetooth: Clean up SBC bitpool calculation

2019-04-06 Thread Pali Rohár
Remove dead code and replace numeric bitpool values by macro definitions. Maximal bitpool value in fill_capabilities() was reduced from 64 to 53 (SBC_BITPOOL_HQ_JOINT_STEREO_44100) because default_bitpool() already set maximal value to 53. This patch does not change SBC behavior as maximal

[pulseaudio-discuss] [PATCH v8 08/12] bluetooth: Add A2DP FastStream codec support

2019-04-06 Thread Pali Rohár
This patch provides support for FastStream codec in bluetooth A2DP profile. FastStream codec is bi-directional, which means that support both music playback and microphone voice at the same time. FastStream codec is just SBC codec with fixed parameters. For playback are used following parameters:

[pulseaudio-discuss] [PATCH v8 01/12] bluetooth: policy: Remove BlueZ 4 related code

2019-04-06 Thread Pali Rohár
--- src/modules/bluetooth/module-bluetooth-policy.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/src/modules/bluetooth/module-bluetooth-policy.c b/src/modules/bluetooth/module-bluetooth-policy.c index 4d3126f65..0a6d59d28 100644 ---

[pulseaudio-discuss] [PATCH v8 04/12] bluetooth: Add missing validations for SBC codec parameters

2019-04-06 Thread Pali Rohár
--- src/modules/bluetooth/a2dp-codec-sbc.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/src/modules/bluetooth/a2dp-codec-sbc.c b/src/modules/bluetooth/a2dp-codec-sbc.c index b735d385c..e590cf01a 100644 --- a/src/modules/bluetooth/a2dp-codec-sbc.c +++

[pulseaudio-discuss] [PATCH v8 11/12] bluetooth: Implement A2DP codec switching and backchannel support

2019-04-06 Thread Pali Rohár
Some A2DP codecs (like FastStream or aptX Low Latency) are bi-directional and can be used for both music playback and audio call. This patch implements usage of backchannel if A2DP codec provided by pulseaudio API supports it. A2DP codec switching needs new version of bluez as older version does

[pulseaudio-discuss] [PATCH v8 02/12] bluetooth: Update a2dp-codecs.h from upstream bluez project

2019-04-06 Thread Pali Rohár
--- src/modules/bluetooth/a2dp-codecs.h | 336 +++- src/modules/bluetooth/bluez5-util.c | 6 +- 2 files changed, 331 insertions(+), 11 deletions(-) diff --git a/src/modules/bluetooth/a2dp-codecs.h b/src/modules/bluetooth/a2dp-codecs.h index 8afcfcb24..93e9d3523

[pulseaudio-discuss] [PATCH v8 03/12] bluetooth: Modular API for A2DP codecs

2019-04-06 Thread Pali Rohár
This patch introduce new modular API for bluetooth A2DP codecs. Its benefits are: * bluez5-util and module-bluez5-device does not contain any codec specific code, they are codec independent. * For adding new A2DP codec it is needed just to adjust one table in a2dp-codec-util.c file. All

[pulseaudio-discuss] [PATCH v8 00/12] New API for Bluetooth A2DP codecs

2019-04-06 Thread Pali Rohár
This is 8th version of my patch series for modular A2DP codec API and aptX support. This version address review comments by Tanu. This patch series provides new modular API for Bluetooth A2DP codecs, clean up module-bluez5-device and bluez5-util to be codec independent and convert SBC codec into