Fix a few whitespace and coding style issues.
[coreboot.git] / src / northbridge / intel / sch / northbridge.c
index bf2870a8aa409abdbf7aaa9b776dd95a80465267..ccd0d331f2927a67696dd5afe3a2434445b35f94 100644 (file)
@@ -41,27 +41,30 @@ static int get_pcie_bar(u32 *base, u32 *len)
        if (!dev)
                return 0;
 
-       // FIXME: determine at runtime
+       /* FIXME: Determine at runtime. */
 #ifdef POULSBO_PRE_B1
-       pciexbar_reg = sch_port_access_read(0,0,4);
+       pciexbar_reg = sch_port_access_read(0, 0, 4);
 #else
-       pciexbar_reg = sch_port_access_read(2,9,4);
+       pciexbar_reg = sch_port_access_read(2, 9, 4);
 #endif
 
        if (!(pciexbar_reg & (1 << 0)))
                return 0;
 
        switch ((pciexbar_reg >> 1) & 3) {
-       case 0: // 256MB
-               *base = pciexbar_reg & ((1 << 31)|(1 << 30)|(1 << 29)|(1 << 28));
+       case 0: /* 256MB */
+               *base = pciexbar_reg & ((1 << 31) | (1 << 30) | (1 << 29) |
+                                       (1 << 28));
                *len = 256 * 1024 * 1024;
                return 1;
-       case 1: // 128M
-               *base = pciexbar_reg & ((1 << 31)|(1 << 30)|(1 << 29)|(1 << 28)|(1 << 27));
+       case 1: /* 128M */
+               *base = pciexbar_reg & ((1 << 31) | (1 << 30) | (1 << 29) |
+                                       (1 << 28) | (1 << 27));
                *len = 128 * 1024 * 1024;
                return 1;
-       case 2: // 64M
-               *base = pciexbar_reg & ((1 << 31)|(1 << 30)|(1 << 29)|(1 << 28)|(1 << 27)|(1 << 26));
+       case 2: /* 64M */
+               *base = pciexbar_reg & ((1 << 31) | (1 << 30) | (1 << 29) |
+                                       (1 << 28) | (1 << 27) | (1 << 26));
                *len = 64 * 1024 * 1024;
                return 1;
        }
@@ -70,7 +73,7 @@ static int get_pcie_bar(u32 *base, u32 *len)
 }
 
 /* IDG memory */
-uint64_t uma_memory_base=0, uma_memory_size=0;
+u64 uma_memory_base = 0, uma_memory_size = 0;
 
 static void add_fixed_resources(struct device *dev, int index)
 {
@@ -86,7 +89,7 @@ static void add_fixed_resources(struct device *dev, int index)
 
        if (get_pcie_bar(&pcie_config_base, &pcie_config_size)) {
                printk(BIOS_DEBUG, "Adding PCIe config bar\n");
-               resource = new_resource(dev, index+1);
+               resource = new_resource(dev, index + 1);
                resource->base = (resource_t) pcie_config_base;
                resource->size = (resource_t) pcie_config_size;
                resource->flags = IORESOURCE_MEM | IORESOURCE_RESERVE |
@@ -94,55 +97,52 @@ static void add_fixed_resources(struct device *dev, int index)
        }
 
        printk(BIOS_DEBUG, "Adding CMC shadow area\n");
-       resource = new_resource(dev, index+1);
+       resource = new_resource(dev, index + 1);
        resource->base = (resource_t) CMC_SHADOW;
        resource->size = (resource_t) (64 * 1024);
        resource->flags = IORESOURCE_MEM | IORESOURCE_RESERVE |
            IORESOURCE_FIXED | IORESOURCE_STORED | IORESOURCE_ASSIGNED;
 }
 
