Commit b4dc06a9 authored by Kevin Barnett's avatar Kevin Barnett Committed by Martin K. Petersen

scsi: smartpqi: Fix a name typo and cleanup code

Rename the function pqi_is_io_high_priority() to pqi_is_io_high_priority().
Remove 2 unnecessary lines from the function, and adjusted the white space.

Link: https://lore.kernel.org/r/164375209818.440833.10908948825731635853.stgit@brunhilda.pdev.netReviewed-by: default avatarMike McGowen <mike.mcgowen@microchip.com>
Reviewed-by: default avatarScott Benesh <scott.benesh@microchip.com>
Reviewed-by: default avatarScott Teel <scott.teel@microchip.com>
Signed-off-by: default avatarKevin Barnett <kevin.barnett@microchip.com>
Signed-off-by: default avatarDon Brace <don.brace@microchip.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 94a68c81
...@@ -5581,18 +5581,19 @@ static void pqi_aio_io_complete(struct pqi_io_request *io_request, ...@@ -5581,18 +5581,19 @@ static void pqi_aio_io_complete(struct pqi_io_request *io_request,
pqi_scsi_done(scmd); pqi_scsi_done(scmd);
} }
static inline bool pqi_is_io_high_prioity(struct pqi_ctrl_info *ctrl_info, static inline bool pqi_is_io_high_priority(struct pqi_ctrl_info *ctrl_info,
struct pqi_scsi_dev *device, struct scsi_cmnd *scmd) struct pqi_scsi_dev *device, struct scsi_cmnd *scmd)
{ {
bool io_high_prio; bool io_high_prio;
int priority_class; int priority_class;
io_high_prio = false; io_high_prio = false;
if (device->ncq_prio_enable) { if (device->ncq_prio_enable) {
priority_class = priority_class =
IOPRIO_PRIO_CLASS(req_get_ioprio(scsi_cmd_to_rq(scmd))); IOPRIO_PRIO_CLASS(req_get_ioprio(scsi_cmd_to_rq(scmd)));
if (priority_class == IOPRIO_CLASS_RT) { if (priority_class == IOPRIO_CLASS_RT) {
/* set NCQ priority for read/write command */ /* Set NCQ priority for read/write commands. */
switch (scmd->cmnd[0]) { switch (scmd->cmnd[0]) {
case WRITE_16: case WRITE_16:
case READ_16: case READ_16:
...@@ -5604,8 +5605,6 @@ static inline bool pqi_is_io_high_prioity(struct pqi_ctrl_info *ctrl_info, ...@@ -5604,8 +5605,6 @@ static inline bool pqi_is_io_high_prioity(struct pqi_ctrl_info *ctrl_info,
case READ_6: case READ_6:
io_high_prio = true; io_high_prio = true;
break; break;
default:
break;
} }
} }
} }
...@@ -5619,7 +5618,8 @@ static inline int pqi_aio_submit_scsi_cmd(struct pqi_ctrl_info *ctrl_info, ...@@ -5619,7 +5618,8 @@ static inline int pqi_aio_submit_scsi_cmd(struct pqi_ctrl_info *ctrl_info,
{ {
bool io_high_prio; bool io_high_prio;
io_high_prio = pqi_is_io_high_prioity(ctrl_info, device, scmd); io_high_prio = pqi_is_io_high_priority(ctrl_info, device, scmd);
return pqi_aio_submit_io(ctrl_info, scmd, device->aio_handle, return pqi_aio_submit_io(ctrl_info, scmd, device->aio_handle,
scmd->cmnd, scmd->cmd_len, queue_group, NULL, scmd->cmnd, scmd->cmd_len, queue_group, NULL,
false, io_high_prio); false, io_high_prio);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment