GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

exynos5250: assign RAM resources in cpu_init() 

This moves the ram resource allocation into cpu_init() so that we
no longer rely on declaring a domain in devicetree.cb (which is kind
of weird for this platform). This does not cause any actual changes
to the coreboot memory table, and paves the way for further updates
to Snow's devicetree.

Change-Id: I141277f59b5d48288f409257bf556a1cfa7a8463
Signed-off-by: David Hendricks <dhendrix@chromium.org>
Reviewed-on: http://review.coreboot.org/2923
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich@gmail.com>
This commit is contained in:
David Hendricks 2013-03-26 16:28:21 -07:00 committed by Ronald G. Minnich
parent 6b0d0d6e14
commit 3cc0d1eb3f
1 changed files with 2 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
src/cpu/samsung/exynos5250/cpu.c
View File

@ -4,32 +4,9 @@
#define RAM_BASE_KB (CONFIG_SYS_SDRAM_BASE >> 10)
#define RAM_SIZE_KB (CONFIG_DRAM_SIZE_MB << 10UL)
static void domain_read_resources(device_t dev)
{
ram_resource(dev, 0, RAM_BASE_KB, RAM_SIZE_KB);
}
static void domain_set_resources(device_t dev)
{
assign_resources(dev->link_list);
}
static unsigned int domain_scan_bus(device_t dev, unsigned int max)
{
return max;
}
static struct device_operations domain_ops = {
.read_resources = domain_read_resources,
.set_resources = domain_set_resources,
.enable_resources = NULL,
.init = NULL,
.scan_bus = domain_scan_bus,
};
static void cpu_init(device_t dev)
{
ram_resource(dev, 0, RAM_BASE_KB, RAM_SIZE_KB);
}
static void cpu_noop(device_t dev)
@ -47,9 +24,7 @@ static struct device_operations cpu_ops = {
static void enable_dev(device_t dev)
{
/* Set the operations if it is a special bus type */
if (dev->path.type == DEVICE_PATH_DOMAIN) {
dev->ops = &domain_ops;
} else if (dev->path.type == DEVICE_PATH_CPU_CLUSTER) {
if (dev->path.type == DEVICE_PATH_CPU_CLUSTER) {
dev->ops = &cpu_ops;
}
}