[U-Boot] [PATCH 1/4] mmc: fsl_esdhc: make GPIO support optional
Y.B. Lu
yangbo.lu at nxp.com
Thu Jan 5 07:14:15 CET 2017
> -----Original Message-----
> From: york sun
> Sent: Thursday, January 05, 2017 3:52 AM
> To: Y.B. Lu
> Cc: u-boot at lists.denx.de; Jaehoon Chung
> Subject: Re: [PATCH 1/4] mmc: fsl_esdhc: make GPIO support optional
>
> 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.
>
> Yangbo,
>
> For MMC or SD patches, please CC Jaehoon.
[Lu Yangbo-B47093] Ok, York. And sorry for that.
>
> York
More information about the U-Boot
mailing list