build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/22980 )
Change subject: sb/intel/i82801ix: Use the common ACPI pirq generator
......................................................................
Patch Set 6:
Build Successful
https://qa.coreboot.org/job/coreboot-checkpatch/19734/ : SUCCESS
https://qa.coreboot.org/job/coreboot-gerrit/64976/ : SUCCESS
--
To view, visit https://review.coreboot.org/22980
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I62e520f53fa3f928a8e6f3b3cf33af2acdd53ed9
Gerrit-Change-Number: 22980
Gerrit-PatchSet: 6
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Sun, 24 Dec 2017 07:58:12 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/22981
to look at the new patch set (#2).
Change subject: sb/intel/i82801{g,j}x: Automatically generate ACPI PIRQ tables
......................................................................
sb/intel/i82801{g,j}x: Automatically generate ACPI PIRQ tables
Both southbridges need to be done at once since this southbridge code
is used for different northbridges, which fails to compile when done
separately.
This needs an acpi_name functions in the northbridge code to be defined.
Change-Id: I286d251ddf8fcae27dd07011a1cd62d8f4847683
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
D src/mainboard/apple/macbook21/acpi/i945_pci_irqs.asl
D src/mainboard/asrock/g41c-gs/acpi/x4x_pci_irqs.asl
D src/mainboard/asus/p5gc-mx/acpi/i945_pci_irqs.asl
D src/mainboard/foxconn/g41s-k/acpi/x4x_pci_irqs.asl
D src/mainboard/getac/p470/acpi/i945_pci_irqs.asl
D src/mainboard/gigabyte/ga-945gcm-s2l/acpi/i945_pci_irqs.asl
D src/mainboard/gigabyte/ga-g41m-es2l/acpi/x4x_pci_irqs.asl
D src/mainboard/ibase/mb899/acpi/i945_pci_irqs.asl
D src/mainboard/intel/d510mo/acpi/pineview_pci_irqs.asl
D src/mainboard/intel/d945gclf/acpi/i945_pci_irqs.asl
D src/mainboard/intel/dg43gt/acpi/x4x_pci_irqs.asl
D src/mainboard/kontron/986lcd-m/acpi/i945_pci_irqs.asl
D src/mainboard/lenovo/t60/acpi/i945_pci_irqs.asl
D src/mainboard/lenovo/x60/acpi/i945_pci_irqs.asl
D src/mainboard/lenovo/z61t/acpi/i945_pci_irqs.asl
D src/mainboard/roda/rk886ex/acpi/i945_pci_irqs.asl
M src/northbridge/intel/i945/acpi/hostbridge.asl
M src/northbridge/intel/i945/northbridge.c
M src/northbridge/intel/pineview/acpi/hostbridge.asl
M src/northbridge/intel/pineview/northbridge.c
M src/northbridge/intel/x4x/acpi/hostbridge.asl
M src/northbridge/intel/x4x/northbridge.c
M src/southbridge/intel/i82801gx/Kconfig
M src/southbridge/intel/i82801gx/i82801gx.h
M src/southbridge/intel/i82801gx/lpc.c
M src/southbridge/intel/i82801jx/Kconfig
M src/southbridge/intel/i82801jx/i82801jx.h
M src/southbridge/intel/i82801jx/lpc.c
28 files changed, 86 insertions(+), 1,191 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/81/22981/2
--
To view, visit https://review.coreboot.org/22981
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I286d251ddf8fcae27dd07011a1cd62d8f4847683
Gerrit-Change-Number: 22981
Gerrit-PatchSet: 2
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/22980
to look at the new patch set (#6).
Change subject: sb/intel/i82801ix: Use the common ACPI pirq generator
......................................................................
sb/intel/i82801ix: Use the common ACPI pirq generator
For this to work the northbridge and lpc bridge device need acpi_name
functions.
This also needs DEFAULT_RCBA being configured in Kconfig.
TESTED on Thinkpad X200, a valid PIRQ routing in SSDT in
/sys/firmware/acpi/tables/SSDT
Change-Id: I62e520f53fa3f928a8e6f3b3cf33af2acdd53ed9
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
D src/mainboard/lenovo/t400/acpi/gm45_pci_irqs.asl
D src/mainboard/lenovo/x200/acpi/gm45_pci_irqs.asl
D src/mainboard/roda/rk9/acpi/gm45_pci_irqs.asl
M src/northbridge/intel/gm45/acpi/hostbridge.asl
M src/northbridge/intel/gm45/northbridge.c
M src/southbridge/intel/i82801ix/Kconfig
M src/southbridge/intel/i82801ix/i82801ix.h
M src/southbridge/intel/i82801ix/lpc.c
8 files changed, 32 insertions(+), 245 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/80/22980/6
--
To view, visit https://review.coreboot.org/22980
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I62e520f53fa3f928a8e6f3b3cf33af2acdd53ed9
Gerrit-Change-Number: 22980
Gerrit-PatchSet: 6
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/22980 )
Change subject: sb/intel/i82801ix: Use the common ACPI pirq generator
......................................................................
Patch Set 5:
Build Successful
https://qa.coreboot.org/job/coreboot-checkpatch/19732/ : SUCCESS
https://qa.coreboot.org/job/coreboot-gerrit/64973/ : SUCCESS
--
To view, visit https://review.coreboot.org/22980
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I62e520f53fa3f928a8e6f3b3cf33af2acdd53ed9
Gerrit-Change-Number: 22980
Gerrit-PatchSet: 5
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Sun, 24 Dec 2017 06:34:32 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No