Matt DeVillier has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40003 )
Change subject: mb/google/cyan: Adjust CID for realtek audio codec ......................................................................
Patch Set 2:
(3 comments)
https://review.coreboot.org/c/coreboot/+/40003/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/40003/1//COMMIT_MSG@11 PS1, Line 11:
Can you elaborate a little bit, where the […]
Done
https://review.coreboot.org/c/coreboot/+/40003/1//COMMIT_MSG@13 PS1, Line 13: Linux
Add version for conveniance.
Done
https://review.coreboot.org/c/coreboot/+/40003/1/src/mainboard/google/cyan/a... File src/mainboard/google/cyan/acpi/codec_realtek.asl:
https://review.coreboot.org/c/coreboot/+/40003/1/src/mainboard/google/cyan/a... PS1, Line 22: INTCCFFD
Where is this ID from? […]
I've elaborated in the commit msg. Yes, both Linux and Windows use the HID/CID for driver attachment matching. The HID (hardware ID) is checked first, then the CID (Compatible ID) which can be a string or an array.
The change in google/buddy was modeled after this one, but got merged first, so the commit msg there is a bit confusing =P