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:
Patch Set 2:
Thanks a lot. I understand it now.
Off-topic:
- Should the ASL Realtek driver code go into some common location? `src/mainboard/intel/strago/acpi/mainboard.asl` has it too for example.
- Should Realtek be contacted to update the drivers?
the hookup is somewhat board/platform specific, so not sure there's a benefit. Strago is just a reference board for some of the cyan variants (all except cyan proper I believe), so we can either change it too or not, but don't believe anyone would care either way