[U-Boot] [PATCH 1/4] mmc: fsl_esdhc: make GPIO support optional
Y.B. Lu
yangbo.lu at nxp.com
Thu Jan 12 02:39:29 CET 2017
Hi York,
Could you help to merge this patchset if there is no any changes needed?
Thanks :)
Best regards,
Yangbo Lu
> -----Original Message-----
> From: Jaehoon Chung [mailto:jh80.chung at samsung.com]
> Sent: Thursday, January 05, 2017 8:33 AM
> To: york sun; Y.B. Lu
> Cc: u-boot at lists.denx.de
> Subject: Re: [PATCH 1/4] mmc: fsl_esdhc: make GPIO support optional
>
> 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