| Message ID | 20221114-narmstrong-sm8550-upstream-mpss_dsm-v2-1-f7c65d6f0e55@linaro.org |
|---|---|
| State | Changes Requested, archived |
| Headers | show |
| Series | soc: qcom: Add support for Qualcomm Modem Processing SubSystem DSM memory | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/patch-applied | success | |
| robh/dtbs-check | warning | build log |
| robh/dt-meta-schema | success |
On 18/11/2022 09:53, Neil Armstrong wrote: > Document the Qualcomm Modem Processing SubSystem DSM shared memory. > > This memory zone is shared between the APPS and the MPSS subsystem, > and must be configured during the whole lifetime of the system. > > Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org> > --- > .../reserved-memory/qcom,mpss-dsm-mem.yaml | 37 ++++++++++++++++++++++ > 1 file changed, 37 insertions(+) > > diff --git a/Documentation/devicetree/bindings/reserved-memory/qcom,mpss-dsm-mem.yaml b/Documentation/devicetree/bindings/reserved-memory/qcom,mpss-dsm-mem.yaml > new file mode 100644 > index 000000000000..226d0dfc422c > --- /dev/null > +++ b/Documentation/devicetree/bindings/reserved-memory/qcom,mpss-dsm-mem.yaml > @@ -0,0 +1,37 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/reserved-memory/qcom,mpss-dsm-mem.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Qualcomm Modem Processing SubSystem DSM Memory > + Discussion in v1 is still going. Memory region is not a device. Best regards, Krzysztof
On 18/11/2022 11:46, Krzysztof Kozlowski wrote: > On 18/11/2022 09:53, Neil Armstrong wrote: >> Document the Qualcomm Modem Processing SubSystem DSM shared memory. >> >> This memory zone is shared between the APPS and the MPSS subsystem, >> and must be configured during the whole lifetime of the system. >> >> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org> >> --- >> .../reserved-memory/qcom,mpss-dsm-mem.yaml | 37 ++++++++++++++++++++++ >> 1 file changed, 37 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/reserved-memory/qcom,mpss-dsm-mem.yaml b/Documentation/devicetree/bindings/reserved-memory/qcom,mpss-dsm-mem.yaml >> new file mode 100644 >> index 000000000000..226d0dfc422c >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/reserved-memory/qcom,mpss-dsm-mem.yaml >> @@ -0,0 +1,37 @@ >> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/reserved-memory/qcom,mpss-dsm-mem.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: Qualcomm Modem Processing SubSystem DSM Memory >> + > > Discussion in v1 is still going. Memory region is not a device. Nowhere is was affirmed this was a device. > > Best regards, > Krzysztof >
diff --git a/Documentation/devicetree/bindings/reserved-memory/qcom,mpss-dsm-mem.yaml b/Documentation/devicetree/bindings/reserved-memory/qcom,mpss-dsm-mem.yaml new file mode 100644 index 000000000000..226d0dfc422c --- /dev/null +++ b/Documentation/devicetree/bindings/reserved-memory/qcom,mpss-dsm-mem.yaml @@ -0,0 +1,37 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/reserved-memory/qcom,mpss-dsm-mem.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Qualcomm Modem Processing SubSystem DSM Memory + +description: | + Qualcomm Modem Processing SubSystem DSM, which serves the purpose of + describing the shared memory region used for MPSS remote processors. + +maintainers: + - Bjorn Andersson <andersson@kernel.org> + +allOf: + - $ref: reserved-memory.yaml + +properties: + compatible: + const: qcom,mpss-dsm-mem + +unevaluatedProperties: false + +examples: + - | + reserved-memory { + #address-cells = <1>; + #size-cells = <1>; + ranges; + + mpss-dsm@86700000 { + compatible = "qcom,mpss-dsm-mem"; + reg = <0x86700000 0xe0000>; + no-map; + }; + };
Document the Qualcomm Modem Processing SubSystem DSM shared memory. This memory zone is shared between the APPS and the MPSS subsystem, and must be configured during the whole lifetime of the system. Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org> --- .../reserved-memory/qcom,mpss-dsm-mem.yaml | 37 ++++++++++++++++++++++ 1 file changed, 37 insertions(+)