Re: [PATCH v5 2/6] net: stmmac: sun8i: force select external PHY when no internal one

From: Sergei Shtylyov
Date: Tue May 21 2019 - 05:30:05 EST


Hello!

On 21.05.2019 2:50, megous@xxxxxxxxxx wrote:

From: Icenowy Zheng <icenowy@xxxxxxx>

The PHY selection bit also exists on SoCs without an internal PHY; if it's
set to 1 (internal PHY, default value) then the MAC will not make use of
any PHY such SoCs.
^ "on" or "with" missing?

This problem appears when adapting for H6, which has no real internal PHY
(the "internal PHY" on H6 is not on-die, but on a co-packaged AC200 chip,
connected via RMII interface at GPIO bank A).

Force the PHY selection bit to 0 when the SOC doesn't have an internal PHY,
to address the problem of a wrong default value.

Signed-off-by: Icenowy Zheng <icenowy@xxxxxxx>
Signed-off-by: Ondrej Jirman <megous@xxxxxxxxxx>
[...]

MBR, Sergei