sravan/system76-coreboot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

mb/google/brox/var/brox: Enable Class-D calibration

Browse Source 
DC offset of class-D amplifier is 7mV in Brox which is larger than the expected 3mV.
Add a section in the verb table to enable class-D calibration based
on the updated verb table provided by Realtek in b:342506575 comment#6.

This improves the offset to be less than 1mV.

BUG=b:342506575
BRANCH=main
TEST=Verify DC offset of speaker amplier output is less than 1mV with a multimeter when \
     playing -100dB sine waves.

Change-Id: I776f5c24ce3c829cbd64840957c1431608cf2b85
Signed-off-by: Terry Cheong <htcheong@chromium.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/82794
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Shelley Chen <shchen@google.com>
Reviewed-by: Karthik Ramasubramanian <kramasub@google.com>
This commit is contained in:
Terry Cheong 2024-06-04 13:15:54 +08:00 committed by Felix Held
parent ef4d562d2f
commit 3396c4027d
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/mainboard/google/brox/variants/brox/include/variant/hda_verb.h
View File

@ -9,7 +9,7 @@ const u32 cim_verb_data[] = {
/* coreboot specific header */
0x10ec0256, // Codec Vendor / Device ID: Realtek ALC256
0x10ec12ac, // Subsystem ID
0x00000013, // Number of jacks (NID entries)
0x00000014, // Number of jacks (NID entries)
AZALIA_RESET(0x1),
/* NID 0x01, HDA Codec Subsystem ID Verb table */
@ -113,6 +113,11 @@ const u32 cim_verb_data[] = {
0x02046A0C,
0x02050008,
0x02046A0C,
/* Class-D calibration */
0x05850000,
0x0584F880,
0x05850000,
0x0584F880,
};
const u32 pc_beep_verbs[] = {