On Tuesday, October 25, 2016 3:41 PM, Lee Jones wrote: > On Tue, 25 Oct 2016, Eric Jeong wrote: > > > > > From: Eric Jeong <eric.jeong.opensource@xxxxxxxxxxx> > > > > This patch adds support for the PV88080 PMIC. > > > > This pathch is done as part of the existing PV88080 regulator driver > > by expanding the driver for GPIO function support. > > > > The MFD core driver provides communication through the I2C interface. > > and contains the following components: > > > > - Regulators > > - Configurable GPIOs > > > > Kconfig and Makefile are updated to reflect support for PV88080 PMIC. > > > > Signed-off-by: Eric Jeong <eric.jeong.opensource@xxxxxxxxxxx> > > > > --- > > This patch applies against linux-next and next-20161024 > > > > Hi, > > > > This change is made as a single patch. Because, to ensure that kernel > > builds and runs properly after this patch. > > > > The regulator device driver for PV88080 IC is submitted to Linux kernel. > > And now, GPIO function is required. In order to add GPIO driver, MFD > > driver is also required. > > > > Changes > > - Add MFD driver. > > - Add GPIO driver. > > - Update regulator driver to reflect the support. > > - Delete pv88080-regulator.h file. > > - Move binding document to mfd directory. > > > > Regards, > > Eric Jeong, Dialog Semiconductor Ltd. > > > > > > Documentation/devicetree/bindings/mfd/pv88080.txt | 63 +++++ > > .../devicetree/bindings/regulator/pv88080.txt | 62 ----- > > drivers/gpio/Kconfig | 11 + > > drivers/gpio/Makefile | 1 + > > drivers/gpio/gpio-pv88080.c | 195 ++++++++++++++ > > drivers/mfd/Kconfig | 12 + > > drivers/mfd/Makefile | 2 + > > drivers/mfd/pv88080-core.c | 270 ++++++++++++++++++++ > > drivers/mfd/pv88080-i2c.c | 99 +++++++ > > drivers/regulator/Kconfig | 5 +- > > drivers/regulator/pv88080-regulator.c | 202 ++++++--------- > > drivers/regulator/pv88080-regulator.h | 118 --------- > > include/linux/mfd/pv88080.h | 236 +++++++++++++++++ > > 13 files changed, 970 insertions(+), 306 deletions(-) create mode > > 100644 Documentation/devicetree/bindings/mfd/pv88080.txt > > delete mode 100644 > > Documentation/devicetree/bindings/regulator/pv88080.txt > > create mode 100644 drivers/gpio/gpio-pv88080.c create mode 100644 > > drivers/mfd/pv88080-core.c create mode 100644 > > drivers/mfd/pv88080-i2c.c delete mode 100644 > > drivers/regulator/pv88080-regulator.h > > create mode 100644 include/linux/mfd/pv88080.h > > You're going to need to split this patch up as much as possible. > > No one is going to want to review a 1200 line patch. > > -- > Lee Jones > Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source > software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog Sorry for inconvenience. I will separate the patch and send again. Regards Eric ��.n��������+%������w��{.n����z�{��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f