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

[Mathieu Poirier <mathieu.poirier@linaro.org>]

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
>