[PATCH] mmc: host: dw_mmc: remove unexpected word "the"

Jiang Jian posted 1 patch 3 years, 10 months ago
drivers/mmc/host/dw_mmc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] mmc: host: dw_mmc: remove unexpected word "the"
Posted by Jiang Jian 3 years, 10 months ago
there is an unexpected word "the" in the comments that need to be removed

 * command hasn't already completed (indicating the the irq

Signed-off-by: Jiang Jian <jiangjian@cdjrlc.com>
---
 drivers/mmc/host/dw_mmc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c
index 581614196a84..7f00d7d7e35e 100644
--- a/drivers/mmc/host/dw_mmc.c
+++ b/drivers/mmc/host/dw_mmc.c
@@ -1363,7 +1363,7 @@ static void __dw_mci_start_request(struct dw_mci *host,
 		 * is just about to roll over.
 		 *
 		 * We do this whole thing under spinlock and only if the
-		 * command hasn't already completed (indicating the the irq
+		 * command hasn't already completed (indicating the irq
 		 * already ran so we don't want the timeout).
 		 */
 		spin_lock_irqsave(&host->irq_lock, irqflags);
-- 
2.17.1
Re: [PATCH] mmc: host: dw_mmc: remove unexpected word "the"
Posted by Ulf Hansson 3 years, 10 months ago
On Tue, 21 Jun 2022 at 11:42, Jiang Jian <jiangjian@cdjrlc.com> wrote:
>
> there is an unexpected word "the" in the comments that need to be removed
>
>  * command hasn't already completed (indicating the the irq

I am fine with picking up patches for these kinds of trivial
spelling/wording mistakes. However, in this case, it seems silly to
have one patch per single line of change. Would you mind walking
through the entire drivers/mmc/host/* directory and make one patch for
all the mmc host drivers instead?

Then if you want to continue with the mmc core files, that could be
another separate patch.

Kind regards
Uffe

>
> Signed-off-by: Jiang Jian <jiangjian@cdjrlc.com>
> ---
>  drivers/mmc/host/dw_mmc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c
> index 581614196a84..7f00d7d7e35e 100644
> --- a/drivers/mmc/host/dw_mmc.c
> +++ b/drivers/mmc/host/dw_mmc.c
> @@ -1363,7 +1363,7 @@ static void __dw_mci_start_request(struct dw_mci *host,
>                  * is just about to roll over.
>                  *
>                  * We do this whole thing under spinlock and only if the
> -                * command hasn't already completed (indicating the the irq
> +                * command hasn't already completed (indicating the irq
>                  * already ran so we don't want the timeout).
>                  */
>                 spin_lock_irqsave(&host->irq_lock, irqflags);
> --
> 2.17.1
>
>