> On 5/15/24 23:51, Avri Altman wrote:
> > void ufshcd_fixup_dev_quirks(struct ufs_hba *hba,
> > const struct ufs_dev_quirk *fixups)
> > {
> > @@ -8278,6 +8312,8 @@ static int ufs_get_device_desc(struct ufs_hba *hba)
> > if (hba->ext_iid_sup)
> > ufshcd_ext_iid_probe(hba, desc_buf);
> >
> > + ufshcd_rtt_set(hba, desc_buf);
> > +
>
> Why does this call occur in ufs_get_device_desc()? ufshcd_rtt_set() sets
> a device parameter. Shouldn't this call be moved one level up into
> ufshcd_device_params_init()?
Because otherwise bDeviceRTTCap is not available.
Please note several device configuration calls in ufs_get_device_desc - all requires some device descriptor fields.
Thanks,
Avri
>
> Otherwise this patch looks good to me.
>
> Thanks,
>
> Bart.
