Re: [PATCH v5 0/5] In-kernel QMI helpers and sysmon

From: Philippe Ombredanne <pombredanne@nexb.com>
To: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: Andy Gross <andy.gross@linaro.org>,
	Ohad Ben-Cohen <ohad@wizery.com>,
	Arun Kumar Neelakantam <aneela@codeaurora.org>,
	Chris Lew <clew@codeaurora.org>,
	LKML <linux-kernel@vger.kernel.org>,
	linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org,
	linux-remoteproc@vger.kernel.org
Subject: Re: [PATCH v5 0/5] In-kernel QMI helpers and sysmon
Date: Thu, 7 Dec 2017 11:20:02 +0100	[thread overview]
Message-ID: <CAOFm3uEG9kXLXMLSp-R94SncNxCOh1oFudKvnPVfxkbzy-3u2Q@mail.gmail.com> (raw)
In-Reply-To: <20171205174310.15799-1-bjorn.andersson@linaro.org>

On Tue, Dec 5, 2017 at 6:43 PM, Bjorn Andersson
<bjorn.andersson@linaro.org> wrote:
> This series introduces a helper library for drivers that needs to implement
> clients or services in the kernel for communicating with QMI encoded messages.
>
> This is used by a set of drivers in order to implement control signaling that
> needs to happen between a driver and a service on a remote processor, such as
> the synchronization of states during a remoteproc shutdown/restart; as seen in
> the sysmon implementation.
>
> Finally a sample driver provides an implementation of the "test" protocol,
> which is a service typically implemented by Qualcomm remoteproc firmware.
>
> Changes since v4:
> - Use SPDX headers
>
> Changes since v3:
> - Don't call QMI handler for response type messages without known txn
> - Kerneldoc updates
> - Style updates
>
> Changes since v2:
> - Fix reported typos
> - Checkpatch fixes
> - Use non-gpl EXPORT_SYMBOL
>
> Changes since v1:
> - Lot of modifications to QMI interface, from feedback and implementation and
>   testing of sysmon.
> - Added sysmon driver.
> - Added patch for remoteproc to pass gracefulness on subdev remove.
> - QRTR patches part of v1 was merged separately
>
> Bjorn Andersson (5):
>   soc: qcom: Introduce QMI encoder/decoder
>   soc: qcom: Introduce QMI helpers
>   remoteproc: Pass type of shutdown to subdev remove
>   remoteproc: qcom: Introduce sysmon
>   samples: Introduce Qualcomm QMI sample client
>
>  drivers/remoteproc/Kconfig           |  17 +
>  drivers/remoteproc/Makefile          |   1 +
>  drivers/remoteproc/qcom_adsp_pil.c   |  12 +
>  drivers/remoteproc/qcom_common.c     |   6 +-
>  drivers/remoteproc/qcom_common.h     |  21 +
>  drivers/remoteproc/qcom_q6v5_pil.c   |   3 +
>  drivers/remoteproc/qcom_sysmon.c     | 579 ++++++++++++++++++++++++
>  drivers/remoteproc/qcom_wcnss.c      |   4 +
>  drivers/remoteproc/remoteproc_core.c |  18 +-
>  drivers/soc/qcom/Kconfig             |   9 +
>  drivers/soc/qcom/Makefile            |   2 +
>  drivers/soc/qcom/qmi_encdec.c        | 816 +++++++++++++++++++++++++++++++++
>  drivers/soc/qcom/qmi_interface.c     | 848 +++++++++++++++++++++++++++++++++++
>  include/linux/remoteproc.h           |   4 +-
>  include/linux/soc/qcom/qmi.h         | 271 +++++++++++
>  samples/Kconfig                      |   9 +
>  samples/Makefile                     |   2 +-
>  samples/qmi/Makefile                 |   1 +
>  samples/qmi/qmi_sample_client.c      | 622 +++++++++++++++++++++++++
>  19 files changed, 3230 insertions(+), 15 deletions(-)
>  create mode 100644 drivers/remoteproc/qcom_sysmon.c
>  create mode 100644 drivers/soc/qcom/qmi_encdec.c
>  create mode 100644 drivers/soc/qcom/qmi_interface.c
>  create mode 100644 include/linux/soc/qcom/qmi.h
>  create mode 100644 samples/qmi/Makefile
>  create mode 100644 samples/qmi/qmi_sample_client.c
>
> --
> 2.15.0
>

For this patch series and the use of SPDX ids: Thanks!

Acked-by:  Philippe Ombredanne <pombredanne@nexB.com>