linux-kernel - Re: [PATCH v3 1/2] dt-bindings: firmware: svc: Add IOMMU support for Agilex5

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <d64add31-5d4d-4dc3-aa09-5deb8e7b6cba@altera.com>
Date: Wed, 5 Nov 2025 01:40:06 +0000
From: "Romli, Khairul Anuar" <khairul.anuar.romli@...era.com>
To: Dinh Nguyen <dinguyen@...nel.org>, Rob Herring <robh@...nel.org>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>,
	"Rao, Mahesh" <mahesh.rao@...era.com>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>, "devicetree@...r.kernel.org"
	<devicetree@...r.kernel.org>, "Ng, Adrian Ho Yin"
	<adrian.ho.yin.ng@...era.com>, "Rabara, Niravkumar Laxmidas"
	<nirav.rabara@...era.com>
Subject: Re: [PATCH v3 1/2] dt-bindings: firmware: svc: Add IOMMU support for
 Agilex5

On 4/11/2025 7:39 am, Romli, Khairul Anuar wrote:
> In Agilex5, the TBU (Translation Buffer Unit) can now operate in non-secure
> mode, enabling Linux to utilize it through the IOMMU framework. This allows
> improved memory management capabilities in non-secure environments. With
> Agilex5 lifting this restriction, we are now extending the device tree
> bindings to support IOMMU for the Agilex5 SVC.
> 
> Signed-off-by: Khairul Anuar Romli <khairul.anuar.romli@...era.com>
> Reviewed-by: Adrian Ng Ho Yin <adrianhoyin.ng@...era.com>
> Reviewed-by: Niravkumar L Rabara <niravkumarlaxmidas.rabara@...era.com>
> ---
> Changes in v3:
> 	- Add iommu property.
> 	- Add logical check where only Agilex5 required iommus
> 	- Rewrite the commit message to explain why this changes is
> 	  needed.
> Changes in v2:
> 	- Reprase commit message to exclude iommu
> ---
>   .../firmware/intel,stratix10-svc.yaml         | 20 +++++++++++++++----
>   1 file changed, 16 insertions(+), 4 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/firmware/intel,stratix10-svc.yaml b/Documentation/devicetree/bindings/firmware/intel,stratix10-svc.yaml
> index fac1e955852e..094973494098 100644
> --- a/Documentation/devicetree/bindings/firmware/intel,stratix10-svc.yaml
> +++ b/Documentation/devicetree/bindings/firmware/intel,stratix10-svc.yaml
> @@ -34,6 +34,7 @@ properties:
>       enum:
>         - intel,stratix10-svc
>         - intel,agilex-svc
> +      - intel,agilex5-svc
>   
>     method:
>       description: |
> @@ -54,14 +55,25 @@ properties:
>         reserved memory region for the service layer driver to
>         communicate with the secure device manager.
>   
> +  iommus:
> +    maxItems: 1
> +
>     fpga-mgr:
>       $ref: /schemas/fpga/intel,stratix10-soc-fpga-mgr.yaml
>       description: Optional child node for fpga manager to perform fabric configuration.
>   
> -required:
> -  - compatible
> -  - method
> -  - memory-region
> +allOf:
> +  - required:
> +      - compatible
> +      - method
> +      - memory-region
> +  - if:
> +      properties:
> +        compatible:
> +          const: intel,agilex5-svc

I am sending the next revision to use contains: enum: instead of const.
> +    then:
> +      required:
> +        - iommus
>   
>   additionalProperties: false
>