On Thu, Jun 14, 2018 at 01:09:02PM +0530, Taniya Das wrote: > Add device tree bindings for Low Power Audio subsystem clock controller for > Qualcomm Technology Inc's SDM845 SoCs. > > Signed-off-by: Taniya Das <tdas@xxxxxxxxxxxxxx> > --- > .../devicetree/bindings/clock/qcom,lpasscc.txt | 46 ++++++++++++++++++++++ > include/dt-bindings/clock/qcom,lpass-sdm845.h | 18 +++++++++ > 2 files changed, 64 insertions(+) > create mode 100644 Documentation/devicetree/bindings/clock/qcom,lpasscc.txt > create mode 100644 include/dt-bindings/clock/qcom,lpass-sdm845.h > > diff --git a/Documentation/devicetree/bindings/clock/qcom,lpasscc.txt b/Documentation/devicetree/bindings/clock/qcom,lpasscc.txt > new file mode 100644 > index 0000000..16cabc4 > --- /dev/null > +++ b/Documentation/devicetree/bindings/clock/qcom,lpasscc.txt > @@ -0,0 +1,46 @@ > +Qualcomm LPASS Clock Controller Binding > +----------------------------------------------- > + > +Required properties : > +- compatible : shall contain "qcom,sdm845-lpasscc" > +- #clock-cells : from common clock binding, shall contain 1. > + > +Note that #address-cells, #size-cells, and ranges shall be present to ensure > +the lpasscc can address the various lpass cc registers. > + > +Child Node Properties : > +The Low Pass Audio clock controller would need to define the following child > +nodes with the properties. > +- compatible : shall contain all of the below for clocks in each LPASS domain > + "qcom,sdm845-lpass-gcc", > + "qcom,sdm845-lpass-cc", > + "qcom,sdm845-lpass-qdsp6ss" > +- reg : shall contain base register address and size, I can't really see any reason to have these child nodes. Just put 3 entries in reg in the parent. Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html