On 3/14/23 04:16, Ziqi Chen wrote:
+static bool ufshcd_print_tr_iter(struct request *req, void *priv) +{ + struct scsi_device *sdev = req->q->queuedata; + struct Scsi_Host *shost = sdev->host; + struct ufs_hba *hba = shost_priv(shost); + + if (!IS_ERR_OR_NULL(hba)) + ufshcd_print_tr(hba, req->tag, *(bool *)priv);
I think that the "if (!IS_ERR_OR_NULL(hba))" code can be removed safely. Otherwise this patch looks good to me.
Thanks, Bart.