-
 #if CONFIG_WRITE_HIGH_TABLES==1
 #include <cbmem.h>
 #endif
 
 static void pci_domain_set_resources(device_t dev)
 {
-       uint32_t pci_tolm;
-       uint8_t  reg8;
-       uint16_t reg16;
+       u32 pci_tolm;
+       u8 reg8;
+       u16 reg16;
        unsigned long long tomk, tolud;
-       /* Can we find out how much memory we can use at most
-        * this way?
-        */
+
+       /* Can we find out how much memory we can use at most this way? */
        pci_tolm = find_pci_tolm(dev->link_list);
        printk(BIOS_DEBUG, "pci_tolm: 0x%x\n", pci_tolm);
        printk(BIOS_SPEW, "Base of stolen memory: 0x%08x\n",
-                   pci_read_config32(dev_find_slot(0, PCI_DEVFN(2, 0)), 0x5c));
+              pci_read_config32(dev_find_slot(0, PCI_DEVFN(2, 0)), 0x5c));
 
        tolud = pci_read_config8(dev_find_slot(0, PCI_DEVFN(0, 0)), 0x9c);
-        printk(BIOS_SPEW, "Top of Low Used DRAM: 0x%08llx\n", tolud << 24);
+       printk(BIOS_SPEW, "Top of Low Used DRAM: 0x%08llx\n", tolud << 24);
 
-        tomk = tolud << 14;
+       tomk = tolud << 14;
 
-       /* Note: subtract IGD device and TSEG */
+       /* Note: subtract IGD device and TSEG. */
        reg8 = pci_read_config8(dev_find_slot(0, PCI_DEVFN(0, 0)), 0x9e);
-       if (reg8 & 1)
-        {
+       if (reg8 & 1) {
                int tseg_size = 0;
                printk(BIOS_DEBUG, "TSEG decoded, subtracting ");
                reg8 >>= 1;
                reg8 &= 3;
                switch (reg8) {
                case 0:
-                       tseg_size = 1024;
-                       break;  /* TSEG = 1M */
+                       tseg_size = 1024; /* TSEG = 1M */
+                       break;
                case 1:
-                       tseg_size = 2048;
-                       break;  /* TSEG = 2M */
+                       tseg_size = 2048; /* TSEG = 2M */
+                       break;
                case 2:
-                       tseg_size = 8192;
-                       break;  /* TSEG = 8M */
+                       tseg_size = 8192; /* TSEG = 8M */
+                       break;
                }
 
                printk(BIOS_DEBUG, "%dM\n", tseg_size >> 10);
@@ -150,70 +150,69 @@ static void pci_domain_set_resources(device_t dev)
        }
 
        reg16 = pci_read_config16(dev_find_slot(0, PCI_DEVFN(0, 0)), GGC);
-       if (!(reg16 & 2))
-       {
+       if (!(reg16 & 2)) {
                int uma_size = 0;
                printk(BIOS_DEBUG, "IGD decoded, subtracting ");
                reg16 >>= 4;
                reg16 &= 7;
-               switch (reg16)
-               {
-                       case 1:
-                               uma_size = 1024;
-                               break;
-                       case 2:
-                               uma_size = 4096;
-                               break;
-                       case 3:
-                               uma_size = 8192;
-                               break;
+               switch (reg16) {
+               case 1:
+                       uma_size = 1024;
+                       break;
+               case 2:
+                       uma_size = 4096;
+                       break;
+               case 3:
+                       uma_size = 8192;
+                       break;
                }
                printk(BIOS_DEBUG, "%dM UMA\n", uma_size >> 10);
                tomk -= uma_size;
 
-               /* For reserving UMA memory in the memory map */
+               /* For reserving UMA memory in the memory map. */
                uma_memory_base = tomk * 1024ULL;
                uma_memory_size = uma_size * 1024ULL;
        }
 
-       /* The following needs to be 2 lines, otherwise the second
-        * number is always 0
+       /*
+        * The following needs to be 2 lines, otherwise the second
+        * number is always 0.
         */
-       printk(BIOS_INFO, "Available memory: %dK", (uint32_t)tomk);
-       printk(BIOS_INFO, " (%dM)\n", (uint32_t)(tomk >> 10));
+       printk(BIOS_INFO, "Available memory: %dK", (u32) tomk);
+       printk(BIOS_INFO, " (%dM)\n", (u32) (tomk >> 10));
 
-       /* Report the memory regions */
+       /* Report the memory regions. */
        ram_resource(dev, 3, 0, 640);
        ram_resource(dev, 4, 768, (tomk - 768));
-       if (tomk > 4 * 1024 * 1024) {
+       if (tomk > 4 * 1024 * 1024)
                ram_resource(dev, 5, 4096 * 1024, tomk - 4 * 1024 * 1024);
-       }
 
        add_fixed_resources(dev, 6);
 
        assign_resources(dev->link_list);
 
 #if CONFIG_WRITE_HIGH_TABLES==1
-       /* Leave some space for ACPI, PIRQ and MP tables */
+       /* Leave some space for ACPI, PIRQ and MP tables. */
        high_tables_base = (tomk * 1024) - HIGH_MEMORY_SIZE;
        high_tables_size = HIGH_MEMORY_SIZE;
 #endif
 }
 
-       /* TODO We could determine how many PCIe busses we need in
-        * the bar. For now that number is hardcoded to a max of 64.
-        * See e7525/northbridge.c for an example.
-        */
+/*
+ * TODO: We could determine how many PCIe busses we need in the bar. For now
+ * that number is hardcoded to a max of 64.
+ * See e7525/northbridge.c for an example.
+ */
 static struct device_operations pci_domain_ops = {
-       .read_resources   = pci_domain_read_resources,
-       .set_resources    = pci_domain_set_resources,
-       .enable_resources = NULL,
-       .init             = NULL,
-       .scan_bus         = pci_domain_scan_bus,
+       .read_resources         = pci_domain_read_resources,
+       .set_resources          = pci_domain_set_resources,
+       .enable_resources       = NULL,
+       .init                   = NULL,
+       .scan_bus               = pci_domain_scan_bus,
 #if CONFIG_MMCONF_SUPPORT_DEFAULT
-       .ops_pci_bus      = &pci_ops_mmconf,
+       .ops_pci_bus            = &pci_ops_mmconf,
 #else
-       .ops_pci_bus      = &pci_cf8_conf1,
+       .ops_pci_bus            = &pci_cf8_conf1,
 #endif
 };
 
@@ -223,32 +222,36 @@ static void mc_read_resources(device_t dev)
 
        pci_dev_read_resources(dev);
 
-       /* So, this is one of the big mysteries in the coreboot resource
+       /*
+        * So, this is one of the big mysteries in the coreboot resource
         * allocator. This resource should make sure that the address space
         * of the PCIe memory mapped config space bar. But it does not.
         */
 
-       /* We use 0xcf as an unused index for our PCIe bar so that we find it again */
+       /*
+        * We use 0xcf as an unused index for our PCIe bar so that we find
+        * it again.
+        */
        resource = new_resource(dev, 0xcf);
-       resource->flags =
-           IORESOURCE_MEM | IORESOURCE_FIXED | IORESOURCE_STORED |
-           IORESOURCE_ASSIGNED;
-       get_pcie_bar((u32*)&resource->base, (u32*)&resource->size);
-       printk(BIOS_DEBUG, "Adding PCIe enhanced config space BAR 0x%08lx-0x%08lx.\n",
-                    (unsigned long)(resource->base), (unsigned long)(resource->base + resource->size));
+       resource->flags = IORESOURCE_MEM | IORESOURCE_FIXED |
+                         IORESOURCE_STORED | IORESOURCE_ASSIGNED;
+       get_pcie_bar((u32 *)&resource->base, (u32 *)&resource->size);
+       printk(BIOS_DEBUG,
+              "Adding PCIe enhanced config space BAR 0x%08lx-0x%08lx.\n",
+              (unsigned long)(resource->base),
+              (unsigned long)(resource->base + resource->size));
 }
 
 static void mc_set_resources(device_t dev)
 {
        struct resource *resource;
 
-       /* Report the PCIe BAR */
+       /* Report the PCIe BAR. */
        resource = find_resource(dev, 0xcf);
-       if (resource) {
+       if (resource)
                report_resource_stored(dev, resource, "<mmconfig>");
-       }
 
-       /* And call the normal set_resources */
+       /* And call the normal set_resources. */
        pci_dev_set_resources(dev);
 }
 
@@ -256,10 +259,10 @@ static void intel_set_subsystem(device_t dev, unsigned vendor, unsigned device)
 {
        if (!vendor || !device) {
                pci_write_config32(dev, PCI_SUBSYSTEM_VENDOR_ID,
-                               pci_read_config32(dev, PCI_VENDOR_ID));
+                                  pci_read_config32(dev, PCI_VENDOR_ID));
        } else {
                pci_write_config32(dev, PCI_SUBSYSTEM_VENDOR_ID,
-                               ((device & 0xffff) << 16) | (vendor & 0xffff));
+                       ((device & 0xffff) << 16) | (vendor & 0xffff));
        }
 }
 
