apps: add Auracast USB sample with LC3 codec #1659
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
KKopyscinski
force-pushed
the
auracast-usb
branch
4 times, most recently
from
0a75ddf
to
9c91f10
Compare
sjanc
reviewed
Dec 11, 2023
KKopyscinski
commented
Dec 11, 2023
KKopyscinski
commented
Dec 11, 2023
KKopyscinski
force-pushed
the
auracast-usb
branch
from
9c91f10
to
8668c54
Compare
|
@sjanc apache/mynewt-core#3121 patch was merged and this builds now, can I merge this PR? |
KKopyscinski
force-pushed
the
auracast-usb
branch
from
8668c54
to
2a1ef73
Compare
|
@andrzej-kaczmarek removed unused and debug code, added serial ID config |
KKopyscinski
force-pushed
the
auracast-usb
branch
from
2a1ef73
to
fccc2bf
Compare
andrzej-kaczmarek
requested changes
Feb 22, 2024
apps/auracast_usb/syscfg.yml
Outdated
| BLE_MAX_BIG: 1 | ||
| BLE_MAX_BIS: 2 | ||
|
|
||
| BLE_PHY_NRF52_HEADERMASK_WORKAROUND: 1 |
There was a problem hiding this comment.
this is only valid for nrf52
There was a problem hiding this comment.
moved under syscfg.vals.BSP_NRF52:
|
|
||
| #define BROADCAST_SID 1 | ||
| #define BROADCAST_SDU_INTVL MYNEWT_VAL(LC3_FRAME_DURATION) | ||
| #define BROADCAST_SAMPLE_RATE BLE_AUDIO_SAMPLING_RATE_48000_HZ |
There was a problem hiding this comment.
this should depend on lc3 configuration
There was a problem hiding this comment.
added separate defines for BLE_AUDIO_SAMPLING_RATE_24000_HZ, BLE_AUDIO_SAMPLING_RATE_48000_HZ and BLE_AUDIO_SAMPLING_RATE_96000_HZ. I feel like we should have some kind of macro that translates sampling rate from number value to the spec, but this can be done in separate PR
KKopyscinski
force-pushed
the
auracast-usb
branch
5 times, most recently
from
eed8067
to
b7ad5c2
Compare
KKopyscinski
force-pushed
the
auracast-usb
branch
4 times, most recently
from
776ade7
to
a8ea520
Compare
KKopyscinski
force-pushed
the
auracast-usb
branch
from
a8ea520
to
694e65c
Compare
This adds sample that acts as USB sound device. Audio signal is coded using LC3 and broadacasted using Auracast package.
Adds build target for auracast_usb application.
Cortex M-33 FPU doesn't support double precision arithmetic. Use float instead of double so the performance can be improved. This allows to use better interpolator in resampler.
Compile using floats instead of doubles, so the FPUs that don't support double-precision arithmetic can be used.
KKopyscinski
force-pushed
the
auracast-usb
branch
from
694e65c
to
0f27618
Compare
|
This needs libsamplerate patch to work properly: libsndfile/libsamplerate#210 |
andrzej-kaczmarek
approved these changes
Apr 15, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This adds sample that acts as USB sound device. Audio signal is coded using LC3 and broadacasted using Auracast package.