> - return readx_poll_timeout(axinet_ior_read_mcr, lp, > + return readx_poll_timeout(ioread32, lp->regs + XAE_MDIO_MCR_OFFSET, > val, val & XAE_MDIO_MCR_READY_MASK, > 1, 20000); I think this change makes axinet_ior_read_mcr() redundant? So it would be good to remove it. Andrew