[PATCH v4] ufs: core: correct ufshcd_shutdown flow

Bart Van Assche bvanassche at acm.org
Tue Jul 26 11:34:29 PDT 2022


On 7/26/22 04:36, peter.wang at mediatek.com wrote:
> In normal case: ufshcd_wl_shutdown -> ufshcd_shtdown
> ufshcd_shtdown should turn off clock/power after ufshcd_wl_shutdown
> which set device power off and link off.

The above sentence is confusing: it mentions "normal case" which 
suggests there is also an abnormal case. Isn't the order in which 
ufshcd_wl_shutdown() and ufshcd_shutdown() are called always the same?

Otherwise this patch looks good to me.

Thanks,

Bart.



More information about the Linux-mediatek mailing list