Hi Can, "Refector" in title shall be "Refactor"? On Mon, 2020-11-23 at 23:28 -0800, Can Guo wrote: > Remove the param skip_ref_clk from __ufshcd_setup_clocks(), but keep a flag > in struct ufs_clk_info to tell whether a clock can be disabled or not while > the link is active. > > Signed-off-by: Can Guo <cang@xxxxxxxxxxxxxx> Otherwise looks good to me. Reviewed-by: Stanley Chu <stanley.chu@xxxxxxxxxxxx>