diff mbox series

mmc: sdhci: Change the code to check auto_cmd23

Message ID 20210825093345.14706-1-cw9316.lee@samsung.com
State New
Headers show
Series mmc: sdhci: Change the code to check auto_cmd23 | expand

Commit Message

Chanwoo Lee Aug. 25, 2021, 9:33 a.m. UTC
From: ChanWoo Lee <cw9316.lee@samsung.com>

It is replaced with a function that is already declared.
//[1/5] mmc: sdhci: Add helpers for the auto-CMD23 flag
//20200412090349.1607-2-adrian.hunter@intel.com

Signed-off-by: ChanWoo Lee <cw9316.lee@samsung.com>
---
 drivers/mmc/host/sdhci.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Adrian Hunter Aug. 31, 2021, 7:37 a.m. UTC | #1
On 25/08/21 12:33 pm, Chanwoo Lee wrote:
> From: ChanWoo Lee <cw9316.lee@samsung.com>

> 

> It is replaced with a function that is already declared.

> //[1/5] mmc: sdhci: Add helpers for the auto-CMD23 flag

> //20200412090349.1607-2-adrian.hunter@intel.com

> 

> Signed-off-by: ChanWoo Lee <cw9316.lee@samsung.com>


Acked-by: Adrian Hunter <adrian.hunter@intel.com>


> ---

>  drivers/mmc/host/sdhci.c | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

> 

> diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c

> index 36f15f81a6af..5782650ddf7d 100644

> --- a/drivers/mmc/host/sdhci.c

> +++ b/drivers/mmc/host/sdhci.c

> @@ -3232,7 +3232,7 @@ static void sdhci_cmd_irq(struct sdhci_host *host, u32 intmask, u32 *intmask_p)

>  			  -ETIMEDOUT :

>  			  -EILSEQ;

>  

> -		if (mrq->sbc && (host->flags & SDHCI_AUTO_CMD23)) {

> +		if (sdhci_auto_cmd23(host, mrq)) {

>  			mrq->sbc->error = err;

>  			__sdhci_finish_mrq(host, mrq);

>  			return;

>
Ulf Hansson Sept. 6, 2021, 5:09 p.m. UTC | #2
On Wed, 25 Aug 2021 at 11:41, Chanwoo Lee <cw9316.lee@samsung.com> wrote:
>

> From: ChanWoo Lee <cw9316.lee@samsung.com>

>

> It is replaced with a function that is already declared.

> //[1/5] mmc: sdhci: Add helpers for the auto-CMD23 flag

> //20200412090349.1607-2-adrian.hunter@intel.com

>

> Signed-off-by: ChanWoo Lee <cw9316.lee@samsung.com>


Queued for v5.16 on the temporary devel branch, thanks!

Kind regards
Uffe


> ---

>  drivers/mmc/host/sdhci.c | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

>

> diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c

> index 36f15f81a6af..5782650ddf7d 100644

> --- a/drivers/mmc/host/sdhci.c

> +++ b/drivers/mmc/host/sdhci.c

> @@ -3232,7 +3232,7 @@ static void sdhci_cmd_irq(struct sdhci_host *host, u32 intmask, u32 *intmask_p)

>                           -ETIMEDOUT :

>                           -EILSEQ;

>

> -               if (mrq->sbc && (host->flags & SDHCI_AUTO_CMD23)) {

> +               if (sdhci_auto_cmd23(host, mrq)) {

>                         mrq->sbc->error = err;

>                         __sdhci_finish_mrq(host, mrq);

>                         return;

> --

> 2.29.0

>
diff mbox series

Patch

diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c
index 36f15f81a6af..5782650ddf7d 100644
--- a/drivers/mmc/host/sdhci.c
+++ b/drivers/mmc/host/sdhci.c
@@ -3232,7 +3232,7 @@  static void sdhci_cmd_irq(struct sdhci_host *host, u32 intmask, u32 *intmask_p)
 			  -ETIMEDOUT :
 			  -EILSEQ;
 
-		if (mrq->sbc && (host->flags & SDHCI_AUTO_CMD23)) {
+		if (sdhci_auto_cmd23(host, mrq)) {
 			mrq->sbc->error = err;
 			__sdhci_finish_mrq(host, mrq);
 			return;