Re: [PATCH net-next 12/12] arm64: dts: qcom: qcs6490-rb3gen2: enable TC9564 with a single QCS8081 phy

[Andrew Lunn]

> However the real reason we jammed this on is because I couldn't find a
> way to get the phy/mdio code to turn one on. However it is possible to
> add regulator support to MDIO devices by extending their existing logic
> to manage resets so it can also manage a regulator. It comes out fairly
> clean so we can add that to the patch set and remove the
> regulator-always-on.

We, I have rejected this before. It might look clean and easy, but it
is not. How do you determine the order of enabling reset, regulators
clocks? How do you specify the need sleeps in between these different
operations?

There is nothing in particular MDIO specific here, and there is
generic power sequencing code in the kernel. And a while back,
somebody said they would look at what is needed to make MDIO busses
and Ethernet PHYs make use of that generic power sequencing code. That
is the better way to do this.

	Andrew