2023-03-27 15:31:21

by Devarsh Thakkar

Subject: [PATCH v8 0/3] Add single core R5F IPC for AM62 SoC family

AM62 SoC family don't have a multicore R5F cluster,
instead they have a single core R5F.
This enables IPC support with single core R5F for AM62
family of SoCs.

While at it, also simplify the cluster mode setting usage
by using soc_data specific checks only at probe.

Devarsh Thakkar (3):
remoteproc: k3-r5: Simplify cluster mode setting usage
dt-bindings: remoteproc: ti: Add new compatible for AM62 SoC family
remoteproc: k3-r5: Use separate compatible string for TI AM62x SoC
family

.../bindings/remoteproc/ti,k3-r5f-rproc.yaml | 76 ++++++++---
drivers/remoteproc/ti_k3_r5_remoteproc.c | 127 ++++++++++++------
2 files changed, 138 insertions(+), 65 deletions(-)
---
V2:
- dt-bindings: Avoid acronyms, use "Device Manager" instead of "DM"
V3:
- dt-bindings: Use separate if block for each compatible for
ti,cluster-mode property
- dt-bindings: Rearrange compatibles as per alphabatical order
V4:
- dt-bindings: Place each enum in separate line in allOf
V5:
- No change (fixing typo in email address)
V6:
- dt-bindings: Remove reviewed-by due to new modifications to use
cluster-mode=3
Introduce Simplify cluster-mode setting preamble patch per review
comments
- Use CLUSTER_MODE_SINGLECORE for AM62x
- Set PROC_BOOT_CFG_FLAG_R5_SINGLE_CORE for single core.
V7:
- Override to appropriate cluster-mode per firmware status flag
without checking soc_data
- Set appropriate mode as default if not provided in DT
- Check mode validity against SoC data during probe
- Rebase on top of 6.3 linux-next
V8:
- Avoid using soc_data while overriding cluster mode for IPC-only mode
--
2.34.1

2023-03-29 23:23:06

by Mathieu Poirier

Subject: Re: [PATCH v8 0/3] Add single core R5F IPC for AM62 SoC family

On Mon, Mar 27, 2023 at 08:58:29PM +0530, Devarsh Thakkar wrote:
> AM62 SoC family don't have a multicore R5F cluster,
> instead they have a single core R5F.
> This enables IPC support with single core R5F for AM62
> family of SoCs.
>
> While at it, also simplify the cluster mode setting usage
> by using soc_data specific checks only at probe.
>
> Devarsh Thakkar (3):
> remoteproc: k3-r5: Simplify cluster mode setting usage
> dt-bindings: remoteproc: ti: Add new compatible for AM62 SoC family
> remoteproc: k3-r5: Use separate compatible string for TI AM62x SoC
> family
>
> .../bindings/remoteproc/ti,k3-r5f-rproc.yaml | 76 ++++++++---
> drivers/remoteproc/ti_k3_r5_remoteproc.c | 127 ++++++++++++------
> 2 files changed, 138 insertions(+), 65 deletions(-)

I have applied this set.

Thanks,
Mathieu

> ---
> V2:
> - dt-bindings: Avoid acronyms, use "Device Manager" instead of "DM"
> V3:
> - dt-bindings: Use separate if block for each compatible for
> ti,cluster-mode property
> - dt-bindings: Rearrange compatibles as per alphabatical order
> V4:
> - dt-bindings: Place each enum in separate line in allOf
> V5:
> - No change (fixing typo in email address)
> V6:
> - dt-bindings: Remove reviewed-by due to new modifications to use
> cluster-mode=3
> Introduce Simplify cluster-mode setting preamble patch per review
> comments
> - Use CLUSTER_MODE_SINGLECORE for AM62x
> - Set PROC_BOOT_CFG_FLAG_R5_SINGLE_CORE for single core.
> V7:
> - Override to appropriate cluster-mode per firmware status flag
> without checking soc_data
> - Set appropriate mode as default if not provided in DT
> - Check mode validity against SoC data during probe
> - Rebase on top of 6.3 linux-next
> V8:
> - Avoid using soc_data while overriding cluster mode for IPC-only mode
> --
> 2.34.1
>