On 8/23/2023 2:50 AM, Dmitry Antipov wrote:
Commit 25d0dbcbd5c7 ("ath10k: split ce initialization and allocation")
changes 'ath10k_ce_init_src_ring()' and 'ath10k_ce_init_dest_ring()'
so these functions can't return -ENOMEM but always returns 0. This way
both of the above may be converted to 'void' and related code may be
simplified as well.
Found by Linux Verification Center (linuxtesting.org) with SVACE.
Signed-off-by: Dmitry Antipov <dmantipov@xxxxxxxxx>
IMO you are doing too many changes in a single patch which makes it
difficult to quickly review. Please keep it simple. Suggest you change a
single function signature per patch. Any related code simplification
should be in a separate patch.
/jeff
