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: linux-pci@vger.kernel.org, linux-pm@lists.linux-foundation.org,
	ACPI Devel Maling List <linux-acpi@vger.kernel.org>,
	Jesse Barnes <jbarnes@virtuousgeek.org>,
	Kenji Kaneshige <kaneshige.kenji@jp.fujitsu.com>
Subject: [PATCH 1/10] ACPI / PCI: Introduce function for querying PCI root _OSC
Date: Fri, 6 Aug 2010 03:05:19 +0200	[thread overview]
Message-ID: <201008060305.19383.rjw__7494.8635338389$1281058010$gmane$org@sisk.pl> (raw)
In-Reply-To: <201008060303.51224.rjw@sisk.pl>

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

Introduce a function allowing the caller to obtain a mask of _OSC
control bits the BIOS will allow the kernel to control for a given
PCI root bridge.

Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
---
 drivers/acpi/pci_root.c |   77 +++++++++++++++++++++++++++++++++++++++++-------
 include/linux/acpi.h    |    1 
 2 files changed, 67 insertions(+), 11 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
@@ -225,22 +225,35 @@ static acpi_status acpi_pci_run_osc(acpi
 	return status;
 }
 
-static acpi_status acpi_pci_query_osc(struct acpi_pci_root *root, u32 flags)
+static acpi_status acpi_pci_query_osc(struct acpi_pci_root *root,
+					u32 support,
+					u32 *control)
 {
 	acpi_status status;
-	u32 support_set, result, capbuf[3];
+	u32 result, capbuf[3];
+
+	support &= OSC_PCI_SUPPORT_MASKS;
+	support |= root->osc_support_set;
 
-	/* do _OSC query for all possible controls */
-	support_set = root->osc_support_set | (flags & OSC_PCI_SUPPORT_MASKS);
 	capbuf[OSC_QUERY_TYPE] = OSC_QUERY_ENABLE;
-	capbuf[OSC_SUPPORT_TYPE] = support_set;
-	capbuf[OSC_CONTROL_TYPE] = OSC_PCI_CONTROL_MASKS;
+	capbuf[OSC_SUPPORT_TYPE] = support;
+	if (control) {
+		*control &= OSC_PCI_CONTROL_MASKS;
+		capbuf[OSC_CONTROL_TYPE] = *control | root->osc_control_set;
+	} else {
+		/* Run _OSC query for all possible controls. */
+		capbuf[OSC_CONTROL_TYPE] = OSC_PCI_CONTROL_MASKS;
+	}
 
 	status = acpi_pci_run_osc(root->device->handle, capbuf, &result);
 	if (ACPI_SUCCESS(status)) {
-		root->osc_support_set = support_set;
-		root->osc_control_qry = result;
-		root->osc_queried = 1;
+		root->osc_support_set = support;
+		if (control) {
+			*control = result;
+		} else {
+			root->osc_control_qry = result;
+			root->osc_queried = 1;
+		}
 	}
 	return status;
 }
@@ -254,7 +267,7 @@ static acpi_status acpi_pci_osc_support(
 	if (ACPI_FAILURE(status))
 		return status;
 	mutex_lock(&osc_lock);
-	status = acpi_pci_query_osc(root, flags);
+	status = acpi_pci_query_osc(root, flags, NULL);
 	mutex_unlock(&osc_lock);
 	return status;
 }
@@ -363,6 +376,48 @@ out:
 }
 EXPORT_SYMBOL_GPL(acpi_get_pci_dev);
 
+ /**
+ * acpi_pci_osc_control_query - Get the _OSC bits the kernel can control.
+ * @handle: ACPI handle of a PCI root bridge (or PCIe Root Complex).
+ * @mask: Mask of _OSC bits to query and the place to put the result into.
+ **/
+acpi_status acpi_pci_osc_control_query(acpi_handle handle, u32 *mask)
+{
+	struct acpi_pci_root *root;
+	acpi_handle tmp;
+	acpi_status status = AE_OK;
+	u32 ctrl = *mask;
+
+	if (!(ctrl & OSC_PCI_CONTROL_MASKS))
+		return AE_BAD_PARAMETER;
+
+	root = acpi_pci_find_root(handle);
+	if (!root)
+		return AE_NOT_EXIST;
+
+	status = acpi_get_handle(handle, "_OSC", &tmp);
+	if (ACPI_FAILURE(status))
+		return status;
+
+	mutex_lock(&osc_lock);
+
+	*mask |= root->osc_control_set;
+	if ((ctrl & root->osc_control_set) == ctrl)
+		goto out;
+
+	for (ctrl = 0; *mask != ctrl; ctrl = *mask) {
+		status = acpi_pci_query_osc(root, root->osc_support_set, mask);
+		if (ACPI_FAILURE(status))
+			break;
+	}
+
+ out:
+	mutex_unlock(&osc_lock);
+
+	return status;
+}
+EXPORT_SYMBOL(acpi_pci_osc_control_query);
+
 /**
  * acpi_pci_osc_control_set - commit requested control to Firmware
  * @handle: acpi_handle for the target ACPI object
@@ -396,7 +451,7 @@ acpi_status acpi_pci_osc_control_set(acp
 
 	/* Need to query controls first before requesting them */
 	if (!root->osc_queried) {
-		status = acpi_pci_query_osc(root, root->osc_support_set);
+		status = acpi_pci_query_osc(root, root->osc_support_set, NULL);
 		if (ACPI_FAILURE(status))
 			goto out;
 	}
Index: linux-2.6/include/linux/acpi.h
===================================================================
--- linux-2.6.orig/include/linux/acpi.h
+++ linux-2.6/include/linux/acpi.h
@@ -305,6 +305,7 @@ acpi_status acpi_run_osc(acpi_handle han
 				OSC_PCI_EXPRESS_AER_CONTROL |		\
 				OSC_PCI_EXPRESS_CAP_STRUCTURE_CONTROL)
 
+extern acpi_status acpi_pci_osc_control_query(acpi_handle handle, u32 *mask);
 extern acpi_status acpi_pci_osc_control_set(acpi_handle handle, u32 flags);
 extern void acpi_early_init(void);
 

  parent reply	other threads:[~2010-08-06  1:05 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 [this message]
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
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='201008060305.19383.rjw__7494.8635338389$1281058010$gmane$org@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.