Re: [PATCH v2] arm64: dts: mt8183: Enable CPU idle-states

From: James Liao
Date: Wed Jan 15 2020 - 00:35:50 EST


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