Re: [PATCH] mmc: It is not an error for the card to be removed while suspended

[Jaehoon Chung <jh80.chung@samsung.com>]

Hi, Adrian.

On 12/14/2015 10:51 PM, Adrian Hunter wrote:
> A card can be removed while it is runtime suspended.
> Do not print an error message.

Well, if card is non-removable (in case of eMMC, card can be non-removable),
 it needs to print the error message?
I'm not sure what's correct..but just my opinion.

Best Regards,
Jaehoon Chung

> 
> Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
> ---
>  drivers/mmc/core/mmc.c | 2 +-
>  drivers/mmc/core/sd.c  | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c
> index 549c56e8cf6b..bf49e44571f2 100644
> --- a/drivers/mmc/core/mmc.c
> +++ b/drivers/mmc/core/mmc.c
> @@ -1935,7 +1935,7 @@ static int mmc_runtime_resume(struct mmc_host *host)
>  	int err;
>  
>  	err = _mmc_resume(host);
> -	if (err)
> +	if (err && err != -ENOMEDIUM)
>  		pr_err("%s: error %d doing runtime resume\n",
>  			mmc_hostname(host), err);
>  
> diff --git a/drivers/mmc/core/sd.c b/drivers/mmc/core/sd.c
> index 8f3b46a56b3d..f2b164b214ae 100644
> --- a/drivers/mmc/core/sd.c
> +++ b/drivers/mmc/core/sd.c
> @@ -1158,7 +1158,7 @@ static int mmc_sd_runtime_resume(struct mmc_host *host)
>  	int err;
>  
>  	err = _mmc_sd_resume(host);
> -	if (err)
> +	if (err && err != -ENOMEDIUM)
>  		pr_err("%s: error %d doing runtime resume\n",
>  			mmc_hostname(host), err);
>  
>