Hi Bart,

> > @@ -147,6 +152,8 @@ static int ufs_mtk_init(struct ufs_hba *hba)
> >     if (err)
> >             goto out_variant_clear;
> >  
> > +   ufs_mtk_set_caps(hba);
> > +
> >     /*
> >      * ufshcd_vops_init() is invoked after
> >      * ufshcd_setup_clock(true) in ufshcd_hba_init() thus
> 
> Please inline the ufs_mtk_set_caps() function. Introducing single line
> functions like is done in this patch doesn't improve readability.
> 

OK! Will be fixed in next version.

Thanks,
Stanley


Reply via email to