[PATCH v6 4/9] mfd: axp20x: Split the driver into core and i2c bits

Hi,

On Mon, Jan 11, 2016 at 5:25 PM, Lee Jones [off-list ref] wrote:

On Thu, 17 Dec 2015, Chen-Yu Tsai wrote:

quoted

The axp20x driver assumes the device is i2c based. This is not the
case with later chips, which use a proprietary 2 wire serial bus
by Allwinner called "Reduced Serial Bus".

This patch follows the example of mfd/wm831x and splits it into
an interface independent core, and an i2c specific glue layer.
MFD_AXP20X and the new MFD_AXP20X_I2C are changed to tristate
symbols, allowing the driver to be built as modules.

Whitespace and other style errors in the moved i2c specific code
have been fixed. Included but unused header files are removed as
well.

Signed-off-by: Chen-Yu Tsai <redacted>
---
 drivers/mfd/Kconfig        |  14 ++++---
 drivers/mfd/Makefile       |   1 +
 drivers/mfd/axp20x-i2c.c   | 102 +++++++++++++++++++++++++++++++++++++++++++++
 drivers/mfd/axp20x.c       |  88 +++++++-------------------------------
 include/linux/mfd/axp20x.h |  33 ++++++++++++++-
 5 files changed, 158 insertions(+), 80 deletions(-)
 create mode 100644 drivers/mfd/axp20x-i2c.c

Acked-by: Lee Jones <redacted>

[..]

quoted

diff --git a/drivers/mfd/axp20x-i2c.c b/drivers/mfd/axp20x-i2c.c
new file mode 100644
index 000000000000..b54205677bb2
--- /dev/null
+++ b/drivers/mfd/axp20x-i2c.c

@@ -0,0 +1,102 @@
+/*
+ * axp20x-i2c.c - I2C driver for the X-Powers' Power Management ICs

Do you want me to remove the filenames from these 2 files (axp20x.c and
axp20x-i2c.c) as well?

quoted

+ *
+ * AXP20x typically comprises an adaptive USB-Compatible PWM charger, BUCK DC-DC
+ * converters, LDOs, multiple 12-bit ADCs of voltage, current and temperature
+ * as well as configurable GPIOs.
+ *
+ * This driver supports the I2C variants.
+ *
+ * Author: Carlo Caione [off-list ref]

Not sure about the copyright, since it's not mine.

quoted

+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ */
+

[..]

Regards
ChenYu