On Mon, Sep 14, 2015 at 11:42:54AM +0100, Russell King - ARM Linux wrote: > Andrew, > > I think you're the current maintainer of the Marvell DSA code, as being > the most recent author of changes to it. :)
Hi Russell Sorry for the slow reply, i've been on vacation. Humm, i suppose i might be the defacto Maintainer for Marvell parts, but i've no NDA with Marvell, so no access to the data sheets etc. > I've noticed in my testing that the Marvell DSA code seems to poll the > internal phy link status in mv88e6xxx_poll_link(), and set the network > device carrier status according to the results. Peter Korsgaard comment might be correct, the switch needs to know what the PHY has negotiated. Hence the use of the PPU. There are also comments in the code that the PPU is needed for indirect access to the PHY. So we probably need to keep the PPU, but disable it from changing the networks stacks idea of the link state, etc. I will add this to my TODO list to play with it. Andrew -- 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
