Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-18 Thread Mathieu Poirier
On Wed, 18 Nov 2020 at 00:08, Guennadi Liakhovetski wrote: > > On Tue, Nov 17, 2020 at 05:06:47PM -0700, Mathieu Poirier wrote: > > [snip] > > > I confirm that all this is working as expected - I will send a new revision > > of > > this set tomorrow afternoon. > > > > Guennadi, can I add a

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-17 Thread Guennadi Liakhovetski
On Tue, Nov 17, 2020 at 05:06:47PM -0700, Mathieu Poirier wrote: [snip] > I confirm that all this is working as expected - I will send a new revision of > this set tomorrow afternoon. > > Guennadi, can I add a Co-developed-by and Signed-off-by with your name on the > patch? You can add the

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-17 Thread Mathieu Poirier
On Tue, Nov 17, 2020 at 05:44:05PM +0100, Arnaud POULIQUEN wrote: > > > On 11/17/20 5:03 PM, Guennadi Liakhovetski wrote: > > On Tue, Nov 17, 2020 at 12:42:30PM +0100, Arnaud POULIQUEN wrote: > > > > [snip] > > > >> diff --git a/drivers/rpmsg/rpmsg_ns.c b/drivers/rpmsg/rpmsg_ns.c > >> index

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-17 Thread Guennadi Liakhovetski
On Tue, Nov 17, 2020 at 06:30:54PM +0100, Arnaud POULIQUEN wrote: [snip] > It's not a good day for me today... it seems I read your explanation too > quickly > this morning, which is, however, very clear. > My apologies Oh, I did the same to one of your earlier emails one of these days - I

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-17 Thread Arnaud POULIQUEN
On 11/17/20 5:58 PM, Guennadi Liakhovetski wrote: > On Tue, Nov 17, 2020 at 05:44:05PM +0100, Arnaud POULIQUEN wrote: >> >> >> On 11/17/20 5:03 PM, Guennadi Liakhovetski wrote: >>> On Tue, Nov 17, 2020 at 12:42:30PM +0100, Arnaud POULIQUEN wrote: >>> >>> [snip] >>> diff --git

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-17 Thread Guennadi Liakhovetski
On Tue, Nov 17, 2020 at 05:44:05PM +0100, Arnaud POULIQUEN wrote: > > > On 11/17/20 5:03 PM, Guennadi Liakhovetski wrote: > > On Tue, Nov 17, 2020 at 12:42:30PM +0100, Arnaud POULIQUEN wrote: > > > > [snip] > > > >> diff --git a/drivers/rpmsg/rpmsg_ns.c b/drivers/rpmsg/rpmsg_ns.c > >> index

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-17 Thread Arnaud POULIQUEN
On 11/17/20 5:03 PM, Guennadi Liakhovetski wrote: > On Tue, Nov 17, 2020 at 12:42:30PM +0100, Arnaud POULIQUEN wrote: > > [snip] > >> diff --git a/drivers/rpmsg/rpmsg_ns.c b/drivers/rpmsg/rpmsg_ns.c >> index 5bda7cb44618..80c2cc23bada 100644 >> --- a/drivers/rpmsg/rpmsg_ns.c >> +++

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-17 Thread Guennadi Liakhovetski
On Tue, Nov 17, 2020 at 12:42:30PM +0100, Arnaud POULIQUEN wrote: [snip] > diff --git a/drivers/rpmsg/rpmsg_ns.c b/drivers/rpmsg/rpmsg_ns.c > index 5bda7cb44618..80c2cc23bada 100644 > --- a/drivers/rpmsg/rpmsg_ns.c > +++ b/drivers/rpmsg/rpmsg_ns.c > @@ -55,6 +55,39 @@ static int

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-17 Thread Arnaud POULIQUEN
Hi Mathieu, On 11/16/20 11:40 PM, Mathieu Poirier wrote: > On Mon, Nov 16, 2020 at 04:51:52PM +0100, Arnaud POULIQUEN wrote: >> Hi Guennadi, >> >> On 11/16/20 4:10 PM, Guennadi Liakhovetski wrote: [snip] > I did a lot of probing, went deep in the bowels of the user mode helper > subsystem and

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-16 Thread Guennadi Liakhovetski
Hi Mathieu, On Mon, Nov 16, 2020 at 03:40:03PM -0700, Mathieu Poirier wrote: > On Mon, Nov 16, 2020 at 04:51:52PM +0100, Arnaud POULIQUEN wrote: [snip] > > Having said that, does this guarantee the probe, a good question! > > Maybe you or Mathieu have the answer, not me... > > I did a lot of

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-16 Thread Mathieu Poirier
On Mon, Nov 16, 2020 at 04:51:52PM +0100, Arnaud POULIQUEN wrote: > Hi Guennadi, > > On 11/16/20 4:10 PM, Guennadi Liakhovetski wrote: > > Hi Arnaud, > > > > On Mon, Nov 16, 2020 at 03:43:35PM +0100, Arnaud POULIQUEN wrote: > >> Hi Guennadi, Mathieu, > > > > [snip] > > > >> I tried the

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-16 Thread Guennadi Liakhovetski
On Mon, Nov 16, 2020 at 04:51:52PM +0100, Arnaud POULIQUEN wrote: > Hi Guennadi, > > On 11/16/20 4:10 PM, Guennadi Liakhovetski wrote: > > Hi Arnaud, > > > > On Mon, Nov 16, 2020 at 03:43:35PM +0100, Arnaud POULIQUEN wrote: > >> Hi Guennadi, Mathieu, > > > > [snip] > > > >> I tried the

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-16 Thread Arnaud POULIQUEN
Hi Guennadi, On 11/16/20 4:10 PM, Guennadi Liakhovetski wrote: > Hi Arnaud, > > On Mon, Nov 16, 2020 at 03:43:35PM +0100, Arnaud POULIQUEN wrote: >> Hi Guennadi, Mathieu, > > [snip] > >> I tried the find_module API, it's quite simple and seems to work well. just >> need >> to be protected by

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-16 Thread Guennadi Liakhovetski
Hi Arnaud, On Mon, Nov 16, 2020 at 03:43:35PM +0100, Arnaud POULIQUEN wrote: > Hi Guennadi, Mathieu, [snip] > I tried the find_module API, it's quite simple and seems to work well. just > need > to be protected by #ifdef MODULE > > I also added a select RMPS_NS in the RPMSG_VIRTIO for

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-16 Thread Arnaud POULIQUEN
Hi Guennadi, Mathieu, On 11/12/20 2:27 PM, Arnaud POULIQUEN wrote: > > > On 11/12/20 12:51 PM, Guennadi Liakhovetski wrote: >> Hi Arnaud, >> >> On Thu, Nov 12, 2020 at 11:17:54AM +0100, Arnaud POULIQUEN wrote: >>> Hi Guennadi, >>> >>> On 11/11/20 3:49 PM, Guennadi Liakhovetski wrote: Hi

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-14 Thread Mathieu Poirier
On Thu, Nov 12, 2020 at 10:04:17AM +0100, Arnaud POULIQUEN wrote: > > > On 11/11/20 1:37 AM, Mathieu Poirier wrote: > > On Tue, 10 Nov 2020 at 11:18, Arnaud POULIQUEN > > wrote: > >> > >> Hi Mathieu, Guennadi, > >> > >> On 11/9/20 6:55 PM, Mathieu Poirier wrote: > >>> On Mon, Nov 09, 2020 at

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-12 Thread Arnaud POULIQUEN
On 11/12/20 12:51 PM, Guennadi Liakhovetski wrote: > Hi Arnaud, > > On Thu, Nov 12, 2020 at 11:17:54AM +0100, Arnaud POULIQUEN wrote: >> Hi Guennadi, >> >> On 11/11/20 3:49 PM, Guennadi Liakhovetski wrote: >>> Hi Arnaud, > > [snip] > >>> From: Guennadi Liakhovetski >>> Subject: [PATCH]

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-12 Thread Guennadi Liakhovetski
Hi Arnaud, On Thu, Nov 12, 2020 at 11:17:54AM +0100, Arnaud POULIQUEN wrote: > Hi Guennadi, > > On 11/11/20 3:49 PM, Guennadi Liakhovetski wrote: > > Hi Arnaud, [snip] > > From: Guennadi Liakhovetski > > Subject: [PATCH] fixup! rpmsg: Turn name service into a stand alone driver > > > > Link

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-12 Thread Arnaud POULIQUEN
Hi Guennadi, On 11/11/20 3:49 PM, Guennadi Liakhovetski wrote: > Hi Arnaud, > > On Tue, Nov 10, 2020 at 07:18:45PM +0100, Arnaud POULIQUEN wrote: >> Hi Mathieu, Guennadi, >> >> On 11/9/20 6:55 PM, Mathieu Poirier wrote: >>> On Mon, Nov 09, 2020 at 11:20:24AM +0100, Guennadi Liakhovetski wrote:

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-12 Thread Arnaud POULIQUEN
On 11/11/20 1:37 AM, Mathieu Poirier wrote: > On Tue, 10 Nov 2020 at 11:18, Arnaud POULIQUEN > wrote: >> >> Hi Mathieu, Guennadi, >> >> On 11/9/20 6:55 PM, Mathieu Poirier wrote: >>> On Mon, Nov 09, 2020 at 11:20:24AM +0100, Guennadi Liakhovetski wrote: Hi Arnaud, On Mon, Nov

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-11 Thread Guennadi Liakhovetski
Hi Arnaud, On Tue, Nov 10, 2020 at 07:18:45PM +0100, Arnaud POULIQUEN wrote: > Hi Mathieu, Guennadi, > > On 11/9/20 6:55 PM, Mathieu Poirier wrote: > > On Mon, Nov 09, 2020 at 11:20:24AM +0100, Guennadi Liakhovetski wrote: > >> Hi Arnaud, > >> > >> On Mon, Nov 09, 2020 at 09:48:37AM +0100,

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-10 Thread Mathieu Poirier
On Tue, 10 Nov 2020 at 11:18, Arnaud POULIQUEN wrote: > > Hi Mathieu, Guennadi, > > On 11/9/20 6:55 PM, Mathieu Poirier wrote: > > On Mon, Nov 09, 2020 at 11:20:24AM +0100, Guennadi Liakhovetski wrote: > >> Hi Arnaud, > >> > >> On Mon, Nov 09, 2020 at 09:48:37AM +0100, Arnaud POULIQUEN wrote: >

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-10 Thread Arnaud POULIQUEN
Hi Mathieu, Guennadi, On 11/9/20 6:55 PM, Mathieu Poirier wrote: > On Mon, Nov 09, 2020 at 11:20:24AM +0100, Guennadi Liakhovetski wrote: >> Hi Arnaud, >> >> On Mon, Nov 09, 2020 at 09:48:37AM +0100, Arnaud POULIQUEN wrote: >>> Hi Guennadi, Mathieu, >>> >>> On 11/6/20 6:53 PM, Mathieu Poirier

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-09 Thread Mathieu Poirier
On Mon, Nov 09, 2020 at 11:20:24AM +0100, Guennadi Liakhovetski wrote: > Hi Arnaud, > > On Mon, Nov 09, 2020 at 09:48:37AM +0100, Arnaud POULIQUEN wrote: > > Hi Guennadi, Mathieu, > > > > On 11/6/20 6:53 PM, Mathieu Poirier wrote: > > > On Fri, Nov 06, 2020 at 03:00:28PM +0100, Guennadi

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-09 Thread Guennadi Liakhovetski
Hi Arnaud, On Mon, Nov 09, 2020 at 09:48:37AM +0100, Arnaud POULIQUEN wrote: > Hi Guennadi, Mathieu, > > On 11/6/20 6:53 PM, Mathieu Poirier wrote: > > On Fri, Nov 06, 2020 at 03:00:28PM +0100, Guennadi Liakhovetski wrote: > >> On Fri, Nov 06, 2020 at 02:15:45PM +0100, Guennadi Liakhovetski

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-09 Thread Arnaud POULIQUEN
Hi Guennadi, Mathieu, On 11/6/20 6:53 PM, Mathieu Poirier wrote: > On Fri, Nov 06, 2020 at 03:00:28PM +0100, Guennadi Liakhovetski wrote: >> On Fri, Nov 06, 2020 at 02:15:45PM +0100, Guennadi Liakhovetski wrote: >>> Hi Mathieu, Arnaud, >>> >>> On Thu, Nov 05, 2020 at 03:50:28PM -0700, Mathieu

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-06 Thread Mathieu Poirier
On Fri, Nov 06, 2020 at 03:00:28PM +0100, Guennadi Liakhovetski wrote: > On Fri, Nov 06, 2020 at 02:15:45PM +0100, Guennadi Liakhovetski wrote: > > Hi Mathieu, Arnaud, > > > > On Thu, Nov 05, 2020 at 03:50:28PM -0700, Mathieu Poirier wrote: > > > From: Arnaud Pouliquen > > > > > > Make the

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-06 Thread Guennadi Liakhovetski
On Fri, Nov 06, 2020 at 02:15:45PM +0100, Guennadi Liakhovetski wrote: > Hi Mathieu, Arnaud, > > On Thu, Nov 05, 2020 at 03:50:28PM -0700, Mathieu Poirier wrote: > > From: Arnaud Pouliquen > > > > Make the RPMSG name service announcement a stand alone driver so that it > > can be reused by

Re: [PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-06 Thread Guennadi Liakhovetski
Hi Mathieu, Arnaud, On Thu, Nov 05, 2020 at 03:50:28PM -0700, Mathieu Poirier wrote: > From: Arnaud Pouliquen > > Make the RPMSG name service announcement a stand alone driver so that it > can be reused by other subsystems. It is also the first step in making the > functionatlity transport

[PATCH v5 8/8] rpmsg: Turn name service into a stand alone driver

2020-11-05 Thread Mathieu Poirier
From: Arnaud Pouliquen Make the RPMSG name service announcement a stand alone driver so that it can be reused by other subsystems. It is also the first step in making the functionatlity transport independent, i.e that is not tied to virtIO. Co-developed-by: Mathieu Poirier Signed-off-by: