From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: Fabrizio Castro Subject: [PATCH v2 1/2] ARM: dts: iwg20d-q7-common: Fix SDHI1 VccQ regularor Date: Mon, 3 Jun 2019 10:53:51 +0100 Message-Id: <1559555632-2258-2-git-send-email-fabrizio.castro@bp.renesas.com> In-Reply-To: <1559555632-2258-1-git-send-email-fabrizio.castro@bp.renesas.com> References: <1559555632-2258-1-git-send-email-fabrizio.castro@bp.renesas.com> To: Simon Horman , Geert Uytterhoeven , Rob Herring , Mark Rutland Cc: Fabrizio Castro , Magnus Damm , linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org, Chris Paterson , Biju Das List-ID: SDR50 isn't working anymore because the GPIO regulator driver is using descriptors since commit d6cd33ad7102 ("regulator: gpio: Convert to use descriptors") which in turn causes the system to use the polarity of the GPIOs (as specified in the DT) for selecting the states, but the polarity specified in the DT is wrong. This patch fixes the regulator DT definition, and that fixes SDR50. Fixes: 029efb3a03c5 ("ARM: dts: iwg20d-q7: Add SDHI1 support") Signed-off-by: Fabrizio Castro --- v1->v2: * Removed enable-active-high property --- arch/arm/boot/dts/iwg20d-q7-common.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/iwg20d-q7-common.dtsi b/arch/arm/boot/dts/iwg20d-q7-common.dtsi index e2b1ab9..ae75a1db 100644 --- a/arch/arm/boot/dts/iwg20d-q7-common.dtsi +++ b/arch/arm/boot/dts/iwg20d-q7-common.dtsi @@ -87,7 +87,7 @@ regulator-min-microvolt = <1800000>; regulator-max-microvolt = <3300000>; - gpios = <&gpio2 30 GPIO_ACTIVE_LOW>; + gpios = <&gpio2 30 GPIO_ACTIVE_HIGH>; gpios-states = <1>; states = <3300000 1 1800000 0>; -- 2.7.4