scsi: ufs: core: Remove a ufshcd_add_command_trace() call

bvanassche · martinkpetersen · commit 72554035b979 · 2023-06-07T21:20:21.000-04:00
ufshcd_add_command_trace() traces SCSI commands. Remove a ufshcd_add_command_trace() call from a code path that is not related to SCSI commands. Signed-off-by: Bart Van Assche <bvanassche@acm.org> Link: https://lore.kernel.org/r/20230531224050.25554-1-bvanassche@acm.org Reviewed-by: Avri Altman <avri.altman@wdc.com> Reviewed-by: Bean Huo <beanhuo@micron.com> Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
diff --git a/drivers/ufs/core/ufshcd.c b/drivers/ufs/core/ufshcd.c
@@ -5447,7 +5447,6 @@ void ufshcd_compl_one_cqe(struct ufs_hba *hba, int task_tag,
 		   lrbp->command_type == UTP_CMD_TYPE_UFS_STORAGE) {
 		if (hba->dev_cmd.complete) {
 			hba->dev_cmd.cqe = cqe;
-			ufshcd_add_command_trace(hba, task_tag, UFS_DEV_COMP);
 			complete(hba->dev_cmd.complete);
 			ufshcd_clk_scaling_update_busy(hba);
 		}

Original file line number	Diff line number	Diff line change
`@@ -5447,7 +5447,6 @@ void ufshcd_compl_one_cqe(struct ufs_hba *hba, int task_tag,`
`5447`	`5447`	`lrbp->command_type == UTP_CMD_TYPE_UFS_STORAGE) {`
`5448`	`5448`	`if (hba->dev_cmd.complete) {`
`5449`	`5449`	`hba->dev_cmd.cqe = cqe;`
`5450`		`- ufshcd_add_command_trace(hba, task_tag, UFS_DEV_COMP);`
`5451`	`5450`	`complete(hba->dev_cmd.complete);`
`5452`	`5451`	`ufshcd_clk_scaling_update_busy(hba);`
`5453`	`5452`	`}`