Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20211027130812.426373-1-colin.i.king@gmail.com>
In-Reply-To: <20211027130812.426373-1-colin.i.king@gmail.com>
From:   Ulf Hansson <ulf.hansson@linaro.org>
Date:   Thu, 28 Oct 2021 12:00:01 +0200
Message-ID: <CAPDyKFqXQwfJuU_PPrfeNwh51UkhtVFJzZ71_0R6kcVuSM3bWw@mail.gmail.com>
Subject: Re: [PATCH][next] mmc: dw_mmc: exynos: Fix spelling mistake
 "candiates" -> candidates
To:     Colin Ian King <colin.i.king@googlemail.com>
Cc:     Jaehoon Chung <jh80.chung@samsung.com>,
        Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>,
        linux-mmc@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-samsung-soc@vger.kernel.org, kernel-janitors@vger.kernel.org,
        linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Wed, 27 Oct 2021 at 15:08, Colin Ian King
<colin.i.king@googlemail.com> wrote:
>
> There are several spelling mistakes in variable names and in a dev_warn
> message. Fix these.
>
> Signed-off-by: Colin Ian King <colin.i.king@gmail.com>

Applied for next, thanks!

Kind regards
Uffe


> ---
>  drivers/mmc/host/dw_mmc-exynos.c | 20 ++++++++++----------
>  1 file changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/mmc/host/dw_mmc-exynos.c b/drivers/mmc/host/dw_mmc-exynos.c
> index 1f8a3c0ddfe1..c2dd29ef45c6 100644
> --- a/drivers/mmc/host/dw_mmc-exynos.c
> +++ b/drivers/mmc/host/dw_mmc-exynos.c
> @@ -442,14 +442,14 @@ static inline u8 dw_mci_exynos_move_next_clksmpl(struct dw_mci *host)
>         return sample;
>  }
>
> -static s8 dw_mci_exynos_get_best_clksmpl(u8 candiates)
> +static s8 dw_mci_exynos_get_best_clksmpl(u8 candidates)
>  {
>         const u8 iter = 8;
>         u8 __c;
>         s8 i, loc = -1;
>
>         for (i = 0; i < iter; i++) {
> -               __c = ror8(candiates, i);
> +               __c = ror8(candidates, i);
>                 if ((__c & 0xc7) == 0xc7) {
>                         loc = i;
>                         goto out;
> @@ -457,7 +457,7 @@ static s8 dw_mci_exynos_get_best_clksmpl(u8 candiates)
>         }
>
>         for (i = 0; i < iter; i++) {
> -               __c = ror8(candiates, i);
> +               __c = ror8(candidates, i);
>                 if ((__c & 0x83) == 0x83) {
>                         loc = i;
>                         goto out;
> @@ -466,11 +466,11 @@ static s8 dw_mci_exynos_get_best_clksmpl(u8 candiates)
>
>         /*
>          * If there is no cadiates value, then it needs to return -EIO.
> -        * If there are candiates values and don't find bset clk sample value,
> -        * then use a first candiates clock sample value.
> +        * If there are candidates values and don't find bset clk sample value,
> +        * then use a first candidates clock sample value.
>          */
>         for (i = 0; i < iter; i++) {
> -               __c = ror8(candiates, i);
> +               __c = ror8(candidates, i);
>                 if ((__c & 0x1) == 0x1) {
>                         loc = i;
>                         goto out;
> @@ -485,7 +485,7 @@ static int dw_mci_exynos_execute_tuning(struct dw_mci_slot *slot, u32 opcode)
>         struct dw_mci *host = slot->host;
>         struct dw_mci_exynos_priv_data *priv = host->priv;
>         struct mmc_host *mmc = slot->mmc;
> -       u8 start_smpl, smpl, candiates = 0;
> +       u8 start_smpl, smpl, candidates = 0;
>         s8 found;
>         int ret = 0;
>
> @@ -496,18 +496,18 @@ static int dw_mci_exynos_execute_tuning(struct dw_mci_slot *slot, u32 opcode)
>                 smpl = dw_mci_exynos_move_next_clksmpl(host);
>
>                 if (!mmc_send_tuning(mmc, opcode, NULL))
> -                       candiates |= (1 << smpl);
> +                       candidates |= (1 << smpl);
>
>         } while (start_smpl != smpl);
>
> -       found = dw_mci_exynos_get_best_clksmpl(candiates);
> +       found = dw_mci_exynos_get_best_clksmpl(candidates);
>         if (found >= 0) {
>                 dw_mci_exynos_set_clksmpl(host, found);
>                 priv->tuned_sample = found;
>         } else {
>                 ret = -EIO;
>                 dev_warn(&mmc->class_dev,
> -                       "There is no candiates value about clksmpl!\n");
> +                       "There is no candidates value about clksmpl!\n");
>         }
>
>         return ret;
> --
> 2.32.0
>