]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
nvme: fix multiple spelling and grammar issues in host drivers
authorAlok Tiwari <alok.a.tiwari@oracle.com>
Wed, 25 Jun 2025 04:16:30 +0000 (21:16 -0700)
committerChristoph Hellwig <hch@lst.de>
Thu, 17 Jul 2025 11:38:06 +0000 (13:38 +0200)
This commit fixes several typos and grammatical issues across various
nvme host driver files:

 - correct "glace" to "glance" in a comment in apple.c
 - fix "Idependent" to "Independent" in core.c
 - change "unsucceesful" to "unsuccessful", "they blk-mq" to "the blk-mq",
 - fix "terminaed" to "terminated" and other grammar in fc.c
 - update "O's" to "0's" to clarify meaning in nvme.h
 - fix a function name reference in a comment in zns.c:
   *_transter_len() -> *_transfer_len().
 - fix sysfs_emit() output format in pci.c (replace x%08x with 0x%08x)

These changes improve the code readability and documentation consistency
across the NVMe driver.

Signed-off-by: Alok Tiwari <alok.a.tiwari@oracle.com>
Reviewed-by: Randy Dunlap <rdunlap@infradead.org>
Reviewed-by: Chaitanya Kulkarni <kch@nvidia.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
drivers/nvme/host/apple.c
drivers/nvme/host/core.c
drivers/nvme/host/fc.c
drivers/nvme/host/nvme.h
drivers/nvme/host/pci.c
drivers/nvme/host/rdma.c
drivers/nvme/target/passthru.c
drivers/nvme/target/zns.c

