Skip to content

DMIC capture with 2ch when configured for 4ch gives corrupt audio #329

Closed
Closed
DMIC capture with 2ch when configured for 4ch gives corrupt audio#329
Assignees
singalsu
Labels
P2Critical bugs or normal featuresbugSomething isn't working as expectedverifiedQA/Bug reporter verifies solution worked.

Description

@singalsu
singalsu
opened on Sep 7, 2018

The captured audio sounds like 1/2x speed due to capturing all four microphones but formatting the output as 2ch.

The DMIC FW driver should get the actual capture channels count from DAI and disable for the capture session the non-used PDM controllers for excess channels.

The driver can do only best effort, e.g. capture from microphones 0 and 1 for 2ch capture. It does not currently have information to decide for 0 and 1 vs. 2 and 3.

Activity

  1. singalsu
    self-assigned this
    on Sep 7, 2018
  2. mengdonglin
    added
    bugSomething isn't working as expected
    P2Critical bugs or normal features
    on Sep 17, 2018

  3. singalsu commented on Oct 12, 2018

    @singalsu
    singalsu
    on Oct 12, 2018
    CollaboratorAuthor

    As update the fix is under construction in SOFT PR #90.

  4. singalsu commented on Oct 22, 2018

    @singalsu
    singalsu
    on Oct 22, 2018
    CollaboratorAuthor

    The SOFT PR #90 that contains a fix is merged. I just checked that stereo recording is now rejected if we use four channel DMIC capture topology (and four channels recording sounds OK).

    $ arecord -Dhw:0,6 -c 4 -f S32_LE -r 48000 -d 10 tmp.wav
Recording WAVE 'tmp.wav' : Signed 32 bit Little Endian, Rate 48000 Hz, Channels 4
$ arecord -Dhw:0,6 -c 2 -f S32_LE -r 48000 -d 10 tmp.wav
Recording WAVE 'tmp.wav' : Signed 32 bit Little Endian, Rate 48000 Hz, Stereo
arecord: set_params:1305: Channels count non available

  5. singalsu commented on Oct 22, 2018

    @singalsu
    singalsu
    on Oct 22, 2018
    CollaboratorAuthor

    Closing.

  6. singalsu
    closed this as completedon Oct 22, 2018
  7. mengdonglin
    added
    verifiedQA/Bug reporter verifies solution worked.
    on Oct 24, 2018
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 featuresbugSomething isn't working as expectedverifiedQA/Bug reporter verifies solution worked.

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions