tree da0a838b50c0ad3578ad67a137bee214b916d709
parent 581e43ef5de6cc3369b2496362348e38bc85ded0
author Bjorn Helgaas <bhelgaas@google.com> 1675454249 -0600
committer Bjorn Helgaas <bhelgaas@google.com> 1675717764 -0600

Revert "PCI/ASPM: Save L1 PM Substates Capability for suspend/resume"

This reverts commit 4ff116d0d5fd8a025604b0802d93a2d5f4e465d1.

Thomas Witt reported that 5e85eba6f50d ("PCI/ASPM: Refactor L1 PM Substates
Control Register programming") broke suspend/resume.

4ff116d0d5fd ("PCI/ASPM: Save L1 PM Substates Capability for
suspend/resume") uses a function added by 5e85eba6f50d, so revert it first.

Note that reverting 4ff116d0d5fd means L1 Substates config will be lost on
suspend/resume.  As far as we know the box will use more power but will
still *work* correctly.

Fixes: 5e85eba6f50d ("PCI/ASPM: Refactor L1 PM Substates Control Register programming")
Link: https://bugzilla.kernel.org/show_bug.cgi?id=216877
Reported-by: Thomas Witt <kernel@witt.link>
Tested-by: Thomas Witt <kernel@witt.link>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Cc: Vidya Sagar <vidyas@nvidia.com>