i2c: rk3x: fix some typo

PATCH In review Yangtao Li 4 participants 7 messages Nov 2, 2018 – Nov 9, 2018

Subsystems (2)

Arm Rockchip I2C subsystem host drivers

lore

Share & tools

lore

Yangtao Li

author

2018-11-02 12:45 UTC #0 lore raw

PATCH v2 1/1 i2c: rk3x: fix some typo

Fix multiple instances of a misspelled "Calculates".

Signed-off-by Yangtao Li <[email protected]>

Reviewed-by Heiko Stuebner <[email protected]>

---
Changes in v2:
-add commit message

---
 drivers/i2c/busses/i2c-rk3x.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

drivers/i2c/busses/i2c-rk3x.c +2 -2

index b8a2728dd4b6..fe347e8e2a23 100644
@@ -555,7 +555,7 @@ static const struct i2c_spec_values *rk3x_i2c_get_spec(unsigned int speed)
  *
  * @clk_rate: I2C input clock rate
  * @t: Known I2C timing information
- * @t_calc: Caculated rk3x private timings that would be written into regs
+ * @t_calc: Calculated rk3x private timings that would be written into regs
  *
  * Returns: 0 on success, -EINVAL if the goal SCL rate is too slow. In that case
  * a best-effort divider value is returned in divs. If the target rate is

