Re: [PATCH] staging: rtl8712: rtl871x_security.c: remove unnecessary variable initialization

[Dan Carpenter]

On Fri, Jun 26, 2015 at 05:50:27PM +0200, Luis de Bethencourt wrote:
> On Fri, Jun 26, 2015 at 07:42:24PM +0530, Sunil Shahu wrote:
> > Variable "u32 c" always re-initialize in for loop.
> > Initialized value of "u32 c" is not used in function
> > and is redundant, hence removed.
> > 
> > Suggested-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> > Signed-off-by: Sunil Shahu <shshahu@xxxxxxxxx>
> > ---
> >  drivers/staging/rtl8712/rtl871x_security.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/drivers/staging/rtl8712/rtl871x_security.c b/drivers/staging/rtl8712/rtl871x_security.c
> > index 21465c9..8627928 100644
> > --- a/drivers/staging/rtl8712/rtl871x_security.c
> > +++ b/drivers/staging/rtl8712/rtl871x_security.c
> > @@ -125,7 +125,7 @@ static u8 crc32_reverseBit(u8 data)
> >  static void crc32_init(void)
> >  {
> >  	sint i, j;
> > -	u32 c = 0x12340000;
> > +	u32 c;
> >  	u8 *p = (u8 *)&c, *p1;
> >  	u8 k;
> >  
> > -- 
> > 1.9.1
> 
> The patch is good, the value assigned to c is immediately overwritten.
> 
> Unfortunately the patch fails to apply in staging-testing and linux-next. Could
> you please rebase?

It's based on earlier patches on the list.

Normally only Greg tests that patches apply and it will for him because
he applies them as the order they arrive.  But we should probably start
putting notes on the patches which earlier patches are required...

regards,
dan carpenter

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/