[PATCH 3/9] regulator: ab8500: move to set_optimum_mode
From: broonie@kernel.org (Mark Brown)
Date: 2015-01-29 01:25:50
Also in:
linux-arm-msm, lkml
On Tue, Jan 27, 2015 at 06:46:33PM -0800, Bjorn Andersson wrote:
Signed-off-by: Bjorn Andersson <redacted> --- drivers/regulator/ab8500.c | 48 +++++++++++++++++++++++----------------------- 1 file changed, 24 insertions(+), 24 deletions(-)
These refactorings are surprising and don't seem like an obvious advantage - they mean that we now have two different paths to setting the mode (one explict and the other via this). For regulators that do want to just do a lookup of a mode based on parameters I'd expect to keep the operations separate in the same way that we do for things like voltage setting in case we come up with ways of using the two bits separately. -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 473 bytes Desc: Digital signature URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150128/263cc1e8/attachment-0001.sig>