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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Wed, 29 Mar 2023 17:21:36 -0600
From:   Mathieu Poirier <mathieu.poirier@...aro.org>
To:     Devarsh Thakkar <devarsht@...com>
Cc:     andersson@...nel.org, devicetree@...r.kernel.org,
        p.zabel@...gutronix.de, linux-remoteproc@...r.kernel.org,
        robh+dt@...nel.org, linux-kernel@...r.kernel.org,
        krzysztof.kozlowski+dt@...aro.org, s-anna@...com, hnagalla@...com,
        praneeth@...com, nm@...com, vigneshr@...com, a-bhatia1@...com,
        j-luthra@...com, rogerq@...nel.org
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
>