2021-10-27 21:27:58

by Colin Ian King

[permalink] [raw]
Subject: [PATCH][next] mmc: dw_mmc: exynos: Fix spelling mistake "candiates" -> candidates

There are several spelling mistakes in variable names and in a dev_warn
message. Fix these.

Signed-off-by: Colin Ian King <[email protected]>
---
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


2021-10-28 10:03:31

by Ulf Hansson

[permalink] [raw]
Subject: Re: [PATCH][next] mmc: dw_mmc: exynos: Fix spelling mistake "candiates" -> candidates

On Wed, 27 Oct 2021 at 15:08, Colin Ian King
<[email protected]> wrote:
>
> There are several spelling mistakes in variable names and in a dev_warn
> message. Fix these.
>
> Signed-off-by: Colin Ian King <[email protected]>

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
>

2021-10-28 11:34:43

by Jaehoon Chung

[permalink] [raw]
Subject: Re: [PATCH][next] mmc: dw_mmc: exynos: Fix spelling mistake "candiates" -> candidates

On 10/28/21 7:00 PM, Ulf Hansson wrote:
> On Wed, 27 Oct 2021 at 15:08, Colin Ian King
> <[email protected]> wrote:
>>
>> There are several spelling mistakes in variable names and in a dev_warn
>> message. Fix these.
>>
>> Signed-off-by: Colin Ian King <[email protected]>
>
> Applied for next, thanks!

Thanks for fixing them!

Best Regards,
Jaehoon Chung

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