Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <20220621094146.77216-1-jiangjian@cdjrlc.com>
In-Reply-To: <20220621094146.77216-1-jiangjian@cdjrlc.com>
From:   Ulf Hansson <ulf.hansson@linaro.org>
Date:   Thu, 23 Jun 2022 14:30:21 +0200
Message-ID: <CAPDyKFqFLGeLGk6mupPwNdKSbGk5=F9TzCukq-a2hYuF1LYFtw@mail.gmail.com>
Subject: Re: [PATCH] mmc: host: dw_mmc: remove unexpected word "the"
To:     Jiang Jian <jiangjian@cdjrlc.com>
Cc:     jh80.chung@samsung.com, linux-mmc@vger.kernel.org,
        linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

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
>
>