sravan/system76-coreboot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use lists instead of arrays for resources in devices to reduce memory usage.

Browse Source 
Signed-off-by: Myles Watson <mylesgw@gmail.com>
Acked-by: Patrick Georgi <patrick.georgi@coresystems.de>


git-svn-id: svn://svn.coreboot.org/coreboot/trunk@5576 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
This commit is contained in:
Myles Watson
2010-05-21 14:33:48 +00:00
parent c5b87c8f89
commit c25cc11ae3
25 changed files with 200 additions and 190 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/superio/smsc/lpc47n217/superio.c
View File

@@ -99,15 +99,15 @@ static void enable_dev(device_t dev)
*/
static void lpc47n217_pnp_set_resources(device_t dev)
{
int i;
struct resource *res;
pnp_enter_conf_state(dev);
/* NOTE: Cannot use pnp_set_resources() here because it assumes chip
* support for logical devices, which the LPC47N217 doesn't have
*/
for(i = 0; i < dev->resources; i++)
lpc47n217_pnp_set_resource(dev, &dev->resource[i]);
for(res = dev->resource_list; res; res = res->next)
lpc47n217_pnp_set_resource(dev, res);
/* dump_pnp_device(dev); */

6
src/superio/smsc/lpc47n227/superio.c
View File

@@ -96,14 +96,14 @@ static void enable_dev(device_t dev)
//
void lpc47n227_pnp_set_resources(device_t dev)
{
int i;
struct resource *res;
pnp_enter_conf_state(dev);
// NOTE: Cannot use pnp_set_resources() here because it assumes chip
// support for logical devices, which the LPC47N227 doesn't have
for (i = 0; i < dev->resources; i++)
lpc47n227_pnp_set_resource(dev, &dev->resource[i]);
for (res = dev->resource_list; res; res = res->next)
lpc47n227_pnp_set_resource(dev, res);
pnp_exit_conf_state(dev);
}

39
src/superio/via/vt1211/vt1211.c
View File

@@ -127,15 +127,13 @@ static void vt1211_pnp_enable_resources(device_t dev)
static void vt1211_pnp_set_resources(struct device *dev)
{
int i;
struct resource *resource;
struct resource *res;
#if CONFIG_CONSOLE_SERIAL8250 == 1
if( dev->path.pnp.device == 2 ){
for( i = 0 ; i < dev->resources; i++){
resource = &dev->resource[i];
resource->flags |= IORESOURCE_STORED;
report_resource_stored(dev, resource, "");
for(res = dev->resource_list; res; res = res->next){
res->flags |= IORESOURCE_STORED;
report_resource_stored(dev, res, "");
}
return;
}
@@ -145,34 +143,33 @@ static void vt1211_pnp_set_resources(struct device *dev)
pnp_set_logical_device(dev);
/* Paranoia says I should disable the device here... */
for(i = 0; i < dev->resources; i++) {
resource = &dev->resource[i];
if (!(resource->flags & IORESOURCE_ASSIGNED)) {
for(res = dev->resource_list; res; res = res->next){
if (!(res->flags & IORESOURCE_ASSIGNED)) {
printk(BIOS_ERR, "ERROR: %s %02lx %s size: 0x%010Lx not assigned\n",
dev_path(dev), dev->resource->index,
resource_type(resource),
resource->size);
dev_path(dev), res->index,
resource_type(res),
res->size);
continue;
}
/* Now store the resource */
if (resource->flags & IORESOURCE_IO) {
vt1211_set_iobase(dev, resource->index, resource->base);
if (res->flags & IORESOURCE_IO) {
vt1211_set_iobase(dev, res->index, res->base);
}
else if (resource->flags & IORESOURCE_DRQ) {
pnp_set_drq(dev, resource->index, resource->base);
else if (res->flags & IORESOURCE_DRQ) {
pnp_set_drq(dev, res->index, res->base);
}
else if (resource->flags & IORESOURCE_IRQ) {
pnp_set_irq(dev, resource->index, resource->base);
else if (res->flags & IORESOURCE_IRQ) {
pnp_set_irq(dev, res->index, res->base);
}
else {
printk(BIOS_ERR, "ERROR: %s %02lx unknown resource type\n",
dev_path(dev), resource->index);
dev_path(dev), res->index);
return;
}
resource->flags |= IORESOURCE_STORED;
res->flags |= IORESOURCE_STORED;
report_resource_stored(dev, resource, "");
report_resource_stored(dev, res, "");
}
pnp_exit_ext_func_mode(dev);