Change in coreboot[master]: src/soc/intel/braswell/southcluster.c: Configure IO APIC - coreboot-gerrit

5 Nov 2018


      Frans Hendriks has posted comments on this change. ( https://review.coreboot.org/29423 )
Change subject: src/soc/intel/braswell/southcluster.c: Configure IO APIC
......................................................................
Patch Set 3:
...
Patch Set 3:
(2 comments)
The area need to be reserved to ensure no other use of this area. It might work without this area, but to be ACPI compliant coreboot must claim used areas. 
In comment of CBFS_SIZE mentioned it defaults span the whole ROM.
The use of IO_APIC is in same style of RCBA (at least for braswell). You mean the RCBAxx() macro style used on some other platforms?
-- 
To view, visit https://review.coreboot.org/29423
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: I917c30892b46ac1d964e7bab339082d17a1e706d
Gerrit-Change-Number: 29423
Gerrit-PatchSet: 3
Gerrit-Owner: Frans Hendriks fhendriks@eltan.com
Gerrit-Reviewer: Frans Hendriks fhendriks@eltan.com
Gerrit-Reviewer: Hannah Williams hannah.williams@intel.com
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-CC: Patrick Rudolph siro@das-labor.org
Gerrit-Comment-Date: Mon, 05 Nov 2018 07:33:14 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No