diff options
author | Angel Pons <th3fanbus@gmail.com> | 2021-01-13 18:23:41 +0100 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2021-02-07 22:36:48 +0000 |
commit | 9e58afef598a4efbb6471ec04be5f5979bab9a66 (patch) | |
tree | be5e2df2db1adcc3e5f37615a8accc33b0994a94 | |
parent | ffa2adf2ae3eb7bd17970acc6edb3c9397e8df4b (diff) |
nb/intel/x4x: Update write leveling comment
Fix a typo and do some style improvements.
Change-Id: Ibc7e1869faa6b9ae12a51b1c3d209bbd8e54b0d2
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/49402
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
-rw-r--r-- | src/northbridge/intel/x4x/dq_dqs.c | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/src/northbridge/intel/x4x/dq_dqs.c b/src/northbridge/intel/x4x/dq_dqs.c index 4722dfe7a6..0a65165271 100644 --- a/src/northbridge/intel/x4x/dq_dqs.c +++ b/src/northbridge/intel/x4x/dq_dqs.c @@ -746,19 +746,19 @@ static enum cb_err increment_to_dqs_edge(struct sysinfo *s, u8 channel, u8 rank) * DDR3 uses flyby topology where the clock signal takes a different path * than the data signal, to allow for better signal intergrity. * Therefore the delay on the data signals needs to account for this. - * This is done by sampleling the DQS write (tx) signal back over - * the DQ signal and looking for delay values where the sample transitions + * This is done by sampling the DQS write (tx) signal back over the DQ + * signal and looking for delay values where the sample transitions * from high to low. * Here the following is done: - * - enable write levelling on the first populated rank - * - disable output on other populated ranks - * - start from safe DQS (tx) delays (other transitions can be - * found at different starting values but are generally bad) + * - Enable write levelling on the first populated rank. + * - Disable output on other populated ranks. + * - Start from safe DQS (tx) delays. Other transitions can be + * found at different starting values but are generally bad. * - loop0: decrease DQS (tx) delays until low is sampled, * loop1: increase DQS (tx) delays until high is sampled, - * That way we are sure to hit a low-high transition - * - put all ranks in normal mode of operation again - * - note: All ranks need to be leveled together + * This way, we are sure to have hit a low-high transition. + * - Put all ranks in normal mode of operation again. + * Note: All ranks need to be leveled together. */ void search_write_leveling(struct sysinfo *s) { |