Duncan Laurie has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47682 )
Change subject: drivers/genesyslogic/gl9755: Adjust L1 exit latency to enable ASPM
......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/47682/2//COMMIT_MSG
Commit Message:
https://review.coreboot.org/c/coreboot/+/47682/2//COMMIT_MSG@9
PS2, Line 9: set the latency to <64us
Where is that limit documented, that means, where does it come from?
The PCI spec defines the possible values for the exit latency, 64us just happens to be what was recommended by genesys.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/47682
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ie2b85a6697f164fbe4f84d8cd5acb2b5911ca7a9
Gerrit-Change-Number: 47682
Gerrit-PatchSet: 2
Gerrit-Owner: Duncan Laurie
dlaurie@chromium.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Tue, 17 Nov 2020 21:05:00 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-MessageType: comment