+++ /dev/null
-From 4e39432f4df544d3dfe4fc90a22d87de64d15815 Mon Sep 17 00:00:00 2001
-From: Taku Izumi <izumi.taku@jp.fujitsu.com>
-Date: Fri, 17 Oct 2008 13:49:46 +0900
-Subject: ACPI/PCI: Change pci_osc_control_set() to query control bits first
-Patch-mainline: 2.6.28
-References: bnc#438941
-
-Current pci_osc_control_set() evaluates _OSC without query for control
-bits, unless __pci_osc_support_set() is called beforehand. But as
-strongly recommended in PCI firmware specification, it should query
-control bits first.
-
-This patch changes pci_osc_control_set() to query control bits first
-even if __pci_osc_support_set() is not called beforehand.
-
-Signed-off-by: Kenji Kaneshige <kaneshige.kenji@jp.fujitsu.com>
-Signed-off-by: Taku Izumi <izumi.taku@jp.fujitsu.com>
-Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
-Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
----
- drivers/pci/pci-acpi.c | 47 +++++++++++++++++++++++++++++++----------------
- 1 file changed, 31 insertions(+), 16 deletions(-)
-
---- a/drivers/pci/pci-acpi.c
-+++ b/drivers/pci/pci-acpi.c
-@@ -120,14 +120,35 @@ out_kfree:
- return status;
- }
-
-+static acpi_status __acpi_query_osc(u32 flags, struct acpi_osc_data *osc_data)
-+{
-+ acpi_status status;
-+ u32 support_set;
-+ struct acpi_osc_args osc_args;
-+
-+ /* do _OSC query for all possible controls */
-+ support_set = osc_data->support_set | (flags & OSC_SUPPORT_MASKS);
-+ osc_args.capbuf[OSC_QUERY_TYPE] = OSC_QUERY_ENABLE;
-+ osc_args.capbuf[OSC_SUPPORT_TYPE] = support_set;
-+ osc_args.capbuf[OSC_CONTROL_TYPE] = OSC_CONTROL_MASKS;
-+
-+ status = acpi_run_osc(osc_data->handle, &osc_args);
-+ if (ACPI_SUCCESS(status)) {
-+ osc_data->support_set = support_set;
-+ osc_data->query_result = osc_args.query_result;
-+ osc_data->is_queried = 1;
-+ }
-+
-+ return status;
-+}
-+
- static acpi_status acpi_query_osc(acpi_handle handle,
- u32 level, void *context, void **retval)
- {
- acpi_status status;
- struct acpi_osc_data *osc_data;
-- u32 flags = (unsigned long)context, support_set;
-+ u32 flags = (unsigned long)context;
- acpi_handle tmp;
-- struct acpi_osc_args osc_args;
-
- status = acpi_get_handle(handle, "_OSC", &tmp);
- if (ACPI_FAILURE(status))
-@@ -141,18 +162,7 @@ static acpi_status acpi_query_osc(acpi_h
- goto out;
- }
-
-- /* do _OSC query for all possible controls */
-- support_set = osc_data->support_set | (flags & OSC_SUPPORT_MASKS);
-- osc_args.capbuf[OSC_QUERY_TYPE] = OSC_QUERY_ENABLE;
-- osc_args.capbuf[OSC_SUPPORT_TYPE] = support_set;
-- osc_args.capbuf[OSC_CONTROL_TYPE] = OSC_CONTROL_MASKS;
--
-- status = acpi_run_osc(handle, &osc_args);
-- if (ACPI_SUCCESS(status)) {
-- osc_data->support_set = support_set;
-- osc_data->query_result = osc_args.query_result;
-- osc_data->is_queried = 1;
-- }
-+ status = __acpi_query_osc(flags, osc_data);
- out:
- mutex_unlock(&pci_acpi_lock);
- return status;
-@@ -209,8 +219,13 @@ acpi_status pci_osc_control_set(acpi_han
- goto out;
- }
-
-- if (osc_data->is_queried &&
-- ((osc_data->query_result & ctrlset) != ctrlset)) {
-+ if (!osc_data->is_queried) {
-+ status = __acpi_query_osc(osc_data->support_set, osc_data);
-+ if (ACPI_FAILURE(status))
-+ goto out;
-+ }
-+
-+ if ((osc_data->query_result & ctrlset) != ctrlset) {
- status = AE_SUPPORT;
- goto out;
- }