Messages in this thread | | | Date | Tue, 13 Nov 2018 09:30:37 -0800 | From | Mark Brown <> | Subject | Re: [PATCH 1/3] regulator: dt-bindings: add MCP16502 regulator bindings |
| |
On Tue, Nov 13, 2018 at 11:29:24AM +0000, Andrei.Stefanescu@microchip.com wrote:
> +- lpm-gpio: GPIO for LPM pin. Note that this GPIO *must* remain high during > + suspend-to-ram, keeping the PMIC into HIBERNATE mode.
All GPIO properties should be -gpios even if they can only ever have one GPIO.
> +- regulators: A node that houses a sub-node for each regulator within > + the device. Each sub-node is identified using the node's > + name. The content of each sub-node is defined by the > + standard binding for regulators; see regulator.txt.
You should describe which regulators exist for the device. [unhandled content-type:application/pgp-signature] | |