* [PATCH 1/2] scsi: isci: fix typos in comments
[not found] <20260904112623.4208-1-hemanth.selam@gmail.com>
@ 2026-09-04 11:26 ` Hemanth Selam
2026-09-04 11:26 ` [PATCH 2/2] scsi: isci: fix repeated words " Hemanth Selam
1 sibling, 0 replies; 2+ messages in thread
From: Hemanth Selam @ 2026-09-04 11:26 UTC (permalink / raw)
To: James E.J. Bottomley, Martin K. Petersen; +Cc: linux-kernel, linux-scsi
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/scsi/isci/host.c | 4 ++--
drivers/scsi/isci/phy.h | 2 +-
drivers/scsi/isci/port.c | 2 +-
drivers/scsi/isci/port_config.c | 2 +-
drivers/scsi/isci/registers.h | 14 +++++++-------
drivers/scsi/isci/remote_device.c | 4 ++--
drivers/scsi/isci/remote_device.h | 2 +-
drivers/scsi/isci/remote_node_context.h | 2 +-
drivers/scsi/isci/request.c | 12 ++++++------
drivers/scsi/isci/scu_task_context.h | 12 ++++++------
drivers/scsi/isci/unsolicited_frame_control.c | 2 +-
11 files changed, 29 insertions(+), 29 deletions(-)
diff --git a/drivers/scsi/isci/host.c b/drivers/scsi/isci/host.c
index 1592fc552e6c..bb5e6e302042 100644
--- a/drivers/scsi/isci/host.c
+++ b/drivers/scsi/isci/host.c
@@ -338,7 +338,7 @@ static void sci_controller_unsolicited_frame(struct isci_host *ihost, u32 ent)
if (index == SCIC_SDS_REMOTE_NODE_CONTEXT_INVALID_INDEX) {
/*
* This is a signature fis or a frame from a direct attached SATA
- * device that has not yet been created. In either case forwared
+ * device that has not yet been created. In either case forwarded
* the frame to the PE and let it take care of the frame data. */
index = SCU_GET_PROTOCOL_ENGINE_INDEX(ent);
iphy = &ihost->phys[index];
@@ -374,7 +374,7 @@ static void sci_controller_event_completion(struct isci_host *ihost, u32 ent)
switch (scu_get_event_type(ent)) {
case SCU_EVENT_TYPE_SMU_COMMAND_ERROR:
- /* / @todo The driver did something wrong and we need to fix the condtion. */
+ /* / @todo The driver did something wrong and we need to fix the condition. */
dev_err(&ihost->pdev->dev,
"%s: SCIC Controller 0x%p received SMU command error "
"0x%x\n",
diff --git a/drivers/scsi/isci/phy.h b/drivers/scsi/isci/phy.h
index 5aaf95b14b2e..8c90e89ccced 100644
--- a/drivers/scsi/isci/phy.h
+++ b/drivers/scsi/isci/phy.h
@@ -342,7 +342,7 @@ enum sci_phy_counter_id {
* machine.
* @SCI_PHY_STOPPED: phy has successfully been stopped. In this state
* no new IO operations are permitted on this phy.
- * @SCI_PHY_STARTING: the phy is in the process of becomming ready. In
+ * @SCI_PHY_STARTING: the phy is in the process of becoming ready. In
* this state no new IO operations are permitted on
* this phy.
* @SCI_PHY_SUB_INITIAL: Initial state
diff --git a/drivers/scsi/isci/port.c b/drivers/scsi/isci/port.c
index 0dea0abb9927..7b48d4c00036 100644
--- a/drivers/scsi/isci/port.c
+++ b/drivers/scsi/isci/port.c
@@ -666,7 +666,7 @@ static void sci_port_invalid_link_up(struct isci_port *iport, struct isci_phy *i
struct isci_host *ihost = iport->owning_controller;
/*
- * Check to see if we have alreay reported this link as bad and if
+ * Check to see if we have already reported this link as bad and if
* not go ahead and tell the SCI_USER that we have discovered an
* invalid link.
*/
diff --git a/drivers/scsi/isci/port_config.c b/drivers/scsi/isci/port_config.c
index d709c9df823c..4c4b23fc88a2 100644
--- a/drivers/scsi/isci/port_config.c
+++ b/drivers/scsi/isci/port_config.c
@@ -557,7 +557,7 @@ static void sci_apc_agent_configure_ports(struct isci_host *ihost,
/*
* Check to see if the start timer operations should instead map to an
* add phy operation. This is caused because we have been waiting to
- * add a phy to a port but could not becuase the automatic port
+ * add a phy to a port but could not because the automatic port
* configuration engine had a choice of possible ports for the phy.
* Since we have gone through a timeout we are going to restrict the
* choice to the smallest possible port. */
diff --git a/drivers/scsi/isci/registers.h b/drivers/scsi/isci/registers.h
index 63468cfe3e4a..ecf3b5b2e7ca 100644
--- a/drivers/scsi/isci/registers.h
+++ b/drivers/scsi/isci/registers.h
@@ -909,13 +909,13 @@ struct scu_iit_entry {
* To calculate the offset for other functions use
* BAR0 + FN# * SystemPageSize * 2
*
- * The TCA is only accessable from FN#0 (Physical Function) and each
+ * The TCA is only accessible from FN#0 (Physical Function) and each
* is programmed by (BAR0 + SCU_SMU_TCA_OFFSET + (FN# * 0x04)) or
* TCA0 for FN#0 is at BAR0 + 0x0400
* TCA1 for FN#1 is at BAR0 + 0x0404
* etc.
* ----------------------------------------------------------------------------
- * Accessable to all FN#s */
+ * Accessible to all FN#s */
#define SCU_SMU_PCP_OFFSET 0x0000
#define SCU_SMU_AMR_OFFSET 0x0004
#define SCU_SMU_ISR_OFFSET 0x0010
@@ -929,7 +929,7 @@ struct scu_iit_entry {
#define SCU_SMU_CQPR_OFFSET 0x0040
#define SCU_SMU_CQGR_OFFSET 0x0044
#define SCU_SMU_CQC_OFFSET 0x0048
-/* Accessable to FN#0 only */
+/* Accessible to FN#0 only */
#define SCU_SMU_RNCLBAR_OFFSET 0x0080
#define SCU_SMU_RNCUBAR_OFFSET 0x0084
#define SCU_SMU_DCC_OFFSET 0x0090
@@ -938,9 +938,9 @@ struct scu_iit_entry {
#define SCU_SMU_SCUSRCR_OFFSET 0x009C
#define SCU_SMU_SMAW_OFFSET 0x00A0
#define SCU_SMU_SMDW_OFFSET 0x00A4
-/* Accessable to FN#0 only */
+/* Accessible to FN#0 only */
#define SCU_SMU_TCA_OFFSET 0x0400
-/* Accessable to all FN#s */
+/* Accessible to all FN#s */
#define SCU_SMU_MT_MLAR0_OFFSET 0x2000
#define SCU_SMU_MT_MUAR0_OFFSET 0x2004
#define SCU_SMU_MT_MDR0_OFFSET 0x2008
@@ -994,7 +994,7 @@ struct smu_registers {
u32 reserved_6x[4];
u32 reserved_7x[4];
/*
- * Accessable to FN#0 only
+ * Accessible to FN#0 only
* 0x0080 RNCLBAR */
u32 remote_node_context_lower;
/* 0x0084 RNCUBAR */
@@ -1027,7 +1027,7 @@ struct smu_registers {
u32 reserved_2xx[64];
u32 reserved_3xx[64];
/*
- * Accessable to FN#0 only
+ * Accessible to FN#0 only
* 0x0400 TCA */
u32 task_context_assignment[256];
/* MSI-X registers not included */
diff --git a/drivers/scsi/isci/remote_device.c b/drivers/scsi/isci/remote_device.c
index 4c7462965ea1..33742de2e876 100644
--- a/drivers/scsi/isci/remote_device.c
+++ b/drivers/scsi/isci/remote_device.c
@@ -646,7 +646,7 @@ enum sci_status sci_remote_device_start_io(struct isci_host *ihost,
break;
case SCI_STP_DEV_IDLE: {
/* handle the start io operation for a sata device that is in
- * the command idle state. - Evalute the type of IO request to
+ * the command idle state. - Evaluate the type of IO request to
* be started - If its an NCQ request change to NCQ substate -
* If its any other command change to the CMD substate
*
@@ -964,7 +964,7 @@ static void sci_remote_device_initial_state_enter(struct sci_base_state_machine
* successfully destructed or if some failure occurred. enum sci_status This value
* is returned if the device is successfully destructed.
* SCI_FAILURE_INVALID_REMOTE_DEVICE This value is returned if the supplied
- * device isn't valid (e.g. it's already been destoryed, the handle isn't
+ * device isn't valid (e.g. it's already been destroyed, the handle isn't
* valid, etc.).
*/
static enum sci_status sci_remote_device_destruct(struct isci_remote_device *idev)
diff --git a/drivers/scsi/isci/remote_device.h b/drivers/scsi/isci/remote_device.h
index c1fdf45751cd..e7c923909920 100644
--- a/drivers/scsi/isci/remote_device.h
+++ b/drivers/scsi/isci/remote_device.h
@@ -189,7 +189,7 @@ enum sci_status sci_remote_device_stop(
* requests until this command is complete.
*
* @SCI_STP_DEV_NCQ: This is the NCQ state for the STP remote device.
- * This state is entered when the device is processing an NCQ reuqest.
+ * This state is entered when the device is processing an NCQ request.
* It will remain in this state so long as there is one or more NCQ
* requests being processed.
*
diff --git a/drivers/scsi/isci/remote_node_context.h b/drivers/scsi/isci/remote_node_context.h
index c7ee81d01125..e069d7ca78ef 100644
--- a/drivers/scsi/isci/remote_node_context.h
+++ b/drivers/scsi/isci/remote_node_context.h
@@ -102,7 +102,7 @@ typedef void (*scics_sds_remote_node_context_callback)(void *);
* transition to the posting state.
*
* @SCI_RNC_RESUMING: transition state that will post an RNC resume to the
- * hardare. Once the event notification of resume complete is received the
+ * hardware. Once the event notification of resume complete is received the
* remote node context will transition to the ready state.
*
* @SCI_RNC_READY: state that the remote node context must be in to accept io
diff --git a/drivers/scsi/isci/request.c b/drivers/scsi/isci/request.c
index bb89a2e33eb4..a061e2698316 100644
--- a/drivers/scsi/isci/request.c
+++ b/drivers/scsi/isci/request.c
@@ -539,7 +539,7 @@ static void scu_sata_request_construct_task_context(
/*
* Copy the physical address for the command buffer to the SCU Task
* Context. We must offset the command buffer by 4 bytes because the
- * first 4 bytes are transfered in the body of the TC.
+ * first 4 bytes are transferred in the body of the TC.
*/
dma_addr = sci_io_request_get_dma_addr(ireq,
((char *) &ireq->stp.cmd) +
@@ -613,7 +613,7 @@ static void sci_stp_optimized_request_construct(struct isci_request *ireq,
/* Copy over the SGL elements */
sci_request_build_sgl(ireq);
- /* Copy over the number of bytes to be transfered */
+ /* Copy over the number of bytes to be transferred */
task_context->transfer_length_bytes = len;
if (dir == DMA_TO_DEVICE) {
@@ -1310,7 +1310,7 @@ stp_request_non_data_await_h2d_tc_event(struct isci_request *ireq,
#define SCU_MAX_FRAME_BUFFER_SIZE 0x400 /* 1K is the maximum SCU frame data payload */
/* transmit DATA_FIS from (current sgl + offset) for input
- * parameter length. current sgl and offset is alreay stored in the IO request
+ * parameter length. current sgl and offset is already stored in the IO request
*/
static enum sci_status sci_stp_request_pio_data_out_trasmit_data_frame(
struct isci_request *ireq,
@@ -1458,7 +1458,7 @@ static enum sci_status sci_stp_request_pio_data_in_copy_data(
if (status == SCI_SUCCESS)
stp_req->pio_len = 0;
} else {
- /* We are transfering the whole frame so copy */
+ /* We are transferring the whole frame so copy */
status = sci_stp_request_pio_data_in_copy_data_buffer(
stp_req, data_buffer, SCU_MAX_FRAME_BUFFER_SIZE);
@@ -3298,7 +3298,7 @@ sci_io_request_construct_smp(struct device *dev,
* @ireq: This parameter points to the isci_request allocated in the
* request construct function.
*
- * SCI_SUCCESS on successfull completion, or specific failure code.
+ * SCI_SUCCESS on successful completion, or specific failure code.
*/
static enum sci_status isci_smp_request_build(struct isci_request *ireq)
{
@@ -3324,7 +3324,7 @@ static enum sci_status isci_smp_request_build(struct isci_request *ireq)
* @idev: This parameter is the handle for the sci core's remote device
* object that is the destination for this request.
*
- * SCI_SUCCESS on successfull completion, or specific failure code.
+ * SCI_SUCCESS on successful completion, or specific failure code.
*/
static enum sci_status isci_io_request_build(struct isci_host *ihost,
struct isci_request *request,
diff --git a/drivers/scsi/isci/scu_task_context.h b/drivers/scsi/isci/scu_task_context.h
index f258f4014e7f..ec525f76720a 100644
--- a/drivers/scsi/isci/scu_task_context.h
+++ b/drivers/scsi/isci/scu_task_context.h
@@ -171,7 +171,7 @@ typedef enum {
#define SCU_TASK_REGULAR 0
#define SCU_TASK_ABORTED 1
-/* direction bit defintion */
+/* direction bit definition */
/**
*
*
@@ -265,7 +265,7 @@ typedef enum {
/**
*
*
- * SCU_TASK_CONTEXT_PROTOCOL SCU Task context protocol types this is uesd to
+ * SCU_TASK_CONTEXT_PROTOCOL SCU Task context protocol types this is used to
* program the SCU Task context protocol field in word 0x00.
*/
#define SCU_TASK_CONTEXT_PROTOCOL_SMP 0x00
@@ -452,7 +452,7 @@ struct scu_sgl_element {
/**
* struct scu_sgl_element_pair - This structure is the SCU hardware definition
* of a pair of SGL elements. The SCU hardware always works on SGL pairs.
- * They are refered to in the DS specification as SGL A and SGL B. Each SGL
+ * They are referred to in the DS specification as SGL A and SGL B. Each SGL
* pair is followed by the address of the next pair.
*
*
@@ -599,7 +599,7 @@ struct scu_task_context {
u32 mirrored_node_index:12;
/**
- * This field is programmed with the direction of the SATA reqeust
+ * This field is programmed with the direction of the SATA request
* - SCU_SATA_WRITE_DATA_DIRECTION
* - SCU_SATA_READ_DATA_DIRECTION
*/
@@ -671,7 +671,7 @@ struct scu_task_context {
u32 strict_ordering:1;
/**
- * This field indicates the type of endianess to be utilized for the
+ * This field indicates the type of endianness to be utilized for the
* frame. command, task, and response frames utilized control_frame
* set to 1.
*/
@@ -758,7 +758,7 @@ struct scu_task_context {
/* OFFSET 0x14 */
/**
- * This filed is set to the number of bytes to be transfered in the request.
+ * This filed is set to the number of bytes to be transferred in the request.
*/
u32 transfer_length_bytes:24; /* In terms of bytes */
diff --git a/drivers/scsi/isci/unsolicited_frame_control.c b/drivers/scsi/isci/unsolicited_frame_control.c
index 04a6d0d59a22..fb0006e28cc8 100644
--- a/drivers/scsi/isci/unsolicited_frame_control.c
+++ b/drivers/scsi/isci/unsolicited_frame_control.c
@@ -130,7 +130,7 @@ enum sci_status sci_unsolicited_frame_control_get_header(struct sci_unsolicited_
void **frame_header)
{
if (frame_index < SCU_MAX_UNSOLICITED_FRAMES) {
- /* Skip the first word in the frame since this is a controll word used
+ /* Skip the first word in the frame since this is a control word used
* by the hardware.
*/
*frame_header = &uf_control->buffers.array[frame_index].header->data;
--
2.48.1
^ permalink raw reply related [flat|nested] 2+ messages in thread* [PATCH 2/2] scsi: isci: fix repeated words in comments
[not found] <20260904112623.4208-1-hemanth.selam@gmail.com>
2026-09-04 11:26 ` [PATCH 1/2] scsi: isci: fix typos in comments Hemanth Selam
@ 2026-09-04 11:26 ` Hemanth Selam
1 sibling, 0 replies; 2+ messages in thread
From: Hemanth Selam @ 2026-09-04 11:26 UTC (permalink / raw)
To: James E.J. Bottomley, Martin K. Petersen; +Cc: linux-kernel, linux-scsi
Drop words accidentally written twice, reported by checkpatch.pl as a
possible repeated word. Only touches comments, no code changes.
Assisted-by: Cursor:claude-opus-5
Signed-off-by: Hemanth Selam <hemanth.selam@gmail.com>
---
drivers/scsi/isci/host.h | 2 +-
drivers/scsi/isci/remote_device.h | 4 ++--
drivers/scsi/isci/remote_node_context.h | 2 +-
drivers/scsi/isci/task.c | 2 +-
drivers/scsi/isci/unsolicited_frame_control.h | 2 +-
5 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/scsi/isci/host.h b/drivers/scsi/isci/host.h
index 52388374cf31..e4971ca00769 100644
--- a/drivers/scsi/isci/host.h
+++ b/drivers/scsi/isci/host.h
@@ -244,7 +244,7 @@ enum sci_controller_states {
SCIC_INITIALIZED,
/**
- * This state indicates the the controller is in the process of becoming
+ * This state indicates the controller is in the process of becoming
* ready (i.e. starting). In this state no new IO operations are permitted.
* This state is entered from the INITIALIZED state.
*/
diff --git a/drivers/scsi/isci/remote_device.h b/drivers/scsi/isci/remote_device.h
index e7c923909920..a8175b2fbee3 100644
--- a/drivers/scsi/isci/remote_device.h
+++ b/drivers/scsi/isci/remote_device.h
@@ -170,7 +170,7 @@ enum sci_status sci_remote_device_stop(
* permitted. This state is entered from the INITIAL state. This state
* is entered from the STOPPING state.
*
- * @SCI_DEV_STARTING: This state indicates the the remote device is in
+ * @SCI_DEV_STARTING: This state indicates the remote device is in
* the process of becoming ready (i.e. starting). In this state no new
* IO operations are permitted. This state is entered from the STOPPED
* state.
@@ -180,7 +180,7 @@ enum sci_status sci_remote_device_stop(
* This state is entered from the STARTING state.
*
* @SCI_STP_DEV_IDLE: This is the idle substate for the stp remote
- * device. When there are no active IO for the device it is is in this
+ * device. When there are no active IO for the device it is in this
* state.
*
* @SCI_STP_DEV_CMD: This is the command state for the STP remote
diff --git a/drivers/scsi/isci/remote_node_context.h b/drivers/scsi/isci/remote_node_context.h
index e069d7ca78ef..b5c18d48948b 100644
--- a/drivers/scsi/isci/remote_node_context.h
+++ b/drivers/scsi/isci/remote_node_context.h
@@ -154,7 +154,7 @@ enum sci_remote_node_context_destination_state {
/**
* struct sci_remote_node_context - This structure contains the data
* associated with the remote node context object. The remote node context
- * (RNC) object models the the remote device information necessary to manage
+ * (RNC) object models the remote device information necessary to manage
* the silicon RNC.
*/
struct sci_remote_node_context {
diff --git a/drivers/scsi/isci/task.c b/drivers/scsi/isci/task.c
index 3a25b1a2c52d..aeb2cda92465 100644
--- a/drivers/scsi/isci/task.c
+++ b/drivers/scsi/isci/task.c
@@ -67,7 +67,7 @@
/**
* isci_task_refuse() - complete the request to the upper layer driver in
* the case where an I/O needs to be completed back in the submit path.
-* @ihost: host on which the the request was queued
+* @ihost: host on which the request was queued
* @task: request to complete
* @response: response code for the completed task.
* @status: status code for the completed task.
diff --git a/drivers/scsi/isci/unsolicited_frame_control.h b/drivers/scsi/isci/unsolicited_frame_control.h
index 1bc551ec611f..f9c816d9cd38 100644
--- a/drivers/scsi/isci/unsolicited_frame_control.h
+++ b/drivers/scsi/isci/unsolicited_frame_control.h
@@ -118,7 +118,7 @@ enum unsolicited_frame_state {
UNSOLICITED_FRAME_EMPTY,
/**
- * This state is set when the frame buffer is in use by by some
+ * This state is set when the frame buffer is in use by some
* object in the system.
*/
UNSOLICITED_FRAME_IN_USE,
--
2.48.1
^ permalink raw reply related [flat|nested] 2+ messages in thread