[PATCH V3 6/9] phy: qcom-qmp-ufs: Refactor qmp_ufs_exit callback.

Konrad Dybcio konrad.dybcio at oss.qualcomm.com
Wed Apr 23 04:29:39 PDT 2025


On 4/10/25 11:00 AM, Nitin Rawat wrote:
> Rename qmp_ufs_disable to qmp_ufs_power_off and refactor
> the code to move all the power off sequence to qmp_ufs_power_off.
> 
> Co-developed-by: Ram Kumar Dwivedi <quic_rdwivedi at quicinc.com>
> Signed-off-by: Ram Kumar Dwivedi <quic_rdwivedi at quicinc.com>
> Signed-off-by: Nitin Rawat <quic_nitirawa at quicinc.com>
> ---

So this patch does quite a lot without explaining the context
that isn't visible in just the diff below

- .power_on is altered to no longer reset the PHY (but it only did so
   on docs with !no_pcs_sw_reset?)
- partially inlines com_exit (dropping the reset assert)
- removes .disable in favor of .power_off that we can't tell
  what it does just by looking at this patch in the middle of the
  series

Please improve the commit message and consider splitting this change
in two

Konrad



More information about the linux-phy mailing list