rockchip: veyron: Set vcc33_sd regulator value
authorCarlo Caione <carlo@endlessm.com>
Mon, 11 Jun 2018 19:00:50 +0000 (20:00 +0100)
committerPhilipp Tomsich <philipp.tomsich@theobroma-systems.com>
Fri, 20 Jul 2018 23:55:27 +0000 (01:55 +0200)
On the veyron board the vcc33_sd regulator is used as vmmc-supply for
the SD card. This regulator is powered in the MMC core during power on
but its value is never actually set.

In the veyron platform the reset value for the LDO output is 1.8V while
the standard (min and max) value for this regulator defined in the DTS
is 3.3V. When the MMC core enable the regulator without setting its
value, the output is automatically set to 1.8V instead of 3.3V.

With this patch we preemptively set the value to 3.3V.

Signed-off-by: Carlo Caione <carlo@endlessm.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
arch/arm/mach-rockchip/rk3288-board.c

index 73cd6684aad8e8eae99288bde9ce9e717dfcf03c..9c4f7f219f1359b8d96e1005d1c39ea23215d142 100644 (file)
@@ -122,6 +122,16 @@ static int veyron_init(void)
        if (IS_ERR_VALUE(ret))
                return ret;
 
+       ret = regulator_get_by_platname("vcc33_sd", &dev);
+       if (ret) {
+               debug("Cannot get regulator name\n");
+               return ret;
+       }
+
+       ret = regulator_set_value(dev, 3300000);
+       if (ret)
+               return ret;
+
        ret = regulators_enable_boot_on(false);
        if (ret) {
                debug("%s: Cannot enable boot on regulators\n", __func__);