Messages in this thread | | | From | Jose Abreu <> | Subject | RE: [EXT] [PATCH v2 1/5] scsi: ufs: Allow UFS 3.0 as a valid version | Date | Thu, 30 Apr 2020 08:14:03 +0000 |
| |
From: Bean Huo (beanhuo) <beanhuo@micron.com> Date: Apr/29/2020, 13:59:08 (UTC+00:00)
> > > > @@ -8441,7 +8441,8 @@ int ufshcd_init(struct ufs_hba *hba, void > > > > __iomem *mmio_base, unsigned int irq) > > > > if ((hba->ufs_version != UFSHCI_VERSION_10) && > > > > (hba->ufs_version != UFSHCI_VERSION_11) && > > > > (hba->ufs_version != UFSHCI_VERSION_20) && > > > > - (hba->ufs_version != UFSHCI_VERSION_21)) > > > > + (hba->ufs_version != UFSHCI_VERSION_21) && > > > > + (hba->ufs_version != UFSHCI_VERSION_30)) > > > > > > I don't think these checkups of UFSHCI version is necessary, does the UFSHCI > > have other version number except these? > > > Is there somebody still v1.0 and v1.1? > > > > Probably. I think we can leave them or change the dev_err to a dev_warn. > > This way we have logs in case someone is using a non-supported version. > > > > What do you think ? > > > Hi, Jose > Seems after your patch, all of current released UFS control versions will be supported except the > version suffix is non-zero. Right?
I think we cover all versions with this patch.
--- Thanks, Jose Miguel Abreu
| |