[coreboot-gerrit] Patch set updated for coreboot: purism/librem13: Fix HDA codec verbs. Use correct codec vendor id

Youness Alaoui (snifikino@gmail.com) gerrit at coreboot.org
Mon Feb 20 21:59:31 CET 2017


Youness Alaoui (snifikino at gmail.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18411

-gerrit

commit 8a88f68414be7ba3358bfe60e8feb0db3834e8e2
Author: Youness Alaoui <kakaroto at kakaroto.homelinux.net>
Date:   Wed Feb 15 12:52:50 2017 -0500

    purism/librem13: Fix HDA codec verbs. Use correct codec vendor id
    
    There was a 'typo' where the subsystem id was set instead of the codec
    vendor id. This caused the lynxpoint HDA codecs init to fail to find
    the proper codecid verbs so codecs were never initialized. That caused
    the headphones jack to not work.
    
    Change-Id: I975031643fc42937ecaea2300639b90632543f67
    Signed-off-by: Youness Alaoui <youness.alaoui at puri.sm>
---
 src/mainboard/purism/librem13/hda_verb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mainboard/purism/librem13/hda_verb.c b/src/mainboard/purism/librem13/hda_verb.c
index f553df6..958a939 100644
--- a/src/mainboard/purism/librem13/hda_verb.c
+++ b/src/mainboard/purism/librem13/hda_verb.c
@@ -17,7 +17,7 @@
 
 const u32 cim_verb_data[] = {
 	/* coreboot specific header */
-	0x19910269,	/* Codec Vendor / Device ID: Realtek ALC269 */
+	0x10ec0269,	/* Codec Vendor / Device ID: Realtek ALC269 */
 	0x19910269,	/* Subsystem ID */
 	0x0000000c,	/* Number of jacks (NID entries) */
 



More information about the coreboot-gerrit mailing list