Kyösti Mälkki has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/30731
Change subject: binaryPI: Drop CONFIG_CBB and CONFIG_CDB ......................................................................
binaryPI: Drop CONFIG_CBB and CONFIG_CDB
Static values, copy paste from multi-node fam15 code.
Change-Id: I2401acb9269674bac054fa9a6dd60ca8a21b36a9 Signed-off-by: Kyösti Mälkki kyosti.malkki@gmail.com --- M src/cpu/amd/pi/00630F01/Kconfig M src/cpu/amd/pi/00660F01/Kconfig M src/cpu/amd/pi/00730F01/Kconfig M src/northbridge/amd/pi/00630F01/northbridge.c M src/northbridge/amd/pi/00660F01/northbridge.c M src/northbridge/amd/pi/00730F01/northbridge.c 6 files changed, 15 insertions(+), 39 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/31/30731/1
diff --git a/src/cpu/amd/pi/00630F01/Kconfig b/src/cpu/amd/pi/00630F01/Kconfig index f3ed237..75a8441 100644 --- a/src/cpu/amd/pi/00630F01/Kconfig +++ b/src/cpu/amd/pi/00630F01/Kconfig @@ -23,14 +23,6 @@ int default 48
-config CBB - hex - default 0x0 - -config CDB - hex - default 0x18 - config XIP_ROM_SIZE hex default 0x100000 diff --git a/src/cpu/amd/pi/00660F01/Kconfig b/src/cpu/amd/pi/00660F01/Kconfig index d57cfb1..a6eca07 100644 --- a/src/cpu/amd/pi/00660F01/Kconfig +++ b/src/cpu/amd/pi/00660F01/Kconfig @@ -23,14 +23,6 @@ int default 48
-config CBB - hex - default 0x0 - -config CDB - hex - default 0x18 - config XIP_ROM_SIZE hex default 0x100000 diff --git a/src/cpu/amd/pi/00730F01/Kconfig b/src/cpu/amd/pi/00730F01/Kconfig index a455a3e..054523e 100644 --- a/src/cpu/amd/pi/00730F01/Kconfig +++ b/src/cpu/amd/pi/00730F01/Kconfig @@ -23,14 +23,6 @@ int default 40
-config CBB - hex - default 0x0 - -config CDB - hex - default 0x18 - config XIP_ROM_SIZE hex default 0x100000 diff --git a/src/northbridge/amd/pi/00630F01/northbridge.c b/src/northbridge/amd/pi/00630F01/northbridge.c index 11158d0..db369b0 100644 --- a/src/northbridge/amd/pi/00630F01/northbridge.c +++ b/src/northbridge/amd/pi/00630F01/northbridge.c @@ -104,7 +104,7 @@
static struct device *get_node_pci(u32 nodeid, u32 fn) { - return pcidev_on_root(CONFIG_CDB + nodeid, fn); + return pcidev_on_root(0x18 + nodeid, fn); }
static void get_fx_devs(void) @@ -147,7 +147,7 @@
static u32 amdfam15_nodeid(struct device *dev) { - return (dev->path.pci.devfn >> 3) - CONFIG_CDB; + return (dev->path.pci.devfn >> 3) - 0x18; }
static void set_vga_enable_reg(u32 nodeid, u32 linkn) @@ -855,9 +855,9 @@ printk(BIOS_SPEW, "KaveriPI Debug: AMD Topology Number of Modules (@0x%p) is %d\n", modules_ptr, modules); printk(BIOS_SPEW, "KaveriPI Debug: AMD Topology Number of IOAPICs (@0x%p) is %d\n", options, (int)(options->CfgPlatNumIoApics));
- dev_mc = pcidev_on_root(CONFIG_CDB, 0); + dev_mc = pcidev_on_root(0x18, 0); if (!dev_mc) { - printk(BIOS_ERR, "%02x:%02x.0 not found", CONFIG_CBB, CONFIG_CDB); + printk(BIOS_ERR, "%02x:%02x.0 not found", 0x0, 0x18); die(""); } sysconf_init(dev_mc); @@ -880,7 +880,7 @@ unsigned devn; struct bus *pbus;
- devn = CONFIG_CDB + i; + devn = 0x18 + i; pbus = dev_mc->bus;
/* Find the cpu's pci device */ diff --git a/src/northbridge/amd/pi/00660F01/northbridge.c b/src/northbridge/amd/pi/00660F01/northbridge.c index d107964..a5bfac7 100644 --- a/src/northbridge/amd/pi/00660F01/northbridge.c +++ b/src/northbridge/amd/pi/00660F01/northbridge.c @@ -102,7 +102,7 @@
static struct device *get_node_pci(u32 nodeid, u32 fn) { - return pcidev_on_root(CONFIG_CDB + nodeid, fn); + return pcidev_on_root(0x18 + nodeid, fn); }
static void get_fx_devs(void) @@ -145,7 +145,7 @@
static u32 amdfam15_nodeid(struct device *dev) { - return (dev->path.pci.devfn >> 3) - CONFIG_CDB; + return (dev->path.pci.devfn >> 3) - 0x18; }
static void set_vga_enable_reg(u32 nodeid, u32 linkn) @@ -844,9 +844,9 @@ ioapic_count = (int)options->CfgPlatNumIoApics; ASSERT(ioapic_count > 0);
- dev_mc = pcidev_on_root(CONFIG_CDB, 0); + dev_mc = pcidev_on_root(0x18, 0); if (!dev_mc) { - printk(BIOS_ERR, "%02x:%02x.0 not found", CONFIG_CBB, CONFIG_CDB); + printk(BIOS_ERR, "%02x:%02x.0 not found", 0x0, 0x18); die(""); } sysconf_init(dev_mc); @@ -869,7 +869,7 @@ unsigned devn; struct bus *pbus;
- devn = CONFIG_CDB + i; + devn = 0x18 + i; pbus = dev_mc->bus;
/* Find the cpu's pci device */ diff --git a/src/northbridge/amd/pi/00730F01/northbridge.c b/src/northbridge/amd/pi/00730F01/northbridge.c index 8030a3f..ddaa94f 100644 --- a/src/northbridge/amd/pi/00730F01/northbridge.c +++ b/src/northbridge/amd/pi/00730F01/northbridge.c @@ -104,7 +104,7 @@
static struct device *get_node_pci(u32 nodeid, u32 fn) { - return pcidev_on_root(CONFIG_CDB + nodeid, fn); + return pcidev_on_root(0x18 + nodeid, fn); }
static void get_fx_devs(void) @@ -147,7 +147,7 @@
static u32 amdfam16_nodeid(struct device *dev) { - return (dev->path.pci.devfn >> 3) - CONFIG_CDB; + return (dev->path.pci.devfn >> 3) - 0x18; }
static void set_vga_enable_reg(u32 nodeid, u32 linkn) @@ -1090,9 +1090,9 @@ printk(BIOS_SPEW, "MullinsPI Debug: AMD Topology Number of Modules (@0x%p) is %d\n", modules_ptr, modules); printk(BIOS_SPEW, "MullinsPI Debug: AMD Topology Number of IOAPICs (@0x%p) is %d\n", options, (int)options->CfgPlatNumIoApics);
- dev_mc = pcidev_on_root(CONFIG_CDB, 0); + dev_mc = pcidev_on_root(0x18, 0); if (!dev_mc) { - printk(BIOS_ERR, "%02x:%02x.0 not found", CONFIG_CBB, CONFIG_CDB); + printk(BIOS_ERR, "%02x:%02x.0 not found", 0x0, 0x18); die(""); } sysconf_init(dev_mc); @@ -1115,7 +1115,7 @@ unsigned devn; struct bus *pbus;
- devn = CONFIG_CDB + i; + devn = 0x18 + i; pbus = dev_mc->bus;
/* Find the cpu's pci device */