Re: [PATCH] staging: rtl8723bs: replace magic numbers with named constants
From: Dan Carpenter
Date: Thu Jun 04 2026 - 10:46:15 EST
On Thu, Jun 04, 2026 at 05:31:56PM +0300, Dan Carpenter wrote:
> On Wed, Jun 03, 2026 at 03:31:27PM +0100, Jad Keskes wrote:
> > Remove the magic numbers in rtl8723b_InitBeaconParameters() as requested
> > by the outstanding TODO comment. Replace 0x6404 and 0x660F with named
> > constants defined in rtl8723b_hal.h alongside the existing beacon timing
> > constants.
> >
> > Signed-off-by: Jad Keskes <inasj268@xxxxxxxxx>
>
> Just having the same define but as a goto doesn't add any information
> or value.
Ugh... Sorry, I meant to say "just having the same value but as a define
doesn't add any information."
regards,
dan carpenter