On Sat, 2018-12-15 at 17:46 +0100, Michael Straube wrote: > Replace tabs with spaces and/or remove spaces in declarations. > Remove unused/commented declarations, remove unnecessary comment, > remove blank lines between declarations and add missing lines after > declarations. Also clears some line over 80 characters checkpatch > warnings. [] > diff --git a/drivers/staging/rtl8188eu/core/rtw_mlme_ext.c b/drivers/staging/rtl8188eu/core/rtw_mlme_ext.c [] > @@ -22,14 +22,14 @@ static u8 null_addr[ETH_ALEN] = {0, 0, 0, 0, 0, 0}; > /************************************************** > OUI definitions for the vendor specific IE > ***************************************************/ > -unsigned char RTW_WPA_OUI[] = {0x00, 0x50, 0xf2, 0x01}; > +unsigned char RTW_WPA_OUI[] = {0x00, 0x50, 0xf2, 0x01}; > unsigned char WMM_OUI[] = {0x00, 0x50, 0xf2, 0x02}; > -unsigned char WPS_OUI[] = {0x00, 0x50, 0xf2, 0x04}; > -unsigned char P2P_OUI[] = {0x50, 0x6F, 0x9A, 0x09}; > -unsigned char WFD_OUI[] = {0x50, 0x6F, 0x9A, 0x0A}; > +unsigned char WPS_OUI[] = {0x00, 0x50, 0xf2, 0x04}; > +unsigned char P2P_OUI[] = {0x50, 0x6F, 0x9A, 0x09}; > +unsigned char WFD_OUI[] = {0x50, 0x6F, 0x9A, 0x0A}; > > -unsigned char WMM_INFO_OUI[] = {0x00, 0x50, 0xf2, 0x02, 0x00, 0x01}; > -unsigned char WMM_PARA_OUI[] = {0x00, 0x50, 0xf2, 0x02, 0x01, 0x01}; > +unsigned char WMM_INFO_OUI[] = {0x00, 0x50, 0xf2, 0x02, 0x00, 0x01}; > +unsigned char WMM_PARA_OUI[] = {0x00, 0x50, 0xf2, 0x02, 0x01, 0x01}; > > unsigned char WPA_TKIP_CIPHER[4] = {0x00, 0x50, 0xf2, 0x02}; > unsigned char RSN_TKIP_CIPHER[4] = {0x00, 0x0f, 0xac, 0x02}; unrelated trivia: Might be nice to make some or all of these const or even static const where appropriate. _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel