[pulseaudio-commits] [Git][pulseaudio/pulseaudio][master] bluetooth: ldac: Use format as FLOAT32LE

[PulseAudio Marge Bot]

PulseAudio Marge Bot pushed to branch master at PulseAudio / pulseaudio


Commits:
c6862152 by Sanchayan Maity at 2021-01-22T20:10:54+05:30
bluetooth: ldac: Use format as FLOAT32LE

LDAC encoder already supports S16, S24, S32 and F32LE. Using FLOAT32LE
for the sample format would avoid the additional call for conversion to
pa_sconv_s32le_from_float32ne. perf tool shows this as being the function
called frequently after encode. So, just avoid this by using sample format
as F32LE.

Part-of: <https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/merge_requests/486>

- - - - -


2 changed files:

- src/modules/bluetooth/a2dp-codec-gst.c
- src/modules/bluetooth/a2dp-codec-ldac-gst.c


Changes:

=====================================
src/modules/bluetooth/a2dp-codec-gst.c
=====================================
@@ -321,6 +321,9 @@ static GstCaps *gst_create_caps_from_sample_spec(const pa_sample_spec *ss) {
         case PA_SAMPLE_S32LE:
             sample_format = "S32LE";
             break;
+        case PA_SAMPLE_FLOAT32LE:
+            sample_format = "F32LE";
+            break;
         default:
             pa_assert_not_reached();
             break;


=====================================
src/modules/bluetooth/a2dp-codec-ldac-gst.c
=====================================
@@ -208,7 +208,7 @@ bool gst_init_ldac(struct gst_info *info, pa_sample_spec *ss, bool for_encoding)
         return false;
     }
 
-    ss->format = PA_SAMPLE_S32LE;
+    ss->format = PA_SAMPLE_FLOAT32LE;
 
     switch (info->a2dp_codec_t.ldac_config->frequency) {
         case LDAC_SAMPLING_FREQ_44100:



View it on GitLab: https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/commit/c686215268452765803f1445a8e4db16a323c1d3

-- 
View it on GitLab: https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/commit/c686215268452765803f1445a8e4db16a323c1d3
You're receiving this email because of your account on gitlab.freedesktop.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/pulseaudio-commits/attachments/20210122/877d5e39/attachment.htm>