diff options
| author | Marek Vasut <marex@denx.de> | 2015-07-19 03:48:30 (GMT) |
|---|---|---|
| committer | Marek Vasut <marex@denx.de> | 2015-08-08 12:14:19 (GMT) |
| commit | 49891df661acf164460ba3ada60fccfc5a53ea71 (patch) | |
| tree | ee833a3c45f6b4e97cdc285bfeb883916abeb0d8 /drivers/ddr | |
| parent | a80f45346e77eee313df65187f3e1a8d2dcca971 (diff) | |
| download | u-boot-49891df661acf164460ba3ada60fccfc5a53ea71.tar.xz | |
ddr: altera: Clean up sdr_*_phase() part 9
The "d" variable is set to 0 in sdr_backup_phase() and is not used
at all in sdr_nonworking_phase(). Make it local and zap it respectively.
Signed-off-by: Marek Vasut <marex@denx.de>
Diffstat (limited to 'drivers/ddr')
| -rw-r--r-- | drivers/ddr/altera/sequencer.c | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/drivers/ddr/altera/sequencer.c b/drivers/ddr/altera/sequencer.c index e6fb576..e5dd81b 100644 --- a/drivers/ddr/altera/sequencer.c +++ b/drivers/ddr/altera/sequencer.c @@ -1403,12 +1403,11 @@ static int sdr_working_phase(uint32_t grp, uint32_t *work_bgn, return -EINVAL; } -static void sdr_backup_phase(uint32_t grp, - uint32_t *work_bgn, uint32_t *v, uint32_t *d, - uint32_t *p) +static void sdr_backup_phase(uint32_t grp, uint32_t *work_bgn, + uint32_t *v, uint32_t *p) { - uint32_t tmp_delay; - u32 bit_chk; + u32 tmp_delay; + u32 bit_chk, d; /* Special case code for backing up a phase */ if (*p == 0) { @@ -1420,9 +1419,8 @@ static void sdr_backup_phase(uint32_t grp, tmp_delay = *work_bgn - IO_DELAY_PER_OPA_TAP; scc_mgr_set_dqs_en_phase_all_ranks(grp, *p); - for (*d = 0; *d <= IO_DQS_EN_DELAY_MAX && tmp_delay < *work_bgn; - (*d)++, tmp_delay += IO_DELAY_PER_DQS_EN_DCHAIN_TAP) { - scc_mgr_set_dqs_en_delay_all_ranks(grp, *d); + for (d = 0; d <= IO_DQS_EN_DELAY_MAX && tmp_delay < *work_bgn; d++) { + scc_mgr_set_dqs_en_delay_all_ranks(grp, d); if (rw_mgr_mem_calibrate_read_test_all_ranks(grp, 1, PASS_ONE_BIT, @@ -1430,6 +1428,8 @@ static void sdr_backup_phase(uint32_t grp, *work_bgn = tmp_delay; break; } + + tmp_delay += IO_DELAY_PER_DQS_EN_DCHAIN_TAP; } /* @@ -1445,7 +1445,7 @@ static void sdr_backup_phase(uint32_t grp, scc_mgr_set_dqs_en_delay_all_ranks(grp, 0); } -static int sdr_nonworking_phase(uint32_t grp, uint32_t *v, uint32_t *d, +static int sdr_nonworking_phase(uint32_t grp, uint32_t *v, uint32_t *p, uint32_t *i, uint32_t *work_end) { int ret; @@ -1579,12 +1579,12 @@ static uint32_t rw_mgr_mem_calibrate_vfifo_find_dqs_en_phase(uint32_t grp) /* * step 3a: if we have room, back off by one and increment in dtaps * */ - sdr_backup_phase(grp, &work_bgn, &v, &d, &p); + sdr_backup_phase(grp, &work_bgn, &v, &p); /* ********************************************************* */ /* * step 4a: go forward from working phase to non working phase, increment in ptaps * */ - if (sdr_nonworking_phase(grp, &v, &d, &p, &i, &work_end)) + if (sdr_nonworking_phase(grp, &v, &p, &i, &work_end)) return 0; /* ********************************************************* */ |