On 31 July 2015 at 21:11, Ryan Harkin ryan.harkin@linaro.org wrote:
On 31 July 2015 at 10:41, Viresh Kumar viresh.kumar@linaro.org wrote:
On 31-07-15, 10:38, Jon Medhurst (Tixy) wrote:
I too originally used the words 'scary' and 'hack' in a draft email then toned it down, I needn’t have bothered ;-)
As I'll soon be getting big.LITTLE MP (aka HMP) and Intelligent Power Allocator (IPA) from ARM to integrate, I can't help but think that Google's efforts in this area are going to cause us problems even if it wasn't buggy. So I'm sorely tempted to just revert the whole lot in the ARM Landing Team tree. I'll have to get peoples opinions on that.
I'm of the opinion that they should not be included into LSK at all.
That is, they should be dropped from Amit's branch and someone should go back to original authors and ask them to either improve them, make them CONFIG_ conditional and so on.
CONFIG_ conditional sounds like a good plan for the time being. I'll propose a fix in AOSP for that.
FWIW I have already dropped these patches from 4.1 onwards. I've been told that mainline(4.2+) already has replacement framework for AOSP's "persistent cpu_freq history on hotplug" patches and as Viresh mentioned they are also working on new opp-bindings using which we can track powers (current/voltage) stats.
Regards, Amit Pundir
If or once they're in better shape and we can be sure they aren't going to break things, then we should think about including them.
I haven't seen them all, so can't really ask you to drop them :)
-- viresh _______________________________________________ linaro-kernel mailing list linaro-kernel@lists.linaro.org https://lists.linaro.org/mailman/listinfo/linaro-kernel
linaro-kernel mailing list linaro-kernel@lists.linaro.org https://lists.linaro.org/mailman/listinfo/linaro-kernel