[coreboot] [commit] r5962 - in trunk/src: devices include/device

repository service svn at coreboot.org
Mon Oct 18 02:00:57 CEST 2010


Author: uwe
Date: Mon Oct 18 02:00:57 2010
New Revision: 5962
URL: https://tracker.coreboot.org/trac/coreboot/changeset/5962

Log:
Cosmetics and coding style fixes in devices/*.

 - Whitespace and indentation fixes in various places.
 
 - Fix various typos.
 
 - Use u8, u16 etc. everywhere.

Abuild-tested.

Signed-off-by: Uwe Hermann <uwe at hermann-uwe.de>
Acked-by: Uwe Hermann <uwe at hermann-uwe.de>

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

Modified: trunk/src/devices/cardbus_device.c
==============================================================================
--- trunk/src/devices/cardbus_device.c	Mon Oct 18 01:55:17 2010	(r5961)
+++ trunk/src/devices/cardbus_device.c	Mon Oct 18 02:00:57 2010	(r5962)
@@ -25,28 +25,30 @@
 #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 @@
 		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 @@
 {
 	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 @@
 	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_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 @@
 			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 @@
 	.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,
 };

Modified: trunk/src/devices/pciexp_device.c
==============================================================================
--- trunk/src/devices/pciexp_device.c	Mon Oct 18 01:55:17 2010	(r5961)
+++ trunk/src/devices/pciexp_device.c	Mon Oct 18 02:00:57 2010	(r5962)
@@ -26,13 +26,12 @@
 
 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 @@
 #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 @@
 	.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,

Modified: trunk/src/devices/pcix_device.c
==============================================================================
--- trunk/src/devices/pcix_device.c	Mon Oct 18 01:55:17 2010	(r5961)
+++ trunk/src/devices/pcix_device.c	Mon Oct 18 02:00:57 2010	(r5962)
@@ -26,20 +26,21 @@
 
 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 @@
 		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 @@
 	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 @@
 	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 @@
 
 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 @@
 	.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,

Modified: trunk/src/devices/pnp_device.c
==============================================================================
--- trunk/src/devices/pnp_device.c	Mon Oct 18 01:55:17 2010	(r5961)
+++ trunk/src/devices/pnp_device.c	Mon Oct 18 02:00:57 2010	(r5962)
@@ -32,13 +32,13 @@
 
 /* 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_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 @@
 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 @@
 {
 	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 @@
 
 /* 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 @@
 		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 @@
 }
 
 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 @@
 		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]);
 	}
 }

Modified: trunk/src/devices/root_device.c
==============================================================================
--- trunk/src/devices/root_device.c	Mon Oct 18 01:55:17 2010	(r5961)
+++ trunk/src/devices/root_device.c	Mon Oct 18 02:00:57 2010	(r5962)
@@ -71,40 +71,42 @@
  *
  * @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 bus* link;
+	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 @@
  *
  * @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_reset(struct bus *bus)
 {
-	printk(BIOS_INFO, "Reseting board...\n");
+	printk(BIOS_INFO, "Resetting board...\n");
 	hard_reset();
 }
 

Modified: trunk/src/devices/smbus_ops.c
==============================================================================
--- trunk/src/devices/smbus_ops.c	Mon Oct 18 01:55:17 2010	(r5961)
+++ trunk/src/devices/smbus_ops.c	Mon Oct 18 02:00:57 2010	(r5962)
@@ -28,13 +28,17 @@
 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 @@
 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);
 }

Modified: trunk/src/include/device/pciexp.h
==============================================================================
--- trunk/src/include/device/pciexp.h	Mon Oct 18 01:55:17 2010	(r5961)
+++ trunk/src/include/device/pciexp.h	Mon Oct 18 02:00:57 2010	(r5962)
@@ -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;

Modified: trunk/src/include/device/pcix.h
==============================================================================
--- trunk/src/include/device/pcix.h	Mon Oct 18 01:55:17 2010	(r5961)
+++ trunk/src/include/device/pcix.h	Mon Oct 18 02:00:57 2010	(r5962)
@@ -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;
 

Modified: trunk/src/include/device/pnp.h
==============================================================================
--- trunk/src/include/device/pnp.h	Mon Oct 18 01:55:17 2010	(r5961)
+++ trunk/src/include/device/pnp.h	Mon Oct 18 02:00:57 2010	(r5962)
@@ -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 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 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 */

Modified: trunk/src/include/device/smbus.h
==============================================================================
--- trunk/src/include/device/smbus.h	Mon Oct 18 01:55:17 2010	(r5961)
+++ trunk/src/include/device/smbus.h	Mon Oct 18 02:00:57 2010	(r5962)
@@ -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 */




More information about the coreboot mailing list