Re: [PATCH v2 1/2] staging: rtl8723bs: Improve code readability

[Erick Karanja]

On Wed, 2025-04-09 at 14:11 +0300, Dan Carpenter wrote:
> On Wed, Apr 09, 2025 at 07:04:18AM -0400, Julia Lawall wrote:
> > > @@ -729,10 +721,9 @@ static void hal_ReadEFuse_WiFi(
> > >  		}
> > > 
> > >  		if (offset < EFUSE_MAX_SECTION_8723B) {
> > > -			u16 addr;
> > > +			u16 addr = offset * PGPKT_DATA_SIZE;
> > >  			/*  Get word enable value from PG header
> > > */
> > > 
> > > -			addr = offset * PGPKT_DATA_SIZE;
> > 
> > I'm not sure about this one, due to the comment.
> > 
> > julia
> > 
> 
> I feel like it works.  I wish there were a blank line after the
> declaration but that's something for a different patch.  Both of
> these are okay with me.  Try to be more creative with the subjects
> next time.  (Just put the file name in the subject because that's
> the only difference).
Thank you for the review.
Erick
> 
> Reviewed-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> 
> regards,
> dan carpenter
>