On Thu, Oct 28, 2021 at 03:08:24PM -0700, Stephen Boyd wrote: > Quoting Rakesh Pillai (2021-10-28 01:13:36) > > diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,sc7280-wpss-pil.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,sc7280-wpss-pil.yaml > > new file mode 100644 > > index 0000000..96d11a4 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/remoteproc/qcom,sc7280-wpss-pil.yaml > > @@ -0,0 +1,194 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/remoteproc/qcom,sc7280-wpss-pil.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Qualcomm SC7280 WPSS Peripheral Image Loader > > + > > +maintainers: > > + - Bjorn Andersson <bjorn.andersson@xxxxxxxxxx> > > + > > +description: > > + This document defines the binding for a component that loads and boots firmware > > + on the Qualcomm Technology Inc. WPSS. > > + > > +properties: > > + compatible: > > + enum: > > + - qcom,sc7280-wpss-pil > > + > > + reg: > > + maxItems: 1 > > + description: > > + The base address and size of the qdsp6ss register > > + > > + interrupts: > > + items: > > + - description: Watchdog interrupt > > + - description: Fatal interrupt > > + - description: Ready interrupt > > + - description: Handover interrupt > > + - description: Stop acknowledge interrupt > > + - description: Shutdown acknowledge interrupt > > + > > + interrupt-names: > > + items: > > + - const: wdog > > + - const: fatal > > + - const: ready > > + - const: handover > > + - const: stop-ack > > + - const: shutdown-ack > > + > > + clocks: > > + items: > > + - description: GCC WPSS AHB BDG Master clock > > + - description: GCC WPSS AHB clock > > + - description: GCC WPSS RSCP clock > > + - description: XO clock > > + > > + clock-names: > > + items: > > + - const: ahb_bdg > > + - const: ahb > > + - const: rscp > > + - const: xo > > + > > + power-domains: > > + items: > > + - description: CX power domain > > + - description: MX power domain > > + > > + power-domain-names: > > + items: > > + - const: cx > > + - const: mx > > + > > + resets: > > + items: > > + - description: AOSS restart > > + - description: PDC SYNC > > + > > + reset-names: > > + items: > > + - const: restart > > + - const: pdc_sync > > + > > + memory-region: > > Does it need > > $ref: /schemas/types.yaml#/definitions/phandle > > because it's a phandle? No, standard property that already has a type. Rob
