Hi, On Sat, Mar 05 2011, Linus Walleij wrote: > From: Ulf Hansson <[email protected]> > > When using mmc_try_claim_host the corresponding release > function is mmc_do_release_host, which then also must > be exported. > > Reviewed-by: Jonas Aberg <[email protected]> > Reviewed-by: Sebastian Rasmussen <[email protected]> > Signed-off-by: Ulf Hansson <[email protected]> > Signed-off-by: Linus Walleij <[email protected]> > --- > drivers/mmc/core/core.c | 3 ++- > include/linux/mmc/core.h | 1 + > 2 files changed, 3 insertions(+), 1 deletions(-) > > diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c > index 579ba1e..2521794 100644 > --- a/drivers/mmc/core/core.c > +++ b/drivers/mmc/core/core.c > @@ -527,7 +527,7 @@ int mmc_try_claim_host(struct mmc_host *host) > } > EXPORT_SYMBOL(mmc_try_claim_host); > > -static void mmc_do_release_host(struct mmc_host *host) > +void mmc_do_release_host(struct mmc_host *host) > { > unsigned long flags; > > @@ -542,6 +542,7 @@ static void mmc_do_release_host(struct mmc_host *host) > wake_up(&host->wq); > } > } > +EXPORT_SYMBOL(mmc_do_release_host); > > void mmc_host_deeper_disable(struct work_struct *work) > {
Would you mind adding a kerneldoc comment for mmc_do_release_host()? The other public locking functions have them, so this should gain one too when it becomes public. Thanks, - Chris. -- Chris Ball <[email protected]> <http://printf.net/> One Laptop Per Child -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
