Re: [PATCH V6 0/3] rpmsg signaling/flowcontrol patches

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 4/20/2023 11:51 AM, Sarannya S wrote:
Added new IOCTLS- RPMSG_GET_OUTGOING_FLOWCONTROL and RPMSG_SET_INCOMING_FLOWCONTROL
to get/set the rpmsg char device's flow control signal.
Updated the cmd name 'RPM_CMD_SIGNALS' to 'GLINK_CMD_SIGNALS'.
Changed 'remote_flow' to bool type, and updated it's evaluation.
Updated evaluation of 'set' in rpmsg_char as 'set = !!arg'.
Add destination address of endpoint as parameter in rpmsg_set_flow_control.
Addressed review comments to change variable names/descriptions.

Something to notice, you are not changing the version number as you have sent v6 here again.

Also maintain the changes happened from the last patch series properly.
Like

Changes in v6:
 - ..
 - ..

Change in v5: (a lore kernel link will be helpful here for reviewer to remind the conversation happened during last review)
 - ..
 - ..



--Mukesh


Chris Lew (2):
   rpmsg: glink: Add support to handle signals command
   rpmsg: char: Add RPMSG GET/SET FLOWCONTROL IOCTL support

Deepak Kumar Singh (1):
   rpmsg: core: Add signal API support

  drivers/rpmsg/qcom_glink_native.c | 64 +++++++++++++++++++++++++++++++++++++++
  drivers/rpmsg/rpmsg_char.c        | 49 ++++++++++++++++++++++++++----
  drivers/rpmsg/rpmsg_core.c        | 21 +++++++++++++
  drivers/rpmsg/rpmsg_internal.h    |  2 ++
  include/linux/rpmsg.h             | 15 +++++++++
  include/uapi/linux/rpmsg.h        | 11 ++++++-
  6 files changed, 155 insertions(+), 7 deletions(-)




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux