diff options
author | Rajesh Shah <rajesh.shah@intel.com> | 2005-04-28 03:25:55 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2005-06-28 00:52:43 -0400 |
commit | 2f523b15901f654a9448bbd47ebe1e783ec3195b (patch) | |
tree | 74270f9c16021a5b4accbaadddb50475e3e44701 /drivers/pci | |
parent | 364d5094a43ff2ceff3d19e40c4199771cb6cb8f (diff) |
[PATCH] acpi hotplug: fix slot power-down problem with acpiphp
Earlier I reported that Matthew's acpiphp rewrite had problem in powering down
slot on my i386 system. The following patch is needed to get the acpiphp
rewrite properly powering down the slot.
Signed-off-by: Dely Sy <dely.l.sy@intel.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/pci')
-rw-r--r-- | drivers/pci/hotplug/acpiphp_glue.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/pci/hotplug/acpiphp_glue.c b/drivers/pci/hotplug/acpiphp_glue.c index 2f72e50eca2f..1501eb26af33 100644 --- a/drivers/pci/hotplug/acpiphp_glue.c +++ b/drivers/pci/hotplug/acpiphp_glue.c | |||
@@ -600,7 +600,7 @@ static int power_off_slot(struct acpiphp_slot *slot) | |||
600 | list_for_each (l, &slot->funcs) { | 600 | list_for_each (l, &slot->funcs) { |
601 | func = list_entry(l, struct acpiphp_func, sibling); | 601 | func = list_entry(l, struct acpiphp_func, sibling); |
602 | 602 | ||
603 | if (func->pci_dev && (func->flags & FUNC_HAS_PS3)) { | 603 | if (func->flags & FUNC_HAS_PS3) { |
604 | status = acpi_evaluate_object(func->handle, "_PS3", NULL, NULL); | 604 | status = acpi_evaluate_object(func->handle, "_PS3", NULL, NULL); |
605 | if (ACPI_FAILURE(status)) { | 605 | if (ACPI_FAILURE(status)) { |
606 | warn("%s: _PS3 failed\n", __FUNCTION__); | 606 | warn("%s: _PS3 failed\n", __FUNCTION__); |
@@ -615,7 +615,7 @@ static int power_off_slot(struct acpiphp_slot *slot) | |||
615 | func = list_entry(l, struct acpiphp_func, sibling); | 615 | func = list_entry(l, struct acpiphp_func, sibling); |
616 | 616 | ||
617 | /* We don't want to call _EJ0 on non-existing functions. */ | 617 | /* We don't want to call _EJ0 on non-existing functions. */ |
618 | if (func->pci_dev && (func->flags & FUNC_HAS_EJ0)) { | 618 | if (func->flags & FUNC_HAS_EJ0) { |
619 | /* _EJ0 method take one argument */ | 619 | /* _EJ0 method take one argument */ |
620 | arg_list.count = 1; | 620 | arg_list.count = 1; |
621 | arg_list.pointer = &arg; | 621 | arg_list.pointer = &arg; |