You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@plbossart@ranj063 FYI. Topology binaries for ACE and cAVS platforms are going to be splitted.
It does not mean that the splitting must break existing builds.
Looks to me that 784bce7 has not even compile tested (for topologies at least) and I do wonder why CI let that patch slip through. I know that we only test IPC3/topology1 in CI, but the build system should have complained at least to block it.
It is one thing to have separate topology2 files for CAVS/ACE, but I don't really see the reason why they have to be exclusive.
Activity
[-][BUG][TPLG] topology2 fails to build[/-][+][BUG][TPLG] IPC4 topology2 fails to build[/+]mengdonglin commentedon Sep 22, 2022
@softwarecki @mwasko Can you help on this issue?
@plbossart @ranj063 FYI. Topology binaries for ACE and cAVS platforms are going to be splitted.
marc-hb commentedon Sep 23, 2022
Fix:
git show 784bce763c0e -- tools/ | git apply -Rujfalusi commentedon Sep 23, 2022
Submitted a fix: #6325
ujfalusi commentedon Sep 23, 2022
It does not mean that the splitting must break existing builds.
Looks to me that 784bce7 has not even compile tested (for topologies at least) and I do wonder why CI let that patch slip through. I know that we only test IPC3/topology1 in CI, but the build system should have complained at least to block it.
It is one thing to have separate topology2 files for CAVS/ACE, but I don't really see the reason why they have to be exclusive.
keqiaozhang commentedon Sep 25, 2022
Fixed by PR#6325