On Thu, Aug 29, 2019 at 8:47 PM Lina Iyer <ilina@xxxxxxxxxxxxxx> wrote: > +- qcom,scm-spi-cfg: > + Usage: optional > + Value type: <bool> > + Definition: Specifies if the SPI configuration registers have to be > + written from the firmware. > + > Example: > > pdc: interrupt-controller@b220000 { > compatible = "qcom,sdm845-pdc"; > - reg = <0xb220000 0x30000>; > + reg = <0xb220000 0x30000>, <0x179900f0 0x60>; > qcom,pdc-ranges = <0 512 94>, <94 641 15>, <115 662 7>; > #interrupt-cells = <2>; > interrupt-parent = <&intc>; > interrupt-controller; > + qcom,scm-spi-cfg; You can probably drop this bool if you just give names to the registers. Like reg = <0xb220000 0x30000>, <0x179900f0 0x60>; reg-names = "gic", "pdc"; Then jus check explicitly for a "pdc" register and in that case initialize the PDC. Yours, Linus Walleij