Commit d9df0187 authored by Gregory CLEMENT's avatar Gregory CLEMENT Committed by Mark Brown

regulator: twl6030: workaround the VMMC reset behavior

During reset the VMMC regulator doesn't reach 0V and only drops to
1.8V, furthermore the pulse width is under 200us whereas the SD
specification expect 1ms.

The WR_S bit allows the TWL6030 to no reset at all the VMMC during warm
reset and keep the current voltage. Thanks to this workaround the SD
card doesn't reach a undefined reset stage.

Actually this behavior is available for all the LDO regulator, so the
driver will also allow to use it with any of these regulator.
Signed-off-by: default avatarGregory CLEMENT <gregory.clement@bootlin.com>
Link: https://lore.kernel.org/r/20190725094542.16547-4-gregory.clement@bootlin.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
parent 621d3ce8
...@@ -57,6 +57,9 @@ struct twlreg_info { ...@@ -57,6 +57,9 @@ struct twlreg_info {
#define VREG_BC_PROC 3 #define VREG_BC_PROC 3
#define VREG_BC_CLK_RST 4 #define VREG_BC_CLK_RST 4
/* TWL6030 LDO register values for VREG_VOLTAGE */
#define TWL6030_VREG_VOLTAGE_WR_S BIT(7)
/* TWL6030 LDO register values for CFG_STATE */ /* TWL6030 LDO register values for CFG_STATE */
#define TWL6030_CFG_STATE_OFF 0x00 #define TWL6030_CFG_STATE_OFF 0x00
#define TWL6030_CFG_STATE_ON 0x01 #define TWL6030_CFG_STATE_ON 0x01
...@@ -68,9 +71,10 @@ struct twlreg_info { ...@@ -68,9 +71,10 @@ struct twlreg_info {
#define TWL6030_CFG_STATE_APP(v) (((v) & TWL6030_CFG_STATE_APP_MASK) >>\ #define TWL6030_CFG_STATE_APP(v) (((v) & TWL6030_CFG_STATE_APP_MASK) >>\
TWL6030_CFG_STATE_APP_SHIFT) TWL6030_CFG_STATE_APP_SHIFT)
/* Flags for SMPS Voltage reading */ /* Flags for SMPS Voltage reading and LDO reading*/
#define SMPS_OFFSET_EN BIT(0) #define SMPS_OFFSET_EN BIT(0)
#define SMPS_EXTENDED_EN BIT(1) #define SMPS_EXTENDED_EN BIT(1)
#define TWL_6030_WARM_RESET BIT(3)
/* twl6032 SMPS EPROM values */ /* twl6032 SMPS EPROM values */
#define TWL6030_SMPS_OFFSET 0xB0 #define TWL6030_SMPS_OFFSET 0xB0
...@@ -250,6 +254,9 @@ twl6030ldo_set_voltage_sel(struct regulator_dev *rdev, unsigned selector) ...@@ -250,6 +254,9 @@ twl6030ldo_set_voltage_sel(struct regulator_dev *rdev, unsigned selector)
{ {
struct twlreg_info *info = rdev_get_drvdata(rdev); struct twlreg_info *info = rdev_get_drvdata(rdev);
if (info->flags & TWL_6030_WARM_RESET)
selector |= TWL6030_VREG_VOLTAGE_WR_S;
return twlreg_write(info, TWL_MODULE_PM_RECEIVER, VREG_VOLTAGE, return twlreg_write(info, TWL_MODULE_PM_RECEIVER, VREG_VOLTAGE,
selector); selector);
} }
...@@ -259,6 +266,9 @@ static int twl6030ldo_get_voltage_sel(struct regulator_dev *rdev) ...@@ -259,6 +266,9 @@ static int twl6030ldo_get_voltage_sel(struct regulator_dev *rdev)
struct twlreg_info *info = rdev_get_drvdata(rdev); struct twlreg_info *info = rdev_get_drvdata(rdev);
int vsel = twlreg_read(info, TWL_MODULE_PM_RECEIVER, VREG_VOLTAGE); int vsel = twlreg_read(info, TWL_MODULE_PM_RECEIVER, VREG_VOLTAGE);
if (info->flags & TWL_6030_WARM_RESET)
vsel &= ~TWL6030_VREG_VOLTAGE_WR_S;
return vsel; return vsel;
} }
...@@ -710,6 +720,9 @@ static int twlreg_probe(struct platform_device *pdev) ...@@ -710,6 +720,9 @@ static int twlreg_probe(struct platform_device *pdev)
break; break;
} }
if (of_get_property(np, "ti,retain-on-reset", NULL))
info->flags |= TWL_6030_WARM_RESET;
config.dev = &pdev->dev; config.dev = &pdev->dev;
config.init_data = initdata; config.init_data = initdata;
config.driver_data = info; config.driver_data = info;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment