From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bjorn Andersson Subject: Re: [PATCH] ARM: dts: qcom: msm8974-hammerhead: increase load on l20 for sdhci Date: Tue, 31 Jul 2018 15:29:28 -0700 Message-ID: <20180731222928.GA24465@builder> References: <20180709091628.GA8854@orion> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20180709091628.GA8854@orion> Sender: linux-kernel-owner@vger.kernel.org To: Bhushan Shah Cc: Andy Gross , David Brown , Rob Herring , Mark Rutland , linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Brian Masney List-Id: devicetree@vger.kernel.org On Mon 09 Jul 02:16 PDT 2018, Bhushan Shah wrote: > The kernel would not boot on the hammerhead hardware due to the > following error: > > mmc0: Timeout waiting for hardware interrupt. > mmc0: sdhci: ============ SDHCI REGISTER DUMP =========== > mmc0: sdhci: Sys addr: 0x00000200 | Version: 0x00003802 > mmc0: sdhci: Blk size: 0x00000200 | Blk cnt: 0x00000200 > mmc0: sdhci: Argument: 0x00000000 | Trn mode: 0x00000023 > mmc0: sdhci: Present: 0x03e80000 | Host ctl: 0x00000034 > mmc0: sdhci: Power: 0x00000001 | Blk gap: 0x00000000 > mmc0: sdhci: Wake-up: 0x00000000 | Clock: 0x00000007 > mmc0: sdhci: Timeout: 0x0000000e | Int stat: 0x00000000 > mmc0: sdhci: Int enab: 0x02ff900b | Sig enab: 0x02ff100b > mmc0: sdhci: AC12 err: 0x00000000 | Slot int: 0x00000000 > mmc0: sdhci: Caps: 0x642dc8b2 | Caps_1: 0x00008007 > mmc0: sdhci: Cmd: 0x00000c1b | Max curr: 0x00000000 > mmc0: sdhci: Resp[0]: 0x00000c00 | Resp[1]: 0x00000000 > mmc0: sdhci: Resp[2]: 0x00000000 | Resp[3]: 0x00000000 > mmc0: sdhci: Host ctl2: 0x00000008 > mmc0: sdhci: ADMA Err: 0x00000000 | ADMA Ptr: 0x70040220 > mmc0: sdhci: ============================================ > mmc0: Card stuck in wrong state! mmcblk0 card_busy_detect status: 0xe00 > mmc0: cache flush error -110 > mmc0: Reset 0x1 never completed. > > This patch increases the load on l20 to 0.2 amps for the sdhci > and allows the device to boot normally. > > Signed-off-by: Bhushan Shah > Suggested-by: Bjorn Andersson > Tested-by: Brian Masney Reviewed-by: Bjorn Andersson Regards, Bjorn > --- > arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dts | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dts b/arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dts > index 4dc0b347b1ee..c2dc9d09484a 100644 > --- a/arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dts > +++ b/arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dts > @@ -189,6 +189,8 @@ > regulator-max-microvolt = <2950000>; > > regulator-boot-on; > + regulator-system-load = <200000>; > + regulator-allow-set-load; > }; > > l21 { > -- > 2.17.1 >