dt-bindings: firmware: Add Agilex5 SVC compatible string

Enable support for the Agilex5 in Stratix10 SoC service layer (SVC) by adding the new compatible string "intel,agilex5-svc" in the device tree bindings.

This is needed to differentiate Agilex5 from previous SoC generations, as Agilex5 uses a different mechanism for mapping reserved memory regions, which rely on IOMMU support.

[You don't often get email from [email protected]. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]

Quoted 4 lines · Khairul Anuar Romli

Enable support for the Agilex5 in Stratix10 SoC service layer (SVC) by adding the new compatible string "intel,agilex5-svc" in the device tree bindings.

Quoted 3 lines

This is needed to differentiate Agilex5 from previous SoC generations, as Agilex5 uses a different mechanism for mapping reserved memory regions, which rely on IOMMU support.

There are no iommus here, so this feels inaccurate.

Best regards,
Krzysztof

Quoted 11 lines · Krzysztof Kozlowski

[You don't often get email from [email protected]. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]

Quoted 7 lines · Khairul Anuar Romli

Enable support for the Agilex5 in Stratix10 SoC service layer (SVC) by adding the new compatible string "intel,agilex5-svc" in the device tree bindings.

This is needed to differentiate Agilex5 from previous SoC generations, as Agilex5 uses a different mechanism for mapping reserved memory regions, which rely on IOMMU support.

Quoted 4 lines

There are no iommus here, so this feels inaccurate.

Best regards,
Krzysztof

I will remove this on the v2 revision as the iommu is part on future patch series.

Introduce the Stratix10 SoC service layer (SVC) node for Agilex5 SoCs. The node includes the compatible string "intel,agilex5-svc" and references a reserved memory region required for communication with the Secure Device Manager (SDM).

Agilex5 introduces a dependency on IOMMU-based translation for reserved memory, unlike prior Agilex platforms. This commit introduces the structural changes needed to support this feature once the IOMMU driver is upstreamed.

Signed-off-by Khairul Anuar Romli <[email protected]>

---
 arch/arm64/boot/dts/intel/socfpga_agilex5.dtsi | 9 +++++++++
 1 file changed, 9 insertions(+)

arch/arm64/boot/dts/intel/socfpga_agilex5.dtsi +0 -0

index a13ccee3c4c3..15284092897e 100644

Quoted 20 lines · Khairul Anuar Romli

Introduce the Stratix10 SoC service layer (SVC) node for Agilex5 SoCs. The node includes the compatible string "intel,agilex5-svc" and references a reserved memory region required for communication with the Secure Device Manager (SDM).

Agilex5 introduces a dependency on IOMMU-based translation for reserved memory, unlike prior Agilex platforms. This commit introduces the structural changes needed to support this feature once the IOMMU driver is upstreamed.

Signed-off-by Khairul Anuar Romli <[email protected]>

---
  arch/arm64/boot/dts/intel/socfpga_agilex5.dtsi | 9 +++++++++
  1 file changed, 9 insertions(+)

arch/arm64/boot/dts/intel/socfpga_agilex5.dtsi +0 -0

index a13ccee3c4c3..15284092897e 100644

arch/arm64/boot/dts/intel/socfpga_agilex5.dtsi queue7 { · line 850

Quoted 10 lines

arch/arm64/boot/dts/intel/socfpga_agilex5.dtsi +7 -0

@@ -841,5 +841,14 @@ queue7 {
  				};
  			};
  		};
+
+		firmware {
+			svc {
+				compatible = "intel,agilex5-svc";
+				method = "smc";
+				memory-region = <&service_reserved>;
+				iommus = <&smmu 10>;

You did not test your code.

Plus, where is the driver? Please read submitting patches in DT directory.

Best regards,
Krzysztof