Re: [PATCH V5 REPOST] mfd: add support for ams AS3722 PMIC

From: Grant Likely
Date: Fri Oct 25 2013 - 18:51:54 EST

On Tue, 22 Oct 2013 13:08:46 +0530, Laxman Dewangan <ldewangan@xxxxxxxxxx> wrote:
> The ams AS3722 is a compact system PMU suitable for mobile phones,
> tablets etc. It has 4 DC/DC step-down regulators, 3 DC/DC step-down
> controller, 11 LDOs, RTC, automatic battery, temperature and
> over-current monitoring, 8 GPIOs, ADC and a watchdog.
>
> Add MFD core driver for the AS3722 to support core functionality.
>
> Signed-off-by: Laxman Dewangan <ldewangan@xxxxxxxxxx>
> Signed-off-by: Florian Lobmaier <florian.lobmaier@xxxxxxx>
> Acked-by: Lee Jones <lee.jones@xxxxxxxxxx>
> Acked-by: Stephen Warren <swarren@xxxxxxxxxx>
> ---
> Changes from V1:
> - Remove compatible string from DT for subnode.
> - Nit cleanups in driver and use module_i2c_driver
>
> Changes from V2:
> - Change DT file to reflect the changes in gpio/pincntrl driver.
> Now there is no extra subnode.
>
> Changes from V3:
> - Combine DT of pincontrol and regulator to mfd.
> - Nits cleanups.
> - AMS to ams.
>
> Changes from V4:
> - Rephrased the dt binding documents.
> - Added dt-binding header for DTS file.
> - Fix some of indention issue in dt documents.
>
> Reposting V5 as per Lee's.
>
> Documentation/devicetree/bindings/mfd/as3722.txt | 194 ++++++++++
> drivers/mfd/Kconfig | 12 +
> drivers/mfd/Makefile | 1 +
> drivers/mfd/as3722.c | 449 ++++++++++++++++++++++
> include/dt-bindings/mfd/as3722.h | 52 +++
> include/linux/mfd/as3722.h | 423 ++++++++++++++++++++
> 6 files changed, 1131 insertions(+), 0 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/mfd/as3722.txt
> create mode 100644 drivers/mfd/as3722.c
> create mode 100644 include/dt-bindings/mfd/as3722.h
> create mode 100644 include/linux/mfd/as3722.h
>
> diff --git a/Documentation/devicetree/bindings/mfd/as3722.txt b/Documentation/devicetree/bindings/mfd/as3722.txt
> new file mode 100644
> index 0000000..fc2191e
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/mfd/as3722.txt
> @@ -0,0 +1,194 @@
> +* ams AS3722 Power management IC.
> +
> +Required properties:
> +-------------------
> +- compatible: Must be "ams,as3722".
> +- reg: I2C device address.
> +- interrupt-controller: AS3722 has internal interrupt controller which takes the
> + interrupt request from internal sub-blocks like RTC, regulators, GPIOs as well
> + as external input.
> +- #interrupt-cells: Should be set to 2 for IRQ number and flags.
> + The first cell is the IRQ number. IRQ numbers for different interrupt source
> + of AS3722 are defined at dt-bindings/mfd/as3722.h
> + The second cell is the flags, encoded as the trigger masks from binding document
> + interrupts.txt, using dt-bindings/irq.
> +
> +Optional submodule and their properties:
> +=======================================
> +
> +Pinmux and GPIO:
> +===============
> +Device has 8 GPIO pins which can be configured as GPIO as well as the special IO
> +functions.
> +
> +Please refer to pinctrl-bindings.txt in this directory for details of the
> +common pinctrl bindings used by client devices, including the meaning of the
> +phrase "pin configuration node".
> +
> +Following are properties which is needed if GPIO and pinmux functionality
> +is required:
> + Required properties:
> + -------------------
> + - gpio-controller: Marks the device node as a GPIO controller.
> + - #gpio-cells: Number of GPIO cells. Refer to binding document
> + gpio/gpio.txt
> +
> + Optional properties:
> + --------------------
> + Following properties are require if pin control setting is required
> + at boot.
> + - pinctrl-names: A pinctrl state named "default" be defined, using the
> + bindings in pinctrl/pinctrl-binding.txt.
> + - pinctrl[0...n]: Properties to contain the phandle that refer to
> + different nodes of pin control settings. These nodes represents
> + the pin control setting of state 0 to state n. Each of these
> + nodes contains different subnodes to represents some desired
> + configuration for a list of pins. This configuration can
> + include the mux function to select on those pin(s), and
> + various pin configuration parameters, such as pull-up,
> + open drain.
> +
> + Each subnode have following properties:
> + Required properties:
> + - pins: List of pins. Valid values of pins properties are:
> + gpio0, gpio1, gpio2, gpio3, gpio4, gpio5,
> + gpio6, gpio7
> +
> + Optional properties:
> + function, bias-disable, bias-pull-up, bias-pull-down,
> + bias-high-impedance, drive-open-drain.
> +
> + Valid values for function properties are:
> + gpio, interrupt-out, gpio-in-interrupt,
> + vsup-vbat-low-undebounce-out,
> + vsup-vbat-low-debounce-out,
> + voltage-in-standby, oc-pg-sd0, oc-pg-sd6,
> + powergood-out, pwm-in, pwm-out, clk32k-out,
> + watchdog-in, soft-reset-in
> +
> +Regulators:
> +===========
> +Device has multiple DCDC and LDOs. The node "regulators" is require if regulator
> +functionality is needed.
> +
> +Following are properties of regulator subnode.
> +
> + Optional properties:
> + -------------------
> + The input supply of regulators are the optional properties on the
> + regulator node. The input supply of these regulators are provided
> + through following properties:
> + vsup-sd2-supply: Input supply for SD2.
> + vsup-sd3-supply: Input supply for SD3.
> + vsup-sd4-supply: Input supply for SD4.
> + vsup-sd5-supply: Input supply for SD5.
> + vin-ldo0-supply: Input supply for LDO0.
> + vin-ldo1-6-supply: Input supply for LDO1 and LDO6.
> + vin-ldo2-5-7-supply: Input supply for LDO2, LDO5 and LDO7.
> + vin-ldo3-4-supply: Input supply for LDO3 and LDO4.
> + vin-ldo9-10-supply: Input supply for LDO9 and LDO10.
> + vin-ldo11-supply: Input supply for LDO11.

What is the data in these properties? The documentation isn't clear.

In principle, the binding looks okay to me.

g.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/