aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/soc/amd/genoa_poc/domain.c2
-rw-r--r--src/vendorcode/amd/opensil/genoa_poc/memmap.c32
-rw-r--r--src/vendorcode/amd/opensil/genoa_poc/opensil.h2
3 files changed, 17 insertions, 19 deletions
diff --git a/src/soc/amd/genoa_poc/domain.c b/src/soc/amd/genoa_poc/domain.c
index b34d2e36bb..711dfc3d79 100644
--- a/src/soc/amd/genoa_poc/domain.c
+++ b/src/soc/amd/genoa_poc/domain.c
@@ -15,7 +15,7 @@
void read_soc_memmap_resources(struct device *domain, unsigned long *idx)
{
- *idx = add_opensil_memmap(domain, *idx);
+ add_opensil_memmap(domain, idx);
}
static void genoa_domain_set_resources(struct device *domain)
diff --git a/src/vendorcode/amd/opensil/genoa_poc/memmap.c b/src/vendorcode/amd/opensil/genoa_poc/memmap.c
index 8b54b5f567..1e633d9d7d 100644
--- a/src/vendorcode/amd/opensil/genoa_poc/memmap.c
+++ b/src/vendorcode/amd/opensil/genoa_poc/memmap.c
@@ -84,11 +84,11 @@ static void print_memory_holes(void *unused)
BOOT_STATE_INIT_ENTRY(BS_DEV_RESOURCES, BS_ON_ENTRY, print_memory_holes, NULL);
// This assumes holes are allocated
-unsigned long add_opensil_memmap(struct device *dev, unsigned long idx)
+void add_opensil_memmap(struct device *dev, unsigned long *idx)
{
- ram_from_to(dev, idx++, 0, 0xa0000);
- mmio_from_to(dev, idx++, 0xa0000, 0xc0000); // legacy VGA
- reserved_ram_from_to(dev, idx++, 0xc0000, 1 * MiB); // Option ROM
+ ram_from_to(dev, (*idx)++, 0, 0xa0000);
+ mmio_from_to(dev, (*idx)++, 0xa0000, 0xc0000); // legacy VGA
+ reserved_ram_from_to(dev, (*idx)++, 0xc0000, 1 * MiB); // Option ROM
uint32_t mem_usable = (uintptr_t)cbmem_top();
uintptr_t early_reserved_dram_start, early_reserved_dram_end;
@@ -98,33 +98,33 @@ unsigned long add_opensil_memmap(struct device *dev, unsigned long idx)
early_reserved_dram_end = e->base + e->size;
// 1MB - bottom of DRAM reserved for early coreboot usage
- ram_from_to(dev, idx++, 1 * MiB, early_reserved_dram_start);
+ ram_from_to(dev, (*idx)++, 1 * MiB, early_reserved_dram_start);
// DRAM reserved for early coreboot usage
- reserved_ram_from_to(dev, idx++, early_reserved_dram_start,
+ reserved_ram_from_to(dev, (*idx)++, early_reserved_dram_start,
early_reserved_dram_end);
// top of DRAM consumed early - low top usable RAM
// cbmem_top() accounts for low UMA and TSEG if they are used.
- ram_from_to(dev, idx++, early_reserved_dram_end,
+ ram_from_to(dev, (*idx)++, early_reserved_dram_end,
mem_usable);
// Account for UMA and TSEG
const uint32_t top_mem = ALIGN_DOWN(rdmsr(TOP_MEM).lo, 1 * MiB);
if (mem_usable != top_mem)
- reserved_ram_from_to(dev, idx++, mem_usable, top_mem);
+ reserved_ram_from_to(dev, (*idx)++, mem_usable, top_mem);
- mmconf_resource(dev, idx++);
+ mmconf_resource(dev, (*idx)++);
// Check if we're done
if (top_of_mem <= 0x100000000)
- return idx;
+ return;
// Holes in upper DRAM
// This assumes all the holes in upper DRAM are continuous
get_hole_info();
if (hole_info == NULL)
- return idx;
+ return;
uint64_t lowest_upper_hole_base = top_of_mem;
uint64_t highest_upper_hole_end = 0x100000000;
for (int hole = 0; hole < n_holes; hole++) {
@@ -135,16 +135,14 @@ unsigned long add_opensil_memmap(struct device *dev, unsigned long idx)
lowest_upper_hole_base = MIN(lowest_upper_hole_base, hole_info[hole].Base);
highest_upper_hole_end = MAX(highest_upper_hole_end, hole_info[hole].Base + hole_info[hole].Size);
if (hole_info[hole].Type == UMA)
- mmio_range(dev, idx++, hole_info[hole].Base, hole_info[hole].Size);
+ mmio_range(dev, (*idx)++, hole_info[hole].Base, hole_info[hole].Size);
else
- reserved_ram_range(dev, idx++, hole_info[hole].Base, hole_info[hole].Size);
+ reserved_ram_range(dev, (*idx)++, hole_info[hole].Base, hole_info[hole].Size);
}
- ram_from_to(dev, idx++, 0x100000000, lowest_upper_hole_base);
+ ram_from_to(dev, (*idx)++, 0x100000000, lowest_upper_hole_base);
// Do we need this?
if (top_of_mem > highest_upper_hole_end)
- ram_from_to(dev, idx++, highest_upper_hole_end, top_of_mem);
-
- return idx;
+ ram_from_to(dev, (*idx)++, highest_upper_hole_end, top_of_mem);
}
diff --git a/src/vendorcode/amd/opensil/genoa_poc/opensil.h b/src/vendorcode/amd/opensil/genoa_poc/opensil.h
index 77b3c4b692..e22552233b 100644
--- a/src/vendorcode/amd/opensil/genoa_poc/opensil.h
+++ b/src/vendorcode/amd/opensil/genoa_poc/opensil.h
@@ -8,7 +8,7 @@
void SIL_STATUS_report(const char *function, const int status);
// Add the memory map to dev, starting at index idx, returns last use idx
-unsigned long add_opensil_memmap(struct device *dev, unsigned long idx);
+void add_opensil_memmap(struct device *dev, unsigned long *idx);
// Fill in FADT from openSIL
void opensil_fill_fadt_io_ports(acpi_fadt_t *fadt);