[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CANLsYky00+sD-MsGK40E79xe8v8vtxji8O0LdgEOHYTF5PPpZA@mail.gmail.com>
Date: Thu, 20 Apr 2023 09:41:09 -0600
From: Mathieu Poirier <mathieu.poirier@...aro.org>
To: Sarannya S <quic_sarannya@...cinc.com>
Cc: quic_bjorande@...cinc.com, arnaud.pouliquen@...s.st.com,
swboyd@...omium.org, quic_clew@...cinc.com,
linux-kernel@...r.kernel.org, linux-arm-msm@...r.kernel.org,
linux-remoteproc@...r.kernel.org
Subject: Re: [PATCH V6 0/3] rpmsg signaling/flowcontrol patches
On Thu, 20 Apr 2023 at 00:22, Sarannya S <quic_sarannya@...cinc.com> 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.
>
> 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(-)
>
It is not possible to follow what is happening with this patchset.
Please send a proper V7 that takes into account the comments that were
received _and_ a summary of what changed between the revisions.
> --
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> a Linux Foundation Collaborative Project
>
Powered by blists - more mailing lists