Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48079 )
Change subject: mainboard/intel/adlrvp/spd: Update SPD for LP4x SKU
......................................................................
Patch Set 2: Code-Review+1
(1 comment)
https://review.coreboot.org/c/coreboot/+/48079/2//COMMIT_MSG
Commit Message:
https://review.coreboot.org/c/coreboot/+/48079/2//COMMIT_MSG@10
PS2, Line 10: 1. tCKAVGmax
: 2. CAS Latencies supported (First Byte)
: 3. CAS Latencies supported (Second Byte)
: 4. CAS Latencies supported (Third Byte)
: 5. Minimum CAS Latency (tAAmin)
: 6. Read and Write Latency Set options
: 7. FTB for tAAmin
: 8. FTB for tCKAVGmax
: 9. FTB for tCKAVGmin
Instead of listing the low-level changes, how about explaining the high-level changes? For example:
1. Raise/Lower tCKAVGmax from X ns to Y ns
--
To view, visit
https://review.coreboot.org/c/coreboot/+/48079
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iae7f2693e87bffb2dfa20bd07b22f4a4768c56cb
Gerrit-Change-Number: 48079
Gerrit-PatchSet: 2
Gerrit-Owner: Subrata Banik
subrata.banik@intel.com
Gerrit-Reviewer: Aamir Bohra
aamir.bohra@intel.com
Gerrit-Reviewer: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: Furquan Shaikh
furquan@google.com
Gerrit-Reviewer: Sridhar Siricilla
sridhar.siricilla@intel.com
Gerrit-Reviewer: Tim Wawrzynczak
twawrzynczak@chromium.org
Gerrit-Reviewer: V Sowmya
v.sowmya@intel.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Comment-Date: Sat, 28 Nov 2020 14:44:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment