I've been pointed at commit number 2bf69b5fe90b3246ab50064c5a690a363e8c53e2

http://www.kernel.org/git/gitweb.cgi?p=linux/kernel/git/jgarzik/ netdev-2.6.git;a=commit;h=2bf69b5fe90b3246ab50064c5a690a363e8c53e2

The commit message says:

phy subsystem: more cleanups

- unexport symbols never used outside of home module
- remove dead code
- remove CONFIG_PHYCONTROL, make it unconditionally enabled


And in the patches have been removed all the functions which allow PHY busses to be set up, and ethernet devices to be informed of when changes occur in the link state. Specifically:

mdiobus_register(): Used to register an MDIO bus. Did some code replace this?
mdiobus_unregister()
phy_print_status(): Merely a convenience function, with the idea of standardizing PHY status output (one of the requests from the community)
phy_interrupt(): Services PHY interrupts
phy_start_interrupts(): Interestingly, phy_stop_interrupts was left
phy_change()
phy_stop()
phy_start()
phy_connect()
phy_attach(): Without this function, ethernet controllers do not have access to their PHYs, and visa-versa

There are others. Frankly, I'm not convinced that any of that is dead code. Certainly it's all used by the gianfar ethernet driver (once updated to use the PHY Layer). I'd be happy to work with you to get this code up to... um... code.

Andy Fleming
Freescale Open Source Team
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to