[U-Boot] [PATCH 6/6] i.mx: i.mx6q: Add the initial support for i.mx6q ARM2 board
Jason Hui
jason.hui at linaro.org
Mon Nov 14 10:10:36 CET 2011
On Sun, Nov 13, 2011 at 4:13 AM, Fabio Estevam <festevam at gmail.com> wrote:
> On Sat, Nov 12, 2011 at 8:36 AM, Jason Liu <jason.hui at linaro.org> wrote:
>> +int board_mmc_getcd(u8 *cd, struct mmc *mmc)
>> +{
>> + struct fsl_esdhc_cfg *cfg = (struct fsl_esdhc_cfg *)mmc->priv;
>> +
>> + if (cfg->esdhc_base == USDHC3_BASE_ADDR)
>> + *cd = gpio_get_value(107); /*GPIO6_11*/
>
> GPIO6_11 number is 5*32 + 11 = 171 , not 107.
yes, some mistake. I will fix it later. thanks;
>
> Also, in order to use gpio_get_value you need to configure the pin as
> GPIO first.
yes, correct.
>
> ...
>> +#define CONFIG_SYS_SDRAM_BASE (PHYS_SDRAM)
>> +#define CONFIG_SYS_INIT_RAM_ADDR (IRAM_BASE_ADDR)
>> +#define CONFIG_SYS_INIT_RAM_SIZE (IRAM_SIZE)
>
> Unneeded parentheses.
Ok, I will remove it. Thanks for the review.
Jason Liu
>
> Regards,
>
> Fabio Estevam
>
More information about the U-Boot
mailing list