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.