On 08/13/2013 11:44 PM, York Sun wrote:
> On 07/26/2013 03:02 AM, Valentin Longchamp wrote:
>> +
>> +phys_size_t initdram(int board_type)
>> +{
>> + phys_size_t dram_size = 0;
>> +
>> + puts("Initializing....");
>> +
>> + if (fsl_use_spd()) {
>> + puts("using SPD\n");
>> + dram_size = fsl_ddr_sdram();
>> + } else {
>> + puts("no SPD and fixed parameters\n");
>> + return dram_size;
>> + }
>
>
> How does the else clause work? You probably want to put an error or
> panic here if you don't have other way to initialize it.
>
OK I agree.
Valentin
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
