> One question: are we happy with failing the probe like this, or would it
> be better to allow the probe to suceed?
> 
> As has been pointed out in the C45 MII access patch, we need the PHY
> to bind to the network driver for the MII bus to be accessible to
> userspace, so if we're going to have userspace tools to upload the
> firmware, rather than using u-boot, we need the PHY to be present and
> bound to the network interface.

Hi Russell

It is an interesting question. Failing the probe is the simple
solution. If we don't fail the probe, we then need to allow the
attach, but fail all normal operations, with a noisy kernel log.  That
probably means adding a new state to the state machine, PHY_BROKEN.
Enter that state if phy_start_aneg() returns an error?

      Andrew

Reply via email to