On Tue, Jan 27, 2015 at 06:46:33PM -0800, Bjorn Andersson wrote: > Signed-off-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxxxxxx> > --- > 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.
Attachment:
signature.asc
Description: Digital signature
