[PATCH 4/4] sdhci-s3c: add regulator support

Mark Brown broonie at opensource.wolfsonmicro.com
Wed Jul 28 11:41:49 EDT 2010


On 28 Jul 2010, at 07:48, Maurus Cuelenaere <mcuelenaere at gmail.com> wrote:
>> 
>> +	struct sdhci_s3c *sc = sdhci_priv(host);
>> +
>> +	if (sc->vmmc) {
>> +		int ret = regulator_disable(sc->vmmc);
>> +		if (ret)
>> +			return ret;
>> +		mdelay(2);
> 
> Shouldn't these delays be handled in the regulator framework itself?

A 2ms delay on power down seems suspicious for a regulator. I'm not sure why this is required but if it is I suspect it's due to a large cap on the regulator output and light load rather than something that's always true for whatever regulator is providing the supply.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20100728/cb7a4997/attachment-0001.html>


More information about the linux-arm-kernel mailing list