[PATCH 0/4] Regulator delays and AB8500 init

[Linus Walleij]

From: Linus Walleij <linus.walleij@xxxxxxxxxx>

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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/