Skip to content

Auxiliary devices in SOF break SOFCARD #466

Closed

Description

@xiulipan

Auxiliary devices in SOF thesofproject/linux#2501 break most SOF TEST https://sof-ci.01.org/sofpr/PR3545/build7503/devicetest/

After some quick check root cause to be SOFCARD

ubuntu@sh-cml-mantis-hda-03:~$ grep 'sof-[a-z]' /proc/asound/cards|awk '{print $1;}'
0
2
ubuntu@sh-cml-mantis-hda-03:~$ cat /proc/asound/cards
 0 [sofprobes      ]: sof-probes - sof-probes
                      DellInc.-Inspiron5590--
 1 [Device         ]: USB-Audio - USB Audio Device
                      C-Media Electronics Inc. USB Audio Device at usb-0000:00:14.0-1, full speed
 2 [sofhdadsp      ]: sof-hda-dsp - sof-hda-dsp
                      DellInc.-Inspiron5590--

Activity

  1. xiulipan commented on Oct 23, 2020

    @xiulipan
    ContributorAuthor

    @aiChaoSONG Could you help to have a quick fix for this?
    @ranj063 Why there is a new card for the probe? I do not see any new PCM. Will the name be changed? Can we just ignore the card by probe keyword.

  2. aiChaoSONG commented on Oct 23, 2020

    @aiChaoSONG

    @xiulipan I just want to exclude sof-probes with another grep, but we need @ranj063 to confirm on the name.

  3. xiulipan commented on Oct 23, 2020

    @xiulipan
    ContributorAuthor

    @aiChaoSONG Let's wait for @ranj063 confirmation. I see thesofproject/linux#2501 is closed now, maybe it is not the final solution.

  4. fredoh9 commented on Oct 23, 2020

    @fredoh9
    Contributor

    sof-probes card can be any number by probing order. We need to ignore it.
    2501 is final solution but not going to merge to topic/sof-dev now. Acillary bus should be upstream first and then this will be merged to our github project. We should be ready with the change.

  5. plbossart commented on Oct 23, 2020

    @plbossart
    Member

    Agree with @fredoh9 we need to have sof-test ready (and sof-ci is relevant) for the addition of probes.

    Let's not repeat the fiasco I started with the ACPI transition...

  6. ranj063 commented on Oct 23, 2020

    @ranj063
    Contributor

    Im sorry I dont understand what you mean by ignore. We want to test probes as well. If the problem is that we assume that the pcm devices in topology will always be in Card 0, we need to fix that logic.

    We was discussiong the basic playback and capture test failures. Current sof-test chose sof-probes card, so we need to ignore sof-probes card for current device test cases. For new Probe test case, this will not be ignored.

  7. ranj063 commented on Oct 23, 2020

    @ranj063
    Contributor

    @aiChaoSONG Let's wait for @ranj063 confirmation. I see thesofproject/linux#2501 is closed now, maybe it is not the final solution.

    the aux bus stuff is being reviewed upstream and will land sof-dev after it is accepted upstream

  8. aiChaoSONG commented on Oct 26, 2020

    @aiChaoSONG

    @ranj063 Thanks for the confirmation, I can add a grep to ignore the sof-probes now.

    This is how we get the card nunber : SOFCARD=$(grep 'sof-[a-z]' /proc/asound/cards|awk '{print $1;}'), if there is no PCM in sof-probes, a grep -v should help

  9. aiChaoSONG commented on Jan 26, 2021

    @aiChaoSONG

    confirmed fixed by @xiulipan ,close

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

Metadata

Metadata

Assignees

Labels

type:bugSomething doesn't work as expected

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions