Cosmetics and coding style fixes in devices/*.
authorUwe Hermann <uwe@hermann-uwe.de>
Mon, 18 Oct 2010 00:00:57 +0000 (00:00 +0000)
committerUwe Hermann <uwe@hermann-uwe.de>
Mon, 18 Oct 2010 00:00:57 +0000 (00:00 +0000)
 - Whitespace and indentation fixes in various places.

 - Fix various typos.

 - Use u8, u16 etc. everywhere.

Abuild-tested.

Signed-off-by: Uwe Hermann <uwe@hermann-uwe.de>
Acked-by: Uwe Hermann <uwe@hermann-uwe.de>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@5962 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1

src/devices/cardbus_device.c
src/devices/pciexp_device.c
src/devices/pcix_device.c
src/devices/pnp_device.c
src/devices/root_device.c
src/devices/smbus_ops.c
src/include/device/pciexp.h
src/include/device/pcix.h
src/include/device/pnp.h
src/include/device/smbus.h

index ccd78eb414b318e58c26e00f902b601a87a4e9e0..e3acf856ecb1d778a8cc5248c6ce789661bd6127 100644 (file)
 #include <device/pci_ids.h>
 #include <device/cardbus.h>
 
-/* I don't think this code is quite correct but it is close.
+/*
+ * I don't think this code is quite correct but it is close.
  * Anyone with a cardbus bridge and a little time should be able
  * to make it usable quickly. -- Eric Biederman 24 March 2005
  */
 
 /*
- * IO should be max 256 bytes.  However, since we may
- * have a P2P bridge below a cardbus bridge, we need 4K.
+ * IO should be max 256 bytes. However, since we may have a P2P bridge below
+ * a cardbus bridge, we need 4K.
  */
-#define CARDBUS_IO_SIZE                (4096)
-#define CARDBUS_MEM_SIZE       (32*1024*1024)
+#define CARDBUS_IO_SIZE                4096
+#define CARDBUS_MEM_SIZE       (32 * 1024 * 1024)
 
-static void cardbus_record_bridge_resource(
-       device_t dev, resource_t moving, resource_t min_size,
-       unsigned index, unsigned long type)
+static void cardbus_record_bridge_resource(device_t dev, resource_t moving,
+               resource_t min_size, unsigned int index, unsigned long type)
 {
-       /* Initialize the constraints on the current bus. */
        struct resource *resource;
+
+       /* Initialize the constraints on the current bus. */
        resource = NULL;
        if (moving) {
                unsigned long gran;
                resource_t step;
+
                resource = new_resource(dev, index);
                resource->size = 0;
                gran = 0;
@@ -59,30 +61,33 @@ static void cardbus_record_bridge_resource(
                resource->align = gran;
                resource->limit = moving | (step - 1);
                resource->flags = type;
-               /* Don't let the minimum size exceed what we
+
+               /*
+                * Don't let the minimum size exceed what we
                 * can put in the resource.
                 */
-               if ((min_size - 1) > resource->limit) {
+               if ((min_size - 1) > resource->limit)
                        min_size = resource->limit + 1;
-               }
+
                resource->size = min_size;
        }
        return;
 }
 
-static void cardbus_size_bridge_resource(device_t dev, unsigned index)
+static void cardbus_size_bridge_resource(device_t dev, unsigned int index)
 {
        struct resource *resource;
        resource_t min_size;
+
        resource = find_resource(dev, index);
        if (resource) {
                min_size = resource->size;
-               /* Allways allocate at least the miniumum size to a
+               /*
+                * Always allocate at least the miniumum size to a
                 * cardbus bridge in case a new card is plugged in.
                 */
-               if (resource->size < min_size) {
+               if (resource->size < min_size)
                        resource->size = min_size;
-               }
        }
 }
 
@@ -90,34 +95,34 @@ void cardbus_read_resources(device_t dev)
 {
        resource_t moving_base, moving_limit, moving;
        unsigned long type;
-       uint16_t ctl;
+       u16 ctl;
 
-       /* See if needs a card control registers base address */
+       /* See if needs a card control registers base address. */
 
        pci_get_resource(dev, PCI_BASE_ADDRESS_0);
 
        compact_resources(dev);
 
-       /* See which bridge I/O resources are implemented */
-       moving_base  = pci_moving_config32(dev, PCI_CB_IO_BASE_0);
+       /* See which bridge I/O resources are implemented. */
+       moving_base = pci_moving_config32(dev, PCI_CB_IO_BASE_0);
        moving_limit = pci_moving_config32(dev, PCI_CB_IO_LIMIT_0);
        moving = moving_base & moving_limit;
 
-       /* Initialize the io space constraints on the current bus */
+       /* Initialize the I/O space constraints on the current bus. */
        cardbus_record_bridge_resource(dev, moving, CARDBUS_IO_SIZE,
-               PCI_CB_IO_BASE_0, IORESOURCE_IO);
+                                      PCI_CB_IO_BASE_0, IORESOURCE_IO);
        cardbus_size_bridge_resource(dev, PCI_CB_IO_BASE_0);
 
-       /* See which bridge I/O resources are implemented */
-       moving_base  = pci_moving_config32(dev, PCI_CB_IO_BASE_1);
+       /* See which bridge I/O resources are implemented. */
+       moving_base = pci_moving_config32(dev, PCI_CB_IO_BASE_1);
        moving_limit = pci_moving_config32(dev, PCI_CB_IO_LIMIT_1);
        moving = moving_base & moving_limit;
 
-       /* Initialize the io space constraints on the current bus */
+       /* Initialize the I/O space constraints on the current bus. */
        cardbus_record_bridge_resource(dev, moving, CARDBUS_IO_SIZE,
-               PCI_CB_IO_BASE_1, IORESOURCE_IO);
+                                      PCI_CB_IO_BASE_1, IORESOURCE_IO);
 
