From: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
To: Mathieu Poirier <mathieu.poirier@linaro.org>
Cc: ohad@wizery.com, bjorn.andersson@linaro.org,
arnaud.pouliquen@st.com, linux-remoteproc@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v7 0/8] rpmsg: Make RPMSG name service modular
Date: Wed, 2 Dec 2020 12:05:55 +0100 [thread overview]
Message-ID: <20201202110555.GA65230@ubuntu> (raw)
In-Reply-To: <20201123160610.GA19108@ubuntu>
Hi Mathieu,
I'd like to resume reviewing and begin upstreaming of the next steps of
my Audio DSP Virtualisation work, based on this your patch set. How
confident are we that it's going to be upstreamed in its present form?
What's the plan to push it to "next?"
Thanks
Guennadi
On Mon, Nov 23, 2020 at 05:06:10PM +0100, Guennadi Liakhovetski wrote:
> Hi Mathieu,
>
> Thanks for bringing all the stuff together and for polishing it!
>
> For the entire series:
>
> Tested-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
> Reviewed-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
>
> Thanks
> Guennadi
>
> On Fri, Nov 20, 2020 at 02:42:37PM -0700, Mathieu Poirier wrote:
> > This revision addresses comments received from the previous revision,
> > i.e V6. Please see details below.
> >
> > It starts by making the RPMSG protocol transport agnostic by
> > moving the headers it uses to generic types and using those in the
> > current implementation. From there it re-uses the work that Arnaud
> > published[1] to make the name service modular.
> >
> > Tested on stm32mp157 with the RPMSG client sample application. Applies
> > cleanly on rpmsg-next.
> >
> > Thanks,
> > Mathieu
> >
> > [1]. https://patchwork.kernel.org/project/linux-remoteproc/list/?series=338335
> >
> > -------
> > New for V7:
> > - Fixed error path in rpmsg_probe() as reported by Guennadi
> >
> > Arnaud Pouliquen (4):
> > rpmsg: virtio: Rename rpmsg_create_channel
> > rpmsg: core: Add channel creation internal API
> > rpmsg: virtio: Add rpmsg channel device ops
> > rpmsg: Turn name service into a stand alone driver
> >
> > Mathieu Poirier (4):
> > rpmsg: Introduce __rpmsg{16|32|64} types
> > rpmsg: virtio: Move from virtio to rpmsg byte conversion
> > rpmsg: Move structure rpmsg_ns_msg to header file
> > rpmsg: Make rpmsg_{register|unregister}_device() public
> >
> > drivers/rpmsg/Kconfig | 9 ++
> > drivers/rpmsg/Makefile | 1 +
> > drivers/rpmsg/rpmsg_core.c | 44 ++++++++
> > drivers/rpmsg/rpmsg_internal.h | 14 ++-
> > drivers/rpmsg/rpmsg_ns.c | 126 +++++++++++++++++++++
> > drivers/rpmsg/virtio_rpmsg_bus.c | 186 +++++++++++--------------------
> > include/linux/rpmsg.h | 63 ++++++++++-
> > include/linux/rpmsg/byteorder.h | 67 +++++++++++
> > include/linux/rpmsg/ns.h | 45 ++++++++
> > include/uapi/linux/rpmsg_types.h | 11 ++
> > 10 files changed, 439 insertions(+), 127 deletions(-)
> > create mode 100644 drivers/rpmsg/rpmsg_ns.c
> > create mode 100644 include/linux/rpmsg/byteorder.h
> > create mode 100644 include/linux/rpmsg/ns.h
> > create mode 100644 include/uapi/linux/rpmsg_types.h
> >
> > --
> > 2.25.1
> >
next prev parent reply other threads:[~2020-12-02 11:07 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-20 21:42 [PATCH v7 0/8] rpmsg: Make RPMSG name service modular Mathieu Poirier
2020-11-20 21:42 ` [PATCH v7 1/8] rpmsg: Introduce __rpmsg{16|32|64} types Mathieu Poirier
2020-11-20 21:42 ` [PATCH v7 2/8] rpmsg: virtio: Move from virtio to rpmsg byte conversion Mathieu Poirier
2020-11-20 21:42 ` [PATCH v7 3/8] rpmsg: Move structure rpmsg_ns_msg to header file Mathieu Poirier
2020-11-20 21:42 ` [PATCH v7 4/8] rpmsg: virtio: Rename rpmsg_create_channel Mathieu Poirier
2020-11-20 21:42 ` [PATCH v7 5/8] rpmsg: core: Add channel creation internal API Mathieu Poirier
2020-11-20 21:42 ` [PATCH v7 6/8] rpmsg: virtio: Add rpmsg channel device ops Mathieu Poirier
2020-11-20 21:42 ` [PATCH v7 7/8] rpmsg: Make rpmsg_{register|unregister}_device() public Mathieu Poirier
2020-11-20 21:42 ` [PATCH v7 8/8] rpmsg: Turn name service into a stand alone driver Mathieu Poirier
2020-11-23 16:06 ` [PATCH v7 0/8] rpmsg: Make RPMSG name service modular Guennadi Liakhovetski
2020-12-02 11:05 ` Guennadi Liakhovetski [this message]
2020-12-02 20:39 ` Mathieu Poirier
2020-12-03 20:42 ` Guennadi Liakhovetski
2020-12-04 17:51 ` Mathieu Poirier
2020-12-07 15:35 ` Kishon Vijay Abraham I
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20201202110555.GA65230@ubuntu \
--to=guennadi.liakhovetski@linux.intel.com \
--cc=arnaud.pouliquen@st.com \
--cc=bjorn.andersson@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-remoteproc@vger.kernel.org \
--cc=mathieu.poirier@linaro.org \
--cc=ohad@wizery.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox