Hi Andrew, On 25 July 2015 at 13:38, Andrew Lunn <and...@lunn.ch> wrote: > When a switch is attached to the mdio bus, the mdio bus can be used > while the interface is not open. If the IPG clock is not enabled, MDIO > reads/writes will simply time out. > > Add support for runtime PM to control this clock. Enable/disable this > clock using runtime PM, with open()/close() and mdio read()/write() > function triggering runtime PM operations. Since PM is optional, the > IPG clock is enabled at probe and is no longer modified by > fec_enet_clk_enable(), thus if PM is not enabled in the kernel, it is > guaranteed the clock is running when MDIO operations are performed. > > Signed-off-by: Andrew Lunn <and...@lunn.ch> > Signed-off-by: Lucas Stach <l.st...@pengutronix.de> > Cc: tyler.ba...@linaro.org > Cc: fabio.este...@freescale.com > Cc: shawn....@linaro.org > --- > > v5 was accepted and then reverted, because it broke imx6. Lucas Stach > debugged what was going wrong and provided a patch. Thanks Lucas. This > has been squashed into v6. > > It would be good if this version had more testing on various platforms > before it is accepted. If you reported a problem last time, please do > test and send a Tested-by:
I've tested this single patch (v6) on top of next-20150724 using the kernelci.org bot to build and boot test[1][2] on a broad range of platforms. With these results, I can confirm that the imx6 platforms tested are booting fine, and that no new boot/build regressions have been introduced when comparing these results to the next-20150724 baseline[3][4]. Tested-by: Tyler Baker <tyler.ba...@linaro.org> Cheers, Tyler [1] http://kernelci.org/boot/all/job/tbaker/kernel/v4.2-rc3-4216-g8c8efbc4423a/ [2] http://kernelci.org/build/tbaker/kernel/v4.2-rc3-4216-g8c8efbc4423a/ [3] http://kernelci.org/boot/all/job/next/kernel/next-20150724/ [4] http://kernelci.org/build/next/kernel/next-20150724/ -- 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
