summaryrefslogtreecommitdiffstats
path: root/drivers/staging/wilc1000/host_interface.c
Commit message (Collapse)AuthorAgeFilesLines
* staging: wilc1000: fix build warning with setup_timer()Greg Kroah-Hartman2015-08-171-1/+2
| | | | | | | | | | | I forgot to properly cast a pointer to unsigned long in a conversion to setup_timer() which resulted in a build warning. Fix that up. Cc: Johnny Kim <johnny.kim@atmel.com> Cc: Rachel Kim <rachel.kim@atmel.com> Cc: Dean Lee <dean.lee@atmel.com> Cc: Chris Park <chris.park@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove void function return statements that are not usefulRaphaël Beamonte2015-08-171-4/+0
| | | | | Signed-off-by: Raphaël Beamonte <raphael.beamonte@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove unnecessary void pointer castChaehyun Lim2015-08-171-39/+39
| | | | | | | This patch removes unnecessary void pointer cast of WILC_MALLOC. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove unneeded tstrWILC_MsgQueueAttrs typedefGreg Kroah-Hartman2015-08-171-57/+53
| | | | | | | | | | | No one uses it, so remove it and all of the NULL parameters being used to pass it into the msg code. Cc: Johnny Kim <johnny.kim@atmel.com> Cc: Rachel Kim <rachel.kim@atmel.com> Cc: Dean Lee <dean.lee@atmel.com> Cc: Chris Park <chris.park@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove WILC_TimerStart()Greg Kroah-Hartman2015-08-171-9/+16
| | | | | | | | | | | It was a wrapper around mod_timer() so replace it with the real timer call and remove wilc_timer.c as it's now empty. Cc: Johnny Kim <johnny.kim@atmel.com> Cc: Rachel Kim <rachel.kim@atmel.com> Cc: Dean Lee <dean.lee@atmel.com> Cc: Chris Park <chris.park@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove WILC_TimerCreate()Greg Kroah-Hartman2015-08-141-30/+12
| | | | | | | | | | | | It was just a wrapper around setup_timer() and could never fail, so just call the real function, and fix up the function arguments of the callbacks to be proper timer callback functions. Cc: Johnny Kim <johnny.kim@atmel.com> Cc: Rachel Kim <rachel.kim@atmel.com> Cc: Dean Lee <dean.lee@atmel.com> Cc: Chris Park <chris.park@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove WILC_TimerDestroy()Greg Kroah-Hartman2015-08-141-8/+8
| | | | | | | | | | It was just a wrapper around del_timer_sync() so call that instead. Cc: Johnny Kim <johnny.kim@atmel.com> Cc: Rachel Kim <rachel.kim@atmel.com> Cc: Dean Lee <dean.lee@atmel.com> Cc: Chris Park <chris.park@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove WILC_TimerStop()Greg Kroah-Hartman2015-08-141-10/+10
| | | | | | | | | | It was just a wrapper around del_timer() so call that instead. Cc: Johnny Kim <johnny.kim@atmel.com> Cc: Rachel Kim <rachel.kim@atmel.com> Cc: Dean Lee <dean.lee@atmel.com> Cc: Chris Park <chris.park@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove tstrWILC_TimerAttrs typedefGreg Kroah-Hartman2015-08-141-28/+28
| | | | | | | | | | | It was not used for anything, so remove it, and the variables in wilc_timer.c that were being passed of its type. Cc: Johnny Kim <johnny.kim@atmel.com> Cc: Rachel Kim <rachel.kim@atmel.com> Cc: Dean Lee <dean.lee@atmel.com> Cc: Chris Park <chris.park@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove WILC_TimerHandle typedefGreg Kroah-Hartman2015-08-141-2/+2
| | | | | | | | | | | Use the proper structure (struct timer_list) instead, which makes things much more readable. Cc: Johnny Kim <johnny.kim@atmel.com> Cc: Rachel Kim <rachel.kim@atmel.com> Cc: Dean Lee <dean.lee@atmel.com> Cc: Chris Park <chris.park@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove WILC_Sleep()Greg Kroah-Hartman2015-08-141-2/+2
| | | | | | | | | | | It was just a wrapper around usleep_range() so call that directly instead and remove the now-empty file. Cc: Johnny Kim <johnny.kim@atmel.com> Cc: Rachel Kim <rachel.kim@atmel.com> Cc: Dean Lee <dean.lee@atmel.com> Cc: Chris Park <chris.park@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove commented out WILC_Sleep callsGreg Kroah-Hartman2015-08-141-2/+0
| | | | | | | | | | It's not being called, so delete these lines. Cc: Johnny Kim <johnny.kim@atmel.com> Cc: Rachel Kim <rachel.kim@atmel.com> Cc: Dean Lee <dean.lee@atmel.com> Cc: Chris Park <chris.park@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove WILC_FREE_IF_TRUE macroGreg Kroah-Hartman2015-08-141-10/+10
| | | | | | | | | | Just use kfree, as that's all it resolves itself to. Cc: Johnny Kim <johnny.kim@atmel.com> Cc: Rachel Kim <rachel.kim@atmel.com> Cc: Dean Lee <dean.lee@atmel.com> Cc: Chris Park <chris.park@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: clarify the argument typeJohnny Kim2015-08-141-39/+39
| | | | | | | | | | | | This patch replaces the void pointer type in the host interface functions which process the message from host thread by the real data type, tstrWILC_WFIDrv because the void pointer type as the arguments is not clear and concise. In addition, typecasting to the void pointer type is removed becasue it is not necessary. Signed-off-by: Johnny Kim <johnny.kim@atmel.com> Signed-off-by: Tony Cho <tony.cho@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: change void pointer type to real typeJohnny Kim2015-08-141-1/+1
| | | | | | | | | | This patch changes the void pointer member of the tstrHostIFmsg to the real data type because the void pointer type is ambiguous and not readable. Signed-off-by: Johnny Kim <johnny.kim@atmel.com> Signed-off-by: Tony Cho <tony.cho@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: replace WILC_WFIDrvHandle by tstrWILC_WFIDrvJohnny Kim2015-08-141-63/+63
| | | | | | | | | | | | | | | | The structure, WILC_WFIDrvHandle is used to save the pointer address for the driver handler which is used throughout the driver but it's not easy to understand what it means. In addition, it doesn't support the 64 bit machine and also causes the warnings for the 64 bit build. This patch replaces the WILC_WFIDrvHandle by the tstrWILC_WFIDrv because the tstrWILC_WFIDrv is real structure to represent the driver handler and reduces the 64 bit compile warnings. Also, typecasting to WILC_WFIDrvHandle is not needed by using tstrWILC_WFIDrv as is. Signed-off-by: Johnny Kim <johnny.kim@atmel.com> Signed-off-by: Tony Cho <tony.cho@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: use kfree instead of WILC_FREEChaehyun Lim2015-08-141-56/+56
| | | | | | | Use kfree instead of WILC_FREE. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: use memcpy instead of WILC_memcpyChaehyun Lim2015-08-141-100/+100
| | | | | | | Use memcpy instead of WILC_memcpy that is a custom function. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove commented codeChaehyun Lim2015-08-141-2/+0
| | | | | | | Remove commented code that is not used. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: Remove braces for single statement 'if' and 'else'Chandra S Gorentla2015-08-081-11/+8
| | | | | | | | Fixes the checkpatch.pl warning - braces {} are not necessary for any arm of this statement Signed-off-by: Chandra S Gorentla <csgorentla@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: Remove unused extern declarationsChandra S Gorentla2015-08-081-1/+0
| | | | | | | | 'extern' declarations which are not referenced within the file are removed. Signed-off-by: Chandra S Gorentla <csgorentla@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: use memcmp instead of WILC_memcmpChaehyun Lim2015-08-071-10/+10
| | | | | | | Use memcmp instead of WILC_memcmp. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: use memset instead of WILC_memsetChaehyun Lim2015-08-071-67/+67
| | | | | | | Use memset instead of WILC_memset. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove dead codes related to SIMULATIONTony Cho2015-07-291-23/+0
| | | | | | | | | | This patch removes preprocessor conditionals (#ifdef or #ifndef) related to SIMULATION definition from the codes becasue that SIMULATION feature is not used anymore. Signed-off-by: Tony Cho <tony.cho@atmel.com> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* wilc1000: host_interface.c: global variables do not need to be explicitly ↵Daniel Machon2015-07-061-2/+2
| | | | | | | | | | | initialized to 0 or NULL. Fixed explicit initialization of global pointer variable. GCC takes care of this implicitly. Signed-off-by: Daniel Machon <dmachon.dev@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: switch printks to vsprintf IPv4 extensionLuis de Bethencourt2015-07-061-3/+3
| | | | | | | | Switch printks with IP addresses to use vsprintf extension %pI4. Suggested-by: Joe Perches <joe@perches.com> Signed-off-by: Luis de Bethencourt <luis@debethencourt.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: fix typos in PRINT_ERR()Luis de Bethencourt2015-07-061-2/+2
| | | | | | | Fix typo "packe" to "packet". Signed-off-by: Luis de Bethencourt <luis@debethencourt.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove whitespaces before quoted newlinesLuis de Bethencourt2015-07-061-42/+42
| | | | | | | | Fix all checkpatch.pl warnings: WARNING: unnecessary whitespace before a quoted newline Signed-off-by: Luis de Bethencourt <luis@debethencourt.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: add blank lines after declarationsLuis de Bethencourt2015-07-061-0/+16
| | | | | | | | Fix all checkpatch.pl warnings: WARNING: Missing a blank line after declarations Signed-off-by: Luis de Bethencourt <luis@debethencourt.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove unnecessary bracesLuis de Bethencourt2015-07-061-184/+92
| | | | | | | | Removing all checkpatch.pl warnings: WARNING: braces {} are not necessary for single statement blocks Signed-off-by: Luis de Bethencourt <luis@debethencourt.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: host_interface: add spaces around '='Sunil Shahu2015-07-061-2/+2
| | | | | | | | | Fix coding style error by placing spaces around '=' as suggested by checkpatch.pl script. Signed-off-by: Sunil Shahu <shshahu@gmail.com> Reviewed-by: Luis de Bethencourt <luis@debethencourt.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove unnecessary typecast in #define macroChaehyun Lim2015-06-181-39/+39
| | | | | | | Remove unnecessary typecast in #define macro. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: align #define macroChaehyun Lim2015-06-181-47/+46
| | | | | | | Align #define macro. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove commented codeChaehyun Lim2015-06-181-76/+0
| | | | | | | Remove the commented codes using #if 0. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: wilc1000: Assign proper boolean valueAbdul Hussain2015-06-161-1/+1
| | | | | | | This patch assign proper boolean value to boolean variable. Signed-off-by: Abdul Hussain <habdul@visteon.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: rework include wilc_oswrapper.hDean Lee2015-06-161-1/+0
| | | | | | | | rework line '#include "wilc_oswrapper.h"' it does not used anywhere after change own data type to common data type. Signed-off-by: Dean Lee <dean.lee@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: change WILC_Char to charDean Lee2015-06-151-44/+44
| | | | | | | change own data type(WILC_Char) to common data type(char) Signed-off-by: Dean Lee <dean.lee@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove itypes.hChaehyun Lim2015-06-131-1/+0
| | | | | | | Remove itypes.h that is not needed. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove BOOL_T typedefChaehyun Lim2015-06-131-3/+3
| | | | | | | Remove BOOL_T typedef. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: change WILC_BOOL to boolDean Lee2015-06-111-83/+83
| | | | | | | | change own data type(WILC_BOOL) to common data type(bool) but that's contain true/false value. so change with them. Signed-off-by: Dean Lee <dean.lee@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: modify odd print messageDean Lee2015-06-111-2/+1
| | | | | | | | This driver has odd message in print string. So this patch removes the data type. Signed-off-by: Dean Lee <dean.lee@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove WILC_Sint32Chaehyun Lim2015-06-111-181/+181
| | | | | | | Use s32 instead of WILC_Sint32. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove WILC_Sint8Chaehyun Lim2015-06-111-74/+74
| | | | | | | Use s8 instead of WILC_Sint8. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove WILC_Uint32Chaehyun Lim2015-06-111-133/+133
| | | | | | | Use u32 instead of WILC_Uint32. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove WILC_Uint16Chaehyun Lim2015-06-111-151/+151
| | | | | | | Use u16 instead of WILC_Uint16. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove UWORD8Chaehyun Lim2015-06-111-1/+1
| | | | | | | Use u8 instead of UWORD8. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: modify printk formatJohnny Kim2015-06-101-4/+4
| | | | | | | This remove compile warnings about printk format. Signed-off-by: Johnny Kim <johnny.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: remove WILC_Uint8Greg Kroah-Hartman2015-06-021-219/+219
| | | | | | | | | | Just use u8, as that's what you really want in a kernel driver. Cc: Johnny Kim <johnny.kim@atmel.com> Cc: Rachel Kim <rachel.kim@atmel.com> Cc: Dean Lee <dean.lee@atmel.com> Cc: Chris Park <chris.park@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc100: remove WILC_NULL usageGreg Kroah-Hartman2015-06-021-149/+149
| | | | | | | | | | | Use the "real" NULL value, don't try to be cute and define your own value for something that the compiler obviously supports. Cc: Johnny Kim <johnny.kim@atmel.com> Cc: Rachel Kim <rachel.kim@atmel.com> Cc: Dean Lee <dean.lee@atmel.com> Cc: Chris Park <chris.park@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wilc1000: fix const cast warningsArnd Bergmann2015-06-021-15/+17
| | | | | | | | | | | | | The wilc1000 driver produces a lot of warnings about invalid casts between const and non-const variables. This reworks the code to avoid all those warnings, by marking variables and function arguments const. A lot of the types use WILC_Uint8, I change them to const u8 for style reasons, as I'm touching them anyway. Signed-off-by: Arnd Bergmann <arnd@arndb.de> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
OpenPOWER on IntegriCloud