Hi<br><br><div class="gmail_quote">On Mon, Jun 18, 2012 at 4:47 PM, Johannes Stezenbach <span dir="ltr"><<a href="mailto:js@sig21.net" target="_blank">js@sig21.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Check if the PHY is really accessible (e.g. the<br>
PHY address is correct) during probe.<br>
<br>
Signed-off-by: Johannes Stezenbach <<a href="mailto:js@sig21.net">js@sig21.net</a>><br>
---<br>
 drivers/net/miidev.c |   12 ++++++++++++<br>
 1 file changed, 12 insertions(+)<br>
<br>
diff --git a/drivers/net/miidev.c b/drivers/net/miidev.c<br>
index 272234e..3545889 100644<br>
--- a/drivers/net/miidev.c<br>
+++ b/drivers/net/miidev.c<br>
@@ -225,6 +225,14 @@ static struct file_operations miidev_ops = {<br>
 static int miidev_probe(struct device_d *dev)<br>
 {<br>
        struct mii_device *mdev = dev->priv;<br>
+       int val;<br>
+<br>
+       val = mii_read(mdev, mdev->address, MII_PHYSID1);<br>
+       if (val < 0 || val == 0xffff)<br>
+               goto err_out;<br>
+       val = mii_read(mdev, mdev->address, MII_PHYSID2);<br>
+       if (val < 0 || val == 0xffff)<br>
+               goto err_out;<br>
<br></blockquote><div><br></div><div>Suppose this has no influence on drivers/net/fec_imx.c:fec_set_hwaddr() returning -1?</div><div><br></div><div>Otherwise I believe this is fine.</div><div><br></div><div>Cheers</div><div>

Roberto</div></div>