-       /* If I can enable prefetch for mem0 */
+       /* If I can, enable prefetch for mem0. */
        ctl = pci_read_config16(dev, PCI_CB_BRIDGE_CONTROL);
        ctl &= ~PCI_CB_BRIDGE_CTL_PREFETCH_MEM0;
        ctl &= ~PCI_CB_BRIDGE_CTL_PREFETCH_MEM1;
@@ -125,30 +130,28 @@ void cardbus_read_resources(device_t dev)
        pci_write_config16(dev, PCI_CB_BRIDGE_CONTROL, ctl);
        ctl = pci_read_config16(dev, PCI_CB_BRIDGE_CONTROL);
 
-       /* See which bridge memory resources are implemented */
-       moving_base  = pci_moving_config32(dev, PCI_CB_MEMORY_BASE_0);
+       /* See which bridge memory resources are implemented. */
+       moving_base = pci_moving_config32(dev, PCI_CB_MEMORY_BASE_0);
        moving_limit = pci_moving_config32(dev, PCI_CB_MEMORY_LIMIT_0);
        moving = moving_base & moving_limit;
 
-       /* Initialize the memory space constraints on the current bus */
+       /* Initialize the memory space constraints on the current bus. */
        type = IORESOURCE_MEM;
-       if (ctl & PCI_CB_BRIDGE_CTL_PREFETCH_MEM0) {
+       if (ctl & PCI_CB_BRIDGE_CTL_PREFETCH_MEM0)
                type |= IORESOURCE_PREFETCH;
-       }
        cardbus_record_bridge_resource(dev, moving, CARDBUS_MEM_SIZE,
-               PCI_CB_MEMORY_BASE_0, type);
-       if (type & IORESOURCE_PREFETCH) {
+                                      PCI_CB_MEMORY_BASE_0, type);
+       if (type & IORESOURCE_PREFETCH)
                cardbus_size_bridge_resource(dev, PCI_CB_MEMORY_BASE_0);
-       }
 
-       /* See which bridge memory resources are implemented */
-       moving_base  = pci_moving_config32(dev, PCI_CB_MEMORY_BASE_1);
+       /* See which bridge memory resources are implemented. */
+       moving_base = pci_moving_config32(dev, PCI_CB_MEMORY_BASE_1);
        moving_limit = pci_moving_config32(dev, PCI_CB_MEMORY_LIMIT_1);
        moving = moving_base & moving_limit;
 
-       /* Initialize the memory space constraints on the current bus */
+       /* Initialize the memory space constraints on the current bus. */
        cardbus_record_bridge_resource(dev, moving, CARDBUS_MEM_SIZE,
-               PCI_CB_MEMORY_BASE_1, IORESOURCE_MEM);
+                                      PCI_CB_MEMORY_BASE_1, IORESOURCE_MEM);
        cardbus_size_bridge_resource(dev, PCI_CB_MEMORY_BASE_1);
 
        compact_resources(dev);
