Repository navigation
Audio ml support for lemans - #2162
qti-sbojja wants to merge 2 commits into
Conversation
🔨 Build Failure Analysis — PR #2162PR: #2162
VerdictThis is not a compilation error but a merge conflict during PR integration. The PR cannot be merged into the base branch without manual conflict resolution. 📎 Detailed analysis: Full report |
🔨 Build Failure Analysis — PR #2162PR: #2162
VerdictThis is a merge conflict, not a compilation error. The PR cannot be merged into the base branch 📎 Detailed analysis: Full report |
PR #2162 — validate-patchPR: #2162
Final Summary
|
PR #2162 — checker-log-analyzerPR: #2162
Detailed report: Full report
|
The ADSP PAS on several Qualcomm SoCs (e.g. SA8775P/lemans, QCS8300, Monaco) needs more than a single SMMU stream ID for the audio ML (machine-learning) offload use case. In addition to the primary ADSP stream ID, the DSP/SPF issues DMA transactions through extra stream IDs that are not covered by the existing single-entry mapping. Relax the iommus constraint to allow between 1 and 4 entries so the additional ML offload stream IDs can be described, fixing dtb_check schema violations on device trees that add them. Signed-off-by: Sairamreddy Bojja <sbojja@qti.qualcomm.com>
… SMMU mappings The audio ML (machine-learning) offload use case requires a contiguous, physically addressable buffer shared with the audio DSP/SPF, and issues DMA transactions through additional SMMU stream IDs that are not covered by the existing ADSP mapping. Add a dedicated reusable shared-dma-pool CMA region (16 MiB, 4 MiB aligned) and wire it to the q6apm DAIs node via memory-region. A dedicated pool guarantees the alignment and contiguity the DSP expects and isolates these allocations from the default CMA region. Under the EL2 (Gunyah/hypervisor) configuration the SMMU is fully enforcing, so the ML transactions are otherwise blocked and faulted: arm-smmu 15000000.iommu: Blocked unknown Stream ID 0x3060 arm-smmu 15000000.iommu: Blocked unknown Stream ID 0x3062 Add the ML-related stream IDs (0x3060 mask 0x9, 0x3062 mask 0x1) to the remoteproc_adsp iommus property in the EL2 overlay so these buffers are translated by the SMMU instead of being rejected. Signed-off-by: Sairamreddy Bojja <sbojja@qti.qualcomm.com>
7931eb7 to
926e65e
Compare
🔨 Build Failure Analysis — PR #2162PR: #2162
VerdictThe build failure is caused by a merge conflict introduced when integrating this PR with the already-merged topic branches. This is a PR-introduced integration issue, not a pre-existing problem. No compilation errors were found — the build never reached the compilation stage due to the unresolved merge conflict. 📎 Detailed analysis: Full report |
🔨 Build Failure Analysis — PR #2162PR: #2162
VerdictThis is not a compilation failure. The build failed during the integration/merge phase due to a pre-existing conflict in the qcom-next integration branch. The PR's changes are not at fault — the integration branch has diverged and contains conflicting modifications to the same binding file. 📎 Detailed analysis: Full report |
PR #2162 — validate-patchPR: #2162
Final Summary
|
PR #2162 — checker-log-analyzerPR: #2162
Detailed report: Full report
|
Shiraz Hashim (shashim-quic)
left a comment
Shiraz Hashim (shashim-quic)
left a comment
There was a problem hiding this comment.
PENDING: dt-bindings: remoteproc: qcom,pas-common: allow up to 4 iommus
downstream changes to fundamental bindings cannot be merged (though I see it is part of exception). Did you discuss with remoteproc folks.
The audio ML (machine-learning) offload use case requires a contiguous,
physically addressable buffer shared with the audio DSP/SPF, and issues
DMA transactions through additional SMMU stream IDs that are not
covered by the existing ADSP mapping.
Add a dedicated reusable shared-dma-pool CMA region (16 MiB, 4 MiB
aligned) and wire it to the q6apm DAIs node via memory-region. A
dedicated pool guarantees the alignment and contiguity the DSP expects
and isolates these allocations from the default CMA region.
Under the EL2 (Gunyah/hypervisor) configuration the SMMU is fully
enforcing, so the ML transactions are otherwise blocked and faulted:
arm-smmu 15000000.iommu: Blocked unknown Stream ID 0x3060
arm-smmu 15000000.iommu: Blocked unknown Stream ID 0x3062
Add the ML-related stream IDs (0x3060 mask 0x9, 0x3062 mask 0x1) to the
remoteproc_adsp iommus property in the EL2 overlay so these buffers are
translated by the SMMU instead of being rejected.
CRs-Fixed: 4727557
JIRA: https://jira-dc.qualcomm.com/jira/browse/QLIJIRA-210