OSDN Git Service

codecs/rt5672: Remove Intel SST specific HeadsetMic workaround
authorHans de Goede <hdegoede@redhat.com>
Tue, 9 Feb 2021 21:01:01 +0000 (22:01 +0100)
committerJaroslav Kysela <perex@perex.cz>
Thu, 11 Feb 2021 18:18:12 +0000 (19:18 +0100)
commita9d3953eef559f58327155576bbbbc068ca448c8
tree2a4ba152ff5d636fe6576d4c79f681aa65120256
parent889c5b9809f1635005d77871bb9abc09df6a3cc3
codecs/rt5672: Remove Intel SST specific HeadsetMic workaround

There used to be an issue with the HeadsetMic on BYT devices with
a rt5672 codec only being recorded on the left channel, while the
right channel recorded silence.

The ucm2/codecs/rt5672/HeadsetMic.conf contains a workaround for
this which changes the SST record-stream demuxer settings to
use "slot 0" for both the left and right channels when recording
from the HeadsetMic.

There are 2 problems with this workaround:

1. It uses SST specific mixer settings causing things to break
when using the SOF driver.

2. Files under uc2m/codecs should be architecture agnostic and should
not depend on platform dependent mixer settings such as the SST driver
mixer settings.

The problem which the workaround used to work around has since been
fixed in the kernel by switching the wire format from TDM/DSP A to I2S:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=0ceb8a36d023d4bb4ffca3474a452fb1dfaa0ef2

This fix has landed in 5.8 and has been backported to all recent
stable kernel releases.

Since this is fixed in the kernel now, we can drop the workaround,
fixing the 2 problems mentioned above.

This was tested on a ThinkPad 8 tablet, which is the same device as
where the HeadsetMic problem was originally noticed.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Jaroslav Kysela <perex@perex.cz>
ucm2/codecs/rt5672/HeadsetMic.conf