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