From mboxrd@z Thu Jan 1 00:00:00 1970 From: festevam@gmail.com (Fabio Estevam) Date: Tue, 25 Apr 2017 14:02:04 -0300 Subject: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override In-Reply-To: <89cc7192100bdc9ce546bf6000446e629457ebc1.1493138693.git.leonard.crestez@nxp.com> References: <89cc7192100bdc9ce546bf6000446e629457ebc1.1493138693.git.leonard.crestez@nxp.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Leonard, On Tue, Apr 25, 2017 at 1:57 PM, Leonard Crestez wrote: > The board file for imx6sx-dbg overrides cpufreq operating points to use > higher voltages. This is done because the board has a shared rail for > VDD_ARM_IN and VDD_SOC_IN and when using LDO bypass the shared voltage > needs to be a value suitable for both ARM and SOC. > > This was introduced in: > > commit 54183bd7f766 ("ARM: imx6sx-sdb: add revb board and make it default") > > This only only applies to LDO bypass mode, a feature not present in > upstream. When LDOs are enabled the effect is to use higher voltages than > necesarry for no good reason. > > Setting these higher voltages can make some boards fail to boot with ugly > semi-random crashes, reminiscent of memory corruption. These failures > happen the first time the lowest idle state is used. Remove the OPP > override in order to fix those crashes. > > Signed-off-by: Leonard Crestez > > --- > It's not clear exactly why the crashes happen. Perhaps waking up from idle > draws more power than is available? Removing this override is a correct > change anyway so maybe there is no need to investigate deeper. Marek just sent a similar one a few minutes ago: http://lists.infradead.org/pipermail/linux-arm-kernel/2017-April/503230.html