On Thu, Mar 12, 2020 at 01:02:49AM -0700, Satya Tangirala wrote: > @@ -2470,6 +2504,9 @@ static int ufshcd_compose_dev_cmd(struct ufs_hba *hba, > lrbp->task_tag = tag; > lrbp->lun = 0; /* device management cmd is not specific to any LUN */ > lrbp->intr_cmd = true; /* No interrupt aggregation */ > +#ifdef CONFIG_SCSI_UFS_CRYPTO > + lrbp->crypto_enable = false; /* No crypto operations */ > +#endif > hba->dev_cmd.type = cmd_type; Doesn't this need to be initialized in ufshcd_issue_devman_upiu_cmd() too?