Messages in this thread | | | Date | Wed, 11 Sep 2019 09:19:48 -0600 | From | Lina Iyer <> | Subject | Re: [PATCH RFC 05/14] dt-bindings/interrupt-controller: pdc: add SPI config register |
| |
On Wed, Sep 11 2019 at 04:05 -0600, Linus Walleij wrote: >On Thu, Aug 29, 2019 at 8:47 PM Lina Iyer <ilina@codeaurora.org> 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. > Well the address remains the same. The bool defines how to access that register address - from linux or from the firmware using SCM calls. But I get your point, I could have different register namess - pdc-linux or pdc-scm and request by name. I can then use that to determine the mode for accessing the register.
Thanks, Lina
| |