diff options
author | York Sun <yorksun@freescale.com> | 2015-07-23 14:04:48 -0700 |
---|---|---|
committer | York Sun <yorksun@freescale.com> | 2015-08-03 12:06:38 -0700 |
commit | 56848428a88f89420ae7acc04bb5805e70c430a3 (patch) | |
tree | dacf689050c9dacae1c7ad899704298f6a7626c3 /drivers/ddr/fsl/options.c | |
parent | 14d5547cf158c18bc340f01424e011b0802a6bb0 (diff) | |
download | u-boot-imx-56848428a88f89420ae7acc04bb5805e70c430a3.zip u-boot-imx-56848428a88f89420ae7acc04bb5805e70c430a3.tar.gz u-boot-imx-56848428a88f89420ae7acc04bb5805e70c430a3.tar.bz2 |
drivers/ddr/fsl: Adjust bstopre value
By default the bstopre value has been set to 0x100, used to be 1/4
value of refint. Modern DDR has increased the refresh time. Adjust
to 1/4 of refresh interval dynamically. Individual board can still
override this value in board ddr file, or to use auto-precharge.
Signed-off-by: York Sun <yorksun@freescale.com>
Diffstat (limited to 'drivers/ddr/fsl/options.c')
-rw-r--r-- | drivers/ddr/fsl/options.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/drivers/ddr/fsl/options.c b/drivers/ddr/fsl/options.c index 3b30fa2..3c09c64 100644 --- a/drivers/ddr/fsl/options.c +++ b/drivers/ddr/fsl/options.c @@ -499,7 +499,7 @@ static inline unsigned int auto_bank_intlv(dimm_params_t *pdimm) return 0; } -unsigned int populate_memctl_options(int all_dimms_registered, +unsigned int populate_memctl_options(const common_timing_params_t *common_dimm, memctl_options_t *popts, dimm_params_t *pdimm, unsigned int ctrl_num) @@ -640,7 +640,7 @@ unsigned int populate_memctl_options(int all_dimms_registered, popts->ba_intlv_ctl = 0; /* Memory Organization Parameters */ - popts->registered_dimm_en = all_dimms_registered; + popts->registered_dimm_en = common_dimm->all_dimms_registered; /* Operational Mode Paramters */ @@ -778,9 +778,11 @@ unsigned int populate_memctl_options(int all_dimms_registered, * Set this to 0 for global auto precharge * The value of 0x100 has been used for DDR1, DDR2, DDR3. * It is not wrong. Any value should be OK. The performance depends on - * applications. There is no one good value for all. + * applications. There is no one good value for all. One way to set + * is to use 1/4 of refint value. */ - popts->bstopre = 0x100; + popts->bstopre = picos_to_mclk(ctrl_num, common_dimm->refresh_rate_ps) + >> 2; /* * Window for four activates -- tFAW |