[U-Boot] [PATCH 1/4] mmc: fsl_esdhc: make GPIO support optional
Jaehoon Chung
jh80.chung at samsung.com
Thu Jan 5 01:33:15 CET 2017
Hi,
On 01/05/2017 04:52 AM, york sun wrote:
> On 12/06/2016 08:08 PM, Yangbo Lu wrote:
>> There would be compiling error as below when enable driver model for esdhc.
>> undefined reference to `dm_gpio_get_value'
>> undefined reference to `gpio_request_by_name_nodev'
>> This patch is to make GPIO support optional with CONFIG_DM_GPIO. Because
>> all boards of QorIQ platform don't need it and they just check register for
>> CD/WP status, only some boards of i.MX platform require this.
>>
>> Signed-off-by: Yangbo Lu <yangbo.lu at nxp.com>
>> ---
>> drivers/mmc/fsl_esdhc.c | 10 ++++++++--
>> 1 file changed, 8 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/mmc/fsl_esdhc.c b/drivers/mmc/fsl_esdhc.c
>> index 9796d39..da3a151 100644
>> --- a/drivers/mmc/fsl_esdhc.c
>> +++ b/drivers/mmc/fsl_esdhc.c
>> @@ -104,8 +104,10 @@ struct fsl_esdhc_priv {
>> struct udevice *dev;
>> int non_removable;
>> int wp_enable;
>> +#ifdef CONFIG_DM_GPIO
>> struct gpio_desc cd_gpio;
>> struct gpio_desc wp_gpio;
>> +#endif
>> };
>>
>> /* Return the XFERTYP flags for a given command and data packet */
>> @@ -687,10 +689,11 @@ static int esdhc_getcd(struct mmc *mmc)
>> #ifdef CONFIG_DM_MMC
>> if (priv->non_removable)
>> return 1;
>> -
>> +#ifdef CONFIG_DM_GPIO
>> if (dm_gpio_is_valid(&priv->cd_gpio))
>> return dm_gpio_get_value(&priv->cd_gpio);
>> #endif
>> +#endif
>>
>> while (!(esdhc_read32(®s->prsstat) & PRSSTAT_CINS) && --timeout)
>> udelay(1000);
>> @@ -967,17 +970,20 @@ static int fsl_esdhc_probe(struct udevice *dev)
>> priv->non_removable = 1;
>> } else {
>> priv->non_removable = 0;
>> +#ifdef CONFIG_DM_GPIO
>> gpio_request_by_name_nodev(fdt, node, "cd-gpios", 0,
>> &priv->cd_gpio, GPIOD_IS_IN);
>> +#endif
>> }
>>
>> priv->wp_enable = 1;
>>
>> +#ifdef CONFIG_DM_GPIO
>> ret = gpio_request_by_name_nodev(fdt, node, "wp-gpios", 0,
>> &priv->wp_gpio, GPIOD_IS_IN);
>> if (ret)
>> priv->wp_enable = 0;
>> -
>> +#endif
>> /*
>> * TODO:
>> * Because lack of clk driver, if SDHC clk is not enabled,
>>
>
> Jaehoon,
>
> This set looks OK to me. Please review and comment. I can merge it with
> your ack.
Sorry for late. I missed this patch. It looks good to me.
Acked-by: Jaehoon Chung <jh80.chung at samsung.com>
Best Regards,
Jaehoon Chung
>
> Yangbo,
>
> For MMC or SD patches, please CC Jaehoon.
>
> York
>
>
>
More information about the U-Boot
mailing list