[PATCH] dt-bindings: firmware: qcom,scm: document Eliza SCM Firmware Interface

Abel Vesa posted 1 patch 1 month, 3 weeks ago
There is a newer version of this series
Documentation/devicetree/bindings/firmware/qcom,scm.yaml | 1 +
1 file changed, 1 insertion(+)
[PATCH] dt-bindings: firmware: qcom,scm: document Eliza SCM Firmware Interface
Posted by Abel Vesa 1 month, 3 weeks ago
Document the SCM Firmware Interface on the Eliza SoC.

Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
---
 Documentation/devicetree/bindings/firmware/qcom,scm.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/firmware/qcom,scm.yaml b/Documentation/devicetree/bindings/firmware/qcom,scm.yaml
index d66459f1d84e..5f6a6b165b82 100644
--- a/Documentation/devicetree/bindings/firmware/qcom,scm.yaml
+++ b/Documentation/devicetree/bindings/firmware/qcom,scm.yaml
@@ -23,6 +23,7 @@ properties:
       - enum:
           - qcom,scm-apq8064
           - qcom,scm-apq8084
+          - qcom,scm-eliza
           - qcom,scm-glymur
           - qcom,scm-ipq4019
           - qcom,scm-ipq5018

---
base-commit: 6de23f81a5e08be8fbf5e8d7e9febc72a5b5f27f
change-id: 20260221-eliza-bindings-scm-7f377bcaf743

Best regards,
--  
Abel Vesa <abel.vesa@oss.qualcomm.com>
Re: [PATCH] dt-bindings: firmware: qcom,scm: document Eliza SCM Firmware Interface
Posted by Krzysztof Kozlowski 1 month, 3 weeks ago
On Mon, Feb 23, 2026 at 10:42:02AM +0200, Abel Vesa wrote:
> Document the SCM Firmware Interface on the Eliza SoC.

So different than sm8750? Then you need to explain, because patch looks
odd (incomplete). Plus I am pretty sure it fails tests on your DTS. The
point of the bindings in DT schema is to actually use them...

Best regards,
Krzysztof
Re: [PATCH] dt-bindings: firmware: qcom,scm: document Eliza SCM Firmware Interface
Posted by Abel Vesa 1 month, 1 week ago
On 26-02-23 11:41:31, Krzysztof Kozlowski wrote:
> On Mon, Feb 23, 2026 at 10:42:02AM +0200, Abel Vesa wrote:
> > Document the SCM Firmware Interface on the Eliza SoC.
> 
> So different than sm8750? Then you need to explain, because patch looks
> odd (incomplete). Plus I am pretty sure it fails tests on your DTS. The
> point of the bindings in DT schema is to actually use them...

So as discussed off-list, schema doesn't fail with this patch because
interrupts property isn't marked as required. Not sure if it should or
not.

Anyway, I'll add the Eliza compatible to the if-clause for the interrupts
false, just like sm8750 is and that's it for now. And I think then the
commit message will be good enough as is, no explaining necessary.

Thanks for reviewing!
Re: [PATCH] dt-bindings: firmware: qcom,scm: document Eliza SCM Firmware Interface
Posted by Konrad Dybcio 1 month ago
On 3/10/26 11:23 AM, Abel Vesa wrote:
> On 26-02-23 11:41:31, Krzysztof Kozlowski wrote:
>> On Mon, Feb 23, 2026 at 10:42:02AM +0200, Abel Vesa wrote:
>>> Document the SCM Firmware Interface on the Eliza SoC.
>>
>> So different than sm8750? Then you need to explain, because patch looks
>> odd (incomplete). Plus I am pretty sure it fails tests on your DTS. The
>> point of the bindings in DT schema is to actually use them...
> 
> So as discussed off-list, schema doesn't fail with this patch because
> interrupts property isn't marked as required. Not sure if it should or
> not.

No, that's for a "value-add" feature when multiple OSes run at the
same time

See 6bf325992236 ("firmware: qcom: scm: Add wait-queue handling logic")

Konrad