@@ -156,7 +159,8 @@ void cardbus_read_resources(device_t dev)
 
 void cardbus_enable_resources(device_t dev)
 {
-       uint16_t ctrl;
+       u16 ctrl;
+
        ctrl = pci_read_config16(dev, PCI_CB_BRIDGE_CONTROL);
        ctrl |= (dev->link_list->bridge_ctrl & (
                        PCI_BRIDGE_CTL_PARITY |
@@ -165,7 +169,8 @@ void cardbus_enable_resources(device_t dev)
                        PCI_BRIDGE_CTL_VGA |
                        PCI_BRIDGE_CTL_MASTER_ABORT |
                        PCI_BRIDGE_CTL_BUS_RESET));
-       ctrl |= (PCI_CB_BRIDGE_CTL_PARITY + PCI_CB_BRIDGE_CTL_SERR); /* error check */
+       /* Error check */
+       ctrl |= (PCI_CB_BRIDGE_CTL_PARITY + PCI_CB_BRIDGE_CTL_SERR);
        printk(BIOS_DEBUG, "%s bridge ctrl <- %04x\n", dev_path(dev), ctrl);
        pci_write_config16(dev, PCI_BRIDGE_CONTROL, ctrl);
 
@@ -176,8 +181,8 @@ struct device_operations default_cardbus_ops_bus = {
        .read_resources   = cardbus_read_resources,
        .set_resources    = pci_dev_set_resources,
        .enable_resources = cardbus_enable_resources,
-       .init                     = 0,
-       .scan_bus                 = pci_scan_bridge,
+       .init             = 0,
+       .scan_bus         = pci_scan_bridge,
        .enable           = 0,
        .reset_bus        = pci_bus_reset,
 };
index f304545b74bdab584cd23d49432722a8d6d94643..0349f96c0c690f7dc28c25abd0bbe7e7e69fa413 100644 (file)
 
 static void pciexp_tune_dev(device_t dev)
 {
-       unsigned cap;
+       unsigned int cap;
 
        cap = pci_find_capability(dev, PCI_CAP_ID_PCIE);
-       if (!cap) {
-               /* error... */
+       if (!cap)
                return;
-       }
+
 #ifdef CONFIG_PCIE_TUNING
        printk(BIOS_DEBUG, "PCIe: tuning %s\n", dev_path(dev));
 
@@ -45,16 +44,16 @@ static void pciexp_tune_dev(device_t dev)
 #endif
 }
 
-unsigned int pciexp_scan_bus(struct bus *bus,
-       unsigned min_devfn, unsigned max_devfn,
-       unsigned int max)
+unsigned int pciexp_scan_bus(struct bus *bus, unsigned int min_devfn,
+                            unsigned int max_devfn, unsigned int max)
 {
        device_t child;
+
        max = pci_scan_bus(bus, min_devfn, max_devfn, max);
-       for(child = bus->children; child; child = child->sibling) {
-               if (    (child->path.pci.devfn < min_devfn) ||
-                       (child->path.pci.devfn > max_devfn))
-               {
+
+       for (child = bus->children; child; child = child->sibling) {
+               if ((child->path.pci.devfn < min_devfn) ||
+                   (child->path.pci.devfn > max_devfn)) {
                        continue;
                }
                pciexp_tune_dev(child);
@@ -76,8 +75,8 @@ struct device_operations default_pciexp_ops_bus = {
        .read_resources   = pci_bus_read_resources,
        .set_resources    = pci_dev_set_resources,
        .enable_resources = pci_bus_enable_resources,
-       .init             = 0,
-       .scan_bus         = pciexp_scan_bridge,
+       .init             = 0,
+       .scan_bus         = pciexp_scan_bridge,
        .enable           = 0,
        .reset_bus        = pci_bus_reset,
        .ops_pci          = &pciexp_bus_ops_pci,
index 42babe79dda6c2215528dfeb0b0d9703864b6797..22dd06e5bdd3956c76b7ff38a11938c860258bf0 100644 (file)
 
 static void pcix_tune_dev(device_t dev)
 {
-       unsigned cap;
-       unsigned status, orig_cmd, cmd;
-       unsigned max_read, max_tran;
+       u32 status;
+       u16 orig_cmd, cmd;
+       unsigned int cap, max_read, max_tran;
 
-       if (dev->hdr_type != PCI_HEADER_TYPE_NORMAL) {
+       if (dev->hdr_type != PCI_HEADER_TYPE_NORMAL)
                return;
-       }
+
        cap = pci_find_capability(dev, PCI_CAP_ID_PCIX);
-       if (!cap) {
+       if (!cap)
                return;
-       }
+
        printk(BIOS_DEBUG, "%s PCI-X tuning\n", dev_path(dev));
+
        status = pci_read_config32(dev, cap + PCI_X_STATUS);
-       orig_cmd = cmd = pci_read_config16(dev,cap + PCI_X_CMD);
+       orig_cmd = cmd = pci_read_config16(dev, cap + PCI_X_CMD);
 
        max_read = (status & PCI_X_STATUS_MAX_READ) >> 21;
        max_tran = (status & PCI_X_STATUS_MAX_SPLIT) >> 23;
@@ -51,23 +52,26 @@ static void pcix_tune_dev(device_t dev)
                cmd &= ~PCI_X_CMD_MAX_SPLIT;
                cmd |= max_tran << 4;
        }
-       /* Don't attempt to handle PCI-X errors */
+
+       /* Don't attempt to handle PCI-X errors. */
        cmd &= ~PCI_X_CMD_DPERR_E;
-       /* Enable Relaxed Ordering */
+
+       /* Enable Relaxed Ordering. */
        cmd |= PCI_X_CMD_ERO;
-       if (orig_cmd != cmd) {
+
+       if (orig_cmd != cmd)
                pci_write_config16(dev, cap + PCI_X_CMD, cmd);
-       }
 }
 
 static void pcix_tune_bus(struct bus *bus)
 {
        device_t child;
-       for(child = bus->children; child; child = child->sibling)
+
+       for (child = bus->children; child; child = child->sibling)
                pcix_tune_dev(child);
 }
 
-const char *pcix_speed(unsigned sstatus)
+const char *pcix_speed(u16 sstatus)
 {
        static const char conventional[] = "Conventional PCI";
        static const char pcix_66mhz[] = "66MHz PCI-X";
@@ -76,10 +80,11 @@ const char *pcix_speed(unsigned sstatus)
        static const char pcix_266mhz[] = "266MHz PCI-X";
        static const char pcix_533mhz[] = "533MHZ PCI-X";
        static const char unknown[] = "Unknown";
-
        const char *result;
+
        result = unknown;
-       switch(PCI_X_SSTATUS_MFREQ(sstatus)) {
+
+       switch (PCI_X_SSTATUS_MFREQ(sstatus)) {
        case PCI_X_SSTATUS_CONVENTIONAL_PCI:
                result = conventional;
                break;
@@ -89,17 +94,14 @@ const char *pcix_speed(unsigned sstatus)
        case PCI_X_SSTATUS_MODE1_100MHZ:
                result = pcix_100mhz;
                break;
-
        case PCI_X_SSTATUS_MODE1_133MHZ:
                result = pcix_133mhz;
                break;
-
        case PCI_X_SSTATUS_MODE2_266MHZ_REF_66MHZ:
        case PCI_X_SSTATUS_MODE2_266MHZ_REF_100MHZ:
        case PCI_X_SSTATUS_MODE2_266MHZ_REF_133MHZ:
                result = pcix_266mhz;
                break;
-
        case PCI_X_SSTATUS_MODE2_533MHZ_REF_66MHZ:
        case PCI_X_SSTATUS_MODE2_533MHZ_REF_100MHZ:
        case PCI_X_SSTATUS_MODE2_533MHZ_REF_133MHZ:
@@ -111,20 +113,21 @@ const char *pcix_speed(unsigned sstatus)
 
 unsigned int pcix_scan_bridge(device_t dev, unsigned int max)
 {
-       unsigned pos;
-       unsigned sstatus;
+       unsigned int pos;
+       u16 sstatus;
 
        max = do_pci_scan_bridge(dev, max, pci_scan_bus);
-       /* Find the PCI-X capability */
+
+       /* Find the PCI-X capability. */
        pos = pci_find_capability(dev, PCI_CAP_ID_PCIX);
        sstatus = pci_read_config16(dev, pos + PCI_X_SEC_STATUS);
 
-       if (PCI_X_SSTATUS_MFREQ(sstatus) != PCI_X_SSTATUS_CONVENTIONAL_PCI) {
+       if (PCI_X_SSTATUS_MFREQ(sstatus) != PCI_X_SSTATUS_CONVENTIONAL_PCI)
                pcix_tune_bus(dev->link_list);
-       }
 
-       /* Print the PCI-X bus speed */
-       printk(BIOS_DEBUG, "PCI: %02x: %s\n", dev->link_list->secondary, pcix_speed(sstatus));
+       /* Print the PCI-X bus speed. */
+       printk(BIOS_DEBUG, "PCI: %02x: %s\n", dev->link_list->secondary,
+              pcix_speed(sstatus));
 
        return max;
 }
@@ -138,8 +141,8 @@ struct device_operations default_pcix_ops_bus = {
        .read_resources   = pci_bus_read_resources,
        .set_resources    = pci_dev_set_resources,
        .enable_resources = pci_bus_enable_resources,
-       .init             = 0,
-       .scan_bus         = pcix_scan_bridge,
+       .init             = 0,
+       .scan_bus         = pcix_scan_bridge,
        .enable           = 0,
        .reset_bus        = pci_bus_reset,
        .ops_pci          = &pcix_bus_ops_pci,
index ad14bbf89af633caff023362366e07796c6a2cb1..702aa9253c3a15fc0b26dcec57bea102df855ef2 100644 (file)
 
 /* PNP fundamental operations */
 
-void pnp_write_config(device_t dev, uint8_t reg, uint8_t value)
+void pnp_write_config(device_t dev, u8 reg, u8 value)
 {
        outb(reg, dev->path.pnp.port);
        outb(value, dev->path.pnp.port + 1);
 }
 
-uint8_t pnp_read_config(device_t dev, uint8_t reg)
+u8 pnp_read_config(device_t dev, u8 reg)
 {
        outb(reg, dev->path.pnp.port);
        return inb(dev->path.pnp.port + 1);
@@ -51,45 +51,49 @@ void pnp_set_logical_device(device_t dev)
 
 void pnp_set_enable(device_t dev, int enable)
 {
-       uint8_t tmp, bitpos;
+       u8 tmp, bitpos;
 
        tmp = pnp_read_config(dev, 0x30);
-       /* handle the virtual devices, which share same LDN register */
+
+       /* Handle virtual devices, which share the same LDN register. */
        bitpos = (dev->path.pnp.device >> 8) & 0x7;
 
-       if (enable) {
+       if (enable)
                tmp |= (1 << bitpos);
-       } else {
+       else
                tmp &= ~(1 << bitpos);
-       }
+
        pnp_write_config(dev, 0x30, tmp);
 }
 
 int pnp_read_enable(device_t dev)
 {
-       uint8_t tmp, bitpos;
+       u8 tmp, bitpos;
+
        tmp = pnp_read_config(dev, 0x30);
-       /* handle the virtual devices, which share same LDN register */
+
+       /* Handle virtual devices, which share the same LDN register. */
        bitpos = (dev->path.pnp.device >> 8) & 0x7;
+
        return !!(tmp & (1 << bitpos));
 }
 
-void pnp_set_iobase(device_t dev, unsigned index, unsigned iobase)
+void pnp_set_iobase(device_t dev, u8 index, u16 iobase)
 {
-       /* Index == 0x60 or 0x62 */
+       /* Index == 0x60 or 0x62. */
        pnp_write_config(dev, index + 0, (iobase >> 8) & 0xff);
        pnp_write_config(dev, index + 1, iobase & 0xff);
 }
 
-void pnp_set_irq(device_t dev, unsigned index, unsigned irq)
+void pnp_set_irq(device_t dev, u8 index, u8 irq)
 {
-       /* Index == 0x70 or 0x72 */
+       /* Index == 0x70 or 0x72. */
        pnp_write_config(dev, index, irq);
 }
 
-void pnp_set_drq(device_t dev, unsigned index, unsigned drq)
+void pnp_set_drq(device_t dev, u8 index, u8 drq)
 {
-       /* Index == 0x74 */
+       /* Index == 0x74. */
        pnp_write_config(dev, index, drq & 0xff);
 }
 
@@ -103,26 +107,22 @@ void pnp_read_resources(device_t dev)
 static void pnp_set_resource(device_t dev, struct resource *resource)
 {
        if (!(resource->flags & IORESOURCE_ASSIGNED)) {
-               printk(BIOS_ERR, "ERROR: %s %02lx %s size: 0x%010Lx not assigned\n",
-                       dev_path(dev), resource->index,
-                       resource_type(resource),
-                       resource->size);
+               printk(BIOS_ERR, "ERROR: %s %02lx %s size: 0x%010Lx "
+                      "not assigned\n", dev_path(dev), resource->index,
+                      resource_type(resource), resource->size);
                return;
        }
 
-       /* Now store the resource */
+       /* Now store the resource. */
        if (resource->flags & IORESOURCE_IO) {
                pnp_set_iobase(dev, resource->index, resource->base);
-       }
-       else if (resource->flags & IORESOURCE_DRQ) {
+       } else if (resource->flags & IORESOURCE_DRQ) {
                pnp_set_drq(dev, resource->index, resource->base);
-       }
-       else if (resource->flags  & IORESOURCE_IRQ) {
+       } else if (resource->flags & IORESOURCE_IRQ) {
                pnp_set_irq(dev, resource->index, resource->base);
-       }
-       else {
+       } else {
                printk(BIOS_ERR, "ERROR: %s %02lx unknown resource type\n",
-                       dev_path(dev), resource->index);
+                      dev_path(dev), resource->index);
                return;
        }
        resource->flags |= IORESOURCE_STORED;
@@ -134,13 +134,12 @@ void pnp_set_resources(device_t dev)
 {
        struct resource *res;
 
-       /* Select the device */
+       /* Select the logical device (LDN). */
        pnp_set_logical_device(dev);
 
        /* Paranoia says I should disable the device here... */
-       for(res = dev->resource_list; res; res = res->next) {
+       for (res = dev->resource_list; res; res = res->next)
                pnp_set_resource(dev, res);
-       }
 }
 
 void pnp_enable_resources(device_t dev)
@@ -166,39 +165,45 @@ struct device_operations pnp_ops = {
 
 /* PNP chip opertations */
 
-static void pnp_get_ioresource(device_t dev, unsigned index, struct io_info *info)
+static void pnp_get_ioresource(device_t dev, u8 index, struct io_info *info)
 {
        struct resource *resource;
        unsigned moving, gran, step;
 
        resource = new_resource(dev, index);
 
-       /* Initilize the resource */
+       /* Initilize the resource. */
        resource->limit = 0xffff;
        resource->flags |= IORESOURCE_IO;
 
-       /* Get the resource size */
+       /* Get the resource size... */
+
        moving = info->mask;
        gran = 15;
        step = 1 << gran;
-       /* Find the first bit that moves */
-       while((moving & step) == 0) {
+
+       /* Find the first bit that moves. */
+       while ((moving & step) == 0) {
                gran--;
                step >>= 1;
        }
-       /* Now find the first bit that does not move */
-       while((moving & step) != 0) {
+
+       /* Now find the first bit that does not move. */
+       while ((moving & step) != 0) {
                gran--;
                step >>= 1;
        }
-       /* Of the moving bits the last bit in the first group,
+
+       /*
+        * Of the moving bits the last bit in the first group,
         * tells us the size of this resource.
         */
        if ((moving & step) == 0) {
                gran++;
                step <<= 1;
        }
-       /* Set the resource size and alignment */
+
+       /* Set the resource size and alignment. */
        resource->gran  = gran;
        resource->align = gran;
        resource->limit = info->mask | (step - 1);
@@ -241,8 +246,10 @@ static void get_resources(device_t dev, struct pnp_info *info)
                resource->size = 1;
                resource->flags |= IORESOURCE_DRQ;
        }
-       /* These are not IRQs, but set the flag to have the
-        * resource allocator do the right thing
+
+       /*
+        * These are not IRQs, but set the flag to have the
+        * resource allocator do the right thing.
         */
        if (info->flags & PNP_EN) {
                resource = new_resource(dev, PNP_IDX_EN);
@@ -262,17 +269,17 @@ static void get_resources(device_t dev, struct pnp_info *info)
 }
 
 void pnp_enable_devices(device_t base_dev, struct device_operations *ops,
-       unsigned functions, struct pnp_info *info)
+                       unsigned int functions, struct pnp_info *info)
 {
        struct device_path path;
        device_t dev;
        int i;
 
-       path.type       = DEVICE_PATH_PNP;
+       path.type = DEVICE_PATH_PNP;
        path.pnp.port = base_dev->path.pnp.port;
 
-       /* Setup the ops and resources on the newly allocated devices */
-       for(i = 0; i < functions; i++) {
+       /* Setup the ops and resources on the newly allocated devices. */
+       for (i = 0; i < functions; i++) {
                /* Skip logical devices this Super I/O doesn't have. */
                if (info[i].function == -1)
                        continue;
@@ -280,15 +287,15 @@ void pnp_enable_devices(device_t base_dev, struct device_operations *ops,
                path.pnp.device = info[i].function;
                dev = alloc_find_dev(base_dev->bus, &path);
 
-               /* Don't initialize a device multiple times */
+               /* Don't initialize a device multiple times. */
                if (dev->ops)
                        continue;
 
-               if (info[i].ops == 0) {
+               if (info[i].ops == 0)
                        dev->ops = ops;
-               } else {
+               else
                        dev->ops = info[i].ops;
-               }
+
                get_resources(dev, &info[i]);
        }
 }
index 2c5cfd84e11f5d651d927f1d8917fae8f03ad6f3..591ba66d5d3c322e41085c0e09b5d99e93cb5ce7 100644 (file)
@@ -71,40 +71,42 @@ static void root_dev_set_resources(device_t root)
  *
  * @param bus Pointer to the device to which the static buses are attached to.
  * @param max Maximum bus number currently used before scanning.
- * @return Largest bus number used.
+ * @return The largest bus number used.
  */
 static int smbus_max = 0;
 unsigned int scan_static_bus(device_t bus, unsigned int max)
 {
        device_t child;
-       struct buslink;
+       struct bus *link;
 
        printk(BIOS_SPEW, "%s for %s\n", __func__, dev_path(bus));
 
-       for(link = bus->link_list; link; link = link->next) {
-               /* for smbus bus enumerate */
+       for (link = bus->link_list; link; link = link->next) {
+               /* For SMBus bus enumerate. */
                child = link->children;
-               if(child && child->path.type == DEVICE_PATH_I2C) {
+
+               if (child && child->path.type == DEVICE_PATH_I2C)
                        link->secondary = ++smbus_max;
-               }
-               for(child =link->children; child; child = child->sibling) {
-                       if (child->chip_ops && child->chip_ops->enable_dev) {
+
+               for (child = link->children; child; child = child->sibling) {
+                       if (child->chip_ops && child->chip_ops->enable_dev)
                                child->chip_ops->enable_dev(child);
-                       }
-                       if (child->ops && child->ops->enable) {
+
+                       if (child->ops && child->ops->enable)
                                child->ops->enable(child);
+
+                       if (child->path.type == DEVICE_PATH_I2C) {
+                               printk(BIOS_DEBUG, "smbus: %s[%d]->",
+                                      dev_path(child->bus->dev),
+                                      child->bus->link_num);
                        }
-                       if (child->path.type == DEVICE_PATH_I2C) {
-                               printk(BIOS_DEBUG, "smbus: %s[%d]->",
-                                       dev_path(child->bus->dev), child->bus->link_num );
-                       }
-                       printk(BIOS_DEBUG, "%s %s\n",
-                               dev_path(child),
-                               child->enabled?"enabled": "disabled");
+                       printk(BIOS_DEBUG, "%s %s\n", dev_path(child),
+                              child->enabled ? "enabled" : "disabled");
                }
        }
-       for(link = bus->link_list; link; link = link->next) {
-               for(child = link->children; child; child = child->sibling) {
+
+       for (link = bus->link_list; link; link = link->next) {
+               for (child = link->children; child; child = child->sibling) {
                        if (!child->ops || !child->ops->scan_bus)
                                continue;
                        printk(BIOS_SPEW, "%s scanning...\n", dev_path(child));
@@ -128,7 +130,7 @@ static void root_dev_enable_resources(device_t dev)
  *
  * @param root The root device structure.
  * @param max The current bus number scanned so far, usually 0x00.
- * @return TODO.
+ * @return The largest bus number used.
  */
 static unsigned int root_dev_scan_bus(device_t root, unsigned int max)
 {
@@ -141,7 +143,7 @@ static void root_dev_init(device_t root)
 
 static void root_dev_reset(struct bus *bus)
 {
-       printk(BIOS_INFO, "Reseting board...\n");
+       printk(BIOS_INFO, "Resetting board...\n");
        hard_reset();
 }
 
index 9d134619b89d771f4e23fbbf083d73940dc09044..bac73986daca24a0208d0ab8700edee2c7c2b7e6 100644 (file)
 struct bus *get_pbus_smbus(device_t dev)
 {
        struct bus *pbus = dev->bus;
-       while (pbus && pbus->dev && !ops_smbus_bus(pbus)) {
+
+       while (pbus && pbus->dev && !ops_smbus_bus(pbus))
                pbus = pbus->dev->bus;
-       }
-       if (!pbus || !pbus->dev || !pbus->dev->ops || !pbus->dev->ops->ops_smbus_bus) {
-               printk(BIOS_ALERT, "%s Cannot find smbus bus operations", dev_path(dev));
+
+       if (!pbus || !pbus->dev || !pbus->dev->ops
+           || !pbus->dev->ops->ops_smbus_bus) {
+               printk(BIOS_ALERT, "%s Cannot find SMBus bus operations",
+                      dev_path(dev));
                die("");
        }
+
        return pbus;
 }
 
@@ -50,67 +54,84 @@ struct bus *get_pbus_smbus(device_t dev)
 int smbus_set_link(device_t dev)
 {
        struct bus *pbus_a[4]; // 4 level mux only. Enough?
-        struct bus *pbus = dev->bus;
-       int pbus_num=0;
+       struct bus *pbus = dev->bus;
+       int pbus_num = 0;
        int i;
-        while(pbus && pbus->dev && (pbus->dev->path.type==DEVICE_PATH_I2C)) {
+
+       while (pbus && pbus->dev && (pbus->dev->path.type == DEVICE_PATH_I2C)) {
                pbus_a[pbus_num++] = pbus;
-                pbus = pbus->dev->bus;
-        }
-//     printk(BIOS_INFO, "smbus_set_link: ");
-       for (i=pbus_num-1; i>=0; i--) {
-//             printk(BIOS_INFO, " %s[%d] -> ", dev_path(pbus_a[i]->dev), pbus_a[i]->link);
-               if (ops_smbus_bus(get_pbus_smbus(pbus_a[i]->dev))) {
-                       if (pbus_a[i]->dev->ops && pbus_a[i]->dev->ops->set_link)
-                               pbus_a[i]->dev->ops->set_link(pbus_a[i]->dev, pbus_a[i]->link_num);
+               pbus = pbus->dev->bus;
+       }
+
+       // printk(BIOS_INFO, "smbus_set_link: ");
+       for (i = pbus_num - 1; i >= 0; i--) {
+               // printk(BIOS_INFO, " %s[%d] -> ", dev_path(pbus_a[i]->dev),
+               // pbus_a[i]->link);
+               if (ops_smbus_bus(get_pbus_smbus(pbus_a[i]->dev))) {
+                       if (pbus_a[i]->dev->ops
+                           && pbus_a[i]->dev->ops->set_link)
+                               pbus_a[i]->dev->ops->set_link(pbus_a[i]->dev,
+                                                       pbus_a[i]->link_num);
                }
        }
-//     printk(BIOS_INFO, " %s\n", dev_path(dev));
+       // printk(BIOS_INFO, " %s\n", dev_path(dev));
 
-        return pbus_num;
+       return pbus_num;
 }
 
 int smbus_quick_read(device_t dev)
 {
        return ops_smbus_bus(get_pbus_smbus(dev))->quick_read(dev);
 }
+
 int smbus_quick_write(device_t dev)
 {
        return ops_smbus_bus(get_pbus_smbus(dev))->quick_write(dev);
 }
+
 int smbus_recv_byte(device_t dev)
 {
        return ops_smbus_bus(get_pbus_smbus(dev))->recv_byte(dev);
 }
-int smbus_send_byte(device_t dev, uint8_t byte)
+
+int smbus_send_byte(device_t dev, u8 byte)
 {
        return ops_smbus_bus(get_pbus_smbus(dev))->send_byte(dev, byte);
 }
-int smbus_read_byte(device_t dev, uint8_t addr)
+
+int smbus_read_byte(device_t dev, u8 addr)
 {
        return ops_smbus_bus(get_pbus_smbus(dev))->read_byte(dev, addr);
 }
-int smbus_write_byte(device_t dev, uint8_t addr, uint8_t val)
+
+int smbus_write_byte(device_t dev, u8 addr, u8 val)
 {
        return ops_smbus_bus(get_pbus_smbus(dev))->write_byte(dev, addr, val);
 }
-int smbus_read_word(device_t dev, uint8_t addr)
+
+int smbus_read_word(device_t dev, u8 addr)
 {
        return ops_smbus_bus(get_pbus_smbus(dev))->read_word(dev, addr);
 }
-int smbus_write_word(device_t dev, uint8_t addr, uint16_t val)
+
+int smbus_write_word(device_t dev, u8 addr, u16 val)
 {
        return ops_smbus_bus(get_pbus_smbus(dev))->write_word(dev, addr, val);
 }
-int smbus_process_call(device_t dev, uint8_t cmd, uint16_t data)
+
+int smbus_process_call(device_t dev, u8 cmd, u16 data)
 {
        return ops_smbus_bus(get_pbus_smbus(dev))->process_call(dev, cmd, data);
 }
-int smbus_block_read(device_t dev, uint8_t cmd, uint8_t bytes, uint8_t *buffer)
+
+int smbus_block_read(device_t dev, u8 cmd, u8 bytes, u8 *buffer)
 {
-       return ops_smbus_bus(get_pbus_smbus(dev))->block_read(dev, cmd, bytes, buffer);
+       return ops_smbus_bus(get_pbus_smbus(dev))->block_read(dev, cmd,
+                                                             bytes, buffer);
 }
-int smbus_block_write(device_t dev, uint8_t cmd, uint8_t bytes, const uint8_t *buffer)
+
+int smbus_block_write(device_t dev, u8 cmd, u8 bytes, const u8 *buffer)
 {
-       return ops_smbus_bus(get_pbus_smbus(dev))->block_write(dev, cmd, bytes, buffer);
+       return ops_smbus_bus(get_pbus_smbus(dev))->block_write(dev, cmd,
+                                                              bytes, buffer);
 }
index becc800934132f7fe08a79e29407f378c898f587..409f211a82f6ecd3b46b6f943b5a0bbba12e317b 100644 (file)
@@ -2,8 +2,8 @@
 #define DEVICE_PCIEXP_H
 /* (c) 2005 Linux Networx GPL see COPYING for details */
 
-unsigned int pciexp_scan_bus(struct bus *bus,
-       unsigned min_devfn, unsigned max_devfn, unsigned int max);
+unsigned int pciexp_scan_bus(struct bus *bus, unsigned int min_devfn,
+                            unsigned int max_devfn, unsigned int max);
 unsigned int pciexp_scan_bridge(device_t dev, unsigned int max);
 
 extern struct device_operations default_pciexp_ops_bus;
index e017922ef1cdf4cccec225608bb136bdb4ac2735..4ffab5bba770f3279e5631b697537702ab2cdd3d 100644 (file)
@@ -2,10 +2,10 @@
 #define DEVICE_PCIX_H
 /* (c) 2005 Linux Networx GPL see COPYING for details */
 
-unsigned int pcix_scan_bus(struct bus *bus,
-       unsigned min_devfn, unsigned max_devfn, unsigned int max);
+unsigned int pcix_scan_bus(struct bus *bus, unsigned int min_devfn,
+                          unsigned int max_devfn, unsigned int max);
 unsigned int pcix_scan_bridge(device_t dev, unsigned int max);
-const char *pcix_speed(unsigned sstatus);
+const char *pcix_speed(u16 sstatus);
 
 extern struct device_operations default_pcix_ops_bus;
 
index 32e9c5eada336ed87657b969f19447b9de879e62..9e94a0c5b0794708d9691105ebb0e77cb6291c90 100644 (file)
@@ -5,15 +5,15 @@
 #include <device/device.h>
 #include <device/pnp_def.h>
 
-/* Primitive pnp resource manipulation */
-void    pnp_write_config(device_t dev, uint8_t reg, uint8_t value);
-uint8_t pnp_read_config(device_t dev, uint8_t reg);
-void    pnp_set_logical_device(device_t dev);
-void    pnp_set_enable(device_t dev, int enable);
-int     pnp_read_enable(device_t dev);
-void    pnp_set_iobase(device_t dev, unsigned index, unsigned iobase);
-void    pnp_set_irq(device_t dev, unsigned index, unsigned irq);
-void    pnp_set_drq(device_t dev, unsigned index, unsigned drq);
+/* Primitive PNP resource manipulation */
+void pnp_write_config(device_t dev, u8 reg, u8 value);
+u8 pnp_read_config(device_t dev, u8 reg);
+void pnp_set_logical_device(device_t dev);
+void pnp_set_enable(device_t dev, int enable);
+int pnp_read_enable(device_t dev);
+void pnp_set_iobase(device_t dev, u8 index, u16 iobase);
+void pnp_set_irq(device_t dev, u8 index, u8 irq);
+void pnp_set_drq(device_t dev, u8 index, u8 drq);
 
 /* PNP device operations */
 void pnp_read_resources(device_t dev);
@@ -31,8 +31,8 @@ struct io_info {
 
 struct pnp_info {
        struct device_operations *ops;
-       unsigned function;
-       unsigned flags;
+       unsigned int function;
+       unsigned int flags;
 #define PNP_IO0  0x001
 #define PNP_IO1  0x002
 #define PNP_IO2  0x004
@@ -48,6 +48,6 @@ struct pnp_info {
 };
 struct resource *pnp_get_resource(device_t dev, unsigned index);
 void pnp_enable_devices(struct device *dev, struct device_operations *ops,
-       unsigned functions, struct pnp_info *info);
+                       unsigned int functions, struct pnp_info *info);
 
 #endif /* DEVICE_PNP_H */
index 148c4d7785d9ce1ab05448c36dfadddfa360cabd..073d7e2c99e3a41454de04f0c96fb2acdf95d924 100644 (file)
@@ -6,44 +6,45 @@
 #include <device/path.h>
 #include <device/smbus_def.h>
 
-/* Common smbus bus operations */
+/* Common SMBus bus operations */
 struct smbus_bus_operations {
        int (*quick_read)  (device_t dev);
        int (*quick_write) (device_t dev);
        int (*recv_byte)   (device_t dev);
-       int (*send_byte)   (device_t dev, uint8_t value);
-       int (*read_byte)   (device_t dev, uint8_t addr);
-       int (*write_byte)  (device_t dev, uint8_t addr, uint8_t value);
-       int (*read_word)   (device_t dev, uint8_t addr);
-       int (*write_word)  (device_t dev, uint8_t addr, uint16_t value);
-       int (*process_call)(device_t dev, uint8_t cmd, uint16_t data);
-       int (*block_read)  (device_t dev, uint8_t cmd, uint8_t bytes, uint8_t *buffer);
-       int (*block_write) (device_t dev, uint8_t cmd, uint8_t bytes, const uint8_t *buffer);
+       int (*send_byte)   (device_t dev, u8 value);
+       int (*read_byte)   (device_t dev, u8 addr);
+       int (*write_byte)  (device_t dev, u8 addr, u8 value);
+       int (*read_word)   (device_t dev, u8 addr);
+       int (*write_word)  (device_t dev, u8 addr, u16 value);
+       int (*process_call)(device_t dev, u8 cmd, u16 data);
+       int (*block_read)  (device_t dev, u8 cmd, u8 bytes, u8 *buffer);
+       int (*block_write) (device_t dev, u8 cmd, u8 bytes, const u8 *buffer);
 };
 
 static inline const struct smbus_bus_operations *ops_smbus_bus(struct bus *bus)
 {
-        const struct smbus_bus_operations *bops;
-        bops = 0;
-        if (bus && bus->dev && bus->dev->ops) {
-                bops = bus->dev->ops->ops_smbus_bus;
-        }
-        return bops;
+       const struct smbus_bus_operations *bops;
+
+       bops = 0;
+       if (bus && bus->dev && bus->dev->ops)
+               bops = bus->dev->ops->ops_smbus_bus;
+
+       return bops;
 }
+
 struct bus *get_pbus_smbus(device_t dev);
 int smbus_set_link(device_t dev);
 
 int smbus_quick_read(device_t dev);
 int smbus_quick_write(device_t dev);
 int smbus_recv_byte(device_t dev);
-int smbus_send_byte(device_t dev, uint8_t byte);
-int smbus_read_byte(device_t dev, uint8_t addr);
-int smbus_write_byte(device_t dev, uint8_t addr, uint8_t val);
-int smbus_read_word(device_t dev, uint8_t addr);
-int smbus_write_word(device_t dev, uint8_t addr, uint16_t val);
-int smbus_process_call(device_t dev, uint8_t cmd, uint16_t data);
-int smbus_block_read(device_t dev, uint8_t cmd, uint8_t bytes, uint8_t *buffer);
-int smbus_block_write(device_t dev, uint8_t cmd, uint8_t bytes, const uint8_t *buffer);
-
+int smbus_send_byte(device_t dev, u8 byte);
+int smbus_read_byte(device_t dev, u8 addr);
+int smbus_write_byte(device_t dev, u8 addr, u8 val);
+int smbus_read_word(device_t dev, u8 addr);
+int smbus_write_word(device_t dev, u8 addr, u16 val);
+int smbus_process_call(device_t dev, u8 cmd, u16 data);
+int smbus_block_read(device_t dev, u8 cmd, u8 bytes, u8 *buffer);
+int smbus_block_write(device_t dev, u8 cmd, u8 bytes, const u8 *buffer);
 
 #endif /* DEVICE_SMBUS_H */