[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAK9yfHwvkZsARpWyZYu7+1_nap_Px8TMYCm+hyubA_peobdQMw@mail.gmail.com>
Date: Wed, 3 Jul 2013 17:16:41 +0530
From: Sachin Kamat <sachin.kamat@...aro.org>
To: Yadwinder Singh Brar <yadi.brar@...sung.com>
Cc: linux-kernel@...r.kernel.org, yadi.brar01@...il.com,
sbkim73@...sung.com, broonie@...nel.org, lgirdwood@...il.com,
sameo@...ux.intel.com, rob.herring@...xeda.com,
grant.likely@...aro.org
Subject: Re: [PATCH 3/4] mfd: s2mps11: Add device tree support
On 24 June 2013 16:50, Yadwinder Singh Brar <yadi.brar@...sung.com> wrote:
> This patch adds DT compatible string for s2mps11 and binding documentation.
>
> Signed-off-by: Yadwinder Singh Brar <yadi.brar@...sung.com>
> ---
> Documentation/devicetree/bindings/mfd/s2mps11.txt | 98 +++++++++++++++++++++
> drivers/mfd/sec-core.c | 3 +
> 2 files changed, 101 insertions(+), 0 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/mfd/s2mps11.txt
>
> diff --git a/Documentation/devicetree/bindings/mfd/s2mps11.txt b/Documentation/devicetree/bindings/mfd/s2mps11.txt
> new file mode 100644
> index 0000000..7984625
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/mfd/s2mps11.txt
> @@ -0,0 +1,98 @@
> +
> +* Samsung S2MPS11 Voltage and Current Regulator
> +
> +The Samsung S2MP211 is a multi-function device which includes volatage and
s/volatage/voltage
> +current regulators, rtc, charger controller and other sub-blocks. It is
s/rtc/RTC
> +interfaced to the host controller using a i2c interface. Each sub-block is
> +addressed by the host system using different i2c slave address.
s/i2c/I2C -> also elsewhere
> +
> +Required properties:
> +- compatible: Should be "samsung,s2mps11-pmic".
> +- reg: Specifies the i2c slave address of the pmic block. It should be 0x66.
> +
> +Optional properties:
> +- interrupt-parent: Specifies the phandle of the interrupt controller to which
> + the interrupts from s2mps11 are delivered to.
> +- interrupts: Interrupt specifiers for interrupt sources.
> +
> +Optional nodes:
> +- regulators: The regulators of s2mps11 that have to be instantiated should be
> +included in a sub-node named 'regulators'. Regulator nodes included in this
> +sub-node should be of the format as listed below.
> +
> + regulator_name {
> + [standard regulator constraints....];
> + [regulator-ramp-disable];
> + };
> +
> + regulator-ramp-delay for BUCKs = [6250/12500/25000(default)/50000] uV/us
> +
> + Optional property for BUCK[2/3/4/6] only:
> + regulator-ramp-disable: boolean, disables ramp delay on hardware.
> +
> +NOTE: Some BUCKs shares the ramp rate setting i.e. same ramp value will be set
s/shares/share
--
With warm regards,
Sachin
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists