On 08/09/2023 14:05, Apurva Nandan wrote:
> From: Hari Nagalla <[email protected]>
>
> The K3 J784S4 SoCs have four dual-core R5F subsystems, one in MCU
> voltage domain and the other three in MAIN voltage domain. These R5F
> clusters are similar to the R5F clusters in J7200 and J721S2 SoCs.
>
> Extend support to the R5F clusters for J784S4 with J721S2 compatible
> string.
>
> Signed-off-by: Hari Nagalla <[email protected]>
> Signed-off-by: Apurva Nandan <[email protected]>
> ---
> drivers/remoteproc/ti_k3_r5f_rproc.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/remoteproc/ti_k3_r5f_rproc.c
> b/drivers/remoteproc/ti_k3_r5f_rproc.c
> index 6f3e12d915..631e548dcc 100644
> --- a/drivers/remoteproc/ti_k3_r5f_rproc.c
> +++ b/drivers/remoteproc/ti_k3_r5f_rproc.c
> @@ -855,7 +855,7 @@ static const struct k3_r5f_ip_data k3_data = {
> .tcm_ecc_autoinit = false,
> };
>
> -static const struct k3_r5f_ip_data j7200_data = {
> +static const struct k3_r5f_ip_data j7200_j721s2_data = {
No need to rename this.
> .tcm_is_double = true,
> .tcm_ecc_autoinit = true,
> };
> @@ -863,7 +863,8 @@ static const struct k3_r5f_ip_data j7200_data = {
> static const struct udevice_id k3_r5f_rproc_ids[] = {
> { .compatible = "ti,am654-r5f", .data = (ulong)&k3_data, },
> { .compatible = "ti,j721e-r5f", .data = (ulong)&k3_data, },
> - { .compatible = "ti,j7200-r5f", .data = (ulong)&j7200_data, },
> + { .compatible = "ti,j7200-r5f", .data = (ulong)&j7200_j721s2_data, },
> + { .compatible = "ti,j721s2-r5f", .data = (ulong)&j7200_j721s2_data, },
Where is DT binding documentation for this?
Don't you need compatible for ti,j728s4-r5f?
> {}
> };
>
> @@ -901,6 +902,7 @@ static const struct udevice_id k3_r5fss_ids[] = {
> { .compatible = "ti,am654-r5fss"},
> { .compatible = "ti,j721e-r5fss"},
> { .compatible = "ti,j7200-r5fss"},
> + { .compatible = "ti,j721s2-r5fss"},
> {}
> };
>
--
cheers,
-roger
