arm64: dts: rockchip: remove wrongly added idle states on rk3368
authorHeiko Stuebner <heiko@sntech.de>
Tue, 16 Feb 2016 07:55:41 +0000 (08:55 +0100)
committerHeiko Stuebner <heiko@sntech.de>
Wed, 22 Mar 2017 10:57:11 +0000 (11:57 +0100)
commit531b3c49b384c29baec7b7b271a7739a23858e3a
tree040fcc079f2edc036b71efa9ddcc92101a20e5b7
parent66aef3cb9125583522d7d8da38d420d77287728a
arm64: dts: rockchip: remove wrongly added idle states on rk3368

As reported by Lorenzo, the residency/latency values defined in the
idle-state for rk3368 "make no sense". When introducing them I
simply took the idle-state node from the vendor kernel in error
as I didn't look up if these values were sane in the first place.

Talking to people and determining why they were used in this way
showed that it was meant to make sure the cpu_suspend callback
got initialized which at the 3.10 time was somehow required even
for wfi-based idle handling.

Of course the generic arch_cpu_idle() now does wfi-based idle-handling
already and the rk3368 does not implement any other idle states than
the default WFI, so these wrong idle-states should go away.

Reported-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Signed-off-by: Heiko Stuebner <heiko@sntech.de>
Acked-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
arch/arm64/boot/dts/rockchip/rk3368.dtsi