mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH v2] misc: fix typos in comments
@ 2026-10-06  6:01 Hemanth Selam
  2026-10-06 17:05 ` Vishnu Dasa
  0 siblings, 1 reply; 2+ messages in thread
From: Hemanth Selam @ 2026-10-06  6:01 UTC (permalink / raw)
  To: Arnd Bergmann, Greg Kroah-Hartman, Frank Haverkamp, Bryan Tan,
	Vishnu Dasa
  Cc: Broadcom internal kernel review list, linux-kernel

Fix typos in comments, reported by scripts/checkpatch.pl using the
misspelling list in scripts/spelling.txt.  One further misspelling of
"respectively" in genwqe/card_base.c is fixed as well: it is not in
scripts/spelling.txt, but it sits a few lines from one that checkpatch
does report.  Only comments are touched, there are no code changes.

Assisted-by: LLM
Signed-off-by: Hemanth Selam <hemanth.selam@gmail.com>
---

Notes (v2):
    v2:
     - Drop the drivers/char/lp.c hunk.  It is not under drivers/misc, which
       is what the "misc:" prefix claimed.  It is now sent separately, as
       "char: lp: fix typo in comment".
     - Also fix a second misspelling in genwqe/card_base.c, a few lines from
       one already being corrected.  checkpatch does not report it because
       it is not listed in scripts/spelling.txt.
    
    v1: https://lore.kernel.org/all/20260904113843.21367-1-hemanth.selam@gmail.com/

 drivers/misc/gehc-achc.c                | 2 +-
 drivers/misc/genwqe/card_base.c         | 8 ++++----
 drivers/misc/genwqe/card_ddcb.c         | 4 ++--
 drivers/misc/genwqe/card_utils.c        | 2 +-
 drivers/misc/vmw_vmci/vmci_datagram.c   | 2 +-
 drivers/misc/vmw_vmci/vmci_queue_pair.c | 2 +-
 6 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/drivers/misc/gehc-achc.c b/drivers/misc/gehc-achc.c
index 9792189c0923..2c1222edb5dc 100644
--- a/drivers/misc/gehc-achc.c
+++ b/drivers/misc/gehc-achc.c
@@ -369,7 +369,7 @@ static int ezport_firmware_flash_data(struct spi_device *spi,
 	dev_dbg(&spi->dev, "EzPort verify flashed data...\n");
 	ret = ezport_firmware_compare_data(spi, data, size);
 
-	/* allow missing FW verfication in secure mode */
+	/* allow missing FW verification in secure mode */
 	if (ret == -EACCES)
 		ret = 0;
 
diff --git a/drivers/misc/genwqe/card_base.c b/drivers/misc/genwqe/card_base.c
index e602eb3c51fa..3eb843af31a5 100644
--- a/drivers/misc/genwqe/card_base.c
+++ b/drivers/misc/genwqe/card_base.c
@@ -286,7 +286,7 @@ int genwqe_flash_readback_fails(struct genwqe_dev *cd)
  * @cd: GenWQE device information
  *
  * Note: From a design perspective it turned out to be a bad idea to
- * use codes here to specifiy the frequency/speed values. An old
+ * use codes here to specify the frequency/speed values. An old
  * driver cannot understand new codes and is therefore always a
  * problem. Better is to measure out the value or put the
  * speed/frequency directly into a register which is always a valid
@@ -904,7 +904,7 @@ static int genwqe_reload_bistream(struct genwqe_dev *cd)
  *
  * Condition for the health-thread to trigger:
  *   a) when a kthread_stop() request comes in or
- *   b) a critical GFIR occured
+ *   b) a critical GFIR occurred
  *
  * Informational GFIRs are checked and potentially printed in
  * GENWQE_HEALTH_CHECK_INTERVAL seconds.
@@ -1210,10 +1210,10 @@ static int genwqe_probe(struct pci_dev *pci_dev,
 }
 
 /**
- * genwqe_remove() - Called when device is removed (hot-plugable)
+ * genwqe_remove() - Called when device is removed (hot-pluggable)
  * @pci_dev:	PCI device information struct
  *
- * Or when driver is unloaded respecitively when unbind is done.
+ * Or when driver is unloaded respectively when unbind is done.
  */
 static void genwqe_remove(struct pci_dev *pci_dev)
 {
diff --git a/drivers/misc/genwqe/card_ddcb.c b/drivers/misc/genwqe/card_ddcb.c
index 969178573940..07e2c1be22d9 100644
--- a/drivers/misc/genwqe/card_ddcb.c
+++ b/drivers/misc/genwqe/card_ddcb.c
@@ -460,7 +460,7 @@ static int genwqe_check_ddcb_queue(struct genwqe_dev *cd,
 /**
  * __genwqe_wait_ddcb(): Waits until DDCB is completed
  * @cd:         pointer to genwqe device descriptor
- * @req:        pointer to requsted DDCB parameters
+ * @req:        pointer to requested DDCB parameters
  *
  * The Service Layer will update the RETC in DDCB when processing is
  * pending or done.
@@ -543,7 +543,7 @@ int __genwqe_wait_ddcb(struct genwqe_dev *cd, struct ddcb_requ *req)
 		return -EINVAL;
 	}
 
-	/* Severe error occured. Driver is forced to stop operation */
+	/* Severe error occurred. Driver is forced to stop operation */
 	if (cd->card_state != GENWQE_CARD_USED) {
 		dev_err(&pci_dev->dev,
 			"[%s] err: DDCB#%d forced to stop (rc=%d)\n",
diff --git a/drivers/misc/genwqe/card_utils.c b/drivers/misc/genwqe/card_utils.c
index e3b5fb337182..7c57cb881d7f 100644
--- a/drivers/misc/genwqe/card_utils.c
+++ b/drivers/misc/genwqe/card_utils.c
@@ -1003,7 +1003,7 @@ u64 genwqe_read_vreg(struct genwqe_dev *cd, u32 reg, int func)
  * @cd:	        genwqe device descriptor
  *
  * Note: From a design perspective it turned out to be a bad idea to
- * use codes here to specifiy the frequency/speed values. An old
+ * use codes here to specify the frequency/speed values. An old
  * driver cannot understand new codes and is therefore always a
  * problem. Better is to measure out the value or put the
  * speed/frequency directly into a register which is always a valid
diff --git a/drivers/misc/vmw_vmci/vmci_datagram.c b/drivers/misc/vmw_vmci/vmci_datagram.c
index 6193a22c3ab5..73b43b097f92 100644
--- a/drivers/misc/vmw_vmci/vmci_datagram.c
+++ b/drivers/misc/vmw_vmci/vmci_datagram.c
@@ -434,7 +434,7 @@ EXPORT_SYMBOL_GPL(vmci_datagram_create_handle_priv);
  * @out_handle: vmci_handle that is populated as a result of this function.
  *
  * Creates a host context datagram endpoint and returns a handle to
- * it.  Same as vmci_datagram_create_handle_priv without the priviledge
+ * it.  Same as vmci_datagram_create_handle_priv without the privilege
  * flags argument.
  */
 int vmci_datagram_create_handle(u32 resource_id,
diff --git a/drivers/misc/vmw_vmci/vmci_queue_pair.c b/drivers/misc/vmw_vmci/vmci_queue_pair.c
index 46bba9bef553..e1a11820be01 100644
--- a/drivers/misc/vmw_vmci/vmci_queue_pair.c
+++ b/drivers/misc/vmw_vmci/vmci_queue_pair.c
@@ -2668,7 +2668,7 @@ static ssize_t qp_dequeue_locked(struct vmci_queue *produce_q,
  * @consume_qsize:      Desired size of the consumer queue.
  * @peer:       ContextID of the peer.
  * @flags:      VMCI flags.
- * @priv_flags: VMCI priviledge flags.
+ * @priv_flags: VMCI privilege flags.
  *
  * This is the client interface for allocating the memory for a
  * vmci_qp structure and then attaching to the underlying
-- 
2.47.3


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH v2] misc: fix typos in comments
  2026-10-06  6:01 [PATCH v2] misc: fix typos in comments Hemanth Selam
@ 2026-10-06 17:05 ` Vishnu Dasa
  0 siblings, 0 replies; 2+ messages in thread
