On Fri, May 06, 2022 at 09:47:08AM +0800, Richard Zhu wrote: > To make it more reasonable, move the phy_power_on/phy_init callbacks to > the proper places. > - move the phy_power_on() out of imx6_pcie_clk_enable(). > - move the phy_init() out of imx6_pcie_deassert_core_reset(). I'm not sure what "make it more reasonable" is telling me. In subject line and commit log, please say something more specific than "the proper places." It's probably more important to say where they are moving *to* than where they're moving *out of*. > In order to save power consumption, turn off the clocks and regulators when > the imx6_pcie_host_init() return error. Is the power savings the *reason* for this change? I can't tell from the commit log.
