| Message ID | 20231130-topic-ddr_sleep_stats-v1-1-5981c2e764b6@linaro.org |
|---|---|
| State | Accepted |
| Commit | fa78d0280fdc984d3dc1209b8c7c7a22ec9735de |
| Headers | show |
| Series | [1/3] dt-bindings: soc: qcom: stats: Add QMP handle | expand |
On 1.12.2023 10:44, Johan Hovold wrote: > On Thu, Nov 30, 2023 at 03:58:21PM +0100, Konrad Dybcio wrote: >> The stats can be expanded by poking the Always-On Subsystem through QMP. >> Allow passing a QMP handle for configurations that support it. > > I know Qualcomm likes their opaque acronyms, but how about mentioning > what QMP stands for here and in the binding below? I'll amend the commit message for v2. > Especially since we also have QMP PHYs (Qualcomm Multi PHY?), which I > doubt this is related to. There's only so many three-letter acronyms that begin with a Q ;) QMP stands for Qualcomm Messaging Protocol, which is a JSON-like way to communicate with the Always-On Subsystem (which is responsible for things such as managing low-power states and general SoC building blocks housekeeping) Konrad
diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom-stats.yaml b/Documentation/devicetree/bindings/soc/qcom/qcom-stats.yaml index 96a7f1822022..686a7ef2f48a 100644 --- a/Documentation/devicetree/bindings/soc/qcom/qcom-stats.yaml +++ b/Documentation/devicetree/bindings/soc/qcom/qcom-stats.yaml @@ -31,10 +31,24 @@ properties: reg: maxItems: 1 + qcom,qmp: + $ref: /schemas/types.yaml#/definitions/phandle + description: Reference to the AOSS side-channel message RAM + required: - compatible - reg +allOf: + - if: + not: + properties: + compatible: + const: qcom,rpmh-stats + then: + properties: + qcom,qmp: false + additionalProperties: false examples:
The stats can be expanded by poking the Always-On Subsystem through QMP. Allow passing a QMP handle for configurations that support it. Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org> --- Documentation/devicetree/bindings/soc/qcom/qcom-stats.yaml | 14 ++++++++++++++ 1 file changed, 14 insertions(+)