> This patch does add additional code I overlooked to cleanup the phy > if it fails, I guess in theory that portion could be a prereq patch, > I will break that portion out. I'm still not sure how to partially > move the MDIO startup...
Hi Jeremy You can add a cleanup patch which replaces these hard coded return values with the value returned by the function calls. Then do the move. In general, it is easier to review lots of small obvious patches, than one big patch with lots of things going on. Andrew