On 2015年05月14日 16:56, Jiang Liu wrote:
Use common interface to simplify ACPI PCI host bridge implementation.
Signed-off-by: Jiang Liu <[email protected]>
---
arch/x86/pci/acpi.c | 292 +++++++++++++++------------------------------------
1 file changed, 85 insertions(+), 207 deletions(-)
[...]
-static void release_pci_root_info(struct pci_host_bridge *bridge)
+static void pci_acpi_root_release_info(struct acpi_pci_root_info *ci)
{
- struct resource *res;
- struct resource_entry *entry;
- struct pci_root_info *info = bridge->release_data;
-
- resource_list_for_each_entry(entry, &bridge->windows) {
- res = entry->res;
- if (res->parent &&
- (res->flags & (IORESOURCE_MEM | IORESOURCE_IO)))
- release_resource(res);
- }
-
- teardown_mcfg_map(info);
- kfree(info);
+ teardown_mcfg_map(ci);
+ kfree(ci);
Implicit pointer cast? I think add a comment here that %p of ci
is same as %p of info would be helpful for review and easy understood.
other than that,
Reviewed-by: Hanjun Guo <[email protected]>
Thanks
Hanjun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
