Change in coreboot[master]: intel/sandybridge: Gather MMCONF_BASE_ADDRESS defaults
Felix Held has posted comments on this change. ( https://review.coreboot.org/20813 ) Change subject: intel/sandybridge: Gather MMCONF_BASE_ADDRESS defaults ...................................................................... Patch Set 2: (1 comment) https://review.coreboot.org/#/c/20813/2/src/northbridge/intel/sandybridge/Kc... File src/northbridge/intel/sandybridge/Kconfig: https://review.coreboot.org/#/c/20813/2/src/northbridge/intel/sandybridge/Kc... PS2, Line 120: default 0xf8000000 if USE_NATIVE_RAMINIT
this is inside the "if !USE_NATIVE_RAMINIT" block, so if I understand this haven't read the code completely, but it's probably a good idea to leave MMCONF_BASE_ADDRESS at 0xf8000000 for the native ram init in this patch...
-- To view, visit https://review.coreboot.org/20813 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I924c43cec1e36e84db40e4b8e1dd0e05cad2b978 Gerrit-Change-Number: 20813 Gerrit-PatchSet: 2 Gerrit-Owner: Nico Huber <nico.h@gmx.de> Gerrit-Reviewer: Alexander Couzens <lynxis@fe80.eu> Gerrit-Reviewer: Felix Held <felix-coreboot@felixheld.de> Gerrit-Reviewer: James Ye <jye836@gmail.com> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-Comment-Date: Sat, 29 Jul 2017 02:41:38 +0000 Gerrit-HasComments: Yes
participants (1)
-
Felix Held (Code Review)