]> git.proxmox.com Git - mirror_ubuntu-disco-kernel.git/commitdiff
PCI hotplug: acpiphp: remove superfluous _HPP/_HPX evaluation
authorBjorn Helgaas <bjorn.helgaas@hp.com>
Mon, 14 Sep 2009 22:35:05 +0000 (16:35 -0600)
committerJesse Barnes <jbarnes@virtuousgeek.org>
Tue, 15 Sep 2009 00:38:19 +0000 (17:38 -0700)
decode_hpp() looks up hotplug PCI parameters from ACPI and saves them
in the acpiphp_bridge structure.  These parameters (in bridge->hpp) are
only used by the acpiphp_set_hpp_values() -> program_hpp() path.  In
that path, we always call decode_hpp() before program_hpp(), so there's
no need to do it in init_bridge_misc().

Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com>
Reviewed-by: Alex Chiang <achiang@hp.com>
Reviewed-by: Kenji Kaneshige <kaneshige.kenji@jp.fujitsu.com>
Acked-by: Kenji Kaneshige <kaneshige.kenji@jp.fujitsu.com>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
drivers/pci/hotplug/acpiphp_glue.c

index 680c33635b6f3a21e3824f81fc2b7e7be678aab3..e0623373615fc5f46cd9c22e4b4b591085cec357 100644 (file)
@@ -300,9 +300,6 @@ static void init_bridge_misc(struct acpiphp_bridge *bridge)
 {
        acpi_status status;
 
-       /* decode ACPI 2.0 _HPP (hot plug parameters) */
-       decode_hpp(bridge);
-
        /* must be added to the list prior to calling register_slot */
        list_add(&bridge->list, &bridge_list);