Re: [PATCH v2 05/11] net: stmmac: dwmac-rk: Add internal phy support

From: David.Wu
Date: Fri Jul 28 2017 - 02:56:15 EST


Hi Florian,

å 2017/7/28 0:54, Florian Fainelli åé:
- if you need knowledge about this PHY connection type prior to binding
the PHY device and its driver (that is, before of_phy_connect()) we
could add a boolean property e.g: "phy-is-internal" that allows us to
know that, or we can have a new phy-mode value, e.g: "internal-rmii"
which describes that, either way would probably be fine, but the former
scales better


Using "phy-is-internal" is very helpful, but it is easy to confuse with the real internal PHY, may we use the other words like phy is inlinedð.

Then again, using phy-mode = "internal" even though this is Reduced MII
is not big of a deal IMHO as long as there is no loss of information and
that internal de-facto means internal reduced MII for instance.
--