mbox series

[0/6] rpmsg: enable the use of the rpmsg_char device for the Virtio backend

Message ID 20210311140413.31725-1-arnaud.pouliquen@foss.st.com (mailing list archive)
Headers show
Series rpmsg: enable the use of the rpmsg_char device for the Virtio backend | expand

Message

Arnaud Pouliquen March 11, 2021, 2:04 p.m. UTC
This series is the first step in the division of the series: 
"Introduce a generic IOCTL interface for RPMsg channels management"[1]

The main goal here is to enable the RPMsg char interface for
the virtio RPMsg backend. 

In addition some patches have been includes in order to document the
interface and rename the rpmsg_char_init function.

It also includes Mathieu Poirier's comments made on [1]

Patchsets that should be the next steps:
 - Extract the control part of the char dev and create the rpmsg_ctrl.c
   file
 - Introduce the RPMSG_CREATE_DEV_IOCTL IOCTL to instantiate RPMsg devices


[1]: https://patchwork.kernel.org/project/linux-remoteproc/list/?series=435523

Arnaud Pouliquen (6):
  rpmsg: char: Rename rpmsg_char_init to rpmsg_chrdev_init
  rpmsg: Move RPMSG_ADDR_ANY in user API
  rpmsg: Add short description of the IOCTL defined in UAPI.
  rpmsg: char: Use rpmsg_sendto to specify the message destination
    address
  rpmsg: virtio: Register the rpmsg_char device
  rpmsg: char: Return an error if device already open

 drivers/rpmsg/qcom_glink_native.c | 16 ++++++++
 drivers/rpmsg/qcom_smd.c          | 16 ++++++++
 drivers/rpmsg/rpmsg_char.c        | 11 ++++--
 drivers/rpmsg/virtio_rpmsg_bus.c  | 62 ++++++++++++++++++++++++++++---
 include/linux/rpmsg.h             |  3 +-
 include/uapi/linux/rpmsg.h        | 13 ++++++-
 6 files changed, 108 insertions(+), 13 deletions(-)

Comments

Bjorn Andersson March 11, 2021, 11:38 p.m. UTC | #1
On Thu 11 Mar 08:04 CST 2021, Arnaud Pouliquen wrote:

> This series is the first step in the division of the series: 
> "Introduce a generic IOCTL interface for RPMsg channels management"[1]
> 
> The main goal here is to enable the RPMsg char interface for
> the virtio RPMsg backend. 
> 
> In addition some patches have been includes in order to document the
> interface and rename the rpmsg_char_init function.
> 
> It also includes Mathieu Poirier's comments made on [1]
> 
> Patchsets that should be the next steps:
>  - Extract the control part of the char dev and create the rpmsg_ctrl.c
>    file
>  - Introduce the RPMSG_CREATE_DEV_IOCTL IOCTL to instantiate RPMsg devices
> 
> 
> [1]: https://patchwork.kernel.org/project/linux-remoteproc/list/?series=435523
> 

Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>

Regards,
Bjorn

> Arnaud Pouliquen (6):
>   rpmsg: char: Rename rpmsg_char_init to rpmsg_chrdev_init
>   rpmsg: Move RPMSG_ADDR_ANY in user API
>   rpmsg: Add short description of the IOCTL defined in UAPI.
>   rpmsg: char: Use rpmsg_sendto to specify the message destination
>     address
>   rpmsg: virtio: Register the rpmsg_char device
>   rpmsg: char: Return an error if device already open
> 
>  drivers/rpmsg/qcom_glink_native.c | 16 ++++++++
>  drivers/rpmsg/qcom_smd.c          | 16 ++++++++
>  drivers/rpmsg/rpmsg_char.c        | 11 ++++--
>  drivers/rpmsg/virtio_rpmsg_bus.c  | 62 ++++++++++++++++++++++++++++---
>  include/linux/rpmsg.h             |  3 +-
>  include/uapi/linux/rpmsg.h        | 13 ++++++-
>  6 files changed, 108 insertions(+), 13 deletions(-)
> 
> -- 
> 2.17.1
>
Mathieu Poirier March 15, 2021, 4:16 p.m. UTC | #2
On Thu, Mar 11, 2021 at 03:04:07PM +0100, Arnaud Pouliquen wrote:
> This series is the first step in the division of the series: 
> "Introduce a generic IOCTL interface for RPMsg channels management"[1]
> 
> The main goal here is to enable the RPMsg char interface for
> the virtio RPMsg backend. 
> 
> In addition some patches have been includes in order to document the
> interface and rename the rpmsg_char_init function.
> 
> It also includes Mathieu Poirier's comments made on [1]
> 
> Patchsets that should be the next steps:
>  - Extract the control part of the char dev and create the rpmsg_ctrl.c
>    file
>  - Introduce the RPMSG_CREATE_DEV_IOCTL IOCTL to instantiate RPMsg devices
> 
> 
> [1]: https://patchwork.kernel.org/project/linux-remoteproc/list/?series=435523
> 
> Arnaud Pouliquen (6):
>   rpmsg: char: Rename rpmsg_char_init to rpmsg_chrdev_init
>   rpmsg: Move RPMSG_ADDR_ANY in user API
>   rpmsg: Add short description of the IOCTL defined in UAPI.
>   rpmsg: char: Use rpmsg_sendto to specify the message destination
>     address
>   rpmsg: virtio: Register the rpmsg_char device
>   rpmsg: char: Return an error if device already open
> 
>  drivers/rpmsg/qcom_glink_native.c | 16 ++++++++
>  drivers/rpmsg/qcom_smd.c          | 16 ++++++++
>  drivers/rpmsg/rpmsg_char.c        | 11 ++++--
>  drivers/rpmsg/virtio_rpmsg_bus.c  | 62 ++++++++++++++++++++++++++++---
>  include/linux/rpmsg.h             |  3 +-
>  include/uapi/linux/rpmsg.h        | 13 ++++++-
>  6 files changed, 108 insertions(+), 13 deletions(-)

Reviewed-by: Mathieu Poirier <mathieu.poirier@linaro.org>

> 
> -- 
> 2.17.1
>
patchwork-bot+linux-arm-msm@kernel.org May 26, 2021, 7:03 p.m. UTC | #3
Hello:

This series was applied to qcom/linux.git (refs/heads/for-next):

On Thu, 11 Mar 2021 15:04:07 +0100 you wrote:
> This series is the first step in the division of the series:
> "Introduce a generic IOCTL interface for RPMsg channels management"[1]
> 
> The main goal here is to enable the RPMsg char interface for
> the virtio RPMsg backend.
> 
> In addition some patches have been includes in order to document the
> interface and rename the rpmsg_char_init function.
> 
> [...]

Here is the summary with links:
  - [1/6] rpmsg: char: Rename rpmsg_char_init to rpmsg_chrdev_init
    https://git.kernel.org/qcom/c/60d7b22d2593
  - [2/6] rpmsg: Move RPMSG_ADDR_ANY in user API
    https://git.kernel.org/qcom/c/3093c3c7c136
  - [3/6] rpmsg: Add short description of the IOCTL defined in UAPI.
    https://git.kernel.org/qcom/c/809328b40cfb
  - [4/6] rpmsg: char: Use rpmsg_sendto to specify the message destination address
    https://git.kernel.org/qcom/c/b4ce7e2ebcc5
  - [5/6] rpmsg: virtio: Register the rpmsg_char device
    https://git.kernel.org/qcom/c/c486682ae1e2
  - [6/6] rpmsg: char: Return an error if device already open
    https://git.kernel.org/qcom/c/964e8bedd5a1

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html