| Message ID | 1559203675-19675-2-git-send-email-fabrizio.castro@bp.renesas.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Simon Horman |
| Headers | show |
| Series | Fix SDR50 on iwg20d and iwg23s | expand |
diff --git a/arch/arm/boot/dts/iwg20d-q7-common.dtsi b/arch/arm/boot/dts/iwg20d-q7-common.dtsi index e2b1ab9..87be070 100644 --- a/arch/arm/boot/dts/iwg20d-q7-common.dtsi +++ b/arch/arm/boot/dts/iwg20d-q7-common.dtsi @@ -87,7 +87,9 @@ regulator-min-microvolt = <1800000>; regulator-max-microvolt = <3300000>; - gpios = <&gpio2 30 GPIO_ACTIVE_LOW>; + enable-active-high; + + gpios = <&gpio2 30 GPIO_ACTIVE_HIGH>; gpios-states = <1>; states = <3300000 1 1800000 0>;
The GPIO regulator is using descriptors since commit d6cd33ad7102 ("regulator: gpio: Convert to use descriptors") leading to calls into gpiolib now, which in turn consider gpio2 30 as active low due to the fact that the corresponding DT node is missing property enable-active-high, breaking the SDR50 functionality. 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 <fabrizio.castro@bp.renesas.com> --- arch/arm/boot/dts/iwg20d-q7-common.dtsi | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)