index b1fddfa33ab97365c9924ab4d766ecf5615bf26d..1286c31320e630cb012009d6b962526e0553869f 100644 (file)
@@ -301,8 +301,8 @@ static void apple_nvme_submit_cmd(struct apple_nvme_queue *q,
        memcpy(&q->sqes[tag], cmd, sizeof(*cmd));
 
        /*
-        * This lock here doesn't make much sense at a first glace but
-        * removing it will result in occasional missed completetion
+        * This lock here doesn't make much sense at a first glance but
+        * removing it will result in occasional missed completion
         * interrupts even though the commands still appear on the CQ.
         * It's unclear why this happens but our best guess is that
         * there is a bug in the firmware triggered when a new command
index e533d791955d137911fc53f4f4de825796f8a43b..5d8638086cba56fc3a97eac35eb0c742a8e4346b 100644 (file)
@@ -4286,7 +4286,7 @@ static void nvme_scan_ns(struct nvme_ctrl *ctrl, unsigned nsid)
        }
 
        /*
-        * If available try to use the Command Set Idependent Identify Namespace
+        * If available try to use the Command Set Independent Identify Namespace
         * data structure to find all the generic information that is needed to
         * set up a namespace.  If not fall back to the legacy version.
         */
index 014b387f1e8b1f1df1b9fcaaaf3fbff879abb564..08a5ea3e93836b9f79dc73d724b07be4076e4473 100644 (file)
@@ -899,7 +899,7 @@ EXPORT_SYMBOL_GPL(nvme_fc_set_remoteport_devloss);
  * may crash.
  *
  * As such:
- * Wrapper all the dma routines and check the dev pointer.
+ * Wrap all the dma routines and check the dev pointer.
  *
  * If simple mappings (return just a dma address, we'll noop them,
  * returning a dma address of 0.
@@ -1955,8 +1955,8 @@ nvme_fc_fcpio_done(struct nvmefc_fcp_req *req)
        }
 
        /*
-        * For the linux implementation, if we have an unsucceesful
-        * status, they blk-mq layer can typically be called with the
+        * For the linux implementation, if we have an unsuccessful
+        * status, the blk-mq layer can typically be called with the
         * non-zero status and the content of the cqe isn't important.
         */
        if (status)
@@ -2429,7 +2429,7 @@ static bool nvme_fc_terminate_exchange(struct request *req, void *data)
 
 /*
  * This routine runs through all outstanding commands on the association
- * and aborts them.  This routine is typically be called by the
+ * and aborts them.  This routine is typically called by the
  * delete_association routine. It is also called due to an error during
  * reconnect. In that scenario, it is most likely a command that initializes
  * the controller, including fabric Connect commands on io queues, that
@@ -2622,7 +2622,7 @@ nvme_fc_unmap_data(struct nvme_fc_ctrl *ctrl, struct request *rq,
  * as part of the exchange.  The CQE is the last thing for the io,
  * which is transferred (explicitly or implicitly) with the RSP IU
  * sent on the exchange. After the CQE is received, the FC exchange is
- * terminaed and the Exchange may be used on a different io.
+ * terminated and the Exchange may be used on a different io.
  *
  * The transport to LLDD api has the transport making a request for a
  * new fcp io request to the LLDD. The LLDD then allocates a FC exchange
index 7df2ea21851f5bdb3ac8630bd99686ab97b2d50b..cfd2b5b90b915728e0c56e8b7c9971b771806c43 100644 (file)
@@ -69,7 +69,7 @@ enum nvme_quirks {
        NVME_QUIRK_IDENTIFY_CNS                 = (1 << 1),
 
        /*
-        * The controller deterministically returns O's on reads to
+        * The controller deterministically returns 0's on reads to
         * logical blocks that deallocate was called on.
         */
        NVME_QUIRK_DEALLOCATE_ZEROES            = (1 << 2),
index 9055741fffd99f80a11f7d38f8fb398e96db3526..16bf4cb2d063fdb011ebc7808bf7bd73fa0679c9 100644 (file)
@@ -2439,7 +2439,7 @@ static ssize_t cmb_show(struct device *dev, struct device_attribute *attr,
 {
        struct nvme_dev *ndev = to_nvme_dev(dev_get_drvdata(dev));
 
-       return sysfs_emit(buf, "cmbloc : x%08x\ncmbsz  : x%08x\n",
+       return sysfs_emit(buf, "cmbloc : 0x%08x\ncmbsz  : 0x%08x\n",
                       ndev->cmbloc, ndev->cmbsz);
 }
 static DEVICE_ATTR_RO(cmb);
index 9bd3646568d03c23d02cb9096d05479aaf3f8f37..190a4cfa8a5ee2e6b97a5a1b304c1338dcd748fa 100644 (file)
@@ -877,7 +877,7 @@ static int nvme_rdma_configure_io_queues(struct nvme_rdma_ctrl *ctrl, bool new)
 
        /*
         * Only start IO queues for which we have allocated the tagset
-        * and limitted it to the available queues. On reconnects, the
+        * and limited it to the available queues. On reconnects, the
         * queue number might have changed.
         */
        nr_queues = min(ctrl->tag_set.nr_hw_queues + 1, ctrl->ctrl.queue_count);
index b7515c53829b89d960ce182f7b09bbdfad89d627..3b4b0df8f879951436f91287bba6d9f4b8d21484 100644 (file)
@@ -106,7 +106,7 @@ static u16 nvmet_passthru_override_id_ctrl(struct nvmet_req *req)
                                      pctrl->max_hw_sectors);
 
        /*
-        * nvmet_passthru_map_sg is limitted to using a single bio so limit
+        * nvmet_passthru_map_sg is limited to using a single bio so limit
         * the mdts based on BIO_MAX_VECS as well
         */
        max_hw_sectors = min_not_zero(BIO_MAX_VECS << PAGE_SECTORS_SHIFT,
@@ -147,7 +147,7 @@ static u16 nvmet_passthru_override_id_ctrl(struct nvmet_req *req)
         * When passthru controller is setup using nvme-loop transport it will
         * export the passthru ctrl subsysnqn (PCIe NVMe ctrl) and will fail in
         * the nvme/host/core.c in the nvme_init_subsystem()->nvme_active_ctrl()
-        * code path with duplicate ctr subsynqn. In order to prevent that we
+        * code path with duplicate ctrl subsysnqn. In order to prevent that we
         * mask the passthru-ctrl subsysnqn with the target ctrl subsysnqn.
         */
        memcpy(id->subnqn, ctrl->subsysnqn, sizeof(id->subnqn));
index 29a60fabfcc8ace2c1fb80e27ad41151a03d45ea..15a579cf528c1e32f3a23a7b6268d6afeb6f8145 100644 (file)
@@ -541,7 +541,7 @@ void nvmet_bdev_execute_zone_append(struct nvmet_req *req)
        struct bio *bio;
        int sg_cnt;
 
-       /* Request is completed on len mismatch in nvmet_check_transter_len() */
+       /* Request is completed on len mismatch in nvmet_check_transfer_len() */
        if (!nvmet_check_transfer_len(req, nvmet_rw_data_len(req)))
                return;