mbox series

[net-next,0/5] net: ipa: kill endpoint stop workaround

Message ID 20200504235345.17118-1-elder@linaro.org
Headers show
Series net: ipa: kill endpoint stop workaround | expand

Message

Alex Elder May 4, 2020, 11:53 p.m. UTC
It turns out that a workaround that performs a small DMA operation
between retried attempts to stop a GSI channel is not needed for any
supported hardware.  The hardware quirk that required the extra DMA
operation was fixed after IPA v3.1.  So this series gets rid of that
workaround code, along with some other code that was only present to
support it.

NOTE:  This series depends on (and includes/duplicates) another patch
       that has already been committed in the net tree:
         713b6ebb4c37 net: ipa: fix a bug in ipa_endpoint_stop()

					-Alex
Alex Elder (5):
  net: ipa: fix a bug in ipa_endpoint_stop()
  net: ipa: get rid of workaround in ipa_endpoint_stop()
  net: ipa: don't retry in ipa_endpoint_stop()
  net: ipa: kill ipa_endpoint_stop()
  net: ipa: kill ipa_cmd_dma_task_32b_addr_add()

 drivers/net/ipa/ipa_cmd.c      | 59 -------------------------
 drivers/net/ipa/ipa_cmd.h      | 11 -----
 drivers/net/ipa/ipa_endpoint.c | 80 +++-------------------------------
 drivers/net/ipa/ipa_endpoint.h |  2 -
 4 files changed, 6 insertions(+), 146 deletions(-)

Comments

David Miller May 7, 2020, 12:42 a.m. UTC | #1
From: Alex Elder <elder@linaro.org>
Date: Mon,  4 May 2020 18:53:40 -0500

> It turns out that a workaround that performs a small DMA operation
> between retried attempts to stop a GSI channel is not needed for any
> supported hardware.  The hardware quirk that required the extra DMA
> operation was fixed after IPA v3.1.  So this series gets rid of that
> workaround code, along with some other code that was only present to
> support it.
> 
> NOTE:  This series depends on (and includes/duplicates) another patch
>        that has already been committed in the net tree:
>          713b6ebb4c37 net: ipa: fix a bug in ipa_endpoint_stop()

Series applied.