[U-Boot] [PATCH] Disable unused chip-select for DDR controller interleaving

Wolfgang Denk wd at denx.de
Wed Oct 13 18:24:07 CEST 2010


Dear York Sun,

In message <1286983650-3911-1-git-send-email-yorksun at freescale.com> you wrote:
> When DDR controller interleaving is eabled and less than all bank (chip-select)
> interleaving is seletected, the unused chip-select should be disabled.
...

> @@ -1250,16 +1251,20 @@ compute_fsl_memctl_config_regs(const memctl_options_t *popts,
>  			 * and each controller uses rank interleaving within
>  			 * itself. Therefore the starting and ending address
>  			 * on each controller is twice the amount present on
> -			 * each controller.
> +			 * each controller.When
>  			 */

Seems you wanted to add some comment here, but then forgot it?

> @@ -1379,8 +1384,10 @@ compute_fsl_memctl_config_regs(const memctl_options_t *popts,
>  			);
>  
>  		debug("FSLDDR: cs[%d]_bnds = 0x%08x\n", i, ddr->cs[i].bnds);
> -		set_csn_config(dimm_number, i, ddr, popts, dimm_params);
> -		set_csn_config_2(i, ddr);
> +		if (cs_en) {
> +			set_csn_config(dimm_number, i, ddr, popts, dimm_params);
> +			set_csn_config_2(i, ddr);
> +		}

Would it not be helpful to inform the user that we disabled a memory
bank? Eventually this wa snot intentional...

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Knowledge, sir, should be free to all!
	-- Harry Mudd, "I, Mudd", stardate 4513.3


More information about the U-Boot mailing list