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

[Andrew Davis]

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?

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