[alsa-devel] [PATCH 0/3] ASoC: Intel: Add support for surface3 tablet

[Stephen J]

On Jul 20, 2016 8:48 AM, "Bastien Nocera" <hadess at hadess.net> wrote:
>
> On Fri, 2016-07-08 at 15:39 +0530, Vinod Koul wrote:
> > Surface3 tablet is CHT based device which used RT5645 codec. But the
> > ACPI ID
> > is 10EC5640 which belong to RT5640 codec :(
> > So we add quirk support to load cht-machine driver for surface tablet
> > combination, add the ACPI id and DMI fix for jack detect in codec and
> > finally machine update.
> >
> > This fixes Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=9800
> > 1
> > Please verify and send your tested-by.
>
>
> Did anyone test this?
At least I did

> I get this output:
> Jul 20 16:20:32 surface kernel: rt5640 i2c-10EC5640:00: Device with ID
register 0x6308 is not rt5640/39
> Jul 20 16:20:34 surface kernel: cht-bsw-rt5645 cht-bsw-rt5645:
snd-soc-dummy-dai <-> media-cpu-dai mapping ok
> Jul 20 16:20:34 surface kernel: cht-bsw-rt5645 cht-bsw-rt5645:
snd-soc-dummy-dai <-> deepbuffer-cpu-dai mapping ok
> Jul 20 16:20:34 surface kernel: compress asoc: snd-soc-dummy-dai <->
compress-cpu-dai mapping ok
> Jul 20 16:20:34 surface kernel: cht-bsw-rt5645 cht-bsw-rt5645:
rt5645-aif1 <-> ssp2-port mapping ok
> Jul 20 16:20:34 surface kernel: input: chtrt5645 Headset as
/devices/pci0000:00/808622A8:00/cht-bsw-rt5645/sound/card0/input25
>
> With the early warning telling me it's probably not going through the
> quirks, though the rest of the output makes me think otherwise.

Both rt5640 and rt5645 drivers pick up the sound chip, but only the correct
one succeeds.

>
> After PulseAudio starts I get:
> Jul 20 16:20:40 surface pulseaudio[1315]: [pulseaudio]
module-alsa-card.c: Failed to find a working profile.
> Jul 20 16:41:32 surface pulseaudio[1315]: [pulseaudio] module.c: Failed
to load module "module-alsa-card" (argument: "device_id="0"
name="platform-cht-bsw-rt5645"
card_name="alsa_card.platform-cht-bsw-rt5645" namereg_fail=false tsched=yes
fixed_latency_range=no ignore_dB=no deferred_volume=yes use_ucm=yes
card_properties="module-udev-detect.discovered=1""): initialization failed.
> Jul 20 16:20:40 surface kernel:  Audio Port: ASoC: no backend DAIs
enabled for Audio Port
> [and plenty more of that last line]
>
> $ journalctl -b | grep "Audio Port: ASoC: no backend DAIs enabled for
Audio Port" | wc -l
> 1802
>
> Ideas?

Are you using the alsa-lib patch to add UCM configuration too [1]?
Pulseaudio needs it to initialize properly (without other manual
intervention described in the bugzilla ticket for these patches).

[1]
http://git.alsa-project.org/?p=alsa-lib.git;a=commit;h=b6c69d7067a1818e53da774bdbdc3023f299d579