17.07.2015 02:25, Florian Fainelli пишет: > On 16/07/15 07:50, Stas Sergeev wrote: >> >> Currently fixed_phy driver recognizes only the link-up state. >> This simple patch adds an implementation of link-down state. >> It fixes the status registers when link is down, and also allows >> to register the fixed-phy with link down without specifying the speed. > > This patch still breaks my setups here, e.g: drivers/net/dsa/bcm_sf2.c, > but I will look into it. > > Do we really need this for now for your two other patches to work > properly, or is it just nicer to have? Yes, absolutely. Otherwise registering fixed phy will return -EINVAL because of the missing link speed (even though the link is down).
Please, see what makes a problem. I can't reproduce what you report. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html