Skip to content

[BUG] update SOF CI docker with new alsa-utils #9790

Closed

Description

@kv2019i

Update SOF docker image used in public CI github flows once alsa-utils alsa-project/alsa-utils#286 is merged upstream.

This is needed to complete work in #9763

FYI @singalsu

Activity

  1. added
    alsa-utilsIssues caused by alsa-utils
    bugSomething isn't working as expected
    on Jan 22, 2025
  2. kv2019i commented on Feb 4, 2025

    @kv2019i
    CollaboratorAuthor

    @cgturner1 Any updates with this one?

  3. cgturner1 commented on Feb 5, 2025

    @cgturner1
    Contributor

    Building a new docker today...once that is complete will need to verify it pulled in the right components before I push upstream hopefully it works first shot for me (since the build takes a few hours) but I think a new docker version should be pushed by end of week

  4. kv2019i commented on Feb 18, 2025

    @kv2019i
    CollaboratorAuthor

    The SOF docker update pushed on Feb 6 has the alsa-utils fix for SSP encoding:
    "Topology: NHLT: Intel: Fix DMA slots config in SSP blob"

    But the fixes for dmic 1ch (and #9763 ) is not yet included, so let's keep this open.

  5. cgturner1 commented on Apr 3, 2025

    @cgturner1
    Contributor

    @kv2019i forgot to circle back on this ... i think we are ok to close since the fixes were included?

  6. kv2019i commented on Apr 3, 2025

    @kv2019i
    CollaboratorAuthor

    @cgturner1 ? Did you do an update yet that includes #9763 fix (alsa-project/alsa-utils#286 )?

    So patch with ```commit fd67c11b6d3d84308db14150997ae773283caec2 (sof/master, origin/master)
    Topology: NHLT: Intel: Improve error message for DMIC enable conflict

    
    This was merged close to the time we did the Feb 5 update. I've not pinged about this as I've been waiting whether https://github.com/thesofproject/sof/pull/9870 matures on time for 2.13 release in May. If #9870 is merged, then we can stop these docker updates for alsa-utils/lib altogether.
    
  7. lgirdwood commented on Apr 3, 2025

    @lgirdwood
    Member

    Should we not stop here and merge the local ALSA build PR and have the Docker use local ALSA to build topology.
    #9870

  8. cgturner1 commented on Apr 3, 2025

    @cgturner1
    Contributor

    Should we not stop here and merge the local ALSA build PR and have the Docker use local ALSA to build topology. #9870

    Yea that makes sense i had thought this had been dealt with on the previous update to the docker but can focus on changing to use the local ALSA instead as the future path

  9. cgturner1 commented on Apr 16, 2025

    @cgturner1
    Contributor

    still working on debugging the work from #9870 in a alsa-less docker...I think I'm running into path issues I expect this to be working soon (within a week) if we need to update the alsa in our current docker ASAP i can do that in parallel so to unblock any issues while i debug and implement the changes to use the local alsa

  10. kv2019i commented on May 13, 2025

    @kv2019i
    CollaboratorAuthor

    @cgturner1 Should we keep this open and track the remaining Jenkins/docker issues separate? Or do we keep this open to track?

  11. cgturner1 commented on May 13, 2025

    @cgturner1
    Contributor

    @cgturner1 Should we keep this open and track the remaining Jenkins/docker issues separate? Or do we keep this open to track?

    I think we should close this as the new docker does have the latest alsa utils and we can open a separate task for fixing the compatability issues on the docker with the new local alsa tools install

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

P2Critical bugs or normal featuresalsa-utilsIssues caused by alsa-utilsbugSomething isn't working as expected

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions