[PATCH v2 04/10] mmc: sdhci: Expose sdhci_init() as non-static
Simon Goldschmidt
simon.k.r.goldschmidt at gmail.com
Wed Jan 29 15:16:01 CET 2020
On Wed, Jan 29, 2020 at 12:00 AM Jaehoon Chung <jh80.chung at samsung.com> wrote:
>
> On 1/24/20 8:52 PM, Faiz Abbas wrote:
> > Expose sdhci_init() as non-static.
>
> Does it need to change to non-static?
And even if it needs to, can we have a reason *why* in the commit message?
Thanks,
Simon
>
> Best Regards,
> Jaehoon Chung
>
> >
> > Signed-off-by: Faiz Abbas <faiz_abbas at ti.com>
> > Signed-off-by: Lokesh Vutla <lokeshvutla at ti.com>
> > ---
> > drivers/mmc/sdhci.c | 2 +-
> > include/sdhci.h | 1 +
> > 2 files changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
> > index 01fa5a9d4d..4fce85a0ea 100644
> > --- a/drivers/mmc/sdhci.c
> > +++ b/drivers/mmc/sdhci.c
> > @@ -618,7 +618,7 @@ static int sdhci_set_ios(struct mmc *mmc)
> > return 0;
> > }
> >
> > -static int sdhci_init(struct mmc *mmc)
> > +int sdhci_init(struct mmc *mmc)
> > {
> > struct sdhci_host *host = mmc->priv;
> > #if CONFIG_IS_ENABLED(DM_MMC) && CONFIG_IS_ENABLED(DM_GPIO)
> > diff --git a/include/sdhci.h b/include/sdhci.h
> > index 01addb7a60..0830e05d53 100644
> > --- a/include/sdhci.h
> > +++ b/include/sdhci.h
> > @@ -487,6 +487,7 @@ void sdhci_set_uhs_timing(struct sdhci_host *host);
> > #ifdef CONFIG_DM_MMC
> > /* Export the operations to drivers */
> > int sdhci_probe(struct udevice *dev);
> > +int sdhci_init(struct mmc *mmc);
> > int sdhci_set_clock(struct mmc *mmc, unsigned int clock);
> > extern const struct dm_mmc_ops sdhci_ops;
> > #else
> >
>
More information about the U-Boot
mailing list