@@ -72,7 +72,12 @@ static void homer_init_chip(struct proc_chip *chip)
if (read_pba_bar(chip, 0, &hbase, &hsize)) {
prlog(PR_DEBUG, " HOMER Image at 0x%llx size %lldMB\n",
hbase, hsize / 0x100000);
- mem_reserve_hw("ibm,homer-image", hbase, hsize);
+
+ if (!mem_range_is_reserved(hbase, hsize)) {
+ prlog(PR_WARNING,
+ "HOMER image is not reserved! Reserving\n");
+ mem_reserve_hw("ibm,homer-image", hbase, hsize);
+ }
chip->homer_base = hbase;
chip->homer_size = hsize;
@@ -90,9 +95,13 @@ static void homer_init_chip(struct proc_chip *chip)
* Only reserve it if we have no homer image or if it
* doesn't fit in it (only check the base).
*/
- if (sbase < hbase || sbase > (hbase + hsize) ||
- (hbase == 0 && sbase > 0))
+ if ((sbase < hbase || sbase > (hbase + hsize) ||
+ (hbase == 0 && sbase > 0)) &&
+ !mem_range_is_reserved(sbase, ssize)) {
+ prlog(PR_WARNING,
+ "SLW image is not reserved! Reserving\n");
mem_reserve_hw("ibm,slw-image", sbase, ssize);
+ }
chip->slw_base = sbase;
chip->slw_bar_size = ssize;
Hostboot is going to cover these ranges with its own reservations (to allow access from the HBRT image), so don't overwrite reservations with our own. Signed-off-by: Jeremy Kerr <jk@ozlabs.org> --- hw/homer.c | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-)