All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: Hidetoshi Seto <seto.hidetoshi@jp.fujitsu.com>
Cc: Kenji Kaneshige <kaneshige.kenji@jp.fujitsu.com>,
	Jesse Barnes <jbarnes@virtuousgeek.org>,
	linux-pci@vger.kernel.org,
	ACPI Devel Maling List <linux-acpi@vger.kernel.org>,
	linux-pm@lists.linux-foundation.org
Subject: [PATCH 8/10] ACPI / PCI: Do not preserve _OSC control bits returned by a query (v2)
Date: Fri, 6 Aug 2010 03:12:54 +0200	[thread overview]
Message-ID: <201008060312.54954.rjw@sisk.pl> (raw)
In-Reply-To: <201008060303.51224.rjw@sisk.pl>

From: Rafael J. Wysocki <rjw@sisk.pl>

All of the existing callers of acpi_pci_osc_control_set() also call
acpi_pci_osc_control_query() right prior to it, so they don't use the
preserved control bits returned by the last _OSC query for the given
PCI root bridge (or PCIe root complex).

For this reason the osc_control_qry and osc_queried members of
struct acpi_pci_root are not necessary any more, so drop them and
remove the remaining code using them.

Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
---
 drivers/acpi/pci_root.c |   19 +------------------
 include/acpi/acpi_bus.h |    3 ---
 2 files changed, 1 insertion(+), 21 deletions(-)

Index: linux-2.6/drivers/acpi/pci_root.c
===================================================================
--- linux-2.6.orig/drivers/acpi/pci_root.c
+++ linux-2.6/drivers/acpi/pci_root.c
@@ -248,12 +248,8 @@ static acpi_status acpi_pci_query_osc(st
 	status = acpi_pci_run_osc(root->device->handle, capbuf, &result);
 	if (ACPI_SUCCESS(status)) {
 		root->osc_support_set = support;
-		if (control) {
+		if (control)
 			*control = result;
-		} else {
-			root->osc_control_qry = result;
-			root->osc_queried = 1;
-		}
 	}
 	return status;
 }
