On Fri, Sep 03, 2010 at 11:00:31AM -0700, Kevin Hilman wrote: > Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx> writes: > > The only thing I can think you might need to do if this is just straight > > voltage setting is re-add support for multiple consumers setting > > voltages simultaneously > Yeah, that sounds like what we need. > re-add? was it there at one point and removed? Any pointers to the old > code? It was present in the pre-merge regulator API which you can find in the bowels of: git://opensource.wolfsonmicro.com/linux-2.6-audioplus but was removed to simplify review during the merge. It's fairly simple to do, it's just that there's been no demand. Essentially all that needs doing is that when regulator_set_voltage() is called instead of merging with the machine constraints and applying the setting immediately we store the constraints that are specified in the consumer then iterate over all enabled consumers applying all the constraints that they've set in addition to those from the machine. This results in a configuration which is the lowest possible voltage which satisfies all the constraints that have been supplied and for supplies with only one consumer it gives the same behaviour as we have currently. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html