Previously, the format of messages printed after negotiating _OSC
depended on whether the entire operation was considered to be a success
or failure. Now, printed messages are homogenized to always show what
was requested versus what was granted.
Previous output (success):
acpi PNP0A08:00: _OSC: OS now controls [PME AER PCIeCapability LTR]
Previous output (failure):
acpi PNP0A08:00: _OSC: OS requested [PME AER PCIeCapability LTR]
acpi PNP0A08:00: _OSC: platform willing to grant []
New output:
acpi PNP0A08:00: _OSC: OS requested [PME AER PCIeCapability LTR]
acpi PNP0A08:00: _OSC: platform granted [PME AER PCIeCapability LTR]
Signed-off-by: Aaron Sierra <asierra@xxxxxxxxxxx>
---
drivers/acpi/pci_root.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/acpi/pci_root.c b/drivers/acpi/pci_root.c
index 0d57f817ef1e..21aa56f9ca54 100644
--- a/drivers/acpi/pci_root.c
+++ b/drivers/acpi/pci_root.c
@@ -504,8 +504,9 @@ static void negotiate_os_control(struct acpi_pci_root *root, int *no_aspm,
requested = control;
status = acpi_pci_osc_control_set(handle, &control,
OSC_PCI_EXPRESS_CAPABILITY_CONTROL);
+ decode_osc_control(root, "OS requested", requested);
+ decode_osc_control(root, "platform granted", control);
if (ACPI_SUCCESS(status)) {
- decode_osc_control(root, "OS now controls", control);
if (acpi_gbl_FADT.boot_flags & ACPI_FADT_NO_ASPM) {
/*
* We have ASPM control, but the FADT indicates that
@@ -516,8 +517,6 @@ static void negotiate_os_control(struct acpi_pci_root *root, int *no_aspm,
*no_aspm = 1;
}
} else {
- decode_osc_control(root, "OS requested", requested);
- decode_osc_control(root, "platform willing to grant", control);
dev_info(&device->dev, "_OSC failed (%s); disabling ASPM\n",
acpi_format_exception(status));
--
2.17.1