@@ -449,19 +445,6 @@ acpi_status acpi_pci_osc_control_set(acp
 	if ((root->osc_control_set & control_req) == control_req)
 		goto out;
 
-	/* Need to query controls first before requesting them */
-	if (!root->osc_queried) {
-		status = acpi_pci_query_osc(root, root->osc_support_set, NULL);
-		if (ACPI_FAILURE(status))
-			goto out;
-	}
-	if ((root->osc_control_qry & control_req) != control_req) {
-		printk(KERN_DEBUG
-		       "Firmware did not grant requested _OSC control\n");
-		status = AE_SUPPORT;
-		goto out;
-	}
-
 	capbuf[OSC_QUERY_TYPE] = 0;
 	capbuf[OSC_SUPPORT_TYPE] = root->osc_support_set;
 	capbuf[OSC_CONTROL_TYPE] = root->osc_control_set | control_req;
Index: linux-2.6/include/acpi/acpi_bus.h
===================================================================
--- linux-2.6.orig/include/acpi/acpi_bus.h
+++ linux-2.6/include/acpi/acpi_bus.h
@@ -377,9 +377,6 @@ struct acpi_pci_root {
 
 	u32 osc_support_set;	/* _OSC state of support bits */
 	u32 osc_control_set;	/* _OSC state of control bits */
-	u32 osc_control_qry;	/* the latest _OSC query result */
-
-	u32 osc_queried:1;	/* has _OSC control been queried? */
 };
 
 /* helper */


  parent reply	other threads:[~2010-08-06  1:17 UTC|newest]

Thread overview: 61+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-06  1:03 [PATCH 0/10] ACPI / PCI / PCIe: Rework _OSC handling (v3) Rafael J. Wysocki
2010-08-06  1:05 ` [PATCH 1/10] ACPI / PCI: Introduce function for querying PCI root _OSC Rafael J. Wysocki
2010-08-06  1:05 ` Rafael J. Wysocki
2010-08-06  1:06 ` [PATCH 2/10] PCI / PCIe/ AER: Introduce pci_aer_available() Rafael J. Wysocki
2010-08-06  1:06 ` Rafael J. Wysocki
2010-08-06  1:07 ` [PATCH 3/10] PCI / PCIe: Introduce commad line switch for disabling port services Rafael J. Wysocki
2010-08-06  1:07 ` Rafael J. Wysocki
2010-08-06  1:08 ` [PATCH 4/10] PCI / PCIe: Ask BIOS for control of all native services at once (v6) Rafael J. Wysocki
2010-08-06  1:08 ` Rafael J. Wysocki
2010-08-06  1:09 ` [PATCH 5/10] PCI / PCIe: Disable PCIe port services during port initialization Rafael J. Wysocki
2010-08-06  1:09 ` Rafael J. Wysocki
2010-08-06  1:10 ` [PATCH 6/10] PCI / PCIe: Remove the port driver module exit routine Rafael J. Wysocki
2010-08-06  1:10 ` Rafael J. Wysocki
2010-08-06  1:11 ` [PATCH 7/10] PCI / Hot-plug: Query _OSC before requesting controls Rafael J. Wysocki
2010-08-06  2:20   ` Hidetoshi Seto
2010-08-06  2:20   ` Hidetoshi Seto
2010-08-06 10:34     ` Rafael J. Wysocki
2010-08-09  1:22       ` Kenji Kaneshige
2010-08-09  1:22       ` Kenji Kaneshige
2010-08-06 10:34     ` Rafael J. Wysocki
2010-08-06  1:11 ` Rafael J. Wysocki
2010-08-06  1:12 ` [PATCH 8/10] ACPI / PCI: Do not preserve _OSC control bits returned by a query (v2) Rafael J. Wysocki
2010-08-06  1:12 ` Rafael J. Wysocki [this message]
2010-08-06  1:13 ` [PATCH 9/10] ACPI / PCI: Reorder checks in acpi_pci_osc_control_set() Rafael J. Wysocki
2010-08-06  1:13 ` Rafael J. Wysocki
2010-08-06  1:15 ` [PATCH 10/10] ACPI / PCI: Merge acpi_pci_osc_control_{query|set}() Rafael J. Wysocki
2010-08-06  1:15 ` Rafael J. Wysocki
2010-08-06  3:25 ` [PATCH 0/10] ACPI / PCI / PCIe: Rework _OSC handling (v3) Hidetoshi Seto
2010-08-06 10:42   ` Rafael J. Wysocki
2010-08-06 10:42   ` Rafael J. Wysocki
2010-08-06  3:25 ` Hidetoshi Seto
2010-08-20 23:49 ` [PATCH 0/10] ACPI / PCI / PCIe: Rework _OSC handling (v4) Rafael J. Wysocki
2010-08-20 23:50   ` [PATCH 1/10] PCI / PCIe/ AER: Introduce pci_aer_available() Rafael J. Wysocki
2010-08-20 23:50   ` Rafael J. Wysocki
2010-08-24 20:49     ` Jesse Barnes
2010-08-24 20:49     ` Jesse Barnes
2010-08-20 23:51   ` [PATCH 2/10] PCI / PCIe: Introduce commad line switch for disabling port services Rafael J. Wysocki
2010-08-20 23:51   ` Rafael J. Wysocki
2010-08-20 23:52   ` [PATCH 3/10] ACPI / PCI: Reorder checks in acpi_pci_osc_control_set() Rafael J. Wysocki
2010-08-20 23:52   ` Rafael J. Wysocki
2010-08-20 23:53   ` [PATCH 4/10] ACPI / PCI: Make acpi_pci_query_osc() return control bits Rafael J. Wysocki
2010-08-20 23:53   ` Rafael J. Wysocki
2010-08-20 23:54   ` [PATCH 5/10] ACPI / PCI: Do not preserve _OSC control bits returned by a query (v4) Rafael J. Wysocki
2010-08-20 23:54   ` Rafael J. Wysocki
2010-08-23 21:55     ` [Updated changelog][PATCH " Rafael J. Wysocki
2010-08-23 21:55     ` Rafael J. Wysocki
2010-08-20 23:55   ` [PATCH 6/10] ACPI / PCI: Negotiate _OSC control bits before requesting them Rafael J. Wysocki
2010-08-20 23:55   ` Rafael J. Wysocki
2010-08-23 21:53     ` [Update][PATCH 6/10] ACPI / PCI: Negotiate _OSC control bits before requesting them (v2) Rafael J. Wysocki
2010-08-23 21:53     ` Rafael J. Wysocki
2010-08-20 23:56   ` [PATCH 7/10] PCI / PCIe: Ask BIOS for control of all native services at once (v7) Rafael J. Wysocki
2010-08-21 20:02     ` Rafael J. Wysocki
2010-08-21 20:02     ` [linux-pm] " Rafael J. Wysocki
2010-08-20 23:56   ` Rafael J. Wysocki
2010-08-20 23:57   ` [PATCH 8/10] PCI / PCIe: Disable PCIe port services during port initialization Rafael J. Wysocki
2010-08-20 23:57   ` Rafael J. Wysocki
2010-08-20 23:58   ` [PATCH 9/10] PCI / PCIe: Move PCIe PME code to the pcie directory Rafael J. Wysocki
2010-08-20 23:58   ` Rafael J. Wysocki
2010-08-20 23:59   ` [PATCH 10/10] PCI / PCIe: Remove the port driver module exit routine Rafael J. Wysocki
2010-08-20 23:59   ` Rafael J. Wysocki
2010-08-20 23:49 ` [PATCH 0/10] ACPI / PCI / PCIe: Rework _OSC handling (v4) Rafael J. Wysocki

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201008060312.54954.rjw@sisk.pl \
    --to=rjw@sisk.pl \
    --cc=jbarnes@virtuousgeek.org \
    --cc=kaneshige.kenji@jp.fujitsu.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=linux-pm@lists.linux-foundation.org \
    --cc=seto.hidetoshi@jp.fujitsu.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.