Re: [PATCH 1/2] mmc: fix switch issue with send_status disabled

[Jaehoon Chung]

On 8/17/21 6:20 PM, Ye Li wrote:
> When send_status is false or wait_dat0 is not supported, the switch
> function should not send CMD13 but directly return.
> 
> Signed-off-by: Ye Li 

Reviewed-by: Jaehoon Chung 

Best Regards,
Jaehoon Chung

> ---
>  drivers/mmc/mmc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c
> index 8078a89..a1fd533 100644
> --- a/drivers/mmc/mmc.c
> +++ b/drivers/mmc/mmc.c
> @@ -823,7 +823,7 @@ static int __mmc_switch(struct mmc *mmc, u8 set, u8 
> index, u8 value,
>* capable of polling by using mmc_wait_dat0, then rely on waiting the
>* stated timeout to be sufficient.
>*/
> - if (ret == -ENOSYS && !send_status) {
> + if (ret == -ENOSYS || !send_status) {
>   mdelay(timeout_ms);
>   return 0;
>   }
> 

[PATCH 1/2] mmc: fix switch issue with send_status disabled

[Ye Li]

When send_status is false or wait_dat0 is not supported, the switch
function should not send CMD13 but directly return.

Signed-off-by: Ye Li 
---
 drivers/mmc/mmc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c
index 8078a89..a1fd533 100644
--- a/drivers/mmc/mmc.c
+++ b/drivers/mmc/mmc.c
@@ -823,7 +823,7 @@ static int __mmc_switch(struct mmc *mmc, u8 set, u8 index, 
u8 value,
 * capable of polling by using mmc_wait_dat0, then rely on waiting the
 * stated timeout to be sufficient.
 */
-   if (ret == -ENOSYS && !send_status) {
+   if (ret == -ENOSYS || !send_status) {
mdelay(timeout_ms);
return 0;
}
-- 
2.7.4