* [PATCH] i2c: npcm7xx: Fix typos
@ 2022-01-29 9:39 Jonathan Neuschäfer
2022-01-29 16:18 ` Randy Dunlap
2022-02-07 16:04 ` Wolfram Sang
0 siblings, 2 replies; 4+ messages in thread
From: Jonathan Neuschäfer @ 2022-01-29 9:39 UTC (permalink / raw)
To: linux-i2c
Cc: Jonathan Neuschäfer, Avi Fishman, Tomer Maimon, Tali Perry,
Patrick Venture, Nancy Yuen, Benjamin Fair, openbmc,
linux-kernel
The comments in this driver have a few typos. Let's fix them.
Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
---
drivers/i2c/busses/i2c-npcm7xx.c | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/drivers/i2c/busses/i2c-npcm7xx.c b/drivers/i2c/busses/i2c-npcm7xx.c
index 2ad166355ec9b..71aad029425d8 100644
--- a/drivers/i2c/busses/i2c-npcm7xx.c
+++ b/drivers/i2c/busses/i2c-npcm7xx.c
@@ -781,7 +781,7 @@ static void npcm_i2c_set_fifo(struct npcm_i2c *bus, int nread, int nwrite)
/*
* if we are about to read the first byte in blk rd mode,
* don't NACK it. If slave returns zero size HW can't NACK
- * it immidiattly, it will read extra byte and then NACK.
+ * it immediately, it will read extra byte and then NACK.
*/
if (bus->rd_ind == 0 && bus->read_block_use) {
/* set fifo to read one byte, no last: */
@@ -981,7 +981,7 @@ static void npcm_i2c_slave_xmit(struct npcm_i2c *bus, u16 nwrite,
/*
* npcm_i2c_slave_wr_buf_sync:
* currently slave IF only supports single byte operations.
- * in order to utilyze the npcm HW FIFO, the driver will ask for 16 bytes
+ * in order to utilize the npcm HW FIFO, the driver will ask for 16 bytes
* at a time, pack them in buffer, and then transmit them all together
* to the FIFO and onward to the bus.
* NACK on read will be once reached to bus->adap->quirks->max_read_len.
@@ -1175,7 +1175,7 @@ static irqreturn_t npcm_i2c_int_slave_handler(struct npcm_i2c *bus)
/*
* the i2c module can response to 10 own SA.
* check which one was addressed by the master.
- * repond to the first one.
+ * respond to the first one.
*/
addr = ((i2ccst3 & 0x07) << 7) |
(i2ccst2 & 0x7F);
@@ -1753,8 +1753,8 @@ static void npcm_i2c_recovery_init(struct i2c_adapter *_adap)
/*
* npcm i2c HW allows direct reading of SCL and SDA.
* However, it does not support setting SCL and SDA directly.
- * The recovery function can togle SCL when SDA is low (but not set)
- * Getter functions used internally, and can be used externaly.
+ * The recovery function can toggle SCL when SDA is low (but not set)
+ * Getter functions used internally, and can be used externally.
*/
rinfo->get_scl = npcm_i2c_get_SCL;
rinfo->get_sda = npcm_i2c_get_SDA;
@@ -1768,10 +1768,10 @@ static void npcm_i2c_recovery_init(struct i2c_adapter *_adap)
/*
* npcm_i2c_init_clk: init HW timing parameters.
- * NPCM7XX i2c module timing parameters are depenent on module core clk (APB)
+ * NPCM7XX i2c module timing parameters are dependent on module core clk (APB)
* and bus frequency.
- * 100kHz bus requires tSCL = 4 * SCLFRQ * tCLK. LT and HT are simetric.
- * 400kHz bus requires assymetric HT and LT. A different equation is recomended
+ * 100kHz bus requires tSCL = 4 * SCLFRQ * tCLK. LT and HT are symmetric.
+ * 400kHz bus requires asymmetric HT and LT. A different equation is recommended
* by the HW designer, given core clock range (equations in comments below).
*
*/
--
2.34.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] i2c: npcm7xx: Fix typos
2022-01-29 9:39 [PATCH] i2c: npcm7xx: Fix typos Jonathan Neuschäfer
@ 2022-01-29 16:18 ` Randy Dunlap
2022-01-30 7:46 ` Tali Perry
2022-02-07 16:04 ` Wolfram Sang
1 sibling, 1 reply; 4+ messages in thread
From: Randy Dunlap @ 2022-01-29 16:18 UTC (permalink / raw)
To: Jonathan Neuschäfer, linux-i2c
Cc: Avi Fishman, Tomer Maimon, Tali Perry, Patrick Venture,
Nancy Yuen, Benjamin Fair, openbmc, linux-kernel
On 1/29/22 01:39, Jonathan Neuschäfer wrote:
> The comments in this driver have a few typos. Let's fix them.
>
> Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
Reviewed-by: Randy Dunlap <rdunlap@infradead.org>
Thanks.
> ---
> drivers/i2c/busses/i2c-npcm7xx.c | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/i2c/busses/i2c-npcm7xx.c b/drivers/i2c/busses/i2c-npcm7xx.c
> index 2ad166355ec9b..71aad029425d8 100644
> --- a/drivers/i2c/busses/i2c-npcm7xx.c
> +++ b/drivers/i2c/busses/i2c-npcm7xx.c
> @@ -781,7 +781,7 @@ static void npcm_i2c_set_fifo(struct npcm_i2c *bus, int nread, int nwrite)
> /*
> * if we are about to read the first byte in blk rd mode,
> * don't NACK it. If slave returns zero size HW can't NACK
> - * it immidiattly, it will read extra byte and then NACK.
> + * it immediately, it will read extra byte and then NACK.
> */
> if (bus->rd_ind == 0 && bus->read_block_use) {
> /* set fifo to read one byte, no last: */
> @@ -981,7 +981,7 @@ static void npcm_i2c_slave_xmit(struct npcm_i2c *bus, u16 nwrite,
> /*
> * npcm_i2c_slave_wr_buf_sync:
> * currently slave IF only supports single byte operations.
> - * in order to utilyze the npcm HW FIFO, the driver will ask for 16 bytes
> + * in order to utilize the npcm HW FIFO, the driver will ask for 16 bytes
> * at a time, pack them in buffer, and then transmit them all together
> * to the FIFO and onward to the bus.
> * NACK on read will be once reached to bus->adap->quirks->max_read_len.
> @@ -1175,7 +1175,7 @@ static irqreturn_t npcm_i2c_int_slave_handler(struct npcm_i2c *bus)
> /*
> * the i2c module can response to 10 own SA.
> * check which one was addressed by the master.
> - * repond to the first one.
> + * respond to the first one.
> */
> addr = ((i2ccst3 & 0x07) << 7) |
> (i2ccst2 & 0x7F);
> @@ -1753,8 +1753,8 @@ static void npcm_i2c_recovery_init(struct i2c_adapter *_adap)
> /*
> * npcm i2c HW allows direct reading of SCL and SDA.
> * However, it does not support setting SCL and SDA directly.
> - * The recovery function can togle SCL when SDA is low (but not set)
> - * Getter functions used internally, and can be used externaly.
> + * The recovery function can toggle SCL when SDA is low (but not set)
> + * Getter functions used internally, and can be used externally.
> */
> rinfo->get_scl = npcm_i2c_get_SCL;
> rinfo->get_sda = npcm_i2c_get_SDA;
> @@ -1768,10 +1768,10 @@ static void npcm_i2c_recovery_init(struct i2c_adapter *_adap)
>
> /*
> * npcm_i2c_init_clk: init HW timing parameters.
> - * NPCM7XX i2c module timing parameters are depenent on module core clk (APB)
> + * NPCM7XX i2c module timing parameters are dependent on module core clk (APB)
> * and bus frequency.
> - * 100kHz bus requires tSCL = 4 * SCLFRQ * tCLK. LT and HT are simetric.
> - * 400kHz bus requires assymetric HT and LT. A different equation is recomended
> + * 100kHz bus requires tSCL = 4 * SCLFRQ * tCLK. LT and HT are symmetric.
> + * 400kHz bus requires asymmetric HT and LT. A different equation is recommended
> * by the HW designer, given core clock range (equations in comments below).
> *
> */
> --
> 2.34.1
>
--
~Randy
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] i2c: npcm7xx: Fix typos
2022-01-29 16:18 ` Randy Dunlap
@ 2022-01-30 7:46 ` Tali Perry
0 siblings, 0 replies; 4+ messages in thread
From: Tali Perry @ 2022-01-30 7:46 UTC (permalink / raw)
To: Randy Dunlap, Jonathan Neuschäfer
Cc: Linux I2C, Avi Fishman, Tomer Maimon, Patrick Venture,
Nancy Yuen, Benjamin Fair, OpenBMC Maillist,
Linux Kernel Mailing List, kfting
Reviewed-by: tali.perry@nuvoton.com
Thank you very much !
On Sat, Jan 29, 2022 at 6:18 PM Randy Dunlap <rdunlap@infradead.org> wrote:
>
>
>
> On 1/29/22 01:39, Jonathan Neuschäfer wrote:
> > The comments in this driver have a few typos. Let's fix them.
> >
> > Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
>
> Reviewed-by: Randy Dunlap <rdunlap@infradead.org>
>
> Thanks.
>
> > ---
> > drivers/i2c/busses/i2c-npcm7xx.c | 16 ++++++++--------
> > 1 file changed, 8 insertions(+), 8 deletions(-)
> >
> > diff --git a/drivers/i2c/busses/i2c-npcm7xx.c b/drivers/i2c/busses/i2c-npcm7xx.c
> > index 2ad166355ec9b..71aad029425d8 100644
> > --- a/drivers/i2c/busses/i2c-npcm7xx.c
> > +++ b/drivers/i2c/busses/i2c-npcm7xx.c
> > @@ -781,7 +781,7 @@ static void npcm_i2c_set_fifo(struct npcm_i2c *bus, int nread, int nwrite)
> > /*
> > * if we are about to read the first byte in blk rd mode,
> > * don't NACK it. If slave returns zero size HW can't NACK
> > - * it immidiattly, it will read extra byte and then NACK.
> > + * it immediately, it will read extra byte and then NACK.
> > */
> > if (bus->rd_ind == 0 && bus->read_block_use) {
> > /* set fifo to read one byte, no last: */
> > @@ -981,7 +981,7 @@ static void npcm_i2c_slave_xmit(struct npcm_i2c *bus, u16 nwrite,
> > /*
> > * npcm_i2c_slave_wr_buf_sync:
> > * currently slave IF only supports single byte operations.
> > - * in order to utilyze the npcm HW FIFO, the driver will ask for 16 bytes
> > + * in order to utilize the npcm HW FIFO, the driver will ask for 16 bytes
> > * at a time, pack them in buffer, and then transmit them all together
> > * to the FIFO and onward to the bus.
> > * NACK on read will be once reached to bus->adap->quirks->max_read_len.
> > @@ -1175,7 +1175,7 @@ static irqreturn_t npcm_i2c_int_slave_handler(struct npcm_i2c *bus)
> > /*
> > * the i2c module can response to 10 own SA.
> > * check which one was addressed by the master.
> > - * repond to the first one.
> > + * respond to the first one.
> > */
> > addr = ((i2ccst3 & 0x07) << 7) |
> > (i2ccst2 & 0x7F);
> > @@ -1753,8 +1753,8 @@ static void npcm_i2c_recovery_init(struct i2c_adapter *_adap)
> > /*
> > * npcm i2c HW allows direct reading of SCL and SDA.
> > * However, it does not support setting SCL and SDA directly.
> > - * The recovery function can togle SCL when SDA is low (but not set)
> > - * Getter functions used internally, and can be used externaly.
> > + * The recovery function can toggle SCL when SDA is low (but not set)
> > + * Getter functions used internally, and can be used externally.
> > */
> > rinfo->get_scl = npcm_i2c_get_SCL;
> > rinfo->get_sda = npcm_i2c_get_SDA;
> > @@ -1768,10 +1768,10 @@ static void npcm_i2c_recovery_init(struct i2c_adapter *_adap)
> >
> > /*
> > * npcm_i2c_init_clk: init HW timing parameters.
> > - * NPCM7XX i2c module timing parameters are depenent on module core clk (APB)
> > + * NPCM7XX i2c module timing parameters are dependent on module core clk (APB)
> > * and bus frequency.
> > - * 100kHz bus requires tSCL = 4 * SCLFRQ * tCLK. LT and HT are simetric.
> > - * 400kHz bus requires assymetric HT and LT. A different equation is recomended
> > + * 100kHz bus requires tSCL = 4 * SCLFRQ * tCLK. LT and HT are symmetric.
> > + * 400kHz bus requires asymmetric HT and LT. A different equation is recommended
> > * by the HW designer, given core clock range (equations in comments below).
> > *
> > */
> > --
> > 2.34.1
> >
>
> --
> ~Randy
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] i2c: npcm7xx: Fix typos
2022-01-29 9:39 [PATCH] i2c: npcm7xx: Fix typos Jonathan Neuschäfer
2022-01-29 16:18 ` Randy Dunlap
@ 2022-02-07 16:04 ` Wolfram Sang
1 sibling, 0 replies; 4+ messages in thread
From: Wolfram Sang @ 2022-02-07 16:04 UTC (permalink / raw)
To: Jonathan Neuschäfer
Cc: linux-i2c, Avi Fishman, Tomer Maimon, Tali Perry,
Patrick Venture, Nancy Yuen, Benjamin Fair, openbmc,
linux-kernel
[-- Attachment #1: Type: text/plain, Size: 237 bytes --]
On Sat, Jan 29, 2022 at 10:39:07AM +0100, Jonathan Neuschäfer wrote:
> The comments in this driver have a few typos. Let's fix them.
>
> Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
Applied to for-next, thanks!
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-02-07 16:16 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-29 9:39 [PATCH] i2c: npcm7xx: Fix typos Jonathan Neuschäfer
2022-01-29 16:18 ` Randy Dunlap
2022-01-30 7:46 ` Tali Perry
2022-02-07 16:04 ` Wolfram Sang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).