Patch Set 2:

Thanks a lot. I understand it now.

Off-topic:

1. Should the ASL Realtek driver code go into some common location? `src/mainboard/intel/strago/acpi/mainboard.asl` has it too for example.
2. 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

View Change

To view, visit change 40003. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Idca5cc86ba1f5ef3978cfba291a0c06e56ef5958
Gerrit-Change-Number: 40003
Gerrit-PatchSet: 2
Gerrit-Owner: Matt DeVillier <matt.devillier@gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Comment-Date: Wed, 01 Apr 2020 16:41:24 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment