diff options
| author | Matt DeVillier <matt.devillier@gmail.com> | 2017-04-12 14:25:56 -0500 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2017-04-15 23:09:58 +0200 |
| commit | 6fa36c9c2ce39d81ad123ef936e1eb945208bcda (patch) | |
| tree | 0841bce86f9b68ba9f76fa526d8e25b173e21d1f | |
| parent | 66d5b924405010b48121dba54b781886546538d7 (diff) | |
| download | coreboot-6fa36c9c2ce39d81ad123ef936e1eb945208bcda.tar.xz | |
google/slippy: fix internal mic for falco/wolf variants
The HDA verb for falco/wolf's internal mic was wrong, preventing the mic
from working properly in Windows and macOS (the Linux driver overrides
the verb table, so wasn't affected). Set the verb connector/jack bits
properly, to no connector / no jack detect, in order to fix.
Also, make (2) small non-functional fixes:
On falco, NID 0x1A was being disabled twice (instead of 0x1A and 0x1B
both being disabled - copy/paste error).
On wolf, NID 0x19 was set to an internal analog mic, where it should have
been disabled (again, copy/paste error).
Both these errors were introduced when consolidating/upstreaming
and were not present in the original Chromium sources.
Test: boot Windows [8/8.1/10] and verify mic functional with Realtek
drivers on both falco and wolf.
Change-Id: I9c343dda4762f0b1f814318c155e22c59d2da8db
Signed-off-by: Matt DeVillier <matt.devillier@gmail.com>
Reviewed-on: https://review.coreboot.org/19262
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martinroth@google.com>
| -rw-r--r-- | src/mainboard/google/slippy/variants/falco/hda_verb.c | 4 | ||||
| -rw-r--r-- | src/mainboard/google/slippy/variants/wolf/hda_verb.c | 10 |
2 files changed, 5 insertions, 9 deletions
diff --git a/src/mainboard/google/slippy/variants/falco/hda_verb.c b/src/mainboard/google/slippy/variants/falco/hda_verb.c index 790d501576..3d12d8aa37 100644 --- a/src/mainboard/google/slippy/variants/falco/hda_verb.c +++ b/src/mainboard/google/slippy/variants/falco/hda_verb.c @@ -32,7 +32,7 @@ const u32 cim_verb_data[] = { /* Pin Widget Verb Table */ /* Pin Complex (NID 0x12) DMIC - interior mobile lid */ - AZALIA_PIN_CFG(0x0, 0x12, 0xb7a61010), + AZALIA_PIN_CFG(0x0, 0x12, 0xb7a60110), /* Pin Complex (NID 0x14) SPKR-OUT PORTD */ // group 1, front left/right @@ -58,7 +58,7 @@ const u32 cim_verb_data[] = { AZALIA_PIN_CFG(0x0, 0x1a, 0x411111f0), /* Pin Complex (NID 0x1B) LINE2 PORTE - Disabled */ - AZALIA_PIN_CFG(0x0, 0x1a, 0x411111f0), + AZALIA_PIN_CFG(0x0, 0x1b, 0x411111f0), /* Pin Complex (NID 0x1d) PCBeep */ // eapd low on ex-amp, laptop, custom enable diff --git a/src/mainboard/google/slippy/variants/wolf/hda_verb.c b/src/mainboard/google/slippy/variants/wolf/hda_verb.c index b043bf52ce..56fe320082 100644 --- a/src/mainboard/google/slippy/variants/wolf/hda_verb.c +++ b/src/mainboard/google/slippy/variants/wolf/hda_verb.c @@ -37,7 +37,7 @@ const u32 cim_verb_data[] = { /* Pin Widget Verb Table */ /* Pin Complex (NID 0x12) DMIC - interior mobile lid */ - AZALIA_PIN_CFG(0x0, 0x12, 0xb7a61010), + AZALIA_PIN_CFG(0x0, 0x12, 0xb7a60110), /* Pin Complex (NID 0x14) SPKR-OUT - Internal Speakers */ // group 1, cap 0 @@ -59,12 +59,8 @@ const u32 cim_verb_data[] = { // connector, External left panel AZALIA_PIN_CFG(0x0, 0x19, 0x03a11020), - /* Pin Complex (NID 0x1A) LINE1 - Internal Mic */ - // group 1, cap 1 - // no connector, no jack detect - // mic in, analog connection - // Fixed function, internal, Location N/A - AZALIA_PIN_CFG(0x0, 0x1a, 0x90a70111), + /* Pin Complex (NID 0x1A) LINE1 PORTC - Disabled */ + AZALIA_PIN_CFG(0x0, 0x1a, 0x411111f0), /* Pin Complex (NID 0x1B) LINE2 - Disabled */ AZALIA_PIN_CFG(0x0, 0x1b, 0x411111f0), |