On Thu, Apr 30, 2020 at 11:59:55AM +0000, Satya Tangirala wrote: > @@ -8541,6 +8568,13 @@ int ufshcd_init(struct ufs_hba *hba, void __iomem *mmio_base, unsigned int irq) > /* Reset the attached device */ > ufshcd_vops_device_reset(hba); > > + /* Init crypto */ > + err = ufshcd_hba_init_crypto(hba); > + if (err) { > + dev_err(hba->dev, "crypto setup failed\n"); > + goto out_remove_scsi_host; > + } > + Due to changes in v5.6, this is jumping to the wrong error label. It should be 'free_tmf_queue'. - Eric