On Sat, 18 Feb 2017, Joe Perches wrote: > On Sat, 2017-02-18 at 20:12 +0100, Julia Lawall wrote: > > On Sun, 19 Feb 2017, simran singhal wrote: > > > Extra parentheses were causing checkpatch issues > > > and were removed. > [] > > > diff --git a/drivers/staging/rtl8192e/rtl819x_HTProc.c b/drivers/staging/rtl8192e/rtl819x_HTProc.c > [] > > > @@ -663,13 +663,13 @@ void HTInitializeHTInfo(struct rtllib_device *ieee) > > > pHTInfo->CurrentMPDUDensity = pHTInfo->MPDU_Density; > > > pHTInfo->CurrentAMPDUFactor = pHTInfo->AMPDU_Factor; > > > > > > - memset((void *)(&(pHTInfo->SelfHTCap)), 0, > > > + memset((void *)(&pHTInfo->SelfHTCap), 0, > > > > You don't need the parens around the argument of the cast either. > > Nor need the cast at all. > > memset(&pHTInfo->SelfHTCap, 0, sizeof(pHTInfo->SelfHTCap)); > > etc... Indeed. Thanks! julia _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
