Hi Matt, On Mon, Sep 27, 2010 at 09:42:19AM +0100, Matt Fleming wrote: > There are two checks that need to be made when determining whether a > card is removable. A host controller may set MMC_CAP_NONREMOVABLE if the > controller does not support removing cards (e.g. eMMC), in which case > the card is physically non-removable. Also the 'mmc_assume_removable' > module parameter can be configured at module load time, in which case > the card may be logically non-removable. > > A helper function keeps the logic in one place so that code always > checks both conditions. > > Because this new function is likely to be called from modules we now > need to export the mmc_assume_removable symbol. > > Signed-off-by: Matt Fleming <matt@xxxxxxxxxxxxxxxxx> > Acked-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> > Tested-by: Jaehoon Chung <jh80.chung@xxxxxxxxxxx> Applied to mmc-next with Wolfram's ACK and the following style change, thanks very much. diff --git a/include/linux/mmc/host.h b/include/linux/mmc/host.h index 23a4864..2e0fe62 100644 --- a/include/linux/mmc/host.h +++ b/include/linux/mmc/host.h @@ -272,7 +272,7 @@ extern int mmc_assume_removable; static inline int mmc_card_is_removable(struct mmc_host *host) { - return (!(host->caps & MMC_CAP_NONREMOVABLE) && mmc_assume_removable); + return !(host->caps & MMC_CAP_NONREMOVABLE) && mmc_assume_removable; } #endif -- Chris Ball <cjb@xxxxxxxxxx> <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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html