[PATCH v4 6/6] configs: am6*_evm_a53_defconfig: Enable remoteproc support

[Judith Mendez]

Hi Andrew,

On 2/6/25 5:07 PM, Andrew Davis wrote:
> On 2/5/25 7:54 PM, Judith Mendez wrote:
>> From: Hari Nagalla <hnagalla at ti.com>
>>
>> Enable configs needed to enable remoteproc drivers for AM62
>> and AM64 devices.
>>
>> While we are here, cleanup some remoteproc configs, there
>> is no DSP subsystem for am62px device.
>>
>> Signed-off-by: Hari Nagalla <hnagalla at ti.com>
>> Signed-off-by: Judith Mendez <jm at ti.com>
>> ---
>> Changes since v3:
>> - Add remoteproc support configs for am62x
>> - Cleanup remoteproc configs for am62p and am64x.
>> ---
>>   configs/am62px_evm_a53_defconfig | 1 -
>>   configs/am62x_evm_a53_defconfig  | 2 ++
>>   configs/am64x_evm_a53_defconfig  | 4 +++-
>>   3 files changed, 5 insertions(+), 2 deletions(-)
>>
>> diff --git a/configs/am62px_evm_a53_defconfig 
>> b/configs/am62px_evm_a53_defconfig
>> index 30b4a24ee9a..10e00519bc1 100644
>> --- a/configs/am62px_evm_a53_defconfig
>> +++ b/configs/am62px_evm_a53_defconfig
>> @@ -147,7 +147,6 @@ CONFIG_DM_REGULATOR_GPIO=y
>>   CONFIG_SPL_DM_REGULATOR_GPIO=y
>>   CONFIG_K3_SYSTEM_CONTROLLER=y
>>   CONFIG_REMOTEPROC_TI_K3_ARM64=y
>> -CONFIG_REMOTEPROC_TI_K3_DSP=y
>>   CONFIG_REMOTEPROC_TI_K3_R5F=y
>>   CONFIG_RESET_TI_SCI=y
>>   CONFIG_DM_RTC=y
>> diff --git a/configs/am62x_evm_a53_defconfig 
>> b/configs/am62x_evm_a53_defconfig
>> index c77e09db156..51b8c0ae3fb 100644
>> --- a/configs/am62x_evm_a53_defconfig
>> +++ b/configs/am62x_evm_a53_defconfig
>> @@ -56,6 +56,7 @@ CONFIG_SPL_YMODEM_SUPPORT=y
>>   CONFIG_CMD_BOOTEFI_SELFTEST=y
>>   CONFIG_CMD_NVEDIT_EFI=y
>>   CONFIG_CMD_MMC=y
>> +CONFIG_CMD_REMOTEPROC=y
>>   CONFIG_CMD_EFIDEBUG=y
>>   CONFIG_MMC_SPEED_MODE_SET=y
>>   CONFIG_OF_CONTROL=y
>> @@ -121,6 +122,7 @@ CONFIG_DM_REGULATOR_GPIO=y
>>   CONFIG_SPL_DM_REGULATOR_GPIO=y
>>   CONFIG_K3_SYSTEM_CONTROLLER=y
>>   CONFIG_REMOTEPROC_TI_K3_ARM64=y
>> +CONFIG_REMOTEPROC_TI_K3_M4F=y
>>   CONFIG_RESET_TI_SCI=y
>>   CONFIG_DM_RTC=y
>>   CONFIG_RTC_EMULATION=y
>> diff --git a/configs/am64x_evm_a53_defconfig 
>> b/configs/am64x_evm_a53_defconfig
>> index a501f3f567e..5528f54549d 100644
>> --- a/configs/am64x_evm_a53_defconfig
>> +++ b/configs/am64x_evm_a53_defconfig
>> @@ -70,6 +70,7 @@ CONFIG_CMD_DFU=y
>>   CONFIG_CMD_GPT=y
>>   CONFIG_CMD_I2C=y
>>   CONFIG_CMD_MMC=y
>> +CONFIG_CMD_REMOTEPROC=y
>>   CONFIG_CMD_USB=y
>>   CONFIG_CMD_EFIDEBUG=y
>>   CONFIG_CMD_TIME=y
>> @@ -149,8 +150,9 @@ CONFIG_DM_REGULATOR=y
>>   CONFIG_DM_REGULATOR_TPS65219=y
>>   CONFIG_K3_SYSTEM_CONTROLLER=y
>>   CONFIG_REMOTEPROC_TI_K3_ARM64=y
>> +CONFIG_REMOTEPROC_TI_K3_M4F=y
>> +CONFIG_REMOTEPROC_TI_K3_R5F=y
> 
> Would it work better to add these to the kconfig for AM64 so
> other boards get this without needing to add to each defconfig?

I can do that.

It would also be nice to do the same for other Sitara devices
so will do that as well.

Will respin the series with this change.

Thanks Andrew,

Judith

> 
> Otherwise whole series looks fine to me,
> 
> Reviewed-by: Andrew Davis <afd at ti.com>
> 
>>   CONFIG_REMOTEPROC_TI_PRU=y
>> -CONFIG_CMD_REMOTEPROC=y
>>   CONFIG_RESET_TI_SCI=y
>>   CONFIG_DM_RTC=y
>>   CONFIG_RTC_EMULATION=y