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... _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
