[PATCH v4 06/10] arm: mach-k3: am62x: am625_init: Probe AM65 CPSW NUSS

[Chintan Vankar]

On 13/08/24 23:57, Andrew Davis wrote:
> On 8/12/24 6:48 AM, Chintan Vankar wrote:
>> From: Kishon Vijay Abraham I <kishon at ti.com>
>>
>> In order to support Ethernet boot on AM62x, probe AM65 CPSW NUSS
>> driver in board_init_f().
>>
>> Signed-off-by: Kishon Vijay Abraham I <kishon at ti.com>
>> Signed-off-by: Siddharth Vadapalli <s-vadapalli at ti.com>
>> Signed-off-by: Chintan Vankar <c-vankar at ti.com>
>> ---
>>
>> Link to v4:
>> https://lore.kernel.org/r/20240705045030.1141934-7-c-vankar@ti.com/
>>
>> Changes from v3 to v4:
>> - No changes.
>>
>>   arch/arm/mach-k3/am62x/am625_init.c | 10 ++++++++++
>>   1 file changed, 10 insertions(+)
>>
>> diff --git a/arch/arm/mach-k3/am62x/am625_init.c 
>> b/arch/arm/mach-k3/am62x/am625_init.c
>> index 72a752d38e..0182c153a6 100644
>> --- a/arch/arm/mach-k3/am62x/am625_init.c
>> +++ b/arch/arm/mach-k3/am62x/am625_init.c
>> @@ -280,6 +280,16 @@ void board_init_f(ulong dummy)
>>           if (ret)
>>               panic("DRAM init failed: %d\n", ret);
>>       }
>> +
>> +    if (IS_ENABLED(CONFIG_SPL_ETH) && 
>> IS_ENABLED(CONFIG_TI_AM65_CPSW_NUSS) &&
>> +        spl_boot_device() == BOOT_DEVICE_ETHERNET) {
>> +        struct udevice *cpswdev;
>> +
>> +        if (uclass_get_device_by_driver(UCLASS_MISC, 
>> DM_DRIVER_GET(am65_cpsw_nuss),
>> +                        &cpswdev))
>> +            printf("Failed to probe am65_cpsw_nuss driver\n");
>> +    }
>> +
>>       spl_enable_cache();
> 
> Any reason you added this before enabling caches?
> 
> Andrew
> 

There is no specific reason for it, I can probe CPSW driver after
enabling cache also.

>>       fixup_a53_cpu_freq_by_speed_grade();