Remove unimplemented AGILEX_L3_MAIN_FREE_CLK in smmu node to prevent probe
failure when smmu is enabled.

Signed-off-by: Adrian Ng Ho Yin <adrian.ho.yin.ng@altera.com>
---
 arch/arm64/boot/dts/intel/socfpga_agilex.dtsi | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm64/boot/dts/intel/socfpga_agilex.dtsi b/arch/arm64/boot/dts/intel/socfpga_agilex.dtsi
index 0dfbafde8822..6bfee8263b3d 100644
--- a/arch/arm64/boot/dts/intel/socfpga_agilex.dtsi
+++ b/arch/arm64/boot/dts/intel/socfpga_agilex.dtsi
@@ -445,7 +445,6 @@ smmu: iommu@fa000000 {
 				<GIC_SPI 169 IRQ_TYPE_LEVEL_HIGH>;
 			stream-match-mask = <0x7ff0>;
 			clocks = <&clkmgr AGILEX_MPU_CCU_CLK>,
-				 <&clkmgr AGILEX_L3_MAIN_FREE_CLK>,
 				 <&clkmgr AGILEX_L4_MAIN_CLK>;
 			status = "disabled";
 		};
-- 
2.49.GIT

On 5/7/26 01:12, Adrian Ng Ho Yin wrote:
> Remove unimplemented AGILEX_L3_MAIN_FREE_CLK in smmu node to prevent probe
> failure when smmu is enabled.
> 
> Signed-off-by: Adrian Ng Ho Yin <adrian.ho.yin.ng@altera.com>
> ---
>   arch/arm64/boot/dts/intel/socfpga_agilex.dtsi | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/intel/socfpga_agilex.dtsi b/arch/arm64/boot/dts/intel/socfpga_agilex.dtsi
> index 0dfbafde8822..6bfee8263b3d 100644
> --- a/arch/arm64/boot/dts/intel/socfpga_agilex.dtsi
> +++ b/arch/arm64/boot/dts/intel/socfpga_agilex.dtsi
> @@ -445,7 +445,6 @@ smmu: iommu@fa000000 {
>   				<GIC_SPI 169 IRQ_TYPE_LEVEL_HIGH>;
>   			stream-match-mask = <0x7ff0>;
>   			clocks = <&clkmgr AGILEX_MPU_CCU_CLK>,
> -				 <&clkmgr AGILEX_L3_MAIN_FREE_CLK>,
>   				 <&clkmgr AGILEX_L4_MAIN_CLK>;
>   			status = "disabled";
>   		};

Hmm. not sure about this. When I added this, the documenation had the 
clock and it's defined in include/dt-bindings/clock/agilex-clock.h. But 
you're right, it's unimplemented in the clock driver, should the fix be 
to implement it in the clock driver?

Dinh

On 5/8/2026 8:43 PM, Dinh Nguyen wrote:
> 
> 
> On 5/7/26 01:12, Adrian Ng Ho Yin wrote:
>> Remove unimplemented AGILEX_L3_MAIN_FREE_CLK in smmu node to prevent 
>> probe
>> failure when smmu is enabled.
>>
>> Signed-off-by: Adrian Ng Ho Yin <adrian.ho.yin.ng@altera.com>
>> ---
>>   arch/arm64/boot/dts/intel/socfpga_agilex.dtsi | 1 -
>>   1 file changed, 1 deletion(-)
>>
>> diff --git a/arch/arm64/boot/dts/intel/socfpga_agilex.dtsi b/arch/ 
>> arm64/boot/dts/intel/socfpga_agilex.dtsi
>> index 0dfbafde8822..6bfee8263b3d 100644
>> --- a/arch/arm64/boot/dts/intel/socfpga_agilex.dtsi
>> +++ b/arch/arm64/boot/dts/intel/socfpga_agilex.dtsi
>> @@ -445,7 +445,6 @@ smmu: iommu@fa000000 {
>>                   <GIC_SPI 169 IRQ_TYPE_LEVEL_HIGH>;
>>               stream-match-mask = <0x7ff0>;
>>               clocks = <&clkmgr AGILEX_MPU_CCU_CLK>,
>> -                 <&clkmgr AGILEX_L3_MAIN_FREE_CLK>,
>>                    <&clkmgr AGILEX_L4_MAIN_CLK>;
>>               status = "disabled";
>>           };
> 
> Hmm. not sure about this. When I added this, the documenation had the 
> clock and it's defined in include/dt-bindings/clock/agilex-clock.h. But 
> you're right, it's unimplemented in the clock driver, should the fix be 
> to implement it in the clock driver?
> 
> Dinh 

Hi Dinh,

Since the clock is documented in the TRM, i'll implement the clock in 
the clock driver submit it in a separate patch. This patch will be dropped.

Thanks.
Adrian