On 3 October 2017 at 12:06, Colin King <colin.king@xxxxxxxxxxxxx> wrote: > From: Colin Ian King <colin.king@xxxxxxxxxxxxx> > > The function sdhci_at91_set_uhs_signaling is local to the source and does > not need to be in global scope, so make it static. > > Cleans up sparse warning: > symbol 'sdhci_at91_set_uhs_signaling' was not declared. Should it be > static? > > Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx> Thanks, applied for next! Kind regards Uffe > --- > drivers/mmc/host/sdhci-of-at91.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/mmc/host/sdhci-of-at91.c b/drivers/mmc/host/sdhci-of-at91.c > index 4e47ed6bc716..682c573e20a7 100644 > --- a/drivers/mmc/host/sdhci-of-at91.c > +++ b/drivers/mmc/host/sdhci-of-at91.c > @@ -114,7 +114,8 @@ static void sdhci_at91_set_power(struct sdhci_host *host, unsigned char mode, > sdhci_set_power_noreg(host, mode, vdd); > } > > -void sdhci_at91_set_uhs_signaling(struct sdhci_host *host, unsigned int timing) > +static void sdhci_at91_set_uhs_signaling(struct sdhci_host *host, > + unsigned int timing) > { > if (timing == MMC_TIMING_MMC_DDR52) > sdhci_writeb(host, SDMMC_MC1R_DDR, SDMMC_MC1R); > -- > 2.14.1 > -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html