lkml.org 
[lkml]   [2011]   [Mar]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Subject[PATCH 0/4] Regulator delays and AB8500 init
Date
From: Linus Walleij <linus.walleij@linaro.org>

This provides some infrastructure to do delays when enabling
or setting the voltage of an active regulator.

There is one thing I'm hesitant about here: our AB8500-local
code was using msleep() whereas the code from the
regulator framework was using mdelay() exclusively. Is
there some generic way to determine a threshold for whether
this should delay or sleep? I've seen msleep(100) frequently
for example but ultimately it depends on system speed
I guess...?

Also, inside the ab8500 driver I always return 0 delay time
for set_voltage_time() if the regulator is not enabled,
I don't know if this clause is proper to move to the core,
I think so but maybe there are corner cases out there?

Linus Walleij


Bengt Jonsson (2):
regulator: initialization for ab8500 regulators
mach-ux500: provide ab8500 init vector

Linus Walleij (2):
regulator: add set_voltage_time[_sel] infrastructure
regulator: add ab8500 enable and raise time delays

arch/arm/mach-ux500/board-mop500-regulators.c | 177 ++++++++++++++++++
arch/arm/mach-ux500/board-mop500-regulators.h | 2 +
arch/arm/mach-ux500/board-mop500.c | 3 +
drivers/regulator/ab8500.c | 246 ++++++++++++++++++++++++-
drivers/regulator/core.c | 43 +++++
include/linux/mfd/ab8500.h | 6 +
include/linux/regulator/ab8500.h | 50 +++++-
include/linux/regulator/driver.h | 15 ++-
8 files changed, 537 insertions(+), 5 deletions(-)

--
1.7.3.2



\
 
 \ /
  Last update: 2011-03-11 11:57    [W:3.941 / U:0.028 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site