From 6fa36c9c2ce39d81ad123ef936e1eb945208bcda Mon Sep 17 00:00:00 2001 From: Matt DeVillier Date: Wed, 12 Apr 2017 14:25:56 -0500 Subject: [PATCH] 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 Reviewed-on: https://review.coreboot.org/19262 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth --- src/mainboard/google/slippy/variants/falco/hda_verb.c | 4 ++-- 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),