[PATCH v3 0/3] rpmsg offchannel deadcoding

linux@treblig.org posted 3 patches 9 months, 1 week ago
Documentation/staging/rpmsg.rst  | 46 -----------------------
drivers/rpmsg/rpmsg_core.c       | 63 --------------------------------
drivers/rpmsg/rpmsg_internal.h   |  6 ---
drivers/rpmsg/virtio_rpmsg_bus.c | 24 +-----------
include/linux/rpmsg.h            | 22 -----------
5 files changed, 1 insertion(+), 160 deletions(-)
[PATCH v3 0/3] rpmsg offchannel deadcoding
Posted by linux@treblig.org 9 months, 1 week ago
From: "Dr. David Alan Gilbert" <linux@treblig.org>

Hi,
  This removes a couple of unused functions in rpmsg,
and (v2) after discussions with Arnaud, follows the thread
and removes code that they would call.

(Build tested only)

Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>

V3
  Fixup comment on rpmsg_send_offchannel_raw(),
  removing the reference to "_offchannel" - as per Mathieu's
  review

Dr. David Alan Gilbert (3):
  rpmsg: core:  Remove deadcode
  rpmsg: virtio: Remove uncallable offchannel functions
  rpmsg: Remove unused method pointers *send_offchannel

 Documentation/staging/rpmsg.rst  | 46 -----------------------
 drivers/rpmsg/rpmsg_core.c       | 63 --------------------------------
 drivers/rpmsg/rpmsg_internal.h   |  6 ---
 drivers/rpmsg/virtio_rpmsg_bus.c | 24 +-----------
 include/linux/rpmsg.h            | 22 -----------
 5 files changed, 1 insertion(+), 160 deletions(-)

-- 
2.49.0
Re: [PATCH v3 0/3] rpmsg offchannel deadcoding
Posted by Mathieu Poirier 9 months ago
On Wed, Apr 30, 2025 at 12:45:56AM +0100, linux@treblig.org wrote:
> From: "Dr. David Alan Gilbert" <linux@treblig.org>
> 
> Hi,
>   This removes a couple of unused functions in rpmsg,
> and (v2) after discussions with Arnaud, follows the thread
> and removes code that they would call.
> 
> (Build tested only)
> 
> Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
> 
> V3
>   Fixup comment on rpmsg_send_offchannel_raw(),
>   removing the reference to "_offchannel" - as per Mathieu's
>   review
> 
> Dr. David Alan Gilbert (3):
>   rpmsg: core:  Remove deadcode
>   rpmsg: virtio: Remove uncallable offchannel functions
>   rpmsg: Remove unused method pointers *send_offchannel
> 
>  Documentation/staging/rpmsg.rst  | 46 -----------------------
>  drivers/rpmsg/rpmsg_core.c       | 63 --------------------------------
>  drivers/rpmsg/rpmsg_internal.h   |  6 ---
>  drivers/rpmsg/virtio_rpmsg_bus.c | 24 +-----------
>  include/linux/rpmsg.h            | 22 -----------
>  5 files changed, 1 insertion(+), 160 deletions(-)

I have applied this patchet.

Thanks,
Mathieu

> 
> -- 
> 2.49.0
>
Re: [PATCH v3 0/3] rpmsg offchannel deadcoding
Posted by Dr. David Alan Gilbert 9 months ago
* Mathieu Poirier (mathieu.poirier@linaro.org) wrote:
> On Wed, Apr 30, 2025 at 12:45:56AM +0100, linux@treblig.org wrote:
> > From: "Dr. David Alan Gilbert" <linux@treblig.org>
> > 
> > Hi,
> >   This removes a couple of unused functions in rpmsg,
> > and (v2) after discussions with Arnaud, follows the thread
> > and removes code that they would call.
> > 
> > (Build tested only)
> > 
> > Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
> > 
> > V3
> >   Fixup comment on rpmsg_send_offchannel_raw(),
> >   removing the reference to "_offchannel" - as per Mathieu's
> >   review
> > 
> > Dr. David Alan Gilbert (3):
> >   rpmsg: core:  Remove deadcode
> >   rpmsg: virtio: Remove uncallable offchannel functions
> >   rpmsg: Remove unused method pointers *send_offchannel
> > 
> >  Documentation/staging/rpmsg.rst  | 46 -----------------------
> >  drivers/rpmsg/rpmsg_core.c       | 63 --------------------------------
> >  drivers/rpmsg/rpmsg_internal.h   |  6 ---
> >  drivers/rpmsg/virtio_rpmsg_bus.c | 24 +-----------
> >  include/linux/rpmsg.h            | 22 -----------
> >  5 files changed, 1 insertion(+), 160 deletions(-)
> 
> I have applied this patchet.

Thanks!

Dave

> Thanks,
> Mathieu
> 
> > 
> > -- 
> > 2.49.0
> > 
> 
-- 
 -----Open up your eyes, open up your mind, open up your code -------   
/ Dr. David Alan Gilbert    |       Running GNU/Linux       | Happy  \ 
\        dave @ treblig.org |                               | In Hex /
 \ _________________________|_____ http://www.treblig.org   |_______/
Re: [PATCH v3 0/3] rpmsg offchannel deadcoding
Posted by Arnaud POULIQUEN 9 months, 1 week ago

On 4/30/25 01:45, linux@treblig.org wrote:
> From: "Dr. David Alan Gilbert" <linux@treblig.org>
> 
> Hi,
>   This removes a couple of unused functions in rpmsg,
> and (v2) after discussions with Arnaud, follows the thread
> and removes code that they would call.

Acked-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>

Thanks,
Arnaud

> 
> (Build tested only)
> 
> Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
> 
> V3
>   Fixup comment on rpmsg_send_offchannel_raw(),
>   removing the reference to "_offchannel" - as per Mathieu's
>   review
> 
> Dr. David Alan Gilbert (3):
>   rpmsg: core:  Remove deadcode
>   rpmsg: virtio: Remove uncallable offchannel functions
>   rpmsg: Remove unused method pointers *send_offchannel
> 
>  Documentation/staging/rpmsg.rst  | 46 -----------------------
>  drivers/rpmsg/rpmsg_core.c       | 63 --------------------------------
>  drivers/rpmsg/rpmsg_internal.h   |  6 ---
>  drivers/rpmsg/virtio_rpmsg_bus.c | 24 +-----------
>  include/linux/rpmsg.h            | 22 -----------
>  5 files changed, 1 insertion(+), 160 deletions(-)
>