From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.7 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1DEB5C64E8A for ; Wed, 2 Dec 2020 11:07:49 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id B721D22203 for ; Wed, 2 Dec 2020 11:07:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726788AbgLBLHr (ORCPT ); Wed, 2 Dec 2020 06:07:47 -0500 Received: from mga14.intel.com ([192.55.52.115]:59460 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725885AbgLBLHr (ORCPT ); Wed, 2 Dec 2020 06:07:47 -0500 IronPort-SDR: fMLlIAqROtKa1+IgkHDH7JK9uxl6JDFvhRIksOti5cA9pWk6Y49nnAKPvPnSP00rV8LbEWvcy+ Q3O8YGNcM4wA== X-IronPort-AV: E=McAfee;i="6000,8403,9822"; a="172214866" X-IronPort-AV: E=Sophos;i="5.78,386,1599548400"; d="scan'208";a="172214866" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Dec 2020 03:06:06 -0800 IronPort-SDR: +Ugtq839jygB/t3ISgQF1iebCozkDdRiJ3To3uxbciljYN/s87/pX7F7i+F/v1FbRkdSWVgXAl 9/g669fvXu1w== X-IronPort-AV: E=Sophos;i="5.78,386,1599548400"; d="scan'208";a="550015781" Received: from gliakhov-mobl2.ger.corp.intel.com (HELO ubuntu) ([10.252.47.1]) by orsmga005-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Dec 2020 03:06:04 -0800 Date: Wed, 2 Dec 2020 12:05:55 +0100 From: Guennadi Liakhovetski To: Mathieu Poirier 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 Message-ID: <20201202110555.GA65230@ubuntu> References: <20201120214245.172963-1-mathieu.poirier@linaro.org> <20201123160610.GA19108@ubuntu> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20201123160610.GA19108@ubuntu> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 > Reviewed-by: Guennadi Liakhovetski > > 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 > >