[coreboot-gerrit] New patch to review 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:15:03 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 b2394ccdd5901d81ff50caa3933d7d882a721823
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
---
 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