From: Rob Herring <robh@kernel.org> To: Alex Elder <elder@linaro.org> Cc: Rob Herring <robh+dt@kernel.org>, Mark Rutland <mark.rutland@arm.com>, davem@davemloft.net, Arnd Bergmann <arnd@arndb.de>, Bjorn Andersson <bjorn.andersson@linaro.org>, ilias.apalodimas@linaro.org, netdev <netdev@vger.kernel.org>, devicetree@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, linux-arm-kernel <linux-arm-kernel@lists.infradead.org>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, syadagir@codeaurora.org, mjavid@codeaurora.org Subject: Re: [RFC PATCH 01/12] dt-bindings: soc: qcom: add IPA bindings Date: Wed, 7 Nov 2018 08:59:08 -0600 [thread overview] Message-ID: <CABGGiswmpmSUmg9jEW7GnNtL2uXAN7jJOqFO5kG8adq71GuZpw@mail.gmail.com> (raw) In-Reply-To: <20181107003250.5832-2-elder@linaro.org> On Tue, Nov 6, 2018 at 6:33 PM Alex Elder <elder@linaro.org> wrote: > > Add the binding definitions for the "qcom,ipa" and "qcom,rmnet-ipa" > device tree nodes. > > Signed-off-by: Alex Elder <elder@linaro.org> > --- > .../devicetree/bindings/soc/qcom/qcom,ipa.txt | 136 ++++++++++++++++++ > .../bindings/soc/qcom/qcom,rmnet-ipa.txt | 15 ++ > 2 files changed, 151 insertions(+) > create mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,ipa.txt > create mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,rmnet-ipa.txt > > diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,ipa.txt b/Documentation/devicetree/bindings/soc/qcom/qcom,ipa.txt > new file mode 100644 > index 000000000000..d4d3d37df029 > --- /dev/null > +++ b/Documentation/devicetree/bindings/soc/qcom/qcom,ipa.txt > @@ -0,0 +1,136 @@ > +Qualcomm IPA (IP Accelerator) Driver Bindings are for h/w not drivers. > + > +This binding describes the Qualcomm IPA. The IPA is capable of offloading > +certain network processing tasks (e.g. filtering, routing, and NAT) from > +the main processor. The IPA currently serves only as a network interface, > +providing access to an LTE network available via a modem. > + > +The IPA sits between multiple independent "execution environments," > +including the AP subsystem (APSS) and the modem. The IPA presents > +a Generic Software Interface (GSI) to each execution environment. > +The GSI is an integral part of the IPA, but it is logically isolated > +and has a distinct interrupt and a separately-defined address space. > + > + ---------- ------------- --------- > + | | |G| |G| | | > + | APSS |===|S| IPA |S|===| Modem | > + | | |I| |I| | | > + ---------- ------------- --------- > + > +See also: > + bindings/interrupt-controller/interrupts.txt > + bindings/interconnect/interconnect.txt > + bindings/soc/qcom/qcom,smp2p.txt > + bindings/reserved-memory/reserved-memory.txt > + bindings/clock/clock-bindings.txt > + > +All properties defined below are required. > + > +- compatible: > + Must be one of the following compatible strings: > + "qcom,ipa-sdm845-modem_init" > + "qcom,ipa-sdm845-tz_init" Normal order is <vendor>,<soc>-<ipblock>. Don't use '_'. What's the difference between these 2? It can't be detected somehow? This might be better expressed as a property. Then if Trustzone initializes things, it can just add a property. > + > +-reg: > + Resources specyfing the physical address spaces of the IPA and GSI. typo > + > +-reg-names: > + The names of the address space ranges defined by the "reg" property. > + Must be "ipa" and "gsi". > + > +- interrupts-extended: Use 'interrupts' here and describe what they are and the order. What they are connected to (and the need for interrupts-extended) is outside the scope of this binding. > + Specifies the IRQs used by the IPA. Four cells are required, > + specifying: the IPA IRQ; the GSI IRQ; the clock query interrupt > + from the modem; and the "ready for stage 2 initialization" > + interrupt from the modem. The first two are hardware IRQs; the > + third and fourth are SMP2P input interrupts. > + > +- interrupt-names: > + The names of the interrupts defined by the "interrupts-extended" > + property. Must be "ipa", "gsi", "ipa-clock-query", and > + "ipa-post-init". Format as one per line. > + > +- clocks: > + Resource that defines the IPA core clock. > + > +- clock-names: > + The name used for the IPA core clock. Must be "core". > + > +- interconnects: > + Specifies the interconnects used by the IPA. Three cells are > + required, specifying: the path from the IPA to memory; from > + IPA to internal (SoC resident) memory; and between the AP > + subsystem and IPA for register access. > + > +- interconnect-names: > + The names of the interconnects defined by the "interconnects" > + property. Must be "memory", "imem", and "config". > + > +- qcom,smem-states > + The state bits used for SMP2P output. Two cells must be specified. > + The first indicates whether the value in the second bit is valid > + (1 means valid). The second, if valid, defines whether the IPA > + clock is enabled (1 means enabled). > + > +- qcom,smem-state-names > + The names of the state bits used for SMP2P output. These must be > + "ipa-clock-enabled-valid" and "ipa-clock-enabled". > + > +- memory-region > + A phandle for a reserved memory area that holds the firmware passed > + to Trust Zone for authentication. (Note, this is required > + only for "qcom,ipa-sdm845-tz_init".) > + > += EXAMPLE > + > +The following example represents the IPA present in the SDM845 SoC. It > +shows portions of the "modem-smp2p" node to indicate its relationship > +with the interrupts and SMEM states used by the IPA. > + > + modem-smp2p { > + compatible = "qcom,smp2p"; > + . . . > + ipa_smp2p_out: ipa-ap-to-modem { > + qcom,entry-name = "ipa"; > + #qcom,smem-state-cells = <1>; > + }; > + > + ipa_smp2p_in: ipa-modem-to-ap { > + qcom,entry-name = "ipa"; > + interrupt-controller; > + #interrupt-cells = <2>; > + }; > + }; > + > + ipa@1e00000 { ipa@1e40000 > + compatible = "qcom,ipa-sdm845-modem_init"; > + > + reg = <0x1e40000 0x34000>, > + <0x1e04000 0x2c000>; > + reg-names = "ipa", > + "gsi"; > + > + interrupts-extended = <&intc 0 311 IRQ_TYPE_LEVEL_HIGH>, > + <&intc 0 432 IRQ_TYPE_LEVEL_HIGH>, > + <&ipa_smp2p_in 0 IRQ_TYPE_EDGE_RISING>, > + <&ipa_smp2p_in 1 IRQ_TYPE_EDGE_RISING>; > + interrupt-names = "ipa", > + "gsi", > + "ipa-clock-query", > + "ipa-post-init"; > + > + clocks = <&rpmhcc RPMH_IPA_CLK>; > + clock-names = "core"; > + > + interconnects = <&qnoc MASTER_IPA &qnoc SLAVE_EBI1>, > + <&qnoc MASTER_IPA &qnoc SLAVE_IMEM>, > + <&qnoc MASTER_APPSS_PROC &qnoc SLAVE_IPA_CFG>; > + interconnect-names = "memory", > + "imem", > + "config"; > + > + qcom,smem-states = <&ipa_smp2p_out 0>, > + <&ipa_smp2p_out 1>; > + qcom,smem-state-names = "ipa-clock-enabled-valid", > + "ipa-clock-enabled"; > + }; > diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,rmnet-ipa.txt b/Documentation/devicetree/bindings/soc/qcom/qcom,rmnet-ipa.txt > new file mode 100644 > index 000000000000..3d0b2aabefc7 > --- /dev/null > +++ b/Documentation/devicetree/bindings/soc/qcom/qcom,rmnet-ipa.txt > @@ -0,0 +1,15 @@ > +Qualcomm IPA RMNet Driver > + > +This binding describes the IPA RMNet driver, which is used to > +represent virtual interfaces available on the modem accessed via > +the IPA. Other than the compatible string there are no properties > +associated with this device. Only a compatible string is a sure sign this is not a h/w device and you are just abusing DT to instantiate drivers. Make the IPA driver instantiate any sub drivers it needs. Rob
WARNING: multiple messages have this Message-ID (diff)
From: robh@kernel.org (Rob Herring) To: linux-arm-kernel@lists.infradead.org Subject: [RFC PATCH 01/12] dt-bindings: soc: qcom: add IPA bindings Date: Wed, 7 Nov 2018 08:59:08 -0600 [thread overview] Message-ID: <CABGGiswmpmSUmg9jEW7GnNtL2uXAN7jJOqFO5kG8adq71GuZpw@mail.gmail.com> (raw) In-Reply-To: <20181107003250.5832-2-elder@linaro.org> On Tue, Nov 6, 2018 at 6:33 PM Alex Elder <elder@linaro.org> wrote: > > Add the binding definitions for the "qcom,ipa" and "qcom,rmnet-ipa" > device tree nodes. > > Signed-off-by: Alex Elder <elder@linaro.org> > --- > .../devicetree/bindings/soc/qcom/qcom,ipa.txt | 136 ++++++++++++++++++ > .../bindings/soc/qcom/qcom,rmnet-ipa.txt | 15 ++ > 2 files changed, 151 insertions(+) > create mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,ipa.txt > create mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,rmnet-ipa.txt > > diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,ipa.txt b/Documentation/devicetree/bindings/soc/qcom/qcom,ipa.txt > new file mode 100644 > index 000000000000..d4d3d37df029 > --- /dev/null > +++ b/Documentation/devicetree/bindings/soc/qcom/qcom,ipa.txt > @@ -0,0 +1,136 @@ > +Qualcomm IPA (IP Accelerator) Driver Bindings are for h/w not drivers. > + > +This binding describes the Qualcomm IPA. The IPA is capable of offloading > +certain network processing tasks (e.g. filtering, routing, and NAT) from > +the main processor. The IPA currently serves only as a network interface, > +providing access to an LTE network available via a modem. > + > +The IPA sits between multiple independent "execution environments," > +including the AP subsystem (APSS) and the modem. The IPA presents > +a Generic Software Interface (GSI) to each execution environment. > +The GSI is an integral part of the IPA, but it is logically isolated > +and has a distinct interrupt and a separately-defined address space. > + > + ---------- ------------- --------- > + | | |G| |G| | | > + | APSS |===|S| IPA |S|===| Modem | > + | | |I| |I| | | > + ---------- ------------- --------- > + > +See also: > + bindings/interrupt-controller/interrupts.txt > + bindings/interconnect/interconnect.txt > + bindings/soc/qcom/qcom,smp2p.txt > + bindings/reserved-memory/reserved-memory.txt > + bindings/clock/clock-bindings.txt > + > +All properties defined below are required. > + > +- compatible: > + Must be one of the following compatible strings: > + "qcom,ipa-sdm845-modem_init" > + "qcom,ipa-sdm845-tz_init" Normal order is <vendor>,<soc>-<ipblock>. Don't use '_'. What's the difference between these 2? It can't be detected somehow? This might be better expressed as a property. Then if Trustzone initializes things, it can just add a property. > + > +-reg: > + Resources specyfing the physical address spaces of the IPA and GSI. typo > + > +-reg-names: > + The names of the address space ranges defined by the "reg" property. > + Must be "ipa" and "gsi". > + > +- interrupts-extended: Use 'interrupts' here and describe what they are and the order. What they are connected to (and the need for interrupts-extended) is outside the scope of this binding. > + Specifies the IRQs used by the IPA. Four cells are required, > + specifying: the IPA IRQ; the GSI IRQ; the clock query interrupt > + from the modem; and the "ready for stage 2 initialization" > + interrupt from the modem. The first two are hardware IRQs; the > + third and fourth are SMP2P input interrupts. > + > +- interrupt-names: > + The names of the interrupts defined by the "interrupts-extended" > + property. Must be "ipa", "gsi", "ipa-clock-query", and > + "ipa-post-init". Format as one per line. > + > +- clocks: > + Resource that defines the IPA core clock. > + > +- clock-names: > + The name used for the IPA core clock. Must be "core". > + > +- interconnects: > + Specifies the interconnects used by the IPA. Three cells are > + required, specifying: the path from the IPA to memory; from > + IPA to internal (SoC resident) memory; and between the AP > + subsystem and IPA for register access. > + > +- interconnect-names: > + The names of the interconnects defined by the "interconnects" > + property. Must be "memory", "imem", and "config". > + > +- qcom,smem-states > + The state bits used for SMP2P output. Two cells must be specified. > + The first indicates whether the value in the second bit is valid > + (1 means valid). The second, if valid, defines whether the IPA > + clock is enabled (1 means enabled). > + > +- qcom,smem-state-names > + The names of the state bits used for SMP2P output. These must be > + "ipa-clock-enabled-valid" and "ipa-clock-enabled". > + > +- memory-region > + A phandle for a reserved memory area that holds the firmware passed > + to Trust Zone for authentication. (Note, this is required > + only for "qcom,ipa-sdm845-tz_init".) > + > += EXAMPLE > + > +The following example represents the IPA present in the SDM845 SoC. It > +shows portions of the "modem-smp2p" node to indicate its relationship > +with the interrupts and SMEM states used by the IPA. > + > + modem-smp2p { > + compatible = "qcom,smp2p"; > + . . . > + ipa_smp2p_out: ipa-ap-to-modem { > + qcom,entry-name = "ipa"; > + #qcom,smem-state-cells = <1>; > + }; > + > + ipa_smp2p_in: ipa-modem-to-ap { > + qcom,entry-name = "ipa"; > + interrupt-controller; > + #interrupt-cells = <2>; > + }; > + }; > + > + ipa at 1e00000 { ipa at 1e40000 > + compatible = "qcom,ipa-sdm845-modem_init"; > + > + reg = <0x1e40000 0x34000>, > + <0x1e04000 0x2c000>; > + reg-names = "ipa", > + "gsi"; > + > + interrupts-extended = <&intc 0 311 IRQ_TYPE_LEVEL_HIGH>, > + <&intc 0 432 IRQ_TYPE_LEVEL_HIGH>, > + <&ipa_smp2p_in 0 IRQ_TYPE_EDGE_RISING>, > + <&ipa_smp2p_in 1 IRQ_TYPE_EDGE_RISING>; > + interrupt-names = "ipa", > + "gsi", > + "ipa-clock-query", > + "ipa-post-init"; > + > + clocks = <&rpmhcc RPMH_IPA_CLK>; > + clock-names = "core"; > + > + interconnects = <&qnoc MASTER_IPA &qnoc SLAVE_EBI1>, > + <&qnoc MASTER_IPA &qnoc SLAVE_IMEM>, > + <&qnoc MASTER_APPSS_PROC &qnoc SLAVE_IPA_CFG>; > + interconnect-names = "memory", > + "imem", > + "config"; > + > + qcom,smem-states = <&ipa_smp2p_out 0>, > + <&ipa_smp2p_out 1>; > + qcom,smem-state-names = "ipa-clock-enabled-valid", > + "ipa-clock-enabled"; > + }; > diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,rmnet-ipa.txt b/Documentation/devicetree/bindings/soc/qcom/qcom,rmnet-ipa.txt > new file mode 100644 > index 000000000000..3d0b2aabefc7 > --- /dev/null > +++ b/Documentation/devicetree/bindings/soc/qcom/qcom,rmnet-ipa.txt > @@ -0,0 +1,15 @@ > +Qualcomm IPA RMNet Driver > + > +This binding describes the IPA RMNet driver, which is used to > +represent virtual interfaces available on the modem accessed via > +the IPA. Other than the compatible string there are no properties > +associated with this device. Only a compatible string is a sure sign this is not a h/w device and you are just abusing DT to instantiate drivers. Make the IPA driver instantiate any sub drivers it needs. Rob
next prev parent reply other threads:[~2018-11-07 14:59 UTC|newest] Thread overview: 76+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-11-07 0:32 [RFC PATCH 00/12] net: introduce Qualcomm IPA driver Alex Elder 2018-11-07 0:32 ` Alex Elder 2018-11-07 0:32 ` [RFC PATCH 01/12] dt-bindings: soc: qcom: add IPA bindings Alex Elder 2018-11-07 0:32 ` Alex Elder 2018-11-07 11:50 ` Arnd Bergmann 2018-11-07 11:50 ` Arnd Bergmann 2018-11-09 22:38 ` Alex Elder 2018-11-09 22:38 ` Alex Elder 2018-11-07 14:59 ` Rob Herring [this message] 2018-11-07 14:59 ` Rob Herring 2018-11-09 22:38 ` Alex Elder 2018-11-09 22:38 ` Alex Elder 2018-11-11 1:40 ` Rob Herring 2018-11-11 1:40 ` Rob Herring 2018-11-11 1:40 ` Rob Herring 2018-11-13 16:28 ` Alex Elder 2018-11-13 16:28 ` Alex Elder 2018-11-07 0:32 ` [RFC PATCH 02/12] soc: qcom: ipa: DMA helpers Alex Elder 2018-11-07 0:32 ` Alex Elder 2018-11-07 12:17 ` Arnd Bergmann 2018-11-07 12:17 ` Arnd Bergmann 2018-11-13 16:33 ` Alex Elder 2018-11-13 16:33 ` Alex Elder 2018-11-07 0:32 ` [RFC PATCH 03/12] soc: qcom: ipa: generic software interface Alex Elder 2018-11-07 0:32 ` Alex Elder 2018-11-07 0:32 ` [RFC PATCH 04/12] soc: qcom: ipa: immediate commands Alex Elder 2018-11-07 0:32 ` Alex Elder 2018-11-07 14:36 ` Arnd Bergmann 2018-11-07 14:36 ` Arnd Bergmann 2018-11-13 16:58 ` Alex Elder 2018-11-13 16:58 ` Alex Elder 2018-11-07 0:32 ` [RFC PATCH 05/12] soc: qcom: ipa: IPA interrupts and the microcontroller Alex Elder 2018-11-07 0:32 ` Alex Elder 2018-11-07 0:32 ` [RFC PATCH 06/12] soc: qcom: ipa: QMI modem communication Alex Elder 2018-11-07 0:32 ` Alex Elder 2018-11-07 0:32 ` [RFC PATCH 07/12] soc: qcom: ipa: IPA register abstraction Alex Elder 2018-11-07 0:32 ` Alex Elder 2018-11-07 15:00 ` Arnd Bergmann 2018-11-07 15:00 ` Arnd Bergmann 2018-11-15 2:48 ` Alex Elder 2018-11-15 2:48 ` Alex Elder 2018-11-15 14:42 ` Arnd Bergmann 2018-11-15 14:42 ` Arnd Bergmann 2018-11-07 0:32 ` [RFC PATCH 08/12] soc: qcom: ipa: utility functions Alex Elder 2018-11-07 0:32 ` Alex Elder 2018-11-07 0:32 ` [RFC PATCH 09/12] soc: qcom: ipa: main IPA source file Alex Elder 2018-11-07 0:32 ` Alex Elder 2018-11-07 14:08 ` Arnd Bergmann 2018-11-07 14:08 ` Arnd Bergmann 2018-11-15 3:11 ` Alex Elder 2018-11-15 3:11 ` Alex Elder 2018-11-07 0:32 ` [RFC PATCH 10/12] soc: qcom: ipa: data path Alex Elder 2018-11-07 0:32 ` Alex Elder 2018-11-07 14:55 ` Arnd Bergmann 2018-11-07 14:55 ` Arnd Bergmann 2018-11-15 3:31 ` Alex Elder 2018-11-15 3:31 ` Alex Elder 2018-11-15 14:48 ` Arnd Bergmann 2018-11-15 14:48 ` Arnd Bergmann 2018-11-07 0:32 ` [RFC PATCH 11/12] soc: qcom: ipa: IPA rmnet interface Alex Elder 2018-11-07 0:32 ` Alex Elder 2018-11-07 13:30 ` Arnd Bergmann 2018-11-07 13:30 ` Arnd Bergmann 2018-11-07 15:26 ` Dan Williams 2018-11-07 15:26 ` Dan Williams 2018-11-07 0:32 ` [RFC PATCH 12/12] soc: qcom: ipa: build and "ipa_i.h" Alex Elder 2018-11-07 0:32 ` Alex Elder 2018-11-07 0:40 ` Randy Dunlap 2018-11-07 0:40 ` Randy Dunlap 2018-11-07 0:40 ` Randy Dunlap 2018-11-08 16:22 ` Alex Elder 2018-11-08 16:22 ` Alex Elder 2018-11-07 12:34 ` Arnd Bergmann 2018-11-07 12:34 ` Arnd Bergmann 2018-11-07 15:46 ` [RFC PATCH 00/12] net: introduce Qualcomm IPA driver Arnd Bergmann 2018-11-07 15:46 ` Arnd Bergmann
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CABGGiswmpmSUmg9jEW7GnNtL2uXAN7jJOqFO5kG8adq71GuZpw@mail.gmail.com \ --to=robh@kernel.org \ --cc=arnd@arndb.de \ --cc=bjorn.andersson@linaro.org \ --cc=davem@davemloft.net \ --cc=devicetree@vger.kernel.org \ --cc=elder@linaro.org \ --cc=ilias.apalodimas@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-arm-msm@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-soc@vger.kernel.org \ --cc=mark.rutland@arm.com \ --cc=mjavid@codeaurora.org \ --cc=netdev@vger.kernel.org \ --cc=robh+dt@kernel.org \ --cc=syadagir@codeaurora.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.