From: Vishnu Dasa @ 2026-10-06 17:05 UTC (permalink / raw)
  To: Hemanth Selam
  Cc: Arnd Bergmann, Greg Kroah-Hartman, Frank Haverkamp, Bryan Tan,
	Broadcom internal kernel review list, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 7000 bytes --]

On Mon, Oct 5, 2026 at 11:01 PM Hemanth Selam <hemanth.selam@gmail.com> wrote:
>
> Fix typos in comments, reported by scripts/checkpatch.pl using the
> misspelling list in scripts/spelling.txt.  One further misspelling of
> "respectively" in genwqe/card_base.c is fixed as well: it is not in
> scripts/spelling.txt, but it sits a few lines from one that checkpatch
> does report.  Only comments are touched, there are no code changes.
>
> Assisted-by: LLM
> Signed-off-by: Hemanth Selam <hemanth.selam@gmail.com>

Acked-by: Vishnu Dasa <vishnu.dasa@broadcom.com>

Thanks. vmci changes look good to me.

> ---
>
> Notes (v2):
>     v2:
>      - Drop the drivers/char/lp.c hunk.  It is not under drivers/misc, which
>        is what the "misc:" prefix claimed.  It is now sent separately, as
>        "char: lp: fix typo in comment".
>      - Also fix a second misspelling in genwqe/card_base.c, a few lines from
>        one already being corrected.  checkpatch does not report it because
>        it is not listed in scripts/spelling.txt.
>
>     v1: https://lore.kernel.org/all/20260904113843.21367-1-hemanth.selam@gmail.com/
>
>  drivers/misc/gehc-achc.c                | 2 +-
>  drivers/misc/genwqe/card_base.c         | 8 ++++----
>  drivers/misc/genwqe/card_ddcb.c         | 4 ++--
>  drivers/misc/genwqe/card_utils.c        | 2 +-
>  drivers/misc/vmw_vmci/vmci_datagram.c   | 2 +-
>  drivers/misc/vmw_vmci/vmci_queue_pair.c | 2 +-
>  6 files changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/misc/gehc-achc.c b/drivers/misc/gehc-achc.c
> index 9792189c0923..2c1222edb5dc 100644
> --- a/drivers/misc/gehc-achc.c
> +++ b/drivers/misc/gehc-achc.c
> @@ -369,7 +369,7 @@ static int ezport_firmware_flash_data(struct spi_device *spi,
>         dev_dbg(&spi->dev, "EzPort verify flashed data...\n");
>         ret = ezport_firmware_compare_data(spi, data, size);
>
> -       /* allow missing FW verfication in secure mode */
> +       /* allow missing FW verification in secure mode */
>         if (ret == -EACCES)
>                 ret = 0;
>
> diff --git a/drivers/misc/genwqe/card_base.c b/drivers/misc/genwqe/card_base.c
> index e602eb3c51fa..3eb843af31a5 100644
> --- a/drivers/misc/genwqe/card_base.c
> +++ b/drivers/misc/genwqe/card_base.c
> @@ -286,7 +286,7 @@ int genwqe_flash_readback_fails(struct genwqe_dev *cd)
>   * @cd: GenWQE device information
>   *
>   * Note: From a design perspective it turned out to be a bad idea to
> - * use codes here to specifiy the frequency/speed values. An old
> + * use codes here to specify the frequency/speed values. An old
>   * driver cannot understand new codes and is therefore always a
>   * problem. Better is to measure out the value or put the
>   * speed/frequency directly into a register which is always a valid
> @@ -904,7 +904,7 @@ static int genwqe_reload_bistream(struct genwqe_dev *cd)
>   *
>   * Condition for the health-thread to trigger:
>   *   a) when a kthread_stop() request comes in or
> - *   b) a critical GFIR occured
> + *   b) a critical GFIR occurred
>   *
>   * Informational GFIRs are checked and potentially printed in
>   * GENWQE_HEALTH_CHECK_INTERVAL seconds.
> @@ -1210,10 +1210,10 @@ static int genwqe_probe(struct pci_dev *pci_dev,
>  }
>
>  /**
> - * genwqe_remove() - Called when device is removed (hot-plugable)
> + * genwqe_remove() - Called when device is removed (hot-pluggable)
>   * @pci_dev:   PCI device information struct
>   *
> - * Or when driver is unloaded respecitively when unbind is done.
> + * Or when driver is unloaded respectively when unbind is done.
>   */
>  static void genwqe_remove(struct pci_dev *pci_dev)
>  {
> diff --git a/drivers/misc/genwqe/card_ddcb.c b/drivers/misc/genwqe/card_ddcb.c
> index 969178573940..07e2c1be22d9 100644
> --- a/drivers/misc/genwqe/card_ddcb.c
> +++ b/drivers/misc/genwqe/card_ddcb.c
> @@ -460,7 +460,7 @@ static int genwqe_check_ddcb_queue(struct genwqe_dev *cd,
>  /**
>   * __genwqe_wait_ddcb(): Waits until DDCB is completed
>   * @cd:         pointer to genwqe device descriptor
> - * @req:        pointer to requsted DDCB parameters
> + * @req:        pointer to requested DDCB parameters
>   *
>   * The Service Layer will update the RETC in DDCB when processing is
>   * pending or done.
> @@ -543,7 +543,7 @@ int __genwqe_wait_ddcb(struct genwqe_dev *cd, struct ddcb_requ *req)
>                 return -EINVAL;
>         }
>
> -       /* Severe error occured. Driver is forced to stop operation */
> +       /* Severe error occurred. Driver is forced to stop operation */
>         if (cd->card_state != GENWQE_CARD_USED) {
>                 dev_err(&pci_dev->dev,
>                         "[%s] err: DDCB#%d forced to stop (rc=%d)\n",
> diff --git a/drivers/misc/genwqe/card_utils.c b/drivers/misc/genwqe/card_utils.c
> index e3b5fb337182..7c57cb881d7f 100644
> --- a/drivers/misc/genwqe/card_utils.c
> +++ b/drivers/misc/genwqe/card_utils.c
> @@ -1003,7 +1003,7 @@ u64 genwqe_read_vreg(struct genwqe_dev *cd, u32 reg, int func)
>   * @cd:                genwqe device descriptor
>   *
>   * Note: From a design perspective it turned out to be a bad idea to
> - * use codes here to specifiy the frequency/speed values. An old
> + * use codes here to specify the frequency/speed values. An old
>   * driver cannot understand new codes and is therefore always a
>   * problem. Better is to measure out the value or put the
>   * speed/frequency directly into a register which is always a valid
> diff --git a/drivers/misc/vmw_vmci/vmci_datagram.c b/drivers/misc/vmw_vmci/vmci_datagram.c
> index 6193a22c3ab5..73b43b097f92 100644
> --- a/drivers/misc/vmw_vmci/vmci_datagram.c
> +++ b/drivers/misc/vmw_vmci/vmci_datagram.c
> @@ -434,7 +434,7 @@ EXPORT_SYMBOL_GPL(vmci_datagram_create_handle_priv);
>   * @out_handle: vmci_handle that is populated as a result of this function.
>   *
>   * Creates a host context datagram endpoint and returns a handle to
> - * it.  Same as vmci_datagram_create_handle_priv without the priviledge
> + * it.  Same as vmci_datagram_create_handle_priv without the privilege
>   * flags argument.
>   */
>  int vmci_datagram_create_handle(u32 resource_id,
> diff --git a/drivers/misc/vmw_vmci/vmci_queue_pair.c b/drivers/misc/vmw_vmci/vmci_queue_pair.c
> index 46bba9bef553..e1a11820be01 100644
> --- a/drivers/misc/vmw_vmci/vmci_queue_pair.c
> +++ b/drivers/misc/vmw_vmci/vmci_queue_pair.c
> @@ -2668,7 +2668,7 @@ static ssize_t qp_dequeue_locked(struct vmci_queue *produce_q,
>   * @consume_qsize:      Desired size of the consumer queue.
>   * @peer:       ContextID of the peer.
>   * @flags:      VMCI flags.
> - * @priv_flags: VMCI priviledge flags.
> + * @priv_flags: VMCI privilege flags.
>   *
>   * This is the client interface for allocating the memory for a
>   * vmci_qp structure and then attaching to the underlying
> --
> 2.47.3
>

[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 5417 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-10-06 17:06 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-10-06  6:01 [PATCH v2] misc: fix typos in comments Hemanth Selam
2026-10-06 17:05 ` Vishnu Dasa

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®