mirror of
https://github.com/torvalds/linux.git
synced 2026-06-07 05:55:44 +02:00
FROMGIT: scsi: ufs: Remove the 'update_scaling' local variable
This patch does not change any functionality but makes the next patch in
this series easier to read.
Link: https://lore.kernel.org/r/20211203231950.193369-12-bvanassche@acm.org
Tested-by: Bean Huo <beanhuo@micron.com>
Reviewed-by: Bean Huo <beanhuo@micron.com>
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
(cherry picked from commit 3eb9dcc027 git://git.kernel.org/pub/scm/linux/kernel/git/mkp/scsi.git for-next)
Bug: 204438323
Change-Id: I5a420ba06517e65aa2cbabf08c2fc78de2490def
Signed-off-by: Bart Van Assche <bvanassche@google.com>
Signed-off-by: Stanley Chu <stanley.chu@mediatek.com>
This commit is contained in:
parent
5f9614157c
commit
d600bdedac
|
|
@ -5193,7 +5193,6 @@ static void __ufshcd_transfer_req_compl(struct ufs_hba *hba,
|
||||||
struct scsi_cmnd *cmd;
|
struct scsi_cmnd *cmd;
|
||||||
int result;
|
int result;
|
||||||
int index;
|
int index;
|
||||||
bool update_scaling = false;
|
|
||||||
|
|
||||||
for_each_set_bit(index, &completed_reqs, hba->nutrs) {
|
for_each_set_bit(index, &completed_reqs, hba->nutrs) {
|
||||||
if (!test_and_clear_bit(index, &hba->outstanding_reqs))
|
if (!test_and_clear_bit(index, &hba->outstanding_reqs))
|
||||||
|
|
@ -5215,7 +5214,7 @@ static void __ufshcd_transfer_req_compl(struct ufs_hba *hba,
|
||||||
/* Do not touch lrbp after scsi done */
|
/* Do not touch lrbp after scsi done */
|
||||||
cmd->scsi_done(cmd);
|
cmd->scsi_done(cmd);
|
||||||
ufshcd_release(hba);
|
ufshcd_release(hba);
|
||||||
update_scaling = true;
|
ufshcd_clk_scaling_update_busy(hba);
|
||||||
} else if (lrbp->command_type == UTP_CMD_TYPE_DEV_MANAGE ||
|
} else if (lrbp->command_type == UTP_CMD_TYPE_DEV_MANAGE ||
|
||||||
lrbp->command_type == UTP_CMD_TYPE_UFS_STORAGE) {
|
lrbp->command_type == UTP_CMD_TYPE_UFS_STORAGE) {
|
||||||
if (hba->dev_cmd.complete) {
|
if (hba->dev_cmd.complete) {
|
||||||
|
|
@ -5223,12 +5222,10 @@ static void __ufshcd_transfer_req_compl(struct ufs_hba *hba,
|
||||||
ufshcd_add_command_trace(hba, index,
|
ufshcd_add_command_trace(hba, index,
|
||||||
"dev_complete");
|
"dev_complete");
|
||||||
complete(hba->dev_cmd.complete);
|
complete(hba->dev_cmd.complete);
|
||||||
update_scaling = true;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
if (update_scaling)
|
|
||||||
ufshcd_clk_scaling_update_busy(hba);
|
ufshcd_clk_scaling_update_busy(hba);
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue
Block a user