integralfx / MemTestHelper

C# WPF to automate HCI MemTest
2.13k stars 214 forks source link

Fix order of tuning timings that depend on other timings #78

Closed HarrySIsNotAvailable closed 6 months ago

HarrySIsNotAvailable commented 6 months ago

Since tWR depends on tCWL and tCWL depends on tCL in the guide, the order of tuning should be changed from:

  1. tWR
  2. tCWL
  3. tCL

to:

  1. tCL
  2. tCWL
  3. tWR

This better aligns with the formulas and recommendations stated in the guide currently. I'm surprised nobody has mentioned this before. I'm sure this would confuse beginners anyway.

HarrySIsNotAvailable commented 6 months ago

As per Eden's guide:

Note that if you do want to tune CWL, you should do so before the later tRDWR and tWRRD timings; “leaving it for last” will complicate things

Hence, I have moved tuning tCWL before tWTR (which involves changing tRDWR and tWRRD)