Messages in this thread | | | Date | Mon, 29 Jun 2020 17:35:17 -0700 | From | Jakub Kicinski <> | Subject | Re: [PATCH net-next 1/5] net: ipa: head-of-line block registers are RX only |
| |
On Mon, 29 Jun 2020 16:49:15 -0500 Alex Elder wrote: > The INIT_HOL_BLOCK_EN and INIT_HOL_BLOCK_TIMER endpoint registers > are only valid for RX endpoints. > > Have ipa_endpoint_modem_hol_block_clear_all() skip writing these > registers for TX endpoints. > > Signed-off-by: Alex Elder <elder@linaro.org> > --- > drivers/net/ipa/ipa_endpoint.c | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/ipa/ipa_endpoint.c b/drivers/net/ipa/ipa_endpoint.c > index 9f50d0d11704..3f5a41fc1997 100644 > --- a/drivers/net/ipa/ipa_endpoint.c > +++ b/drivers/net/ipa/ipa_endpoint.c > @@ -642,6 +642,8 @@ static int ipa_endpoint_init_hol_block_timer(struct ipa_endpoint *endpoint, > u32 offset; > u32 val; > > + /* assert(!endpoint->toward_ipa); */ > + > /* XXX We'll fix this when the register definition is clear */ > if (microseconds) { > struct device *dev = &ipa->pdev->dev; > @@ -671,6 +673,8 @@ ipa_endpoint_init_hol_block_enable(struct ipa_endpoint *endpoint, bool enable) > u32 offset; > u32 val; > > + /* assert(!endpoint->toward_ipa); */
What are these assert comments for? :S
| |