Hi, Alim > > From: Bean Huo <beanhuo@xxxxxxxxxx> > > > > ufshcd_get_max_pwr_mode() only need to be called once while booting, > > take it out from ufshcd_probe_hba() and inline into ufshcd_init_params(). > > > This can be safely squash with the previous patch which introduce > ufshcd_init_params. > I kept this patch because I want you to review the patch easily. The previous patch is exactly to split ufshcd_probe_hba(), doesn't want to do any initialization path flow changing. If I merge this patch to the previous one, that will disorder the original calling process, and some reviewers will have more concern about previous patch. kept them independently, for the previous patch, you only need to check its split-up validity. Thanks, //Bean