From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753398Ab3LCKwu (ORCPT ); Tue, 3 Dec 2013 05:52:50 -0500 Received: from devils.ext.ti.com ([198.47.26.153]:45280 "EHLO devils.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752879Ab3LCKwq (ORCPT ); Tue, 3 Dec 2013 05:52:46 -0500 Message-ID: <529DB7E5.8040702@ti.com> Date: Tue, 3 Dec 2013 16:22:21 +0530 From: Keerthy User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130623 Thunderbird/17.0.7 MIME-Version: 1.0 To: Lee Jones CC: Keerthy , , , , , , , , , , , , , , Subject: Re: [PATCH 1/4] mfd: DT bindings for TPS65218 PMIC References: <1386053006-6480-1-git-send-email-j-keerthy@ti.com> <1386053006-6480-2-git-send-email-j-keerthy@ti.com> <20131203093637.GE11828@lee--X1> In-Reply-To: <20131203093637.GE11828@lee--X1> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tuesday 03 December 2013 03:06 PM, Lee Jones wrote: >> Add DT bindings for TPS65218 PMIC. >> >> Signed-off-by: Keerthy >> --- >> Documentation/devicetree/bindings/mfd/tps65218.txt | 27 ++++++++++++++++++++ >> .../devicetree/bindings/regulator/tps65218.txt | 22 ++++++++++++++++ >> 2 files changed, 49 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/mfd/tps65218.txt >> create mode 100644 Documentation/devicetree/bindings/regulator/tps65218.txt >> >> diff --git a/Documentation/devicetree/bindings/mfd/tps65218.txt b/Documentation/devicetree/bindings/mfd/tps65218.txt >> new file mode 100644 >> index 0000000..87cb7a8 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/mfd/tps65218.txt >> @@ -0,0 +1,27 @@ >> +The TPS65218 Integrated Power Management Chips. >> +These chips are connected to an i2c bus. > I2C Ok. >> +Required properties: >> +- compatible : Must be "ti,tps65218"; >> +- interrupts : This i2c device has an IRQ line connected to the main SoC > I2C Ok. >> +- interrupt-controller : Since the tps65218 support several interrupts > support(s) (hosts?) Yes Supports multiple interrupts from the sub-modules. >> + internally, it is considered as an interrupt controller cascaded to the SoC. >> +- #interrupt-cells = <2>; >> +- interrupt-parent : The parent interrupt controller. > Phandle to ... > >> +Optional node: > node(s): Ok. > >> +- Child nodes contain in the tps65218. >> + It supports a number of features. > Please re-phase the above two sentences to something decipherable. Ok. >> + The children nodes will thus depend of the capability of the variant. > Please, go on ... Ok. >> +Example: >> +/* >> + * Integrated Power Management Chip >> + */ >> +tps@24 { >> + compatible = "ti,tps65218"; >> + reg = <0x24>; >> + interrupt-controller; >> + #interrupt-cells = <2>; >> + interrupt-parent = <&gic>; >> +}; > Perhaps it would be better to centralise the documentation inclusive > of the regulator contingent. Or at least provide a _full_ example in > each document. Ok. > >> diff --git a/Documentation/devicetree/bindings/regulator/tps65218.txt b/Documentation/devicetree/bindings/regulator/tps65218.txt >> new file mode 100644 >> index 0000000..1ccf170 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/regulator/tps65218.txt >> @@ -0,0 +1,22 @@ >> +TPS65218 family of regulators >> + >> +Required properties: >> +For tps65218 regulators/LDOs >> +- compatible: >> + - "ti,tps65218-dcdc1" for DCDC1 >> + - "ti,tps65218-dcdc2" for DCDC2 >> + - "ti,tps65218-dcdc3" for DCDC3 >> + - "ti,tps65218-dcdc4" for DCDC4 >> + - "ti,tps65218-dcdc5" for DCDC5 >> + - "ti,tps65218-dcdc6" for DCDC6 >> + - "ti,tps65218-ldo1" for LDO1 LDO > Why aren't you using 'regulator-compatible'? I referred tps65217.txt the predecessors and used the compatible. I also referred twl-regulator.txt. > >> +Optional properties: >> +- Any optional property defined in bindings/regulator/regulator.txt >> + >> +Example: >> + xyz: regulator@0 { > Genuine question: Is the @0 meaningful? @0 is not the best example. @24 is what i use on the am43x. I will change it to @24. >> + compatible = "ti,tps65218-dcdc1"; >> + regulator-min-microvolt = <1000000>; >> + regulator-max-microvolt = <3000000>; >> + }; Thanks for reviewing. Kind Regards, Keerthy