On 7 March 2013 09:50, Olof Johansson olof@lixom.net wrote:
On Wed, Mar 6, 2013 at 5:46 PM, Olof Johansson olof@lixom.net wrote:
This binding makes no sense to me. It needs to be substantially better documented, not just a couple of sentences that people that understand bit.LITTLE thoroughly can make sense of.
It also duplicates the cpu binding. I suspect this should instead be done through additions of the cpu bindings instead of duplication. So this needs to be substantially reworked.
Actually, i wasn't getting in new bindings for cpu/cluster for big LITTLE but was just trying to get bindings for getting freq-table for cpufreq driver.
Lorenzo (cc'd) actually has a patch for getting the bindings in and he is looking to send them soon. I know these can change after some review and the plan was i will fix cpufreq bindings again once that patch is in. We don't have any mainlined hardware for it for now and so it wouldn't break anything.
Also, _ALWAYS_ cc devicetree-discuss on new bindings. get_maintainer tells you to do so.
I knew it and i messed up with it. Will surely take care of it next time.