Re: [PATCH] staging: rtl8723bs: remove unnecessary else after break/return

From: Dan Carpenter

Date: Thu Jul 23 2026 - 06:23:52 EST


On Tue, Jul 21, 2026 at 04:02:32PM +0000, ColtenOuO wrote:
> checkpatch.pl flags an else clause following a branch that already
> returns, in enqueue_reorder_recvframe(). Restructure the if/else-if/else
> chain into sequential checks to remove the redundant else, with no
> change in behavior.
>
> Signed-off-by: ColtenOuO <jun930436@xxxxxxxxx>

That's not the name you use to sign legal documents.

But really, the original code is fine, just ignore tools when they are
wrong.

regards,
dan carpenter