Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Peng Fan <peng.fan@nxp.com>
To:     Mathieu Poirier <mathieu.poirier@linaro.org>,
        "bjorn.andersson@linaro.org" <bjorn.andersson@linaro.org>,
        "ohad@wizery.com" <ohad@wizery.com>
CC:     "loic.pallardy@st.com" <loic.pallardy@st.com>,
        "arnaud.pouliquen@st.com" <arnaud.pouliquen@st.com>,
        "s-anna@ti.com" <s-anna@ti.com>,
        "linux-remoteproc@vger.kernel.org" <linux-remoteproc@vger.kernel.org>,
        "corbet@lwn.net" <corbet@lwn.net>,
        "linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH v3 00/14] remoteproc: Add support for synchronisaton with
 rproc
Thread-Topic: [PATCH v3 00/14] remoteproc: Add support for synchronisaton with
 rproc
Thread-Index: AQHWGnMtQQcf4J8dJ0yMAJ0O0OhEl6it+yTg
Date:   Mon, 18 May 2020 13:28:11 +0000
Message-ID: <DB6PR0402MB2760B52EDFD1DB20AE0BDE3588B80@DB6PR0402MB2760.eurprd04.prod.outlook.com>
References: <20200424200135.28825-1-mathieu.poirier@linaro.org>
In-Reply-To: <20200424200135.28825-1-mathieu.poirier@linaro.org>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: dc4edfd4-b92b-4a74-3a1f-08d7fb2f4fa1
X-MS-Exchange-CrossTenant-originalarrivaltime: 18 May 2020 13:28:11.2886
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: sAHhwTawZecH8uw88LXWNC3Kd/OEWyATfQrlewU6wAEm/RVJJ92eQ9DI1lu7FGIZkg+3Rs352O7EMJLc7sa+bw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB6PR0402MB2919
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> Subject: [PATCH v3 00/14] remoteproc: Add support for synchronisaton with
> rproc

What's the status of this thread? Will this be applied or requires a new v4=
?

Thanks,
Peng.

>=20
> This is the third revision of this series that tries to address the probl=
em of
> synchronising with a remote processor with as much flexibility as possibl=
e.
>=20
> Two things to pay attention to:
>=20
> 1) Function rproc_actuate() has been abandoned to avoid creating another
>    way to start a remote processor from a kernel driver.  Arnaud expresse=
d
>    the opinion that it is semantically questionnable to synchronise with =
a
>    remote processor when calling rproc_boot().  We could rename
>    rproc_boot() to rproc_actuate() but I'll wait to see what other people
>    think before doing so.
>=20
> 2) The allocation of the synchronisation states has been split from the
>    remote processor allocation.  A new function rproc_set_state_machine()
>    does all the work now.  Proceeding this way has made the patchset a
>    lot more simple.
>=20
> Other than the above I have tried to address all the comments made on the
> second revision.  If a comment was not addressed it simply fell through t=
he
> cracks rather than ignored.  In such a case please reiterate your point
> of view and I'll be sure to address it.
>=20
> Applies cleanly on rproc-next (305ac5a766b1).
>=20
> Best regards,
> Mathieu
>=20
> Mathieu Poirier (14):
>   remoteproc: Make core operations optional
>   remoteproc: Introduce function rproc_alloc_internals()
>   remoteproc: Add new operation and flags for synchronistation
>   remoteproc: Refactor function rproc_boot()
>   remoteproc: Refactor function rproc_fw_boot()
>   remoteproc: Refactor function rproc_trigger_auto_boot()
>   remoteproc: Introducting new start and stop functions
>   remoteproc: Call core functions based on synchronisation flag
>   remoteproc: Deal with synchronisation when crashing
>   remoteproc: Deal with synchronisation when shutting down
>   remoteproc: Deal with synchronisation when changing FW image
>   remoteproc: Introducing function rproc_set_state_machine()
>   remoteproc: Document function rproc_set_state_machine()
>   remoteproc: Expose synchronisation flags via debugfs
>=20
>  Documentation/remoteproc.txt             |  17 ++
>  drivers/remoteproc/remoteproc_core.c     | 197
> +++++++++++++++++++----
>  drivers/remoteproc/remoteproc_debugfs.c  |  21 +++
> drivers/remoteproc/remoteproc_internal.h | 123 +++++++++++++-
>  drivers/remoteproc/remoteproc_sysfs.c    |  24 ++-
>  include/linux/remoteproc.h               |  27 ++++
>  6 files changed, 372 insertions(+), 37 deletions(-)
>=20
> --
> 2.20.1