On 02/20/2013 06:49 AM, J, KEERTHY wrote:

Mark Brown wrote at Wednesday, February 20, 2013 4:57 PM:

quoted

On Wed, Feb 20, 2013 at 09:30:15AM +0530, J Keerthy wrote:

quoted

From: Graeme Gregory <redacted>

Add the DTS definition for the palmas device including the MFD children.

...

quoted

quoted

diff --git a/Documentation/devicetree/bindings/mfd/palmas.txt b/Documentation/devicetree/bindings/mfd/palmas.txt

...

quoted

quoted

+Texas Instruments Palmas family
+
+The Palmas familly are Integrated Power Management Chips.
+These chips are connected to an i2c bus.

I would expect this to enumerate the regulators that the user can
select but it doesn't appear to do that.

Mark,

Thanks for reviewing the patch.

I went through the DT Documentation files in the regulator/ folder.
So shall I add a list of regulators which are in the Palmas device
Under a separate file under Documentation/devicetree/bindings/regulator ?
Is that what you are looking for?

I believe what Mark wants is something like the following in this file
(take from Documentation/devicetree/bindings/regulator/tps6586x.txt):

- regulators: A node that houses a sub-node for each regulator within the
  device. Each sub-node is identified using the node's name (or the deprecated
  regulator-compatible property if present), with valid values listed below.
  The content of each sub-node is defined by the standard binding for
  regulators; see regulator.txt.
  sys, sm[0-2], ldo[0-9] and ldo_rtc