On Thu, Jul 25, 2013 at 11:33:18AM +0800, Robin Gong wrote: > @@ -0,0 +1,113 @@ > +PFUZE100 family of regulators > + > +Required properties: > +- compatible: "fsl,pfuze100" > +- reg: I2C slave address > +- regulators: This is the list of child nodes that specify the regulator > + initialization data for defined regulators. Please refer to below doc > + Documentation/devicetree/bindings/regulator/regulator.txt. I had the comment on v4 patch. The "regulators" should not be documented as a property but a sub-node, because it's not a property but sub-node. Shawn -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html