Subrata Banik has submitted this change. ( https://review.coreboot.org/c/coreboot/+/83920?usp=email )
Change subject: mb/google/brox/var/lotso: Update verb table ......................................................................
mb/google/brox/var/lotso: Update verb table
Update verb table provided by Realtek on 20240710.
Restults: SNR > 90 (spec>=90).
BUG=b:349996984 TEST=emerge-brox sys-boot/coreboot sys-boot/chromeos-bootimage
Change-Id: Ic4f03d09010efa7e32713b2697d5832255f64317 Signed-off-by: Jian Tong tongjian@huaqin.corp-partner.google.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/83920 Reviewed-by: Kun Liu liukun11@huaqin.corp-partner.google.com Reviewed-by: Subrata Banik subratabanik@google.com Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Karthik Ramasubramanian kramasub@google.com --- M src/mainboard/google/brox/variants/lotso/include/variant/hda_verb.h 1 file changed, 6 insertions(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Karthik Ramasubramanian: Looks good to me, approved Kun Liu: Looks good to me, but someone else must approve Subrata Banik: Looks good to me, approved
diff --git a/src/mainboard/google/brox/variants/lotso/include/variant/hda_verb.h b/src/mainboard/google/brox/variants/lotso/include/variant/hda_verb.h index 3e21f6d..48d5cc5 100644 --- a/src/mainboard/google/brox/variants/lotso/include/variant/hda_verb.h +++ b/src/mainboard/google/brox/variants/lotso/include/variant/hda_verb.h @@ -8,7 +8,7 @@ const u32 cim_verb_data[] = { /* coreboot specific header */ 0x10ec0257, // Codec Vendor / Device ID: Realtek ALC257 - 0x10ec3a2c, // Subsystem ID + 0x10ec0257, // Subsystem ID 0x0000000f, // Number of jacks (NID entries)
AZALIA_RESET(0x1), @@ -66,6 +66,11 @@ 0x02044003, 0x0205000A, 0x02047778, + /* Widget node 0x20 - 5 */ + 0x02050037, + 0x0204FE15, + 0x02050030, + 0x02041000, };
const u32 pc_beep_verbs[] = {