On Tue, 2020-01-14 at 12:14 +0100, Matthias Brugger wrote: > > > + idle-states { > > + entry-method = "arm,psci"; > > reading idle-states.txt the entry-method should be "psci". It will be correct in next patch. > > + > > + MCDI_CPU: mcdi-cpu { > > I wonder what MCDI stands for. Normally we have CPU_SLEEP and CLUSTER_SLEEP here. It means MCU Deep Idle. But you are right, CPU/CLUSTER_SLEEP is more readable. So I'll change their name in next patch. Best regards, James