On Thu, Jun 09, 2016 at 01:09:02PM -0500, Andy Gross wrote: > On Thu, Jun 09, 2016 at 09:39:34AM +0200, Ulf Hansson wrote: > > + Daniel > > > > On 19 May 2016 at 07:00, Andy Gross <andy.gross@xxxxxxxxxx> wrote: [ ... ] > > I don't think this will work! > > > > When building a multi defconfig for ARM, you might overwrite the > > suspend_ops (there's only one set) as here you don't know that it's > > actually the QCOM platform that is running, right!? > > > > Perhaps this code actually belongs closer to the cpuidle driver? > > Hmmmm, I might have to get creative. I originally had a DT entry for the pm, > but that doesn't make sense as this is purely a software construct. The db410c > uses the arm cpuidle driver so I can't really hook it in there. I'll have to > come up with something else. IMO, the generic ARM driver should be revisited to add support for suspend to idle. -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html