diff options
author | Bjorn Helgaas <bjorn.helgaas@hp.com> | 2009-09-14 18:35:05 -0400 |
---|---|---|
committer | Jesse Barnes <jbarnes@virtuousgeek.org> | 2009-09-14 20:38:19 -0400 |
commit | dffb4bb560ed73937a52c68c304b232c743b49f7 (patch) | |
tree | f2f0901b3a90c5a66ea140d1d7de52d1c7ba7924 | |
parent | 4b77b0a2ba27d64f58f16d8d4d48d8319dda36ff (diff) |
PCI hotplug: acpiphp: remove superfluous _HPP/_HPX evaluation
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>
-rw-r--r-- | drivers/pci/hotplug/acpiphp_glue.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/pci/hotplug/acpiphp_glue.c b/drivers/pci/hotplug/acpiphp_glue.c index 680c33635b6f..e0623373615f 100644 --- a/drivers/pci/hotplug/acpiphp_glue.c +++ b/drivers/pci/hotplug/acpiphp_glue.c | |||
@@ -300,9 +300,6 @@ static void init_bridge_misc(struct acpiphp_bridge *bridge) | |||
300 | { | 300 | { |
301 | acpi_status status; | 301 | acpi_status status; |
302 | 302 | ||
303 | /* decode ACPI 2.0 _HPP (hot plug parameters) */ | ||
304 | decode_hpp(bridge); | ||
305 | |||
306 | /* must be added to the list prior to calling register_slot */ | 303 | /* must be added to the list prior to calling register_slot */ |
307 | list_add(&bridge->list, &bridge_list); | 304 | list_add(&bridge->list, &bridge_list); |
308 | 305 | ||