- 13 Sep, 2015 40 commits
-
-
Shraddha Barke authored
Remove explicit NULL comparison and write it in its simpler form. Replacement done with coccinelle: @replace_rule@ expression e; @@ -e == NULL + !e Signed-off-by:
Shraddha Barke <shraddha.6596@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Shraddha Barke authored
Remove explicit NULL comparison and write it in its simpler form. Replacement done with coccinelle: @replace_rule@ expression e; @@ -e == NULL + !e Signed-off-by:
-
Shraddha Barke authored
This patch replaces bit shifting on 1 with the BIT(x) macro This was done with coccinelle: @@ int g; @@ -(1 << g) +BIT(g) Signed-off-by:
-
mike dupuis authored
This is a style change to remove whitespace before close parentheses in ieee80211_softmac_wx.c Signed-off-by:
Mike Dupuis <mike.dupuis.0@gmail.com> Signed-off-by:
-
mike dupuis authored
This is a style change to insert spaces before closed braces in ieee80211_softmac_wx.c Signed-off-by:
-
mike dupuis authored
This is a change to add spaces before open braces to ieee80211_softmac_wx.c. Signed-off-by:
-
mike dupuis authored
This is a change to ieee80211_softmac_wx.c to remove spaces after unary & operators. Signed-off-by:
-
Shraddha Barke authored
Staging: rtl8192u: ieee80211: ieee80211_softmac.c: Replace "#include <asm/uaccess>" with "#include <linux/uaccess>" Fix checkpatch.pl warning "Use #include <linux/uaccess.h> instead of <asm/uaccess.h>" Signed-off-by:
-
Shraddha Barke authored
Staging: rtl8192u: ieee80211: ieee80211_module.c: Replace "#include <asm/uaccess>" with "#include <linux/uaccess>" Fix checkpatch.pl warning "Use #include <linux/uaccess.h> instead of <asm/uaccess.h>" Signed-off-by:
-
Shraddha Barke authored
Fix checkpatch.pl warning "Use #include <linux/uaccess.h> instead of <asm/uaccess.h>" Signed-off-by:
-
Alexander Kuleshov authored
The <linux/string.h> provides strpbrk() function that does the same that the dgap_sindex(). Let's use already defined function instead of writing custom. Signed-off-by:
Alexander Kuleshov <kuleshovmail@gmail.com> Signed-off-by:
-
Junesung Lee authored
'syncrhronization' is wrong spell. 'synchronization' is correct. Signed-off-by:
Junesung Lee <junesoung412@gmail.com> Signed-off-by:
-
Sudip Mukherjee authored
These variables were assigned some values but they were never used. Removed them but kept one call to ioread8() as it might affect the hardware. Signed-off-by:
Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by:
-
Sudip Mukherjee authored
The return pointer from dgap_getword() is used in strcmp() where it is dereferenced. But dgap_getword() can return NULL. Lets put a check there and return 0 as error. Signed-off-by:
-
Sakshi Vaid authored
Removed the space between function name and open parenthesis '('. Signed-off-by:Sakshi Vaid <sakshivaid95@gmail.com> Signed-off-by:
-
Shraddha Barke authored
This patch replaces bit shifting on 1 with the BIT(x) macro This was done with coccinelle: @@ int g; @@ -(1 << g) +BIT(g) Signed-off-by:
-
Shraddha Barke authored
This patch replaces bit shifting on 1 with the BIT(x) macro This was done with coccinelle: @@ int g; @@ -(1 << g) +BIT(g) Signed-off-by:
-
Anish Bhatt authored
The BIT() macro is already defined in bitops.h, remove duplicate definitions. Signed-off-by:
Anish Bhatt <anish@chelsio.com> Signed-off-by:
-
Anish Bhatt authored
WLAN_ETHHDR_LEN/WLAN_ETHADDR_LEN/WLAN_ADDR_LEN are unused, duplicate or unnecessary, remove. Signed-off-by:
-
Anish Bhatt authored
WLAN_ETHHDR_LEN/WLAN_ETHADDR_LEN/WLAN_ADDR_LEN are unused, duplicate or unnecessary, remove. Signed-off-by:
-
Shraddha Barke authored
This patch replaces bit shifting on 1 with the BIT(x) macro. This was done with coccinelle: @@ int g; @@ -(1 << g) +BIT(g) Signed-off-by:
-
Shraddha Barke authored
This patch replaces bit shifting on 1 with the BIT(x) macro This was done with coccinelle: @@ int g; @@ -(1 << g) +BIT(g) Signed-off-by:
-
Shraddha Barke authored
This patch replaces bit shifting on 1 with the BIT(x) macro This was done with coccinelle: @@ int g; @@ -(1 << g) +BIT(g) Signed-off-by:
-
Glen Lee authored
This patch removes following unused funtions which will not be used anymore. ascii_hex_to_dec get_hex_char extract_mac_addr create_mac_addr conv_ip_to_int conv_int_to_ip get_wid_type get_type CreateConfigPacket ConfigWaitResponse ConfigProvideResponse ConfigPktReceived ParseWriteResponse CreatePacketHeader ParseResponse ProcessBinWid ProcessAdrWid ProcessStrWid ProcessIPwid ProcessIntWid ProcessShortWid ProcessCharWid As a result, four global variable causes defined but not used compile warnings. So just remove unused varialbs g_seqno, g_wid_num, Res_Len and g_oper_mode. Signed-off-by:
Glen Lee <glen.lee@atmel.com> Signed-off-by:
Tony Cho <tony.cho@atmel.com> Signed-off-by:
-
Glen Lee authored
This patch removes a unused enum tenuFrameClass. Signed-off-by:
-
Tony Cho authored
This patch removes the followings from the driver because they are in debugging purpose but not used anymore. - int_clrd - int_rcvdU - int_rcvdB - android_wifi_priv_cmd structure Signed-off-by:
-
Tony Cho authored
This patch removes the firmware definitions, STA_FIRMWARE, AP_FIRMWARE, and P2P_CONCURRENCY_FIRMWARE from the linux_wlan.c file because they are defined in the Makefile. Signed-off-by:
-
Tony Cho authored
This patch removes unused definition, STATIC_MACADDRESS from the linux_wlan.c file. Signed-off-by:
-
Shraddha Barke authored
Fix checkpatch.pl warning "Use #include <linux/uaccess.h> instead of <asm/uaccess.h>" Signed-off-by:
-
Glen Lee authored
The macro WILC_MALLOC is not used in the driver anymore, so just delete files. Signed-off-by:
-
Glen Lee authored
This patch calls kmalloc() directly. No need to wrap kmalloc(). All is not holding a spin lock or in interrupt context. So use GFP_KERNEL flag for kmalloc. Signed-off-by:
-
Tony Cho authored
This patch removes BROKEN status from the Kconfig. Signed-off-by:
-
Shraddha Barke authored
Remove the if condition, as the code inside the if condition is commented and does not have any FIXME or TODO comment. Signed-off-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
-
Raphaël Beamonte authored
Prefer ether_addr_copy() over memcpy() if the Ethernet addresses are __aligned(2) The values used are stored as dev_addr in net_device (declared in include/linux/netdevice.h) and sa_data in sockaddr (declared in include/linux/socket.h). Both these elements are u16 aligned as shown by using pahole (position must be a multiple of sizeof(u16)): unsigned char * dev_addr; /* 888 8 */ char sa_data[14]; /* 2 14 */ It is thus safe to use ether_addr_copy() instead of memcpy() for that call, as it is already done in multiple files in the Linux kernel sources: drivers/net/ethernet/broadcom/genet/bcmgenet.c drivers/net/ethernet/brocade/bna/bnad.c drivers/net/ethernet/emulex/benet/be_main.c drivers/net/ethernet/ezchip/nps_enet.c drivers/net/ethernet/ibm/ibmveth.c drivers/net/ethernet/intel/fm10k/fm10k_netdev.c drivers/net/ethernet/intel/i40e/i40e_main.c drivers/net/ethernet/mellanox/mlx5/core/en_main.c drivers/net/usb/lan78xx.c net/8021q/vlan_dev.c net/batman-adv/soft-interface.c net/dsa/slave.c Signed-off-by:
Raphaël Beamonte <raphael.beamonte@gmail.com> Signed-off-by:
-
Sudip Mukherjee authored
checkpatch complains if NULL comparison is done as if (var == NULL) Signed-off-by:
-
Sudip Mukherjee authored
We were successfully requesting the firmware but on error it was not being released. Signed-off-by:
-
Sudip Mukherjee authored
Using magic numbers are not good coding practise. Use FBTFT_GPIO_NAME_SIZE as defined in the header files. Signed-off-by:
-
Sudip Mukherjee authored
Using strcpy() is a security risk as the destination buffer size is not checked and we may over-run the buffer. Use strncpy() instead, while mentioning the buffer size leaving place for the NULL termination. Signed-off-by:
-
Shraddha Barke authored
The linux kernel coding style discourages use of braces for single statement blocks. This patch removes the unnecessary braces. The warning was detected using checkpatch.pl. Coccinelle was used to make the change. Signed-off-by:
-
Shraddha Barke authored
The linux kernel coding style discourages use of braces for single statement blocks. This patch removes the unnecessary braces. The warning was detected using checkpatch.pl. Coccinelle was used to make the change. Signed-off-by:
-
