7 Nov
2014
7 Nov
'14
10:44 a.m.
On Fri, Nov 07, 2014 at 09:24:54AM +0000, Lin, Mengdong wrote:
+static int rt5670_runtime_resume(struct device *dev) {
- return 0;
+}
This should be undoing the change done on suspend otherwise we'll be stuck on the internal clock after resuming. Alternatively if something will undo that elsewhere then presumably we don't need to use runtime PM to select the internal clock anyway?
Is it okay to restore default sysclock setting is runtime_resume ops?
Yes, that's fine of course - I'm just assuming that this works already so presumably there is some restore code somewhere already that should be removed?