@@ -716,7 +716,7 @@ static int rk3x_i2c_v0_calc_timings(unsigned long clk_rate,
  *
  * @clk_rate: I2C input clock rate
  * @t: Known I2C timing information
- * @t_calc: Caculated rk3x private timings that would be written into regs
+ * @t_calc: Calculated rk3x private timings that would be written into regs
  *
  * Returns: 0 on success, -EINVAL if the goal SCL rate is too slow. In that case
  * a best-effort divider value is returned in divs. If the target rate is

Signature · 1 line

2.17.0

Peter Rosin

2018-11-02 12:58 UTC #1 lore raw

Quoted 1 line · Yangtao Li

Fix multiple instances of a misspelled "Calculates".

Yes, it sure is hard to spell "Calcluated," it's almost impossible to get it completely right and not introduce another typo somewhere... :-)

Cheers,
Peter

Quoted 33 lines

Signed-off-by Yangtao Li <[email protected]>

Reviewed-by Heiko Stuebner <[email protected]>

---
Changes in v2:

(quoted diff) +0 -2

-add commit message
---
 drivers/i2c/busses/i2c-rk3x.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

drivers/i2c/busses/i2c-rk3x.c +2 -2

index b8a2728dd4b6..fe347e8e2a23 100644
@@ -555,7 +555,7 @@ static const struct i2c_spec_values *rk3x_i2c_get_spec(unsigned int speed)
  *
  * @clk_rate: I2C input clock rate
  * @t: Known I2C timing information
- * @t_calc: Caculated rk3x private timings that would be written into regs
+ * @t_calc: Calculated rk3x private timings that would be written into regs
  *
  * Returns: 0 on success, -EINVAL if the goal SCL rate is too slow. In that case
  * a best-effort divider value is returned in divs. If the target rate is

@@ -716,7 +716,7 @@ static int rk3x_i2c_v0_calc_timings(unsigned long clk_rate,
  *
  * @clk_rate: I2C input clock rate
  * @t: Known I2C timing information
- * @t_calc: Caculated rk3x private timings that would be written into regs
+ * @t_calc: Calculated rk3x private timings that would be written into regs
  *
  * Returns: 0 on success, -EINVAL if the goal SCL rate is too slow. In that case
  * a best-effort divider value is returned in divs. If the target rate is

Frank Lee

author

2018-11-05 15:59 UTC #2 lore raw

I am sorry.
-- Yangtao

Quoted 44 lines · Peter Rosin

Quoted 1 line · Yangtao Li

Fix multiple instances of a misspelled "Calculates".

Yes, it sure is hard to spell "Calcluated," it's almost impossible to get it completely right and not introduce another typo somewhere... :-)

Cheers,
Peter

Quoted 33 lines

Signed-off-by Yangtao Li <[email protected]>

Reviewed-by Heiko Stuebner <[email protected]>

---
Changes in v2:

(quoted diff) +0 -2

-add commit message
---
 drivers/i2c/busses/i2c-rk3x.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

drivers/i2c/busses/i2c-rk3x.c +2 -2

index b8a2728dd4b6..fe347e8e2a23 100644
@@ -555,7 +555,7 @@ static const struct i2c_spec_values *rk3x_i2c_get_spec(unsigned int speed)
  *
  * @clk_rate: I2C input clock rate
  * @t: Known I2C timing information
- * @t_calc: Caculated rk3x private timings that would be written into regs
+ * @t_calc: Calculated rk3x private timings that would be written into regs
  *
  * Returns: 0 on success, -EINVAL if the goal SCL rate is too slow. In that case
  * a best-effort divider value is returned in divs. If the target rate is

@@ -716,7 +716,7 @@ static int rk3x_i2c_v0_calc_timings(unsigned long clk_rate,
  *
  * @clk_rate: I2C input clock rate
  * @t: Known I2C timing information
- * @t_calc: Caculated rk3x private timings that would be written into regs
+ * @t_calc: Calculated rk3x private timings that would be written into regs
  *
  * Returns: 0 on success, -EINVAL if the goal SCL rate is too slow. In that case
  * a best-effort divider value is returned in divs. If the target rate is

Heiko Stuebner

maintainer

2018-11-06 12:01 UTC #3 lore raw

Hi Yangtao, Peter,

Am Montag, 5. November 2018, 16:59:58 CET schrieb Frank Lee:

Quoted 2 lines

I am sorry.
-- Yangtao

If anything, it is me that should be sorry, as I did suggest the patch description, including the new spelling error.

And it took me like the whole weekend to actually realize where the Calculated / Calculates error was hiding ;-) .

Heiko

Quoted 46 lines · Peter Rosin

Quoted 44 lines · Yangtao Li

Quoted 1 line

Fix multiple instances of a misspelled "Calculates".

Yes, it sure is hard to spell "Calcluated," it's almost impossible to get it completely right and not introduce another typo somewhere... :-)

Cheers,
Peter

Quoted 33 lines

Signed-off-by Yangtao Li <[email protected]>

Reviewed-by Heiko Stuebner <[email protected]>

---
Changes in v2:

(quoted diff) +0 -2

-add commit message
---
 drivers/i2c/busses/i2c-rk3x.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

drivers/i2c/busses/i2c-rk3x.c +2 -2

index b8a2728dd4b6..fe347e8e2a23 100644
@@ -555,7 +555,7 @@ static const struct i2c_spec_values *rk3x_i2c_get_spec(unsigned int speed)
  *
  * @clk_rate: I2C input clock rate
  * @t: Known I2C timing information
- * @t_calc: Caculated rk3x private timings that would be written into regs
+ * @t_calc: Calculated rk3x private timings that would be written into regs
  *
  * Returns: 0 on success, -EINVAL if the goal SCL rate is too slow. In that case
  * a best-effort divider value is returned in divs. If the target rate is

@@ -716,7 +716,7 @@ static int rk3x_i2c_v0_calc_timings(unsigned long clk_rate,
  *
  * @clk_rate: I2C input clock rate
  * @t: Known I2C timing information
- * @t_calc: Caculated rk3x private timings that would be written into regs
+ * @t_calc: Calculated rk3x private timings that would be written into regs
  *
  * Returns: 0 on success, -EINVAL if the goal SCL rate is too slow. In that case
  * a best-effort divider value is returned in divs. If the target rate is

Frank Lee

author

2018-11-06 13:01 UTC #4 lore raw

Hi Heiko?

please?correct it in the cimmit message.

--Yangtao

Quoted 66 lines · Heiko Stuebner

Hi Yangtao, Peter,
Am Montag, 5. November 2018, 16:59:58 CET schrieb Frank Lee:
Quoted 2 lines
I am sorry.
-- Yangtao
If anything, it is me that should be sorry, as I did suggest the patch description, including the new spelling error.
And it took me like the whole weekend to actually realize where the Calculated / Calculates error was hiding ;-) .
Heiko
Quoted 46 lines · Peter Rosin
Quoted 44 lines · Yangtao Li
Quoted 1 line
Fix multiple instances of a misspelled "Calculates".
Yes, it sure is hard to spell "Calcluated," it's almost impossible to get it completely right and not introduce another typo somewhere... :-)
Cheers,
Peter
Quoted 33 lines
Signed-off-by Yangtao Li <[email protected]>
Reviewed-by Heiko Stuebner <[email protected]>
---
Changes in v2:
(quoted diff) +0 -2
-add commit message
---
 drivers/i2c/busses/i2c-rk3x.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/i2c/busses/i2c-rk3x.c +2 -2
index b8a2728dd4b6..fe347e8e2a23 100644
@@ -555,7 +555,7 @@ static const struct i2c_spec_values *rk3x_i2c_get_spec(unsigned int speed)
  *
  * @clk_rate: I2C input clock rate
  * @t: Known I2C timing information
- * @t_calc: Caculated rk3x private timings that would be written into regs
+ * @t_calc: Calculated rk3x private timings that would be written into regs
  *
  * Returns: 0 on success, -EINVAL if the goal SCL rate is too slow. In that case
  * a best-effort divider value is returned in divs. If the target rate is
@@ -716,7 +716,7 @@ static int rk3x_i2c_v0_calc_timings(unsigned long clk_rate,
  *
  * @clk_rate: I2C input clock rate
  * @t: Known I2C timing information
- * @t_calc: Caculated rk3x private timings that would be written into regs
+ * @t_calc: Calculated rk3x private timings that would be written into regs
  *
  * Returns: 0 on success, -EINVAL if the goal SCL rate is too slow. In that case
  * a best-effort divider value is returned in divs. If the target rate is

Frank Lee

author

2018-11-06 13:02 UTC #5 lore raw

add myself.

Quoted 73 lines · Frank Lee

Hi Heiko?
please?correct it in the cimmit message.
--Yangtao
Quoted 66 lines · Heiko Stuebner
Hi Yangtao, Peter,
Am Montag, 5. November 2018, 16:59:58 CET schrieb Frank Lee:
Quoted 2 lines
I am sorry.
-- Yangtao
If anything, it is me that should be sorry, as I did suggest the patch description, including the new spelling error.
And it took me like the whole weekend to actually realize where the Calculated / Calculates error was hiding ;-) .
Heiko
Quoted 46 lines · Peter Rosin
Quoted 44 lines · Yangtao Li
Quoted 1 line
Fix multiple instances of a misspelled "Calculates".
Yes, it sure is hard to spell "Calcluated," it's almost impossible to get it completely right and not introduce another typo somewhere... :-)
Cheers,
Peter
Quoted 33 lines
Signed-off-by Yangtao Li <[email protected]>
Reviewed-by Heiko Stuebner <[email protected]>
---
Changes in v2:
(quoted diff) +0 -2
-add commit message
---
 drivers/i2c/busses/i2c-rk3x.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/i2c/busses/i2c-rk3x.c +2 -2
index b8a2728dd4b6..fe347e8e2a23 100644
@@ -555,7 +555,7 @@ static const struct i2c_spec_values *rk3x_i2c_get_spec(unsigned int speed)
  *
  * @clk_rate: I2C input clock rate
  * @t: Known I2C timing information
- * @t_calc: Caculated rk3x private timings that would be written into regs
+ * @t_calc: Calculated rk3x private timings that would be written into regs
  *
  * Returns: 0 on success, -EINVAL if the goal SCL rate is too slow. In that case
  * a best-effort divider value is returned in divs. If the target rate is
@@ -716,7 +716,7 @@ static int rk3x_i2c_v0_calc_timings(unsigned long clk_rate,
  *
  * @clk_rate: I2C input clock rate
  * @t: Known I2C timing information
- * @t_calc: Caculated rk3x private timings that would be written into regs
+ * @t_calc: Calculated rk3x private timings that would be written into regs
  *
  * Returns: 0 on success, -EINVAL if the goal SCL rate is too slow. In that case
  * a best-effort divider value is returned in divs. If the target rate is

Wolfram Sang

2018-11-09 16:09 UTC #6 lore raw

In reply to Heiko Stuebner

Quoted 11 lines · Heiko Stuebner

Hi Yangtao, Peter,
Am Montag, 5. November 2018, 16:59:58 CET schrieb Frank Lee:
Quoted 2 lines
I am sorry.
-- Yangtao
If anything, it is me that should be sorry, as I did suggest the patch description, including the new spelling error.
And it took me like the whole weekend to actually realize where the Calculated / Calculates error was hiding ;-) .

You guys are fun :) Anyhow, $subject is way too generic, that is the real culprit here. I don't mind empty descriptions, so you can basically move the message into the header.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20181109/391a2489/attachment.sig>