@@ -271,39 +274,39 @@ static void northbridge_init(struct device *dev)
        switch (pci_read_config32(dev, SKPAD)) {
        case 0xcafebabe:
                printk(BIOS_DEBUG, "Normal boot.\n");
-               acpi_slp_type=0;
+               acpi_slp_type = 0;
                break;
        case 0xcafed00d:
                printk(BIOS_DEBUG, "S3 Resume.\n");
-               acpi_slp_type=3;
+               acpi_slp_type = 3;
                break;
        default:
                printk(BIOS_DEBUG, "Unknown boot method, assuming normal.\n");
-               acpi_slp_type=0;
+               acpi_slp_type = 0;
                break;
        }
 }
 #endif
 
 static struct pci_operations intel_pci_ops = {
-       .set_subsystem    = intel_set_subsystem,
+       .set_subsystem = intel_set_subsystem,
 };
 
 static struct device_operations mc_ops = {
-       .read_resources   = mc_read_resources,
-       .set_resources    = mc_set_resources,
-       .enable_resources = pci_dev_enable_resources,
+       .read_resources         = mc_read_resources,
+       .set_resources          = mc_set_resources,
+       .enable_resources       = pci_dev_enable_resources,
 #if CONFIG_HAVE_ACPI_RESUME
-       .init             = northbridge_init,
+       .init                   = northbridge_init,
 #endif
-       .scan_bus         = 0,
-       .ops_pci          = &intel_pci_ops,
+       .scan_bus               = 0,
+       .ops_pci                = &intel_pci_ops,
 };
 
 static const struct pci_driver mc_driver __pci_driver = {
-       .ops    = &mc_ops,
-       .vendor = PCI_VENDOR_ID_INTEL,
-       .device = 0x8100,
+       .ops    = &mc_ops,
+       .vendor = PCI_VENDOR_ID_INTEL,
+       .device = 0x8100,
 };
 
 static void cpu_bus_init(device_t dev)
@@ -316,16 +319,16 @@ static void cpu_bus_noop(device_t dev)
 }
 
 static struct device_operations cpu_bus_ops = {
-       .read_resources   = cpu_bus_noop,
-       .set_resources    = cpu_bus_noop,
-       .enable_resources = cpu_bus_noop,
-       .init             = cpu_bus_init,
-       .scan_bus         = 0,
+       .read_resources         = cpu_bus_noop,
+       .set_resources          = cpu_bus_noop,
+       .enable_resources       = cpu_bus_noop,
+       .init                   = cpu_bus_init,
+       .scan_bus               = 0,
 };
 
 static void enable_dev(device_t dev)
 {
-       /* Set the operations if it is a special bus type */
+       /* 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) {