[U-Boot] [NEXT PATCH v1 4/7] MX35: Add soc_boot_mode and soc_boot_device to MX35

Stefano Babic
2012-09-24T08:35:15Z

On 23/09/2012 21:25, Eric B?nard wrote:
> Hi Stefano,
>
> Le Thu, 6 Sep 2012 10:04:57 +0200,
> Stefano Babic <sbabic@denx.de> a ?crit :
>> +#define RCSR_MEM_CTL_WEIM 0
>> +#define RCSR_MEM_CTL_NAND 1
>> +#define RCSR_MEM_CTL_SD 2
>> +#define RCSR_MEM_TYPE_NOR 0
>> +#define RCSR_MEM_TYPE_ONENAND 2
>> +#define RCSR_MEM_TYPE_SD 0
>> +#define RCSR_MEM_TYPE_I2C 2
>> +#define RCSR_MEM_TYPE_SPI 3
>> +
>> +u32 spl_boot_device(void)
>> +{
>> + puts("spl_boot_device\n");
>> + struct ccm_regs *ccm =
>> + (struct ccm_regs *)IMX_CCM_BASE;
>> +
>> +#if 1
>> + return BOOT_DEVICE_MMC1;
>> +#endif
>
> thisseems not clean and seems caused by the fact that the define
> RCSR_MEM_CTL_SD should be 3 and not 2 so in your tests the function
> spl_boot_device was not detecting the right boot mode. Also IMHO this
> define should be named RCSR_MEM_CTL_EXPANSION as in the app note AN3996
> at end of page 3 (there is a typo in the reference manual which seems
> to be a copy'n paste from i.MX25 as it doesn't take in acount the ATA
> HDD case) :
> http://cache.freescale.com/files/dsp/doc/app_note/AN3996.pdf

You're right, thanks ! I will fix in V2

Regards,
Stefano

--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================