Am 21.05.2010 01:08, schrieb Myles Watson: > static void free_resource(device_t dev, struct resource *res, struct resource *prev) > ... > + res->next = free_resources; > + free_resources = res->next;
Shouldn't that be + res->next = free_resources; + free_resources = res; to add res to the free_resources? Other than that, this is great! Smaller, cleaner, faster code, that uses less memory to achieve the same result - wonderful! With the above thing cleared up, this is Acked-by: Patrick Georgi <[email protected]> -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
