Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/84502?usp=email )
Change subject: mb/starlabs/byte_adl/mk_ii: Correct number of jacks in hda_verb.c
......................................................................
mb/starlabs/byte_adl/mk_ii: Correct number of jacks in hda_verb.c
This was found due to the `_Static_assert()` from CB:84360 failing.
Change-Id: Ibb167b8dc379ca331812255c3e7e049556f2b57b
Signed-off-by: Nicholas Sudsgaard <devel+coreboot(a)nsudsgaard.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/84502
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
Reviewed-by: Sean Rhodes <sean(a)starlabs.systems>
---
M src/mainboard/starlabs/byte_adl/variants/mk_ii/hda_verb.c
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
build bot (Jenkins): Verified
Sean Rhodes: Looks good to me, approved
diff --git a/src/mainboard/starlabs/byte_adl/variants/mk_ii/hda_verb.c b/src/mainboard/starlabs/byte_adl/variants/mk_ii/hda_verb.c
index 46c8738..9b87198 100644
--- a/src/mainboard/starlabs/byte_adl/variants/mk_ii/hda_verb.c
+++ b/src/mainboard/starlabs/byte_adl/variants/mk_ii/hda_verb.c
@@ -7,7 +7,7 @@
/* coreboot specific header */
0x14f11f87, /* Codec Vendor / Device ID: SoundWire SN6140 */
0x14f1035e, /* Subsystem ID */
- 17, /* Number of jacks (NID entries) */
+ 19, /* Number of jacks (NID entries) */
/* Reset Codec First */
AZALIA_RESET(0x1),
--
To view, visit https://review.coreboot.org/c/coreboot/+/84502?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ibb167b8dc379ca331812255c3e7e049556f2b57b
Gerrit-Change-Number: 84502
Gerrit-PatchSet: 2
Gerrit-Owner: Nicholas Sudsgaard <devel+coreboot(a)nsudsgaard.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/84414?usp=email )
(
5 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.
)Change subject: mb/google: Correct number of jacks in hda_verb.c
......................................................................
mb/google: Correct number of jacks in hda_verb.c
This corrects the mismatch found in the verb tables of Monroe Chromebase
and Link Chromebook.
The verb data was not aligned to a multiple of 4, therefore an entry was
repeated as padding.
This was found due to the `_Static_assert()` from CB:84360 failing.
TEST=Tested on LINK under Linux and Win11, audio working properly under both.
Change-Id: Id377281af310642a6ba77e5a0002ca1dfca38827
Signed-off-by: Nicholas Sudsgaard <devel+coreboot(a)nsudsgaard.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/84414
Reviewed-by: Matt DeVillier <matt.devillier(a)gmail.com>
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
---
M src/mainboard/google/beltino/variants/monroe/hda_verb.c
M src/mainboard/google/link/hda_verb.c
2 files changed, 5 insertions(+), 2 deletions(-)
Approvals:
build bot (Jenkins): Verified
Matt DeVillier: Looks good to me, approved
diff --git a/src/mainboard/google/beltino/variants/monroe/hda_verb.c b/src/mainboard/google/beltino/variants/monroe/hda_verb.c
index 064bebf..3b529e2 100644
--- a/src/mainboard/google/beltino/variants/monroe/hda_verb.c
+++ b/src/mainboard/google/beltino/variants/monroe/hda_verb.c
@@ -6,7 +6,7 @@
/* coreboot specific header */
0x10ec0283, // Codec Vendor / Device ID: Realtek ALC283
0x10ec0283, // Subsystem ID
- 0x0000000e, // Number of jacks (NID entries)
+ 0x00000010, // Number of jacks (NID entries)
0x0017ff00, // Function Reset
0x0017ff00, // Double Function Reset
diff --git a/src/mainboard/google/link/hda_verb.c b/src/mainboard/google/link/hda_verb.c
index 9381b57..d3309cd 100644
--- a/src/mainboard/google/link/hda_verb.c
+++ b/src/mainboard/google/link/hda_verb.c
@@ -6,7 +6,7 @@
/* coreboot specific header */
0x11020011, // Codec Vendor / Device ID: Creative CA0132
0x10280550, // Subsystem ID
- 0x00000014, // Number of jacks + Number of Malcolm setup blocks.
+ 0x00000015, // Number of jacks + Number of Malcolm setup blocks.
/* Malcolm Setup */
@@ -62,6 +62,9 @@
/* Enable and set EAPD pin for headphone jack */
AZALIA_VERB_12B(0x0, 0x10, 0x78d, 0x2),
+ AZALIA_VERB_12B(0x0, 0x10, 0x78d, 0x2), /* Padding */
+ AZALIA_VERB_12B(0x0, 0x10, 0x78d, 0x2), /* Padding */
+ AZALIA_VERB_12B(0x0, 0x10, 0x78d, 0x2), /* Padding */
/* Pin Widget Verb Table */
--
To view, visit https://review.coreboot.org/c/coreboot/+/84414?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Id377281af310642a6ba77e5a0002ca1dfca38827
Gerrit-Change-Number: 84414
Gerrit-PatchSet: 7
Gerrit-Owner: Nicholas Sudsgaard <devel+coreboot(a)nsudsgaard.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/84395?usp=email )
(
4 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.
)Change subject: mb/samsung/lumpy: Correct number of jacks in hda_verb.c
......................................................................
mb/samsung/lumpy: Correct number of jacks in hda_verb.c
The verb data was not aligned to a multiple of 4, therefore an entry was
repeated as padding. This has not been tested.
This was found due to the `_Static_assert()` from CB:84360 failing.
Change-Id: I3a40e6229419ee7d1a238916ee6d49cf9314f6ab
Signed-off-by: Nicholas Sudsgaard <devel+coreboot(a)nsudsgaard.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/84395
Reviewed-by: Paul Menzel <paulepanter(a)mailbox.org>
Reviewed-by: Matt DeVillier <matt.devillier(a)gmail.com>
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
---
M src/mainboard/samsung/lumpy/hda_verb.c
1 file changed, 4 insertions(+), 1 deletion(-)
Approvals:
Paul Menzel: Looks good to me, but someone else must approve
build bot (Jenkins): Verified
Matt DeVillier: Looks good to me, approved
diff --git a/src/mainboard/samsung/lumpy/hda_verb.c b/src/mainboard/samsung/lumpy/hda_verb.c
index d18d8d2..b3350a1 100644
--- a/src/mainboard/samsung/lumpy/hda_verb.c
+++ b/src/mainboard/samsung/lumpy/hda_verb.c
@@ -6,12 +6,15 @@
/* coreboot specific header */
0x10134210, // Codec Vendor / Device ID: Cirrus Logic CS4210
0x152D0924, // Subsystem ID
- 0x00000007, // Number of jacks
+ 0x00000008, // Number of jacks
/* NID 0x01, HDA Codec Subsystem ID Verb Table: 0x152D0924 */
AZALIA_SUBVENDOR(0, 0x152D0924),
0x00170500,
+ 0x00170500, /* Padding */
+ 0x00170500, /* Padding */
+ 0x00170500, /* Padding */
/* Pin Widget Verb Table */
--
To view, visit https://review.coreboot.org/c/coreboot/+/84395?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I3a40e6229419ee7d1a238916ee6d49cf9314f6ab
Gerrit-Change-Number: 84395
Gerrit-PatchSet: 6
Gerrit-Owner: Nicholas Sudsgaard <devel+coreboot(a)nsudsgaard.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>