Hello Mark,

On 03/20/2018 07:43 PM, Mark Brown wrote:

On Fri, Mar 16, 2018 at 06:09:11PM -0700, David Collins wrote:

quoted

Introduce bindings for RPMh regulator devices found on some
Qualcomm Technlogies, Inc. SoCs.  These devices allow a given

Please use subject lines matching the style for the subsystem.  This
makes it easier for people to identify relevant patches.

Looking through the DT binding patches sent to devicetree at vger.kernel.org,
it seemed like the preferred style is:
[PATCH] dt-bindings: subsystem: ...

I can change the subject line for this patch to be:
"[PATCH 2/2] regulator: dt-bindings: add QCOM RPMh regulator bindings"
if you prefer.

quoted

+=========================================
+Second Level Nodes - Regulators
+=========================================
+
+- regulator-name

This document appears to be reproducing the standard regulator bindings;
there is no need to do that, just reference them.

Sure, I'll remove this.

Thanks,
David

-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project