* [PATCH] dmaengine: fix typos in comments
@ 2026-09-07 6:33 Hemanth Selam
2026-09-30 20:22 ` Frank Li
2026-10-05 13:58 ` Vinod Koul
0 siblings, 2 replies; 6+ messages in thread
From: Hemanth Selam @ 2026-09-07 6:33 UTC (permalink / raw)
To: Vinod Koul, Frank Li, Laurent Pinchart, Michal Simek
Cc: dmaengine, linux-kernel, linux-arm-kernel
Fix typos in comments, reported by scripts/checkpatch.pl using the
misspelling list in scripts/spelling.txt. Only touches comments, no code
changes.
Assisted-by: Cursor:claude-opus-5
Signed-off-by: Hemanth Selam <hemanth.selam@gmail.com>
---
drivers/dma/bcm-sba-raid.c | 8 ++++----
drivers/dma/ppc4xx/xor.h | 2 +-
drivers/dma/xilinx/xilinx_dpdma.c | 2 +-
3 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/dma/bcm-sba-raid.c b/drivers/dma/bcm-sba-raid.c
index 0de03611252e..485fcde390dd 100644
--- a/drivers/dma/bcm-sba-raid.c
+++ b/drivers/dma/bcm-sba-raid.c
@@ -734,7 +734,7 @@ sba_prep_dma_memcpy(struct dma_chan *dchan, dma_addr_t dst, dma_addr_t src,
struct sba_device *sba = to_sba_device(dchan);
struct sba_request *first = NULL, *req;
- /* Create chained requests where each request is upto hw_buf_size */
+ /* Create chained requests where each request is up to hw_buf_size */
while (len) {
req_len = (len < sba->hw_buf_size) ? len : sba->hw_buf_size;
@@ -877,7 +877,7 @@ sba_prep_dma_xor(struct dma_chan *dchan, dma_addr_t dst, dma_addr_t *src,
if (unlikely(src_cnt > sba->max_xor_srcs))
return NULL;
- /* Create chained requests where each request is upto hw_buf_size */
+ /* Create chained requests where each request is up to hw_buf_size */
while (len) {
req_len = (len < sba->hw_buf_size) ? len : sba->hw_buf_size;
@@ -1356,7 +1356,7 @@ sba_prep_dma_pq(struct dma_chan *dchan, dma_addr_t *dst, dma_addr_t *src,
if (!(flags & DMA_PREP_PQ_DISABLE_Q))
dst_q = &dst[1];
- /* Create chained requests where each request is upto hw_buf_size */
+ /* Create chained requests where each request is up to hw_buf_size */
while (len) {
req_len = (len < sba->hw_buf_size) ? len : sba->hw_buf_size;
@@ -1626,7 +1626,7 @@ static int sba_probe(struct platform_device *pdev)
sba->dev = &pdev->dev;
platform_set_drvdata(pdev, sba);
- /* Number of mailbox channels should be atleast 1 */
+ /* Number of mailbox channels should be at least 1 */
ret = of_count_phandle_with_args(pdev->dev.of_node,
"mboxes", "#mbox-cells");
if (ret <= 0)
diff --git a/drivers/dma/ppc4xx/xor.h b/drivers/dma/ppc4xx/xor.h
index da1230df2817..0931e18cd0d3 100644
--- a/drivers/dma/ppc4xx/xor.h
+++ b/drivers/dma/ppc4xx/xor.h
@@ -12,7 +12,7 @@
#include <linux/types.h>
-/* Number of XOR engines available on the contoller */
+/* Number of XOR engines available on the controller */
#define XOR_ENGINES_NUM 1
/* Number of operands supported in the h/w */
diff --git a/drivers/dma/xilinx/xilinx_dpdma.c b/drivers/dma/xilinx/xilinx_dpdma.c
index d9a3542c4531..bcc188e70441 100644
--- a/drivers/dma/xilinx/xilinx_dpdma.c
+++ b/drivers/dma/xilinx/xilinx_dpdma.c
@@ -1032,7 +1032,7 @@ static int xilinx_dpdma_chan_wait_no_ostand(struct xilinx_dpdma_chan *chan)
{
int ret;
- /* Wait for a no outstanding transaction interrupt upto 50msec */
+ /* Wait for a no outstanding transaction interrupt up to 50msec */
ret = wait_event_interruptible_timeout(chan->wait_to_stop,
!xilinx_dpdma_chan_ostand(chan),
msecs_to_jiffies(50));
--
2.48.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dmaengine: fix typos in comments
2026-09-07 6:33 [PATCH] dmaengine: fix typos in comments Hemanth Selam
@ 2026-09-30 20:22 ` Frank Li
2026-10-05 13:58 ` Vinod Koul
1 sibling, 0 replies; 6+ messages in thread
From: Frank Li @ 2026-09-30 20:22 UTC (permalink / raw)
To: Hemanth Selam
Cc: Vinod Koul, Frank Li, Laurent Pinchart, Michal Simek, dmaengine,
linux-kernel, linux-arm-kernel
On Mon, Sep 07, 2026 at 12:03:43PM +0530, Hemanth Selam wrote:
> Fix typos in comments, reported by scripts/checkpatch.pl using the
> misspelling list in scripts/spelling.txt. Only touches comments, no code
> changes.
>
> Assisted-by: Cursor:claude-opus-5
> Signed-off-by: Hemanth Selam <hemanth.selam@gmail.com>
> ---
Reviewed-by: Frank Li <Frank.Li@nxp.com>
> drivers/dma/bcm-sba-raid.c | 8 ++++----
> drivers/dma/ppc4xx/xor.h | 2 +-
> drivers/dma/xilinx/xilinx_dpdma.c | 2 +-
> 3 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/dma/bcm-sba-raid.c b/drivers/dma/bcm-sba-raid.c
> index 0de03611252e..485fcde390dd 100644
> --- a/drivers/dma/bcm-sba-raid.c
> +++ b/drivers/dma/bcm-sba-raid.c
> @@ -734,7 +734,7 @@ sba_prep_dma_memcpy(struct dma_chan *dchan, dma_addr_t dst, dma_addr_t src,
> struct sba_device *sba = to_sba_device(dchan);
> struct sba_request *first = NULL, *req;
>
> - /* Create chained requests where each request is upto hw_buf_size */
> + /* Create chained requests where each request is up to hw_buf_size */
> while (len) {
> req_len = (len < sba->hw_buf_size) ? len : sba->hw_buf_size;
>
> @@ -877,7 +877,7 @@ sba_prep_dma_xor(struct dma_chan *dchan, dma_addr_t dst, dma_addr_t *src,
> if (unlikely(src_cnt > sba->max_xor_srcs))
> return NULL;
>
> - /* Create chained requests where each request is upto hw_buf_size */
> + /* Create chained requests where each request is up to hw_buf_size */
> while (len) {
> req_len = (len < sba->hw_buf_size) ? len : sba->hw_buf_size;
>
> @@ -1356,7 +1356,7 @@ sba_prep_dma_pq(struct dma_chan *dchan, dma_addr_t *dst, dma_addr_t *src,
> if (!(flags & DMA_PREP_PQ_DISABLE_Q))
> dst_q = &dst[1];
>
> - /* Create chained requests where each request is upto hw_buf_size */
> + /* Create chained requests where each request is up to hw_buf_size */
> while (len) {
> req_len = (len < sba->hw_buf_size) ? len : sba->hw_buf_size;
>
> @@ -1626,7 +1626,7 @@ static int sba_probe(struct platform_device *pdev)
> sba->dev = &pdev->dev;
> platform_set_drvdata(pdev, sba);
>
> - /* Number of mailbox channels should be atleast 1 */
> + /* Number of mailbox channels should be at least 1 */
> ret = of_count_phandle_with_args(pdev->dev.of_node,
> "mboxes", "#mbox-cells");
> if (ret <= 0)
> diff --git a/drivers/dma/ppc4xx/xor.h b/drivers/dma/ppc4xx/xor.h
> index da1230df2817..0931e18cd0d3 100644
> --- a/drivers/dma/ppc4xx/xor.h
> +++ b/drivers/dma/ppc4xx/xor.h
> @@ -12,7 +12,7 @@
>
> #include <linux/types.h>
>
> -/* Number of XOR engines available on the contoller */
> +/* Number of XOR engines available on the controller */
> #define XOR_ENGINES_NUM 1
>
> /* Number of operands supported in the h/w */
> diff --git a/drivers/dma/xilinx/xilinx_dpdma.c b/drivers/dma/xilinx/xilinx_dpdma.c
> index d9a3542c4531..bcc188e70441 100644
> --- a/drivers/dma/xilinx/xilinx_dpdma.c
> +++ b/drivers/dma/xilinx/xilinx_dpdma.c
> @@ -1032,7 +1032,7 @@ static int xilinx_dpdma_chan_wait_no_ostand(struct xilinx_dpdma_chan *chan)
> {
> int ret;
>
> - /* Wait for a no outstanding transaction interrupt upto 50msec */
> + /* Wait for a no outstanding transaction interrupt up to 50msec */
> ret = wait_event_interruptible_timeout(chan->wait_to_stop,
> !xilinx_dpdma_chan_ostand(chan),
> msecs_to_jiffies(50));
> --
> 2.48.1
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dmaengine: fix typos in comments
2026-09-07 6:33 [PATCH] dmaengine: fix typos in comments Hemanth Selam
2026-09-30 20:22 ` Frank Li
@ 2026-10-05 13:58 ` Vinod Koul
2026-10-06 6:02 ` Hemanth Selam
1 sibling, 1 reply; 6+ messages in thread
From: Vinod Koul @ 2026-10-05 13:58 UTC (permalink / raw)
To: Hemanth Selam
Cc: Frank Li, Laurent Pinchart, Michal Simek, dmaengine,
linux-kernel, linux-arm-kernel
On 07-09-26, 12:03, Hemanth Selam wrote:
> Fix typos in comments, reported by scripts/checkpatch.pl using the
> misspelling list in scripts/spelling.txt. Only touches comments, no code
> changes.
>
> Assisted-by: Cursor:claude-opus-5
> Signed-off-by: Hemanth Selam <hemanth.selam@gmail.com>
> ---
> drivers/dma/bcm-sba-raid.c | 8 ++++----
> drivers/dma/ppc4xx/xor.h | 2 +-
> drivers/dma/xilinx/xilinx_dpdma.c | 2 +-
> 3 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/dma/bcm-sba-raid.c b/drivers/dma/bcm-sba-raid.c
> index 0de03611252e..485fcde390dd 100644
> --- a/drivers/dma/bcm-sba-raid.c
> +++ b/drivers/dma/bcm-sba-raid.c
> @@ -734,7 +734,7 @@ sba_prep_dma_memcpy(struct dma_chan *dchan, dma_addr_t dst, dma_addr_t src,
> struct sba_device *sba = to_sba_device(dchan);
> struct sba_request *first = NULL, *req;
>
> - /* Create chained requests where each request is upto hw_buf_size */
> + /* Create chained requests where each request is up to hw_buf_size */
This is bogus, pls drop these... If there are real typos pls fix those
instead! This is not a language book, it is code, upto should be just
fine!
> while (len) {
> req_len = (len < sba->hw_buf_size) ? len : sba->hw_buf_size;
>
> @@ -877,7 +877,7 @@ sba_prep_dma_xor(struct dma_chan *dchan, dma_addr_t dst, dma_addr_t *src,
> if (unlikely(src_cnt > sba->max_xor_srcs))
> return NULL;
>
> - /* Create chained requests where each request is upto hw_buf_size */
> + /* Create chained requests where each request is up to hw_buf_size */
> while (len) {
> req_len = (len < sba->hw_buf_size) ? len : sba->hw_buf_size;
>
> @@ -1356,7 +1356,7 @@ sba_prep_dma_pq(struct dma_chan *dchan, dma_addr_t *dst, dma_addr_t *src,
> if (!(flags & DMA_PREP_PQ_DISABLE_Q))
> dst_q = &dst[1];
>
> - /* Create chained requests where each request is upto hw_buf_size */
> + /* Create chained requests where each request is up to hw_buf_size */
> while (len) {
> req_len = (len < sba->hw_buf_size) ? len : sba->hw_buf_size;
>
> @@ -1626,7 +1626,7 @@ static int sba_probe(struct platform_device *pdev)
> sba->dev = &pdev->dev;
> platform_set_drvdata(pdev, sba);
>
> - /* Number of mailbox channels should be atleast 1 */
> + /* Number of mailbox channels should be at least 1 */
> ret = of_count_phandle_with_args(pdev->dev.of_node,
> "mboxes", "#mbox-cells");
> if (ret <= 0)
> diff --git a/drivers/dma/ppc4xx/xor.h b/drivers/dma/ppc4xx/xor.h
> index da1230df2817..0931e18cd0d3 100644
> --- a/drivers/dma/ppc4xx/xor.h
> +++ b/drivers/dma/ppc4xx/xor.h
> @@ -12,7 +12,7 @@
>
> #include <linux/types.h>
>
> -/* Number of XOR engines available on the contoller */
> +/* Number of XOR engines available on the controller */
> #define XOR_ENGINES_NUM 1
>
> /* Number of operands supported in the h/w */
> diff --git a/drivers/dma/xilinx/xilinx_dpdma.c b/drivers/dma/xilinx/xilinx_dpdma.c
> index d9a3542c4531..bcc188e70441 100644
> --- a/drivers/dma/xilinx/xilinx_dpdma.c
> +++ b/drivers/dma/xilinx/xilinx_dpdma.c
> @@ -1032,7 +1032,7 @@ static int xilinx_dpdma_chan_wait_no_ostand(struct xilinx_dpdma_chan *chan)
> {
> int ret;
>
> - /* Wait for a no outstanding transaction interrupt upto 50msec */
> + /* Wait for a no outstanding transaction interrupt up to 50msec */
> ret = wait_event_interruptible_timeout(chan->wait_to_stop,
> !xilinx_dpdma_chan_ostand(chan),
> msecs_to_jiffies(50));
> --
> 2.48.1
--
~Vinod
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dmaengine: fix typos in comments
2026-10-05 13:58 ` Vinod Koul
@ 2026-10-06 6:02 ` Hemanth Selam
2026-10-06 8:36 ` Vinod Koul
0 siblings, 1 reply; 6+ messages in thread
From: Hemanth Selam @ 2026-10-06 6:02 UTC (permalink / raw)
To: Vinod Koul
Cc: Frank Li, Frank Li, Laurent Pinchart, Michal Simek, dmaengine,
linux-kernel, linux-arm-kernel
On Mon, Oct 05, 2026 at 07:28:42PM +0530, Vinod Koul wrote:
> This is bogus, pls drop these... If there are real typos pls fix those
> instead! This is not a language book, it is code, upto should be just
> fine!
Understood, dropping this one.
For what it is worth, one hunk in it was not an "upto" style change:
drivers/dma/ppc4xx/xor.h has "contoller" where "controller" is meant.
If you would still take that on its own I will send it as a separate
one-line patch; otherwise I will leave it be.
Sorry for the noise.
Thanks,
Hemanth
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dmaengine: fix typos in comments
2026-10-06 6:02 ` Hemanth Selam
@ 2026-10-06 8:36 ` Vinod Koul
0 siblings, 0 replies; 6+ messages in thread
From: Vinod Koul @ 2026-10-06 8:36 UTC (permalink / raw)
To: Hemanth Selam
Cc: Frank Li, Frank Li, Laurent Pinchart, Michal Simek, dmaengine,
linux-kernel, linux-arm-kernel
On 06-10-26, 11:32, Hemanth Selam wrote:
> On Mon, Oct 05, 2026 at 07:28:42PM +0530, Vinod Koul wrote:
> > This is bogus, pls drop these... If there are real typos pls fix those
> > instead! This is not a language book, it is code, upto should be just
> > fine!
>
> Understood, dropping this one.
>
> For what it is worth, one hunk in it was not an "upto" style change:
> drivers/dma/ppc4xx/xor.h has "contoller" where "controller" is meant.
> If you would still take that on its own I will send it as a separate
> one-line patch; otherwise I will leave it be.
Yes real typos are welcome. Pls split and send
--
~Vinod
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] dmaengine: fix typos in comments
@ 2022-05-21 11:11 Julia Lawall
0 siblings, 0 replies; 6+ messages in thread
From: Julia Lawall @ 2022-05-21 11:11 UTC (permalink / raw)
To: Vinod Koul; +Cc: kernel-janitors, dmaengine, linux-kernel
Spelling mistakes (triple letters) in comments.
Detected with the help of Coccinelle.
Signed-off-by: Julia Lawall <Julia.Lawall@inria.fr>
---
drivers/dma/amba-pl08x.c | 2 +-
drivers/dma/mv_xor_v2.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/dma/amba-pl08x.c b/drivers/dma/amba-pl08x.c
index a4a794e62ac2..487a01aa207d 100644
--- a/drivers/dma/amba-pl08x.c
+++ b/drivers/dma/amba-pl08x.c
@@ -231,7 +231,7 @@ enum pl08x_dma_chan_state {
/**
* struct pl08x_dma_chan - this structure wraps a DMA ENGINE channel
- * @vc: wrappped virtual channel
+ * @vc: wrapped virtual channel
* @phychan: the physical channel utilized by this channel, if there is one
* @name: name of channel
* @cd: channel platform data
diff --git a/drivers/dma/mv_xor_v2.c b/drivers/dma/mv_xor_v2.c
index f10b29034da1..f629ef6fd3c2 100644
--- a/drivers/dma/mv_xor_v2.c
+++ b/drivers/dma/mv_xor_v2.c
@@ -313,7 +313,7 @@ mv_xor_v2_tx_submit(struct dma_async_tx_descriptor *tx)
"%s sw_desc %p: async_tx %p\n",
__func__, sw_desc, &sw_desc->async_tx);
- /* assign coookie */
+ /* assign cookie */
spin_lock_bh(&xor_dev->lock);
cookie = dma_cookie_assign(tx);
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2026-10-06 8:36 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-09-07 6:33 [PATCH] dmaengine: fix typos in comments Hemanth Selam
2026-09-30 20:22 ` Frank Li
2026-10-05 13:58 ` Vinod Koul
2026-10-06 6:02 ` Hemanth Selam
2026-10-06 8:36 ` Vinod Koul
-- strict thread matches above, loose matches on Subject: below --
2022-05-21 11:11 Julia Lawall
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®