[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <9d54383c-d653-cddf-7746-21f1001122ea@st.com>
Date: Thu, 9 Jul 2020 18:33:25 +0200
From: Arnaud POULIQUEN <arnaud.pouliquen@...com>
To: Mathieu Poirier <mathieu.poirier@...aro.org>,
"ohad@...ery.com" <ohad@...ery.com>,
"bjorn.andersson@...aro.org" <bjorn.andersson@...aro.org>,
Loic PALLARDY <loic.pallardy@...com>,
"mcoquelin.stm32@...il.com" <mcoquelin.stm32@...il.com>,
Alexandre TORGUE <alexandre.torgue@...com>
CC: "linux-remoteproc@...r.kernel.org" <linux-remoteproc@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-stm32@...md-mailman.stormreply.com"
<linux-stm32@...md-mailman.stormreply.com>
Subject: Re: [PATCH v5 00/11] remoteproc: stm32: Add support for attaching to
M4
On 7/7/20 11:31 PM, Mathieu Poirier wrote:
> This set applies on top of [1] and refactors the STM32 platform code in
> order to attach to the M4 remote processor when it has been started by the
> boot loader.
>
> New to V5:
> 1) Added Bjorn's reviewed-by to patch 06.
> 2) Removed Loic's reviewed-by from patch 08, it has changed too much.
>
> Patches that need to be reviewed: 7, 8, 9 and 10.
>
> Tested on ST's mp157c development board.
>
> Thanks,
> Mathieu
>
> [1]. https://patchwork.kernel.org/project/linux-remoteproc/list/?series=314523
>
> Mathieu Poirier (11):
> remoteproc: stm32: Decouple rproc from memory translation
> remoteproc: stm32: Request IRQ with platform device
> remoteproc: stm32: Decouple rproc from DT parsing
> remoteproc: stm32: Remove memory translation from DT parsing
> remoteproc: stm32: Parse syscon that will manage M4 synchronisation
> remoteproc: stm32: Properly set co-processor state when attaching
> remoteproc: Make function rproc_resource_cleanup() public
> remoteproc: stm32: Split function stm32_rproc_parse_fw()
> remoteproc: stm32: Properly handle the resource table when attaching
> remoteproc: stm32: Introduce new attach() operation
> remoteproc: stm32: Update M4 state in stm32_rproc_stop()
>
> drivers/remoteproc/remoteproc_core.c | 3 +-
> drivers/remoteproc/stm32_rproc.c | 214 ++++++++++++++++++++++++---
> include/linux/remoteproc.h | 1 +
> 3 files changed, 198 insertions(+), 20 deletions(-)
>
I tested the series with the [1]
Minor remarks on patch 6 & 8. After fixing them, for the series:
Acked-by: Arnaud Pouliquen <arnaud.pouliquen@...com>
Thank you very much for your work on this feature.
Arnaud
Powered by blists - more mailing lists