> MAX14577 chip is a multi-function device which includes MUIC, > charger and voltage regulator. The driver is located in drivers/mfd. > > This patch adds regulator driver for MAX14577 chip. There are two > regulators in this chip: > 1. Safeout LDO with constant voltage output of 4.9V. It can be only > enabled or disabled. > 2. Current regulator for the charger. It provides current from 90mA up > to 950mA. > Driver supports Device Tree. > > Signed-off-by: Krzysztof Kozlowski <k.kozlowski@xxxxxxxxxxx> > Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> > --- > drivers/mfd/max14577.c | 3 +- > drivers/regulator/Kconfig | 7 ++ > drivers/regulator/Makefile | 1 + > drivers/regulator/max14577.c | 268 ++++++++++++++++++++++++++++++++++++++++++ > 4 files changed, 278 insertions(+), 1 deletion(-) > create mode 100644 drivers/regulator/max14577.c > > diff --git a/drivers/mfd/max14577.c b/drivers/mfd/max14577.c > index 94b766d..d841f2a 100644 > --- a/drivers/mfd/max14577.c > +++ b/drivers/mfd/max14577.c > @@ -26,7 +26,8 @@ > > static struct mfd_cell max14577_devs[] = { > { .name = "max14577-muic", }, > - { .name = "max14577-regulator", }, > + { .name = "max14577-regulator", > + .of_compatible = "maxim,max14577-regulator", }, > { .name = "max14577-charger", }, > }; If you separate this patch out and make it look 'normal' i.e. sort out the indenting, I'll apply it. Instead of this randomness, I expect to see something like: > + { > + .name = "max14577-regulator", > + .of_compatible = "maxim,max14577-regulator", > + }, -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html