>> >> Add full hardware initialization sequence and link configuration logic > > Hi Igor > > I'm still not convinced the PHY driver should be embedded in the MAC > driver, rather than using phylink. > > If i remember correctly, it was because the MAC is involved in > determining if the link is up? That is nothing new. phylink expects > this. The MAC driver should call phylink_mac_change() when the MACs > SERDES goes up/down. Hi Andrew, I'm actually still in doubt as well on this matter. Of course thats quite possible. Here was my original comment: > Thats again because of this product has tightly integrated MAC+Phy. > MAC FW controls system interface and reports/alters link state > as a joint state on copper and SIF (even in dpa direct phy mode). > > We can't extract phy api into a standalone fully functional phylib therefore. > Also as far as I know this particular phy is not available in the wild. So the point is that MAC firmware is managing SERDES and system interface link. Even if we cut out phy stuff into a separate phylink driver - that driver will never be functional in standalone mode, without aqc111 mac driver. It only may make sense from software decomposition perspective - to separate pieces of phy code. We'll discuss and reconsider this internally and will give you more comments, Thanks, Igor
