From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chanwoo Choi Subject: Re: [PATCH 6/6] Documentation: Add device tree bindings document for max77843 Date: Fri, 23 Jan 2015 15:25:07 +0900 Message-ID: <54C1E943.2020800@samsung.com> References: <1421989367-32721-1-git-send-email-jaewon02.kim@samsung.com> <1421989367-32721-7-git-send-email-jaewon02.kim@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-reply-to: <1421989367-32721-7-git-send-email-jaewon02.kim-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Jaewon Kim Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Inki Dae , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Lee Jones , Sebastian Reichel , Mark Brown , Beomho Seo List-Id: devicetree@vger.kernel.org Hi Jaewon, On 01/23/2015 02:02 PM, Jaewon Kim wrote: > Add document describing device tree bindings for max77843 MFD. > Drivers: MFD core, regulator, extcon, charger and fuelgauge. > > Cc: Rob Herring > Cc: Pawel Moll > Cc: Mark Rutland > Cc: Ian Campbell > Cc: Kumar Gala > Cc: Lee Jones > Cc: Chanwoo Choi > Cc: Sebastian Reichel > Cc: Mark Brown > Signed-off-by: Beomho Seo > Signed-off-by: Jaewon Kim > --- > Documentation/devicetree/bindings/mfd/max77843.txt | 87 ++++++++++++++++++++ > 1 file changed, 87 insertions(+) > create mode 100644 Documentation/devicetree/bindings/mfd/max77843.txt > > diff --git a/Documentation/devicetree/bindings/mfd/max77843.txt b/Documentation/devicetree/bindings/mfd/max77843.txt > new file mode 100644 > index 0000000..6895604 > --- /dev/null > +++ b/Documentation/devicetree/bindings/mfd/max77843.txt > @@ -0,0 +1,87 @@ > +Maxim MAX77843 multi-function device > + > +MAX77843 is a Multi-Function Device with the following submodules: > +- PMIC : 2 SAFEOUT LDOs for USB device > +- CHARGER : Li+ battery charger with Fuel Gauge > +- MUIC : Micro USB Interface Circuit > +- HAPTIC : motor control driver for haptics HAPTIC ? > +- LED : 4 channel RGBW LED LED ? This patchset don't include andy haptic/led drivers. I think it is un-necessary information. When you implement the haptic/led driver, you will add the information for haptic/led driver. > + > +It is interfaced to host controller using I2C. > + > +Required properties: > +- compatible : Must be "maxim,max77843". > +- reg : I2C slave address of PMIC block. > +- interrupts : I2C line for main SoCs. > +- interrupt-parent : The parent of interrupt controller. > + > +Optional properties: > +- regulators : The regulators of max77843 have to be instantiated under subnode > + named "regulators" using the following format. > + > + [*]refer : Documentation/devicetree/bindings/regulator/regulator.txt > + > + regulators { > + SAFEOUT { > + regulator-name = "SAFEOUT"; > + }; > + } > + > + List of valid regulator names: > + - SAFEOUT1, SAFEOUT2, CHARGER. > + > +- max77843-muic : This properties used by extcon consumers. > + Required properties: > + - compatible : Must be "maxim,max77842-muic". > + > +- max77843-charger : There battery charger of MAX77843 have to be instantiated > + under sub-node named "max77843-charger" using the following format. > + Required properties: > + - compatible : Must be "maxim,max77842-charger". > + - maxim,fast-charge-uamp : Fast charge current levels are > + 100 mA to 3150 mA programmed by I2C per 100 mA. > + - maxim,top-off-uamp : Top off current threshold levels are > + 125 mA to 650 mA programmed by I2C per 75 mA. > + - maxim,input-uamp-limit : Input current limit levels are > + 100 mA to 3533 mA programmed by I2C per 33 mA. > +- max77843-fuelgauge : There fuelgauge of MAX77843 have to be instantiated > + under sub-node named "max77843-fuelgauge" using the following format. > + Required properties: > + - compatible : Must be "maxim,max77842-fuelgauge". > + > +Example: > + max77843@66 { > + compatible = "samsung,max77843"; > + reg = <0x66>; > + interrupt-parent = <&gpa1>; > + interrupts = <5 2>; > + > + regulators { > + SAFEOUT1 { > + regulator-name = "SAFEOUT1"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <4950000>; > + }; > + SAFEOUT2 { > + regulator-name = "SAFEOUT2"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <4950000>; > + }; As I knew, max77843 regulator driver supprot 'CHARGER' regulator. I think you have to add the 'CHARGER' dt node for regulators. > + }; > + > + max77843-muic { > + compatible = "maxim,max77843-muic"; > + }; > + > + max77843-charger { > + compatible = "maxim,max77843-charger"; > + maxim,fast-charge-uamp = <450000>; > + maxim,top-off-uamp = <125000>; > + maxim,input-uamp-limit = <500000>; > + }; > + > + max77843-fuelgauge { > + compatible = "maxim,max77843-fuelgauge"; > + }; > + > + }; > Thanks, Chanwoo Choi -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html