- 13 Sep, 2015 40 commits
-
-
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 <shraddha.6596@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
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 <sudip@vectorindia.org> 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:
-
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 error was detected using checkpatch.pl. Coccinelle was used to make the change. Signed-off-by:
-
Anish Bhatt authored
ETH_ALEN/ETH_HLEN can be used instead of WLAN_ETHADDR_LEN & WLAN_ETHHDR_LEN, replace directly or use eth_addr* functions where applicable. Signed-off-by:
-
Nayeemahmed Badebade authored
Fixed coding style issues where kernel types u16,u64,u32 should be preferred over uint16_t,uint64_t,uint32_t Signed-off-by:
Nayeemahmed Badebade <itachi.opsrc@gmail.com> Signed-off-by:
-
Ravi Teja Darbha authored
struct nbu2ss_ep *ep parameter in functions _nbu2ss_ep0_in_transfer() and _nbu2ss_ep0_out_transfer() is not used anywhere inside. Hence, removed. Signed-off-by:
Ravi Teja Darbha <ravi2j@gmail.com> Signed-off-by:
-
Ravi Teja Darbha authored
The cleanup1 label does nothing but return. Better way is to return immediately instead of using goto. Hence, removed. Signed-off-by:
-
Ravi Teja Darbha authored
Fixed NULL comparison style as suggested by checkpatch.pl Signed-off-by:
-
Ravi Teja Darbha authored
Fixed multiple blank lines warning by checkpatch.pl Signed-off-by:
-
Ravi Teja Darbha authored
FSF mailing address is no longer required to be specified. Hence removed. Signed-off-by:
-
Sudip Mukherjee authored
The member qc_idx of struct if_quotactl is unsigned and hence it can never be less than zero. Signed-off-by:
-
Sudip Mukherjee authored
If lov_sub_get() fails then it returns the error code in ERR_PTR, but here we were dereferencing sub without checking if lov_sub_get() has actually succeeded or not. And on error we can directly return the error code from lov_io_fault_start() as it return 0 on success and the error code on error. Signed-off-by:
-
Anders Fridlund authored
Remove sparse warning "symbol 'sptlrpc_plain_init' was not declared" by including ptlrpc_internal.h, which includes sptlrpc_plain_init. Signed-off-by:
Anders Fridlund <anders.fridlund@gmail.com> Signed-off-by:
-
Mike Rapoport authored
Fix the following checkpatch warning: drivers/staging/lustre/lustre/include/linux/lustre_lite.h:46: WARNING:INCLUDE_LINUX: Use #include <linux/statfs.h> instead of <asm/statfs.h> Signed-off-by:
Mike Rapoport <mike.rapoport@gmail.com> Signed-off-by:
-
Matthew Needes authored
Fixed sparse warning by removing unused variable "proc_version". Signed-off-by:
Matthew Needes <mneedes@gmail.com> 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:
-
