From mboxrd@z Thu Jan 1 00:00:00 1970 From: Niklas Cassel Subject: Re: [PATCH v2 7/9] arm64: dts: qcom: msm8998: Add PSCI cpuidle low power states Date: Wed, 2 Oct 2019 11:27:34 +0200 Message-ID: <20191002092734.GA15523@centauri> References: <49cf5d94beb9af9ef4e78d4c52f3b0ad20b7c63f.1558430617.git.amit.kucheria@linaro.org> <20191002091950.GA9393@centauri> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20191002091950.GA9393@centauri> Sender: linux-kernel-owner@vger.kernel.org To: Jeffrey Hugo Cc: Amit Kucheria , lkml , MSM , Bjorn Andersson , Andy Gross , Marc Gonzalez , Sibi Sankar , daniel.lezcano@linaro.org, David Brown , Li Yang , Shawn Guo , DTML List-Id: devicetree@vger.kernel.org On Wed, Oct 02, 2019 at 11:19:50AM +0200, Niklas Cassel wrote: > On Mon, Sep 30, 2019 at 04:20:15PM -0600, Jeffrey Hugo wrote: > > Amit, the merged version of the below change causes a boot failure > > (nasty hang, sometimes with RCU stalls) on the msm8998 laptops. Oddly > > enough, it seems to be resolved if I remove the cpu-idle-states > > property from one of the cpu nodes. > > > > I see no issues with the msm8998 MTP. > > Hello Jeffrey, Amit, > > If the PSCI idle states work properly on the msm8998 devboard (MTP), > but causes crashes on msm8998 laptops, the only logical change is > that the PSCI firmware is different between the two devices. Since the msm8998 laptops boot using ACPI, perhaps these laptops doesn't support PSCI/have any PSCI firmware at all. Kind regards, Niklas