Hello Lucas, On Mon, Aug 03, 2015 at 05:50:11PM +0200, Lucas Stach wrote: > The clocks are initially active and thus the device is marked active. > This still keeps the PM refcount at 0, the pm_runtime_put_autosuspend() > call at the end of probe then leaves us with an invalid refcount of -1, > which in turn leads to the device staying in suspended state even though > netdev open had been called. > > Fix this by initializing the refcount to be coherent with the initial > device status. > > Fixes: > 8fff755e9f8 (net: fec: Ensure clocks are enabled while using mdio bus) > > Signed-off-by: Lucas Stach <l.st...@pengutronix.de> the two \n after "Fixes:" and before the S-o-b are unusual I think.
Other than that I tested the change on i.MX27 and it works now. The power refcount oscillates between 1 and 2 now as expected and booting with NFS-root works fine. So: Tested-by: Uwe Kleine-König <u.kleine-koe...@pengutronix.de> Thanks Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | http://www.pengutronix.de/ | -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
