Uh oh!
There was an error while loading. Please reload this page.
topology2: intel: add generic Bluetooth offload pipelines - #6971
Conversation
Uh oh!
There was an error while loading. Please reload this page.
kv2019i
commented
Jan 23, 2023
V2 (Jan 23rd) update:
|
kv2019i
commented
Jan 23, 2023
ranj063
commented
Jan 23, 2023
Following the long and winded road starting from https://sof-ci.01.org/sofpr/PR6971/build3487/devicetest/index.html (internal ID 20119), I found that the reason why ADLP_RVP_SDW_IPC4ZPH failed is as usual NOT because that model was "Not Available" but because: (/job/testflow/62327) The directory http://sof-storage.ostc.intel.com/pr/sof/PR6971-3487/build3487/tplg/topology2/avs-tplg/ does exist. Could this be caused by this PR? |
Add generic Bluetooth offload pipeline and two example topologies based on cavs-nocodec that are using the new Bluetooth pipelines. 1) cavs-nocodec-bt.conf Example topology that matches configuration used on may Intel SOF boards (SSP2, Bluetooth SCO and A2DP configurations). 2) cavs-nocodec-bt-lbm.conf Variant of first topology that configures all mode with SSP loopback mode, allowing to verify the pipeline operation by capturing the audio back from the same PCM. Requires alsa-utils patch "topology: nhlt: intel: ssp: fix obvious typo for 'codec_provider'" to correctly generate the NHLT blobs for 8/16khz modes. Link: alsa-project/alsa-utils#174 Co-developed-by: Kai Vehmanen <kai.vehmanen@linux.intel.com> Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com> Signed-off-by: Jaska Uimonen <jaska.uimonen@intel.com>
kv2019i
commented
Jan 24, 2023
V3 (Jan 24th) update:
|
juimonen
left a comment
There was a problem hiding this comment.
Pierre's typo patch went upstream, so not sure if needed to mention that in commit message. LGTM.
plbossart
commented
Jan 24, 2023
juimonen
commented
Jan 24, 2023
yes definitely, need update to latest upstream. Just meant that Kai's original message was there perhaps to inform that the needed patch is not yet in upstream alsa-utils head. But now it is -> so we need update also. |
kv2019i
commented
Jan 27, 2023
Let's proceed with merge. |
There was "logical" conflict between this 4476171 and ae8ca22. They both passed separately but not together - and CI never tested them together. This sort of "logical conflict" is very rare and too hard to catch in CI, not worth the effort. The mitigations are:
|
Add generic Bluetooth pipeline definitions that matches what we've had with topology1. Augment passthrough pipelines to allow use of Bluetooth.