Messages in this thread | | | Date | Fri, 13 Jan 2012 21:24:09 +0000 | From | Matthew Garrett <> | Subject | Re: [git pull] PCI changes |
| |
On Wed, Jan 11, 2012 at 07:38:25PM -0800, Linus Torvalds wrote: > On Wed, Jan 11, 2012 at 10:34 AM, Jesse Barnes <jbarnes@virtuousgeek.org> wrote: > > > > Matthew Garrett (1): > > PCI: Rework ASPM disable code > > Yay. > > However, looking this through some more, I'm still a bit confused by > the pcie_clear_aspm() call in acpi_pci_root_add(). > > It seems to be explicitly against the rules elsewhere - we clear ASPM > even though ACPI_FADT_NO_ASPM is set - which the changelog (and the > other parts of that commit) says means "don't touch pre-existing ASPM > state"). > > So why is acpi_pci_root_add() special? Cna you explain that part to me?
pcie_no_aspm() means "Do not permit ASPM to be enabled" - it doesn't alter the existing state. pcie_clear_aspm() does that.
-- Matthew Garrett | mjg59@srcf.ucam.org -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |