- 13 Oct, 2015 40 commits
-
-
Ksenija Stanojevic authored
Braces in single statement blocks are not needed. Found by checkpatch.pl Signed-off-by: Ksenija Stanojevic<ksenija.stanojevic@gmail.com> Signed-off-by:Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Parshuram Thombare authored
Signed-off-by:
Parshuram Thombare <thombarepr@yahoo.com> Signed-off-by:
-
Sudip Mukherjee authored
The IO configuration was not used to configure the board. It was only read from the configuration file. Stop reading it and also remove the other related variables defined for it. Signed-off-by:
Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by:
-
Sudip Mukherjee authored
checkpatch was warning us about extra unneeded parenthesis. Signed-off-by:
-
Sudip Mukherjee authored
checkpatch was warning us that the alignment should match the open parenthesis. Signed-off-by:
-
Sudip Mukherjee authored
Space is not necessary after typecast. Signed-off-by:
-
Sudip Mukherjee authored
Blank lines are not needed after opening braces. checkpatch was giving us warnings about this. Signed-off-by:
-
Sudip Mukherjee authored
Blank lines are not needed before closing braces. checkpatch was giving warning about this. Signed-off-by:
-
Sudip Mukherjee authored
checkpatch warns us about multiple blank lines which are not needed. Remove them. Signed-off-by:
-
Alison Schofield authored
Move open braces to be in line with "if" control flow statements. Addresses checkpatch.pl: ERROR: that open brace { should be on the previous line Signed-off-by:Alison Schofield <amsfield22@gmail.com> Signed-off-by:
-
Shraddha Barke authored
kfree on NULL pointer is a no-op. The semantic patch used - // <smpl> @@ expression E; @@ - if (E != NULL) { kfree(E); } + kfree(E); @@ expression E; @@ - if (E != NULL) { kfree(E); E = NULL; } + kfree(E); + E = NULL; // </smpl>smpl> Signed-off-by:Shraddha Barke <shraddha.6596@gmail.com> Signed-off-by:
-
Shraddha Barke authored
kfree on NULL pointer is a no-op. The semantic patch used- // <smpl> @@ expression E; @@ - if (E != NULL) { kfree(E); } + kfree(E); @@ expression E; @@ - if (E != NULL) { kfree(E); E = NULL; } + kfree(E); + E = NULL; // </smpl>smpl> Signed-off-by: -
Shraddha Barke authored
Replace kmalloc and memcpy with kmemdup. Problem found using coccicheck Signed-off-by:
-
Alison Schofield authored
Remove excess braces as suggested by checkpatch.pl: WARNING: braces {} are not necessary for single statement blocks Signed-off-by: -
Stanislav Kholmanskikh authored
A check using 'sparse' shows warnings in linux_wlan_spi.c: drivers/staging/wilc1000/linux_wlan_spi.c:43:19: warning: symbol 'wilc_spi_dev' was not declared. Should it be static? drivers/staging/wilc1000/linux_wlan_spi.c:71:19: warning: symbol 'wilc_bus' was not declared. Should it be static? drivers/staging/wilc1000/linux_wlan_spi.c:95:5: warning: symbol 'linux_spi_init' was not declared. Should it be static? drivers/staging/wilc1000/linux_wlan_spi.c:195:5: warning: symbol 'linux_spi_write' was not declared. Should it be static? drivers/staging/wilc1000/linux_wlan_spi.c:320:5: warning: symbol 'linux_spi_read' was not declared. Should it be static? drivers/staging/wilc1000/linux_wlan_spi.c:365:5: warning: symbol 'linux_spi_write_read' was not declared. Should it be static? drivers/staging/wilc1000/linux_wlan_spi.c:402:5: warning: symbol 'linux_spi_set_max_speed' was not declared. Should it be static? Let's avoid it by including "linux_wlan_spi.h" header. Signed-off-by:
Stanislav Kholmanskikh <kholmanskikh.s.s@gmail.com> Signed-off-by:
-
Stanislav Kholmanskikh authored
linux_spi_write(), linux_spi_read(), linux_spi_write_read(): nwi->io_func.u.spi.spi_tx = linux_spi_write; nwi->io_func.u.spi.spi_rx = linux_spi_read; nwi->io_func.u.spi.spi_trx = linux_spi_write_read; are expected to accept arguments of 'u8 *', u32 types: struct { int (*spi_max_speed)(void); int (*spi_tx)(u8 *, u32); int (*spi_rx)(u8 *, u32); int (*spi_trx)(u8 *, u8 *, u32); } spi; However, linux_spi_read() and linux_spi_write_read() do not do this, they use 'unsigned char *' and 'unsigned long' instead. Changed the types of their arguments to satisfy the expectations. Signed-off-by: -
Tony Cho authored
This patch renames tenuAuth_type of struct host_if_wep_attr to auth_type to avoid CamelCase naming convention. Signed-off-by:
Tony Cho <tony.cho@atmel.com> Signed-off-by:
-
Tony Cho authored
This patch renames u8mode of struct host_if_wep_attr to mode to avoid CamelCase naming convention. Signed-off-by:
-
Tony Cho authored
This patch renames u8Wepidx of struct host_if_wep_attr to index to avoid CamelCase naming convention. Signed-off-by:
-
Tony Cho authored
This patch renames u8WepKeylen of struct host_if_wep_attr to key_len to avoid CamelCase naming convention. Signed-off-by:
-
Tony Cho authored
This patch renames pu8WepKey of struct host_if_wep_attr to key in order to avoid CamelCase naming convention. Signed-off-by:
-
Tony Cho authored
This patch renames strHostIFpmkidAttr of union host_if_key_attr to pmkid to avoid CamelCase naming convention. Signed-off-by:
-
Tony Cho authored
This patch renames strHostIFwpaAttr of union host_if_key_attr to wpa to avoid CamelCase naming convention. Signed-off-by:
-
Tony Cho authored
This patch renames strHostIFwepAttr of union host_if_key_attr to wep to avoid CamelCase naming convention. Signed-off-by:
-
Tony Cho authored
This patch renames u32SetCfgFlag of struct cfg_param_val to flag to avoid CamelCase naming convention. Signed-off-by:
-
Tony Cho authored
This patch renames a variable of struct cfg_param_val in struct cfg_param_attr, pstrCfgParamVal to cfg_attr_info to avoid CamelCase naming convention. Signed-off-by:
-
Tony Cho authored
This patch replaces the variable names of struct host_if_drv used as the functions' input parameter, drvHandler and hWFIDrv with hif_drv. In addition, the local variable declared in many functions, pstrWFIDrv is removed and hif_drv is directly used. A debug message printing pstrWFIDrv is deleted while removing the local variable because it is not useful as well. Signed-off-by:
-
Tony Cho authored
This patch removes "wilc_wlan.c" from the wilc_wfi_cfgoperation.c file and adds wilc_wlan.o into Makefile to compile it because there is few benefits. This patch also adds "wilc_wfi_netdevice.h" in the wilc_wlan.c file to avoid the compile errors. Signed-off-by:
-
Tony Cho authored
This patch removes unused enum tenuWILC_StaFlag from the host_interface.h. Signed-off-by:
-
Leo Kim authored
This patch removes typedef from the enum tenuScanConnTimer and renames it to scan_conn_timer. Signed-off-by:
Leo Kim <leo.kim@atmel.com> Signed-off-by:
-
Leo Kim authored
This patch removes typedef from the enum tenuConnDisconnEvent and renames it to conn_event. Signed-off-by:
-
Leo Kim authored
This patch rename typedef from the enum tenuCfgParamand rename it to cfg_param. Signed-off-by:
-
Leo Kim authored
This patch removes typedef from the enum tenuScanEvent and rename it to scan_event. Signed-off-by:
-
Leo Kim authored
This patch removes typedef from the struct tstrStatistics and rename it to rf_info. Signed-off-by:
-
Leo Kim authored
This patch removes unused the struct cfg_param_t. Signed-off-by:
-
Leo Kim authored
This patch removes unused the struct WILC_WFIDrvHandle with related commnets. Signed-off-by:
-
Leo Kim authored
This patch removes typedef from the struct tstrHiddenNetworkInfo with related comments and renames it to hidden_net_info. Signed-off-by:
-
Leo Kim authored
This patch removes typedef from the struct tstrFoundNetworkInfo and rename it to found_net_info. Signed-off-by:
-
Leo Kim authored
This patch removes typedef from the enum tenuHostIFstate and rename it to host_if_state. Signed-off-by:
-
Leo Kim authored
This patch removes typedef from the struct tstrWILC_UsrConnReq with the related comments and renames it to user_conn_req. Signed-off-by:
-
