* [PATCH 0/2] Add Agilex5 AXI DMA support
@ 2025-11-20 11:31 Khairul Anuar Romli
2025-11-20 11:31 ` [PATCH 1/2] dt-bindings: dma: snps,dw-axi-dmac: Add compatible string for Agilex5 Khairul Anuar Romli
2025-11-20 11:31 ` [PATCH 2/2] arm64: dts: agilex5: Use platform-specific compatible for AXI DMA Khairul Anuar Romli
0 siblings, 2 replies; 6+ messages in thread
From: Khairul Anuar Romli @ 2025-11-20 11:31 UTC (permalink / raw)
To: Dinh Nguyen, Vinod Koul, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, dmaengine, devicetree, linux-kernel,
Khairul Anuar Romli
This series introduces support for Agilex5 SoC in the Synopsys DesignWare
AXI DMA binding and updates the device tree to use the platform-specific
compatible string.
The Agilex5 only has 40-bit DMA addressable bit instead of 64-bit. Hence,
this specific addition will enable driver to handle this limitation.
---
Notes:
This patch series is applied on socfpga maintainer's tree
https://git.kernel.org/pub/scm/linux/kernel/git/dinguyen/linux.git/log/?h=socfpga_dts_for_v6.19
This changes is validated on:
- intel/socfpga_agilex5_socdk.dtb
- snps,dw-axi-dmac.yaml
- snps,dw-axi-dmac.yaml intel/socfpga_agilex5_socdk.dtb
---
Khairul Anuar Romli (2):
dt-bindings: dma: snps,dw-axi-dmac: Add compatible string for Agilex5
arm64: dts: agilex5: Use platform-specific compatible for AXI DMA
.../devicetree/bindings/dma/snps,dw-axi-dmac.yaml | 14 +++++++++-----
arch/arm64/boot/dts/intel/socfpga_agilex5.dtsi | 6 ++++--
2 files changed, 13 insertions(+), 7 deletions(-)
--
2.43.7
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/2] dt-bindings: dma: snps,dw-axi-dmac: Add compatible string for Agilex5
2025-11-20 11:31 [PATCH 0/2] Add Agilex5 AXI DMA support Khairul Anuar Romli
@ 2025-11-20 11:31 ` Khairul Anuar Romli
2025-11-20 17:36 ` Rob Herring
2025-11-20 11:31 ` [PATCH 2/2] arm64: dts: agilex5: Use platform-specific compatible for AXI DMA Khairul Anuar Romli
1 sibling, 1 reply; 6+ messages in thread
From: Khairul Anuar Romli @ 2025-11-20 11:31 UTC (permalink / raw)
To: Dinh Nguyen, Vinod Koul, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, dmaengine, devicetree, linux-kernel,
Khairul Anuar Romli
The address bus on Agilex5 is limited to 40 bits. When SMMU is enable this
will cause address truncation and translation faults. Hence introducing
"altr,agilex5-axi-dma" to enable platform specific configuration to
configure the dma addressable bit mask.
Add a fallback capability for the compatible property to allow driver to
probe and initialize with a newly added compatible string without requiring
additional entry in the driver.
Signed-off-by: Khairul Anuar Romli <khairul.anuar.romli@altera.com>
---
.../devicetree/bindings/dma/snps,dw-axi-dmac.yaml | 14 +++++++++-----
1 file changed, 9 insertions(+), 5 deletions(-)
diff --git a/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml b/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml
index a393a33c8908..db89ebf2b006 100644
--- a/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml
+++ b/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml
@@ -17,11 +17,15 @@ allOf:
properties:
compatible:
- enum:
- - snps,axi-dma-1.01a
- - intel,kmb-axi-dma
- - starfive,jh7110-axi-dma
- - starfive,jh8100-axi-dma
+ oneOf:
+ - enum:
+ - snps,axi-dma-1.01a
+ - intel,kmb-axi-dma
+ - starfive,jh7110-axi-dma
+ - starfive,jh8100-axi-dma
+ - items:
+ - const: altr,agilex5-axi-dma
+ - const: snps,axi-dma-1.01a
reg:
minItems: 1
--
2.43.7
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/2] arm64: dts: agilex5: Use platform-specific compatible for AXI DMA
2025-11-20 11:31 [PATCH 0/2] Add Agilex5 AXI DMA support Khairul Anuar Romli
2025-11-20 11:31 ` [PATCH 1/2] dt-bindings: dma: snps,dw-axi-dmac: Add compatible string for Agilex5 Khairul Anuar Romli
@ 2025-11-20 11:31 ` Khairul Anuar Romli
1 sibling, 0 replies; 6+ messages in thread
From: Khairul Anuar Romli @ 2025-11-20 11:31 UTC (permalink / raw)
To: Dinh Nguyen, Vinod Koul, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, dmaengine, devicetree, linux-kernel,
Khairul Anuar Romli
Update the compatible string for the DMA controller nodes in the Agilex5
device tree from the generic "snps,axi-dma-1.01a" to the platform-specific
"altr,agilex5-axi-dma". Add fallback capability to ensure driver is able
to initialize properly.
This change enables the use of platform-specific features and constraints
in the driver, such as setting a 40-bit DMA addressable mask, which is
required for Agilex5. It also aligns with the updated device tree bindings
and driver support for this compatible string.
Signed-off-by: Khairul Anuar Romli <khairul.anuar.romli@altera.com>
---
arch/arm64/boot/dts/intel/socfpga_agilex5.dtsi | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/intel/socfpga_agilex5.dtsi b/arch/arm64/boot/dts/intel/socfpga_agilex5.dtsi
index a5c2025a616e..ef33f27b1a4d 100644
--- a/arch/arm64/boot/dts/intel/socfpga_agilex5.dtsi
+++ b/arch/arm64/boot/dts/intel/socfpga_agilex5.dtsi
@@ -324,7 +324,8 @@ ocram: sram@0 {
};
dmac0: dma-controller@10db0000 {
- compatible = "snps,axi-dma-1.01a";
+ compatible = "altr,agilex5-axi-dma",
+ "snps,axi-dma-1.01a";
reg = <0x10db0000 0x500>;
clocks = <&clkmgr AGILEX5_L4_MAIN_CLK>,
<&clkmgr AGILEX5_L4_MP_CLK>;
@@ -342,7 +343,8 @@ dmac0: dma-controller@10db0000 {
};
dmac1: dma-controller@10dc0000 {
- compatible = "snps,axi-dma-1.01a";
+ compatible = "altr,agilex5-axi-dma",
+ "snps,axi-dma-1.01a";
reg = <0x10dc0000 0x500>;
clocks = <&clkmgr AGILEX5_L4_MAIN_CLK>,
<&clkmgr AGILEX5_L4_MP_CLK>;
--
2.43.7
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] dt-bindings: dma: snps,dw-axi-dmac: Add compatible string for Agilex5
2025-11-20 11:31 ` [PATCH 1/2] dt-bindings: dma: snps,dw-axi-dmac: Add compatible string for Agilex5 Khairul Anuar Romli
@ 2025-11-20 17:36 ` Rob Herring
2025-11-26 7:43 ` Romli, Khairul Anuar
0 siblings, 1 reply; 6+ messages in thread
From: Rob Herring @ 2025-11-20 17:36 UTC (permalink / raw)
To: Khairul Anuar Romli
Cc: Dinh Nguyen, Vinod Koul, Krzysztof Kozlowski, Conor Dooley,
dmaengine, devicetree, linux-kernel
On Thu, Nov 20, 2025 at 07:31:10PM +0800, Khairul Anuar Romli wrote:
> The address bus on Agilex5 is limited to 40 bits. When SMMU is enable this
> will cause address truncation and translation faults. Hence introducing
> "altr,agilex5-axi-dma" to enable platform specific configuration to
> configure the dma addressable bit mask.
That's likely a bus limitation, not an IP limitation. So that should be
handled with dma-ranges.
However, adding a specific compatible is perfectly fine.
>
> Add a fallback capability for the compatible property to allow driver to
> probe and initialize with a newly added compatible string without requiring
> additional entry in the driver.
>
> Signed-off-by: Khairul Anuar Romli <khairul.anuar.romli@altera.com>
> ---
> .../devicetree/bindings/dma/snps,dw-axi-dmac.yaml | 14 +++++++++-----
> 1 file changed, 9 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml b/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml
> index a393a33c8908..db89ebf2b006 100644
> --- a/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml
> +++ b/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml
> @@ -17,11 +17,15 @@ allOf:
>
> properties:
> compatible:
> - enum:
> - - snps,axi-dma-1.01a
> - - intel,kmb-axi-dma
> - - starfive,jh7110-axi-dma
> - - starfive,jh8100-axi-dma
> + oneOf:
> + - enum:
> + - snps,axi-dma-1.01a
> + - intel,kmb-axi-dma
> + - starfive,jh7110-axi-dma
> + - starfive,jh8100-axi-dma
> + - items:
> + - const: altr,agilex5-axi-dma
> + - const: snps,axi-dma-1.01a
>
> reg:
> minItems: 1
> --
> 2.43.7
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] dt-bindings: dma: snps,dw-axi-dmac: Add compatible string for Agilex5
2025-11-20 17:36 ` Rob Herring
@ 2025-11-26 7:43 ` Romli, Khairul Anuar
2025-11-26 9:41 ` Krzysztof Kozlowski
0 siblings, 1 reply; 6+ messages in thread
From: Romli, Khairul Anuar @ 2025-11-26 7:43 UTC (permalink / raw)
To: Rob Herring
Cc: Dinh Nguyen, Vinod Koul, Krzysztof Kozlowski, Conor Dooley,
dmaengine@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org
On 21/11/2025 1:36 am, Rob Herring wrote:
> On Thu, Nov 20, 2025 at 07:31:10PM +0800, Khairul Anuar Romli wrote:
>> The address bus on Agilex5 is limited to 40 bits. When SMMU is enable this
>> will cause address truncation and translation faults. Hence introducing
>> "altr,agilex5-axi-dma" to enable platform specific configuration to
>> configure the dma addressable bit mask.
>
> That's likely a bus limitation, not an IP limitation. So that should be
> handled with dma-ranges.
>
> However, adding a specific compatible is perfectly fine.
>
Would it be okay if I rephrase and send the next version with a
correction in the commit message body as your comment in this version?
Thanks.
Regards,
Khairul
>>
>> Add a fallback capability for the compatible property to allow driver to
>> probe and initialize with a newly added compatible string without requiring
>> additional entry in the driver.
>>
>> Signed-off-by: Khairul Anuar Romli <khairul.anuar.romli@altera.com>
>> ---
>> .../devicetree/bindings/dma/snps,dw-axi-dmac.yaml | 14 +++++++++-----
>> 1 file changed, 9 insertions(+), 5 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml b/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml
>> index a393a33c8908..db89ebf2b006 100644
>> --- a/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml
>> +++ b/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml
>> @@ -17,11 +17,15 @@ allOf:
>>
>> properties:
>> compatible:
>> - enum:
>> - - snps,axi-dma-1.01a
>> - - intel,kmb-axi-dma
>> - - starfive,jh7110-axi-dma
>> - - starfive,jh8100-axi-dma
>> + oneOf:
>> + - enum:
>> + - snps,axi-dma-1.01a
>> + - intel,kmb-axi-dma
>> + - starfive,jh7110-axi-dma
>> + - starfive,jh8100-axi-dma
>> + - items:
>> + - const: altr,agilex5-axi-dma
>> + - const: snps,axi-dma-1.01a
>>
>> reg:
>> minItems: 1
>> --
>> 2.43.7
>>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] dt-bindings: dma: snps,dw-axi-dmac: Add compatible string for Agilex5
2025-11-26 7:43 ` Romli, Khairul Anuar
@ 2025-11-26 9:41 ` Krzysztof Kozlowski
0 siblings, 0 replies; 6+ messages in thread
From: Krzysztof Kozlowski @ 2025-11-26 9:41 UTC (permalink / raw)
To: Romli, Khairul Anuar
Cc: Rob Herring, Dinh Nguyen, Vinod Koul, Krzysztof Kozlowski,
Conor Dooley, dmaengine@vger.kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
On Wed, Nov 26, 2025 at 07:43:42AM +0000, Romli, Khairul Anuar wrote:
> On 21/11/2025 1:36 am, Rob Herring wrote:
> > On Thu, Nov 20, 2025 at 07:31:10PM +0800, Khairul Anuar Romli wrote:
> >> The address bus on Agilex5 is limited to 40 bits. When SMMU is enable this
> >> will cause address truncation and translation faults. Hence introducing
> >> "altr,agilex5-axi-dma" to enable platform specific configuration to
> >> configure the dma addressable bit mask.
> >
> > That's likely a bus limitation, not an IP limitation. So that should be
> > handled with dma-ranges.
> >
> > However, adding a specific compatible is perfectly fine.
> >
> Would it be okay if I rephrase and send the next version with a
> correction in the commit message body as your comment in this version?
Yes and your DTS patch might need changes - include proper dma-ranges.
I also do not see how separate compatible without any driver change
solved your case. That's a no-op patch from Linux driver point of view.
See also submitting patches in DT dir about the process how patches are
grouped together.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2025-11-26 9:41 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-11-20 11:31 [PATCH 0/2] Add Agilex5 AXI DMA support Khairul Anuar Romli
2025-11-20 11:31 ` [PATCH 1/2] dt-bindings: dma: snps,dw-axi-dmac: Add compatible string for Agilex5 Khairul Anuar Romli
2025-11-20 17:36 ` Rob Herring
2025-11-26 7:43 ` Romli, Khairul Anuar
2025-11-26 9:41 ` Krzysztof Kozlowski
2025-11-20 11:31 ` [PATCH 2/2] arm64: dts: agilex5: Use platform-specific compatible for AXI DMA Khairul Anuar Romli
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox