10 Dec
2017
10 Dec
'17
6:07 p.m.
Felix Held has posted comments on this change. ( https://review.coreboot.org/22808 )
Change subject: nuvoton/nct6776: Add ACPI declarations ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/22808/1/src/superio/nuvoton/nct6776/acpi/sup... File src/superio/nuvoton/nct6776/acpi/superio.asl:
https://review.coreboot.org/#/c/22808/1/src/superio/nuvoton/nct6776/acpi/sup... PS1, Line 20: NCT6886 NCT6776
https://review.coreboot.org/#/c/22808/1/src/superio/nuvoton/nct6776/acpi/sup... PS1, Line 85: PNP_IRQ0, 8, /* First IRQ */ i'm not sure if this is relevant here, but some LDNs like the KBC have two IRQ registers
--
To view, visit https://review.coreboot.org/22808
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Idb76b2e99e90a213e2695efc1afd4fa9069c134f
Gerrit-Change-Number: 22808
Gerrit-PatchSet: 1
Gerrit-Owner: Tobias Diedrich ranma+coreboot@tdiedrich.de
Gerrit-Reviewer: Felix Held felix-coreboot@felixheld.de
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-Comment-Date: Sun, 10 Dec 2017 18:07:18 +0000
Gerrit-HasComments: Yes
2546
days inactive
2546
days old
0 comments
1 participants
participants (1)
-
Felix Held (Code Review)