mbox series

[net-next,0/2] net: ipa: GSI channel flow control

Message ID 20211124194416.707007-1-elder@linaro.org (mailing list archive)
Headers show
Series net: ipa: GSI channel flow control | expand

Message

Alex Elder Nov. 24, 2021, 7:44 p.m. UTC
Starting with IPA v4.2, endpoint DELAY mode (which prevents data
transfer on TX endpoints) does not work properly.  To address this,
changes were made to allow underlying GSI channels to be put into
a "flow controlled" state, which achieves a similar objective.
The first patch in this series implements the flow controlled
channel state and the commands used to control it.  It arranges
to use the new mechanism--instead of DELAY mode--for IPA v4.2+.

In IPA v4.11, the notion of GSI channel flow control was enhanced,
and implemented in a slightly different way.  For the most part this
doesn't affect the way the IPA driver uses flow control, but the
second patch adds support for the newer mechanism.

					-Alex

Alex Elder (2):
  net: ipa: introduce channel flow control
  net: ipa: support enhanced channel flow control

 drivers/net/ipa/gsi.c          | 70 ++++++++++++++++++++++++++--------
 drivers/net/ipa/gsi.h          | 10 +++++
 drivers/net/ipa/gsi_reg.h      |  4 ++
 drivers/net/ipa/ipa_endpoint.c | 50 ++++++++++++++----------
 4 files changed, 99 insertions(+), 35 deletions(-)

Comments

Jakub Kicinski Nov. 26, 2021, 6:08 p.m. UTC | #1
On Wed, 24 Nov 2021 13:44:14 -0600 Alex Elder wrote:
> Starting with IPA v4.2, endpoint DELAY mode (which prevents data
> transfer on TX endpoints) does not work properly.  To address this,
> changes were made to allow underlying GSI channels to be put into
> a "flow controlled" state, which achieves a similar objective.
> The first patch in this series implements the flow controlled
> channel state and the commands used to control it.  It arranges
> to use the new mechanism--instead of DELAY mode--for IPA v4.2+.
> 
> In IPA v4.11, the notion of GSI channel flow control was enhanced,
> and implemented in a slightly different way.  For the most part this
> doesn't affect the way the IPA driver uses flow control, but the
> second patch adds support for the newer mechanism.

Applied, thanks.