Clean up whitespace in fam14 northbridge.c

Change-Id: Id7947d7f3c67fdda67861065b1bc7a519b97208f
Signed-off-by: Marc Jones <marc.jones@se-eng.com>
Reviewed-on: http://review.coreboot.org/789
Tested-by: build bot (Jenkins)
Reviewed-by: Peter Stuge <peter@stuge.se>
This commit is contained in:
Marc Jones
2012-03-15 12:55:26 -06:00
committed by Peter Stuge
parent 02bb57824c
commit 8d595698bf

View File

@@ -38,7 +38,6 @@
#include <sb_cimx.h>
#endif
//#define FX_DEVS NODE_NUMS
#define FX_DEVS 1
@@ -48,13 +47,11 @@ static device_t __f2_dev[FX_DEVS];
static device_t __f4_dev[FX_DEVS];
static unsigned fx_devs = 0;
device_t get_node_pci(u32 nodeid, u32 fn)
{
return dev_find_slot(CONFIG_CBB, PCI_DEVFN(CONFIG_CDB + nodeid, fn));
}
static void get_fx_devs(void)
{
int i;
@@ -71,7 +68,6 @@ static void get_fx_devs(void)
}
}
static u32 f1_read_config32(unsigned reg)
{
if (fx_devs == 0)
@@ -79,7 +75,6 @@ static u32 f1_read_config32(unsigned reg)
return pci_read_config32(__f1_dev[0], reg);
}
static void f1_write_config32(unsigned reg, u32 value)
{
int i;
@@ -94,22 +89,18 @@ static void f1_write_config32(unsigned reg, u32 value)
}
}
static u32 amdfam14_nodeid(device_t dev)
{
return (dev->path.pci.devfn >> 3) - CONFIG_CDB;
}
#include "amdfam14_conf.c"
static void northbridge_init(device_t dev)
{
printk(BIOS_DEBUG, "Northbridge init\n");
}
static void set_vga_enable_reg(u32 nodeid, u32 linkn)
{
u32 val;
@@ -121,7 +112,6 @@ static void set_vga_enable_reg(u32 nodeid, u32 linkn)
}
static int reg_useable(unsigned reg, device_t goal_dev, unsigned goal_nodeid,
unsigned goal_link)
{
@@ -142,15 +132,15 @@ static int reg_useable(unsigned reg, device_t goal_dev, unsigned goal_nodeid,
if (res) {
result = 0;
if ((goal_link == (link - 1)) &&
(goal_nodeid == (nodeid - 1)) &&
(res->flags <= 1)) {
(goal_nodeid == (nodeid - 1)) && (res->flags <= 1)) {
result = 1;
}
}
return result;
}
static struct resource *amdfam14_find_iopair(device_t dev, unsigned nodeid, unsigned link)
static struct resource *amdfam14_find_iopair(device_t dev, unsigned nodeid,
unsigned link)
{
struct resource *resource;
u32 result, reg;
@@ -161,10 +151,12 @@ static struct resource *amdfam14_find_iopair(device_t dev, unsigned nodeid, unsi
/* I have been allocated this one */
reg = 0xc0;
}
//Ext conf space
/* Ext conf space */
if (!reg) {
//because of Extend conf space, we will never run out of reg, but we need one index to differ them. so same node and same link can have multi range
/* Because of Extend conf space, we will never run out of reg,
* but we need one index to differ them. So ,same node and same
* link can have multi range
*/
u32 index = get_io_addr_index(nodeid, link);
reg = 0x110 + (index << 24) + (4 << 20); // index could be 0, 255
}
@@ -174,7 +166,8 @@ static struct resource *amdfam14_find_iopair(device_t dev, unsigned nodeid, unsi
return resource;
}
static struct resource *amdfam14_find_mempair(device_t dev, u32 nodeid, u32 link)
static struct resource *amdfam14_find_mempair(device_t dev, u32 nodeid,
u32 link)
{
struct resource *resource;
u32 free_reg, reg;
@@ -186,8 +179,7 @@ static struct resource *amdfam14_find_mempair(device_t dev, u32 nodeid, u32 link
if (result == 1) {
/* I have been allocated this one */
break;
}
else if (result > 1) {
} else if (result > 1) {
/* I have a free register pair */
free_reg = reg;
}
@@ -195,12 +187,12 @@ static struct resource *amdfam14_find_mempair(device_t dev, u32 nodeid, u32 link
if (reg > 0xb8) {
reg = free_reg;
}
//Ext conf space
/* Ext conf space */
if (!reg) {
//because of Extend conf space, we will never run out of reg,
// but we need one index to differ them. so same node and
// same link can have multi range
/* Because of Extend conf space, we will never run out of reg,
* but we need one index to differ them. So ,same node and same
* link can have multi range
*/
u32 index = get_mmio_addr_index(nodeid, link);
reg = 0x110 + (index << 24) + (6 << 20); // index could be 0, 63
@@ -209,7 +201,6 @@ static struct resource *amdfam14_find_mempair(device_t dev, u32 nodeid, u32 link
return resource;
}
static void amdfam14_link_read_bases(device_t dev, u32 nodeid, u32 link)
{
struct resource *resource;
@@ -221,8 +212,7 @@ static void amdfam14_link_read_bases(device_t dev, u32 nodeid, u32 link)
#if CONFIG_EXT_CONF_SUPPORT == 1
if ((resource->index & 0x1fff) == 0x1110) { // ext
align = 8;
}
else
} else
#endif
align = log2(HT_IO_HOST_ALIGN);
resource->base = 0;
@@ -273,7 +263,8 @@ static u32 my_find_pci_tolm(struct bus *bus, u32 tolm)
{
struct resource *min;
min = 0;
search_bus_resources(bus, IORESOURCE_MEM, IORESOURCE_MEM, tolm_test, &min);
search_bus_resources(bus, IORESOURCE_MEM, IORESOURCE_MEM, tolm_test,
&min);
if (min && tolm > min->base) {
tolm = min->base;
}
@@ -304,10 +295,10 @@ static struct hw_mem_hole_info get_hw_mem_hole_info(void)
mem_hole.node_id = 0; // record the node No with hole
}
}
#if 0
// We need to double check if there is speical set on base reg and limit reg
// are not continous instead of hole, it will find out it's hole_startk
/* We need to double check if there is speical set on base reg and limit reg
* are not continous instead of hole, it will find out it's hole_startk
*/
if (mem_hole.node_id == -1) {
resource_t limitk_pri = 0;
struct dram_base_mask_t d;
@@ -322,7 +313,9 @@ static struct hw_mem_hole_info get_hw_mem_hole_info(void)
}
}
limit_k = ((resource_t)((d.mask + 0x00000100) & 0x1fffff00)) << 9;
limit_k =
((resource_t) ((d.mask + 0x00000100) & 0x1fffff00))
<< 9;
limitk_pri = limit_k;
}
}
@@ -369,9 +362,7 @@ static void read_resources(device_t dev)
}
}
static void set_resource(device_t dev, struct resource *resource,
u32 nodeid)
static void set_resource(device_t dev, struct resource *resource, u32 nodeid)
{
resource_t rbase, rend;
unsigned reg, link_num;
@@ -408,18 +399,17 @@ static void set_resource(device_t dev, struct resource *resource,
link_num = IOINDEX_LINK(resource->index);
if (resource->flags & IORESOURCE_IO) {
set_io_addr_reg(dev, nodeid, link_num, reg, rbase>>8, rend>>8);
}
else if (resource->flags & IORESOURCE_MEM) {
set_mmio_addr_reg(nodeid, link_num, reg, (resource->index >>24), rbase>>8, rend>>8, 1) ;// [39:8]
set_io_addr_reg(dev, nodeid, link_num, reg, rbase >> 8,
rend >> 8);
} else if (resource->flags & IORESOURCE_MEM) {
set_mmio_addr_reg(nodeid, link_num, reg, (resource->index >> 24),
rbase >> 8, rend >> 8, 1); // [39:8]
}
resource->flags |= IORESOURCE_STORED;
sprintf(buf, " <node %x link %x>",
nodeid, link_num);
sprintf(buf, " <node %x link %x>", nodeid, link_num);
report_resource_stored(dev, resource, buf);
}
#if CONFIG_CONSOLE_VGA_MULTI
extern device_t vga_pri; // the primary vga device, defined in device.c
#endif
@@ -435,12 +425,13 @@ static void create_vga_resource(device_t dev, unsigned nodeid)
for (link = dev->link_list; link; link = link->next) {
if (link->bridge_ctrl & PCI_BRIDGE_CTL_VGA) {
#if CONFIG_CONSOLE_VGA_MULTI
printk(BIOS_DEBUG, "VGA: vga_pri bus num = %d bus range [%d,%d]\n", vga_pri->bus->secondary,
link->secondary,link->subordinate);
printk(BIOS_DEBUG,
"VGA: vga_pri bus num = %d bus range [%d,%d]\n",
vga_pri->bus->secondary, link->secondary,
link->subordinate);
/* We need to make sure the vga_pri is under the link */
if ((vga_pri->bus->secondary >= link->secondary) &&
(vga_pri->bus->secondary <= link->subordinate )
)
(vga_pri->bus->secondary <= link->subordinate))
#endif
break;
}
@@ -450,11 +441,11 @@ static void create_vga_resource(device_t dev, unsigned nodeid)
if (link == NULL)
return;
printk(BIOS_DEBUG, "VGA: %s (aka node %d) link %d has VGA device\n", dev_path(dev), nodeid, link->link_num);
printk(BIOS_DEBUG, "VGA: %s (aka node %d) link %d has VGA device\n",
dev_path(dev), nodeid, link->link_num);
set_vga_enable_reg(nodeid, link->link_num);
}
static void set_resources(device_t dev)
{
unsigned nodeid;
@@ -480,7 +471,6 @@ static void set_resources(device_t dev)
}
}
/* Domain/Root Complex related code */
static void domain_read_resources(device_t dev)
@@ -509,7 +499,10 @@ static void domain_read_resources(device_t dev)
if (reg_dev) {
/* Reserve the resource */
struct resource *res;
res = new_resource(reg_dev, IOINDEX(0x1000 + reg, reg_link));
res =
new_resource(reg_dev,
IOINDEX(0x1000 + reg,
reg_link));
if (res) {
res->flags = 1;
}
@@ -544,7 +537,6 @@ static void domain_read_resources(device_t dev)
#endif
}
static void domain_set_resources(device_t dev)
{
printk(BIOS_DEBUG, "\nFam14h - domain_set_resources.\n");
@@ -573,15 +565,19 @@ printk(BIOS_DEBUG, "adsr - CONFIG_PCI_64BIT_PREF_MEM is true.\n");
mem1 = find_resource(dev, 1 | (link->link_num << 2));
mem2 = find_resource(dev, 2 | (link->link_num << 2));
printk(BIOS_DEBUG, "base1: 0x%08Lx limit1: 0x%08Lx size: 0x%08Lx align: %d\n",
(u32)(mem1->base), (u32)(mem1->limit), (u32)(mem1->size), u32)(mem1->align));
printk(BIOS_DEBUG, "base2: 0x%08Lx limit2: 0x%08Lx size: 0x%08Lx align: %d\n",
(u32)(mem2->base), (u32)(mem2->limit), (u32)(mem2->size), (u32)(mem2->align));
printk(BIOS_DEBUG,
"base1: 0x%08Lx limit1: 0x%08Lx size: 0x%08Lx align: %d\n",
(u32) (mem1->base), (u32) (mem1->limit),
(u32) (mem1->size), u32) (mem1->align));
printk(BIOS_DEBUG,
"base2: 0x%08Lx limit2: 0x%08Lx size: 0x%08Lx align: %d\n",
(u32) (mem2->base), (u32) (mem2->limit),
(u32) (mem2->size), (u32) (mem2->align));
/* See if both resources have roughly the same limits */
if (((mem1->limit <= 0xffffffff) && (mem2->limit <= 0xffffffff)) ||
((mem1->limit > 0xffffffff) && (mem2->limit > 0xffffffff)))
{
if (((mem1->limit <= 0xffffffff) && (mem2->limit <= 0xffffffff))
|| ((mem1->limit > 0xffffffff)
&& (mem2->limit > 0xffffffff))) {
/* If so place the one with the most stringent alignment first
*/
if (mem2->align > mem1->align) {
@@ -594,21 +590,21 @@ printk(BIOS_DEBUG, "adsr - CONFIG_PCI_64BIT_PREF_MEM is true.\n");
mem2->base = resource_max(mem2);
mem1->limit = mem2->base - 1;
mem1->base = resource_max(mem1);
}
else {
} else {
/* Place the resources as high up as they will go */
mem2->base = resource_max(mem2);
mem1->base = resource_max(mem1);
}
printk(BIOS_DEBUG, "base1: 0x%08Lx limit1: 0x%08Lx size: 0x%08Lx align: %d\n",
printk(BIOS_DEBUG,
"base1: 0x%08Lx limit1: 0x%08Lx size: 0x%08Lx align: %d\n",
mem1->base, mem1->limit, mem1->size, mem1->align);
printk(BIOS_DEBUG, "base2: 0x%08Lx limit2: 0x%08Lx size: 0x%08Lx align: %d\n",
printk(BIOS_DEBUG,
"base2: 0x%08Lx limit2: 0x%08Lx size: 0x%08Lx align: %d\n",
mem2->base, mem2->limit, mem2->size, mem2->align);
}
for(res = &dev->resource_list; res; res = res->next)
{
for (res = &dev->resource_list; res; res = res->next) {
res->flags |= IORESOURCE_ASSIGNED;
res->flags |= IORESOURCE_STORED;
report_resource_stored(dev, res, "");
@@ -657,14 +653,18 @@ printk(BIOS_DEBUG, "adsr - CONFIG_PCI_64BIT_PREF_MEM is true.\n");
if (d.mask & 1) {
basek = ((resource_t) ((u64) d.base)) << 8;
limitk = (resource_t) (((u64) d.mask << 8) | 0xFFFFFF);
printk(BIOS_DEBUG, "adsr: (before) basek = %llx, limitk = %llx.\n",basek,limitk);
printk(BIOS_DEBUG,
"adsr: (before) basek = %llx, limitk = %llx.\n", basek,
limitk);
/* Convert these values to multiples of 1K for ease of math. */
basek >>= 10;
limitk >>= 10;
sizek = limitk - basek + 1;
printk(BIOS_DEBUG, "adsr: (after) basek = %llx, limitk = %llx, sizek = %llx.\n",basek,limitk,sizek);
printk(BIOS_DEBUG,
"adsr: (after) basek = %llx, limitk = %llx, sizek = %llx.\n",
basek, limitk, sizek);
/* see if we need a hole from 0xa0000 to 0xbffff */
if ((basek < 640) && (sizek > 768)) {
@@ -675,8 +675,9 @@ printk(BIOS_DEBUG, "adsr - 0xa0000 to 0xbffff resource.\n");
sizek = limitk - 768;
}
printk(BIOS_DEBUG, "adsr: mmio_basek=%08lx, basek=%08llx, limitk=%08llx\n", mmio_basek, basek, limitk);
printk(BIOS_DEBUG,
"adsr: mmio_basek=%08lx, basek=%08llx, limitk=%08llx\n",
mmio_basek, basek, limitk);
/* split the region to accomodate pci memory space */
if ((basek < 4 * 1024 * 1024) && (limitk > mmio_basek)) {
@@ -684,20 +685,24 @@ printk(BIOS_DEBUG, "adsr: mmio_basek=%08lx, basek=%08llx, limitk=%08llx\n", mmi
unsigned pre_sizek;
pre_sizek = mmio_basek - basek;
if (pre_sizek > 0) {
ram_resource(dev, idx, basek, pre_sizek);
ram_resource(dev, idx, basek,
pre_sizek);
idx += 0x10;
sizek -= pre_sizek;
#if CONFIG_WRITE_HIGH_TABLES==1
if (high_tables_base == 0) {
/* Leave some space for ACPI, PIRQ and MP tables */
#if CONFIG_GFXUMA == 1
high_tables_base = uma_memory_base - (HIGH_TABLES_SIZE * 1024);
high_tables_base =
uma_memory_base -
(HIGH_TABLES_SIZE * 1024);
#else
high_tables_base = (mmio_basek - HIGH_TABLES_SIZE) * 1024;
#endif
high_tables_size = HIGH_TABLES_SIZE * 1024;
printk(BIOS_DEBUG, " split: %dK table at =%08llx\n", HIGH_TABLES_SIZE,
high_tables_base);
printk(BIOS_DEBUG,
" split: %dK table at =%08llx\n",
HIGH_TABLES_SIZE, high_tables_base);
}
#endif
}
@@ -706,8 +711,7 @@ printk(BIOS_DEBUG, "adsr: mmio_basek=%08lx, basek=%08llx, limitk=%08llx\n", mmi
}
if ((basek + sizek) <= 4 * 1024 * 1024) {
sizek = 0;
}
else {
} else {
basek = 4 * 1024 * 1024;
sizek -= (4 * 1024 * 1024 - mmio_basek);
}
@@ -716,8 +720,9 @@ printk(BIOS_DEBUG, "adsr: mmio_basek=%08lx, basek=%08llx, limitk=%08llx\n", mmi
ram_resource(dev, (idx | 0), basek, sizek);
idx += 0x10;
#if CONFIG_WRITE_HIGH_TABLES==1
printk(BIOS_DEBUG, "%d: mmio_basek=%08lx, basek=%08llx, limitk=%08llx\n",
0, mmio_basek, basek, limitk);
printk(BIOS_DEBUG,
"%d: mmio_basek=%08lx, basek=%08llx, limitk=%08llx\n", 0,
mmio_basek, basek, limitk);
if (high_tables_base == 0) {
/* Leave some space for ACPI, PIRQ and MP tables */
#if CONFIG_GFXUMA == 1
@@ -731,7 +736,8 @@ printk(BIOS_DEBUG, "adsr: mmio_basek=%08lx, basek=%08llx, limitk=%08llx\n", mmi
#endif
}
printk(BIOS_DEBUG, " adsr - mmio_basek = %lx.\n", mmio_basek);
printk(BIOS_DEBUG, " adsr - high_tables_size = %llx.\n",high_tables_size);
printk(BIOS_DEBUG, " adsr - high_tables_size = %llx.\n",
high_tables_size);
#if CONFIG_GFXUMA == 1
printk(BIOS_DEBUG, "adsr - adding uma resource.\n");
@@ -746,9 +752,7 @@ printk(BIOS_DEBUG, " adsr - high_tables_size = %llx.\n",high_tables_size);
printk(BIOS_DEBUG, " adsr - leaving this lovely routine.\n");
}
static void domain_enable_resources(device_t dev)
{
static void domain_enable_resources(device_t dev) {
u32 val;
#if CONFIG_AMD_SB_CIMX
@@ -766,12 +770,9 @@ static void domain_enable_resources(device_t dev)
printk(BIOS_DEBUG, " ader - leaving domain_enable_resources.\n");
}
/* Bus related code */
static void cpu_bus_read_resources(device_t dev)
{
static void cpu_bus_read_resources(device_t dev) {
printk(BIOS_DEBUG, "\nFam14h - cpu_bus_read_resources.\n");
#if CONFIG_MMCONF_SUPPORT
@@ -783,8 +784,7 @@ static void cpu_bus_read_resources(device_t dev)
#endif
}
static void cpu_bus_set_resources(device_t dev)
{
static void cpu_bus_set_resources(device_t dev) {
struct resource *resource = find_resource(dev, 0xc0010058);
printk(BIOS_DEBUG, "\nFam14h - cpu_bus_set_resources.\n");
@@ -795,8 +795,7 @@ static void cpu_bus_set_resources(device_t dev)
pci_dev_set_resources(dev);
}
static void cpu_bus_init(device_t dev)
{
static void cpu_bus_init(device_t dev) {
struct device_path cpu_path;
device_t cpu;
int apic_id;
@@ -808,14 +807,14 @@ static void cpu_bus_init(device_t dev)
cpu_path.type = DEVICE_PATH_APIC;
cpu_path.apic.apic_id = apic_id;
cpu = alloc_dev(dev->link_list, &cpu_path);
if (!cpu) return;
if (!cpu)
return;
cpu->enabled = 1;
cpu->path.apic.node_id = 0;
cpu->path.apic.core_id = apic_id;
}
}
/* North Bridge Structures */
static struct device_operations northbridge_operations = {
@@ -823,27 +822,22 @@ static struct device_operations northbridge_operations = {
.set_resources = set_resources,
.enable_resources = pci_dev_enable_resources,
.init = northbridge_init,
.enable = 0,
.ops_pci = 0,
.enable = 0,.ops_pci = 0,
};
static const struct pci_driver northbridge_driver __pci_driver = {
.ops = &northbridge_operations,
.vendor = PCI_VENDOR_ID_AMD,
.device = 0x1510,
};
struct chip_operations northbridge_amd_agesa_family14_ops = {
CHIP_NAME("AMD Family 14h Northbridge")
.enable_dev = 0,
};
/* Root Complex Structures */
static struct device_operations pci_domain_ops = {
.read_resources = domain_read_resources,
.set_resources = domain_set_resources,
@@ -852,7 +846,6 @@ static struct device_operations pci_domain_ops = {
.scan_bus = pci_domain_scan_bus,
};
static struct device_operations cpu_bus_ops = {
.read_resources = cpu_bus_read_resources,
.set_resources = cpu_bus_set_resources,
@@ -861,19 +854,15 @@ static struct device_operations cpu_bus_ops = {
.scan_bus = NULL,
};
static void root_complex_enable_dev(struct device *dev)
{
static void root_complex_enable_dev(struct device *dev) {
/* Set the operations if it is a special bus type */
if (dev->path.type == DEVICE_PATH_PCI_DOMAIN) {
dev->ops = &pci_domain_ops;
}
else if (dev->path.type == DEVICE_PATH_APIC_CLUSTER) {
} else if (dev->path.type == DEVICE_PATH_APIC_CLUSTER) {
dev->ops = &cpu_bus_ops;
}
}
struct chip_operations northbridge_amd_agesa_family14_root_complex_ops = {
CHIP_NAME("AMD Family 14h Root Complex")
.enable_dev = root_complex_enable_dev,