Hi Kevin, On Wed, Oct 31, 2012 at 6:49 AM, Kevin Hilman <khil...@deeprootsystems.com> wrote: > Hi Jean, > > Jean Pihet <jean.pi...@newoldbits.com> writes: > > [...] > >>> Based on a very quick look, I'd say the original patch 3db11fe is broken. >>> I don't see how it can ensure that its PM_QOS_CPU_DMA_LATENCY request is >>> honored when CONFIG_CPU_IDLE=n. CONFIG_CPU_IDLE=n is the default for >>> omap2plus_defconfig. >> >> Withtout CPU_IDLE set the PM QoS has no influence on the power domains >> states. > > Exactly, which means there is *no* constraint set when CPUidle is > disabled, Correct. With CPU_IDLE disabled PM QoS manages the constraints list but cpuidle does not request the aggregated value nor does it restrict the CPU and CORE power domains states.
> and it's exactly this that is different from the behavior > before your patch. No. > Before your patch, the constraint would be set whether or not CPUidle > was enabled, correct? No. In the linux-omap source tree the OMAP PM API for the latency constraints simply is a no-op. The only difference with the code after the patch is that PM QoS manages the constraints list, which should be neglictable in terms of CPU execution time. Paul, Could you please check with the 2 calls to PM QoS from the I2C code commented out? This will rule out the PM QoS impact. > The solution to this will probably be to make OMAPs non-CPUidle idle path > check the constraints. This is the idea behind the per-device PM QoS support, which allows to set a constraint on any device and so on any power domain (note that cpuidle influences CPU and CORE only). However in the current context -the I2C timeouts issue- there is no apparent link between the issue and the patch 3db11fef [1]. [1] http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=commit;h=3db11feffc1ad2ab9dea27789e6b5b3032827adc > Kevin Thanks, Jean -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
