Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752026AbcCYFpe (ORCPT ); Fri, 25 Mar 2016 01:45:34 -0400 Received: from lists.s-osg.org ([54.187.51.154]:57091 "EHLO lists.s-osg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751382AbcCYFpd (ORCPT ); Fri, 25 Mar 2016 01:45:33 -0400 Subject: Re: [PATCH] ARM: dts: exynos: Fix invalid maximum voltage for buck9 supplying SD card To: Krzysztof Kozlowski , Kukjin Kim , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mmc@vger.kernel.org References: <1458807125-13580-1-git-send-email-k.kozlowski@samsung.com> From: Javier Martinez Canillas Cc: Ulf Hansson , Mark Brown , Ivaylo Dimitrov Message-ID: <56F4D070.4020403@osg.samsung.com> Date: Fri, 25 Mar 2016 02:45:20 -0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 MIME-Version: 1.0 In-Reply-To: <1458807125-13580-1-git-send-email-k.kozlowski@samsung.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2340 Lines: 57 Hello Krzysztof, On 03/24/2016 05:12 AM, Krzysztof Kozlowski wrote: > The buck9 regulator of S2MPS11 PMIC had incorrect maximum voltage - > lower by 5 mV than specified by datasheet. This buck9 provides power to > other regulators, including LDO13 and LDO19 which supply the MMC2 (SD > card). > > When constraints were applied, the depending regulators were not > providing enough power, leading to SD card detection errors: > mmc1: card never left busy state > mmc1: error -110 whilst initialising SD card > > Fixes: 86a2d2ac5e5d ("ARM: dts: Add dts file for Odroid XU3 board") > Signed-off-by: Krzysztof Kozlowski > > --- > > The issue can be reproduced on next-20160324 with > bae4fdc88d7f7dda1 (regulator: core: Ensure we are at least in bounds for our constraints). > --- > arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi b/arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi > index 1bd507bfa750..82b2ba632cf6 100644 > --- a/arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi > +++ b/arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi > @@ -299,7 +299,7 @@ > buck9_reg: BUCK9 { > regulator-name = "vdd_2.8v_ldo"; > regulator-min-microvolt = <3000000>; > - regulator-max-microvolt = <3750000>; > + regulator-max-microvolt = <3755000>; > regulator-always-on; > regulator-boot-on; > }; > I tested next-20160324 + $SUBJECT on an Odroid XU4 (with exynos_defconfig) and while it's true that the patch makes the SD card error to go away, I see that the s2mps11 regulator driver fails to probe due an -EINVAL being return by machine_constraints_voltage() -> _regulator_do_set_voltage(): [ 1.710300] vdd_2.8v_ldo: failed to apply 3755000-3755000uV constraint(-22) [ 1.716387] s2mps11-pmic s2mps11-regulator: regulator init failed for 46 [ 1.734878] s2mps11-pmic: probe of s2mps11-regulator failed with error -22 I wonder how neither Markus nor you saw this issue when testing so maybe I'm doing something wrong? Since AFAICT it's only working because the driver fails to probe so the same effect can be achieved disabling CONFIG_REGULATOR_S2MPS11. Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America