On 5/28/2020 7:42 AM, Rafael J. Wysocki wrote: >> diff --git a/drivers/acpi/pci_root.c b/drivers/acpi/pci_root.c >> index ac8ad6c..5140b26 100644 >> --- a/drivers/acpi/pci_root.c >> +++ b/drivers/acpi/pci_root.c >> @@ -456,7 +456,7 @@ static void negotiate_os_control(struct acpi_pci_root *root, int *no_aspm, >> >> dev_info(&device->dev, "_OSC failed (%s)%s\n", >> acpi_format_exception(status), >> - pcie_aspm_support_enabled() ? "; disabling ASPM" : ""); >> + pcie_aspm_support_enabled() ? "" : "; disabling ASPM"); >> return; >> } >> >> -- > Applied as 5.8 material under the "ACPI: PCI: Fix the ASPM part of the > _OSC failure message" subject and with a different changelog. I'm confused. The original change would print ASPM is getting disabled only when ASPM is supported. Now, we are printing disabling ASPM when ASPM is not supported. Now, we reverted the change and went back to incorrect behavior again. Am I missing something?
