mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: firmware: qcom,scm: document Eliza SCM Firmware Interface
@ 2026-02-23  8:42 Abel Vesa
  2026-02-23 10:41 ` Krzysztof Kozlowski
  0 siblings, 1 reply; 4+ messages in thread
From: Abel Vesa @ 2026-02-23  8:42 UTC (permalink / raw)
  To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Robert Marko, Das Srinagesh
  Cc: linux-arm-msm, devicetree, linux-kernel, Abel Vesa

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>


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] dt-bindings: firmware: qcom,scm: document Eliza SCM Firmware Interface
  2026-02-23  8:42 [PATCH] dt-bindings: firmware: qcom,scm: document Eliza SCM Firmware Interface Abel Vesa
@ 2026-02-23 10:41 ` Krzysztof Kozlowski
  2026-03-10 10:23   ` Abel Vesa
  0 siblings, 1 reply; 4+ messages in thread
From: Krzysztof Kozlowski @ 2026-02-23 10:41 UTC (permalink / raw)
  To: Abel Vesa
  Cc: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Robert Marko, Das Srinagesh, linux-arm-msm,
	devicetree, linux-kernel

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


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] dt-bindings: firmware: qcom,scm: document Eliza SCM Firmware Interface
  2026-02-23 10:41 ` Krzysztof Kozlowski
@ 2026-03-10 10:23   ` Abel Vesa
  2026-03-16 10:17     ` Konrad Dybcio
  0 siblings, 1 reply; 4+ messages in thread
From: Abel Vesa @ 2026-03-10 10:23 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Robert Marko, Das Srinagesh, linux-arm-msm,
	devicetree, linux-kernel

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!

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] dt-bindings: firmware: qcom,scm: document Eliza SCM Firmware Interface
  2026-03-10 10:23   ` Abel Vesa
@ 2026-03-16 10:17     ` Konrad Dybcio
  0 siblings, 0 replies; 4+ messages in thread
From: Konrad Dybcio @ 2026-03-16 10:17 UTC (permalink / raw)
  To: Abel Vesa, Krzysztof Kozlowski
  Cc: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Robert Marko, Das Srinagesh, linux-arm-msm,
	devicetree, linux-kernel

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2026-03-16 10:17 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-02-23  8:42 [PATCH] dt-bindings: firmware: qcom,scm: document Eliza SCM Firmware Interface Abel Vesa
2026-02-23 10:41 ` Krzysztof Kozlowski
2026-03-10 10:23   ` Abel Vesa
2026-03-16 10:17     ` Konrad Dybcio

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®