On Sunday 27 March 2016 11:30:58 Thomas Petazzoni wrote: > + /* Wait until the link becomes active again */ > + while (timeout) { > + if (armada8k_pcie_link_up(pp)) > + break; > + udelay(1); > + timeout--; > + } > Why the busy-loop here? Maybe just do a single msleep(1) here to wait for the link to come up instead? Arnd -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html