From ec38570ead077f6fe37c017c857915ea6cca3951 Mon Sep 17 00:00:00 2001 From: Angel Pons Date: Tue, 12 Jan 2021 21:08:27 +0100 Subject: nb/intel/sandybridge: Correct description of QCLK QCLK means "quadrature clock", and is equivalent to one half of a full clock cycle (tCK). Fix the comment. The `QCLK_PI` value is still valid. Change-Id: I7089fc32381addc280a71761a377075f107b5c62 Signed-off-by: Angel Pons Reviewed-on: https://review.coreboot.org/c/coreboot/+/49363 Reviewed-by: Patrick Rudolph Reviewed-by: Paul Menzel Tested-by: build bot (Jenkins) --- src/northbridge/intel/sandybridge/raminit_common.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'src/northbridge/intel/sandybridge') diff --git a/src/northbridge/intel/sandybridge/raminit_common.h b/src/northbridge/intel/sandybridge/raminit_common.h index c43378d7f1..b93902aa38 100644 --- a/src/northbridge/intel/sandybridge/raminit_common.h +++ b/src/northbridge/intel/sandybridge/raminit_common.h @@ -433,7 +433,8 @@ typedef struct ramctr_timing_st { #define MIN_CAS 4 /* - * 1 QCLK (quarter of a clock cycle) equals 64 PI (phase interpolator) ticks. + * 1 QCLK (quadrature clock) is one half of a full clock cycle (tCK). + * In addition, 64 PI (phase interpolator) ticks are equal to 1 QCLK. * Logic delay values in I/O register bitfields are expressed in QCLKs. */ #define QCLK_PI 64 -- cgit v1.2.3