net: ipa: initialize ring indexes to 0
authorAlex Elder <elder@linaro.org>
Tue, 19 Jul 2022 14:18:55 +0000 (09:18 -0500)
committerDavid S. Miller <davem@davemloft.net>
Wed, 20 Jul 2022 10:12:20 +0000 (11:12 +0100)
commit5fb859f79f4f49d9df16bac2b3a84a6fa3aaccf1
tree4331292e9ec4ce51a41616a3dbbd928e6a541f03
parent52323ef75414d60b17f683076833eb55a6bffa2b
net: ipa: initialize ring indexes to 0

When a GSI channel is initially allocated, and after it has been
reset, the hardware assumes its ring index is 0.  And although we
do initialize channels this way, the comments in the IPA code don't
really explain this.  For event rings, it doesn't matter what value
we use initially, so using 0 is just fine.

Add some information about the assumptions made by hardware above
the definition of the gsi_ring structure in "gsi.h".

Zero the index field for all rings (channel and event) when the ring
is allocated.  As a result, that function initializes all fields in
the structure.

Stop zeroing the index the top of gsi_channel_program().  Initially
we'll use the index value set when the channel ring was allocated.
And we'll explicitly zero the index value in gsi_channel_reset()
before programming the hardware, adding a comment explaining why
it's required.

For event rings, use the index initialized by gsi_ring_alloc()
rather than 0 when ringing the doorbell in gsi_evt_ring_program().
(It'll still be zero, but we won't assume that to be the case.)

Use a local variable in gsi_evt_ring_program() that represents the
address of the event ring's ring structure.

Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ipa/gsi.c
drivers/net/ipa/gsi.h