| Message ID | 1559203675-19675-3-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/r8a77470-iwg23s-sbc.dts b/arch/arm/boot/dts/r8a77470-iwg23s-sbc.dts index af78e77..a199ded 100644 --- a/arch/arm/boot/dts/r8a77470-iwg23s-sbc.dts +++ b/arch/arm/boot/dts/r8a77470-iwg23s-sbc.dts @@ -63,7 +63,9 @@ regulator-min-microvolt = <1800000>; regulator-max-microvolt = <3300000>; - gpios = <&gpio2 24 GPIO_ACTIVE_LOW>; + enable-active-high; + + gpios = <&gpio2 24 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 24 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: 9eb36b945b5c ("ARM: dts: iwg23s-sbc: Add uSD and eMMC support") Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> --- arch/arm/boot/dts/r8a77470-iwg23s-sbc.dts | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)