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 <dmanti...@yandex.ru>
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 _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k
