On Thursday 17 August 2017 02:25 AM, Franklin S Cooper Jr wrote: > 66AK2G UART instances are not apart of the ALWAYS_ON power domain. > Therefore, pm_runtime calls must be made to properly insure the appropriate > power domains needed by UART are on. Keep legacy clk api calls since other > users of this driver may not support PM runtime. Do we really have users like that? And even if there are, cant they use PM clock handling support available in drivers/base/power/clock_ops.c ? The clock enable support itself was added pretty "recently" - about 5 years back with 0bbeb3c3e84b ("of serial port driver - add clk_get_rate() support"). So I doubt any really legacy platforms relied on clock support being there. It was added by Murali, I assume for Keystone devices. Keystone devices can work with runtime PM using the PM clock support pointed to above. Perhaps linux-arm-kernel list should be copied on this submission too, since most users of this driver are likely to be there on that list. > > Signed-off-by: Franklin S Cooper Jr <fcooper@xxxxxx> Thanks, Sekhar -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html