On Wed, Aug 02, 2023 at 07:07:19PM +0300, Dmitry Antipov wrote: > In 'mwifiex_process_sta_txpd()' and 'mwifiex_process_uap_txpd()', > replace 'BUG_ON()' with runtime check, and move all these checks > to 'mwifiex_process_tx()'. This way, both callees may be converted > to 'void', and the caller may be simplified as well. > > Suggested-by: Brian Norris <briannorris@xxxxxxxxxxxx> > Signed-off-by: Dmitry Antipov <dmantipov@xxxxxxxxx> > --- > v3: drop some overengineered bits, fix a few nits reported by > checkpatch.pl, and reorder series by making this patch the last > one, thus hopefully simplify the landing Thanks, I think this version is pretty clear, concise, and an overall improvement. For the whole series (though you rightly carried my Acked-by on the others already): Acked-by: Brian Norris <briannorris@xxxxxxxxxxxx>
