whitespace error in some of commits

2016-02-15 Thread Tony Cho
Hi Janani, I can find the whitespace errors in your commits which is already accepted in staging-testing branch. The commit number is c611d48e65e25af2dc0176e9ac135116095ed03d and c611d48e65e25af2dc0176e9ac135116095ed03d. I don't know why I couldn't find your patch emails from the community. P

Re: whitespace error in some of commits

2016-02-15 Thread Tony Cho
Hi Janani, Please refer to the commit numbers again in the wilc1000 driver: c611d48e65e25af2dc0176e9ac135116095ed03d 24c6c29d85af3748b2d79b46f72188b335566340 Thanks, Tony. On 2016년 02월 16일 15:47, Tony Cho wrote: Hi Janani, I can find the whitespace errors in your commits which is already

[PATCH 2/2] staging: wilc1000: wilc_wlan.c: remove multiple blank line

2016-02-24 Thread Tony Cho
This patch removes the multiple blank line reported by checkpatch.pl. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_wlan.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/wilc1000/wilc_wlan.c b/drivers/staging/wilc1000/wilc_wlan.c index f142e61..17e39cc 100644

[PATCH 1/2] staging: wilc1000: wilc_wlan.c: remove unnecessary blank lines

2016-02-24 Thread Tony Cho
This patch removes unnecessary blank lines found by checkpatch.pl. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_wlan.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_wlan.c b/drivers/staging/wilc1000/wilc_wlan.c index d50ddc6..f142e61 100644

[PATCH] staging: wilc1000: wilc_wlan_if.h: remove unused functions

2016-02-29 Thread Tony Cho
This patch removes the unused functions anymore related to the configuration of the bus interface clock speed. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_wlan_if.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_wlan_if.h b/drivers/staging

Re: [PATCH V2 1/3] staging: wilc1000: coreconfigurator.c: remove WILC_MALLOC

2015-07-26 Thread tony . cho
Hi, This is not called in IRQ context and not even held in the spin lock. All of the macros in the driver for the Linux APIs will be removed soon. I expect all of concerns will be also removed at that time. Thanks, Tony. On 2015년 07월 24일 09:44, Dan Carpenter wrote: On Fri, Jul 24, 2015 at 08

[PATCH 00/21] cover letter of Atmel WILC1000 patches

2015-07-28 Thread Tony Cho
multiple blank lines staging: wilc1000: alignment should match open parenthesis Kim, Leo (3): staging: wilc1000: remove braces {} for single statement blocks staging: wilc1000: remove warnings on the multiple blank lines uses staging: wilc1000: remove unnecessary spcae Tony Cho (16

[PATCH 04/21] staging: wilc1000: remove warnings on the multiple line uses

2015-07-28 Thread Tony Cho
This patch removes the warnings reported by checkpatch.pl for using multiple blank lines. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_sdio.c | 14 -- 1 file changed, 14 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_sdio.c b/drivers/staging/wilc1000

[PATCH 03/21] staging: wilc1000: remove dead codes related to SIMULATION

2015-07-28 Thread Tony Cho
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 --- drivers/staging/wilc1000/coreconfigurator.c | 86 --- drivers/staging

[PATCH 07/21] staging: wilc1000: remove preprocessor conditionals unused

2015-07-28 Thread Tony Cho
This patch removes unused preprocessor conditionals for the PLAT_AML8726_M3_BACKUP and PLAT_AML8726_M3 which are not used and so dead codes. They are also platform-dependent codes. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_sdio.c | 174 +-- 1 file

[PATCH 01/21] staging: wilc1000: remove unnecessary files

2015-07-28 Thread Tony Cho
This patch removes the following files which are not used anymore. - fifo_buffer.c - fifo_buffer.h - coreconfigsimulator.h - wilc_wfi_netdevice.c Signed-off-by: Robin Hwang Signed-off-by: Tony Cho --- drivers/staging/wilc1000/Makefile | 4

[PATCH 06/21] staging: wilc1000: remove the warnings on unnecessary braces

2015-07-28 Thread Tony Cho
This patch removes the warnings reported by checkpatch.pl on {} not necessary for the single statement blocks. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_sdio.c | 28 ++-- 1 file changed, 10 insertions(+), 18 deletions(-) diff --git a/drivers/staging

[PATCH 02/21] staging: wilc1000: describe the config symbol fully

2015-07-28 Thread Tony Cho
This patch removes the warnings reported by checkpatch.pl on the short description for the config symbol in the Kconfig by adding more comments to describe the config symbol in more detail. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/Kconfig | 28 1 file

[PATCH 05/21] staging: wilc1000: remove the warnings on missing blank line

2015-07-28 Thread Tony Cho
This patch removes the warnings reported by checkpatch.pl on missing a blank line after declaration. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_sdio.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/drivers/staging/wilc1000/wilc_sdio.c b/drivers/staging/wilc1000

[PATCH 14/21] staging: wilc1000: remove warnings on unnecessary braces

2015-07-28 Thread Tony Cho
This patch removes the warnings reported by checkpatch.pl on the braces {} not necessary for any arm of this statement. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_sdio.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_sdio.c

[PATCH 09/21] staging: wilc1000: #ifdef conditionals cover entire functions

2015-07-28 Thread Tony Cho
This patch lets preprocessor conditionals (#ifdef) related to WILC_SDIO_IRQ_GPIO to compile out the entire functions. Compiling out the entire functions is preferred rather than portions of functions or expressions becausue doing so makes code harder to read. Signed-off-by: Tony Cho --- drivers

[PATCH 11/21] staging: wilc1000: remove multiple blank lines

2015-07-28 Thread Tony Cho
From: "Jude.Lee" This patch removes the warnings reported by checkpatch.pl for using multiple blank lines. Signed-off-by: Jude.Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/coreconfigurator.h | 8 1 file changed, 8 deletions(-) diff --git a/drivers/stagin

[PATCH 08/21] staging: wilc1000: remove a dead preprocessor conditionals

2015-07-28 Thread Tony Cho
This patch removes the preprocessor conditionals which are related to the WILC1000_SINGLE_TRANSFER definition becasue this is not used. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_sdio.c | 26 -- 1 file changed, 26 deletions(-) diff --git a/drivers/staging

[PATCH 12/21] staging: wilc1000: alignment should match open parenthesis

2015-07-28 Thread Tony Cho
From: "Jude.Lee" This patch fixes the checks reported by checkpatch.pl alignment should match open parenthesis Signed-off-by: Jude.Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/coreconfigurator.h | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git

[PATCH 15/21] staging: wilc1000: remove warnings on missing blank line

2015-07-28 Thread Tony Cho
This patch removes the warnings reported by checkpatch.pl on missing a blank line after declarations. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_sdio.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/wilc1000/wilc_sdio.c b/drivers/staging/wilc1000/wilc_sdio.c

[PATCH 13/21] staging: wilc1000: remove errors on required space

2015-07-28 Thread Tony Cho
This patch removes the errors reported by checkpatch.pl, which is the space required before the open parenthesis '('. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_sdio.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/wilc

[PATCH 21/21] staging: wilc1000: update TODO list of WILC1000

2015-07-28 Thread Tony Cho
The TODO list is updated for the WILC1000 kernel driver. The TODO list contains many items listed in time sequence. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/TODO | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/drivers/staging/wilc1000/TODO b/drivers

[PATCH 17/21] staging: wilc1000: remove unnecessary blank lines

2015-07-28 Thread Tony Cho
This patch removes the following warnings reported by checkpatch.pl. - Blank lines aren't necessary after an open brace '{' - Blank lines aren't necessary before a close brace '}' Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_sdio.c | 3 --- 1 file

[PATCH 19/21] staging: wilc1000: remove warnings on the multiple blank lines uses

2015-07-28 Thread Tony Cho
From: "Kim, Leo" This patch removes the warnings reported by checkpatch.pl for using multiple blank lines. Signed-off-by: Kim, Leo Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan.c | 69 --- 1 file changed, 69 deletions(-) diff --git

[PATCH 10/21] staging: wilc1000: remove unused functions

2015-07-28 Thread Tony Cho
This patch removes the dead functions which are not used anymore. - sdio_check_bs - sdio_xfer_cnt Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_sdio.c | 56 drivers/staging/wilc1000/wilc_wlan.c | 1 - 2 files changed, 57

[PATCH 20/21] staging: wilc1000: remove unnecessary spcae

2015-07-28 Thread Tony Cho
From: "Kim, Leo" This patch removes the warnings reported by checkpatch.pl on space prohibited between function name and open parenthesis '(' Signed-off-by: Kim, Leo Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan.c | 20 ++-- 1 file ch

[PATCH 18/21] staging: wilc1000: remove braces {} for single statement blocks

2015-07-28 Thread Tony Cho
From: "Kim, Leo" This patch removes the warnings reported by checkpatch.pl on braces {} not necessary for the single statement blocks. Signed-off-by: Kim, Leo Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan.c | 78 --- 1 file c

[PATCH 16/21] staging: wilc1000: remove unnecessary inner braces

2015-07-28 Thread Tony Cho
This patch removes unnecessary inner braces. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_sdio.c | 30 ++ 1 file changed, 14 insertions(+), 16 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_sdio.c b/drivers/staging/wilc1000/wilc_sdio.c index

Re: [PATCH 06/21] staging: wilc1000: remove the warnings on unnecessary braces

2015-07-29 Thread tony . cho
On 2015년 07월 30일 05:50, Greg KH wrote: On Tue, Jul 28, 2015 at 05:47:25PM +0900, Tony Cho wrote: This patch removes the warnings reported by checkpatch.pl on {} not necessary for the single statement blocks. Signed-off-by: Tony Cho Reviewed-by: Dan Carpenter --- drivers/staging/wilc1000

Re: [PATCH 21/21] staging: wilc1000: update TODO list of WILC1000

2015-07-29 Thread tony . cho
On 2015년 07월 30일 06:01, Greg KH wrote: On Tue, Jul 28, 2015 at 05:47:40PM +0900, Tony Cho wrote: The TODO list is updated for the WILC1000 kernel driver. The TODO list contains many items listed in time sequence. I don't understand what you mean by "in time sequence" So

Re: [PATCH 09/21] staging: wilc1000: #ifdef conditionals cover entire functions

2015-07-29 Thread tony . cho
On 2015년 07월 30일 06:14, Dan Carpenter wrote: On Wed, Jul 29, 2015 at 01:54:54PM -0700, Greg KH wrote: On Tue, Jul 28, 2015 at 05:47:28PM +0900, Tony Cho wrote: /** * Disable power sequencer **/ - if (!sdio_read_reg(WILC_MISC, ®)) { + if

Re: [PATCH 09/21] staging: wilc1000: #ifdef conditionals cover entire functions

2015-07-29 Thread tony . cho
On 2015년 07월 30일 06:29, Greg KH wrote: On Thu, Jul 30, 2015 at 12:14:08AM +0300, Dan Carpenter wrote: On Wed, Jul 29, 2015 at 01:54:54PM -0700, Greg KH wrote: On Tue, Jul 28, 2015 at 05:47:28PM +0900, Tony Cho wrote: /** * Disable power sequencer

Re: [PATCH 09/21] staging: wilc1000: #ifdef conditionals cover entire functions

2015-07-29 Thread tony . cho
On 2015년 07월 30일 05:54, Greg KH wrote: On Tue, Jul 28, 2015 at 05:47:28PM +0900, Tony Cho wrote: This patch lets preprocessor conditionals (#ifdef) related to WILC_SDIO_IRQ_GPIO to compile out the entire functions. Compiling out the entire functions is preferred rather than portions of

[PATCH 0/5] 2nd patch for illegal coding style

2015-07-30 Thread Tony Cho
include new fix introduced on the previous patch due to no sync-up issue. The original 13th patch fixed the same warnings but in different place. The 64bit compile warnings will be resolved after sending these patches. Then, ambiguous sdio/spi parts including preprocessor conditionals will be trie

[PATCH V2 2/5] staging: wilc1000: remove unnecessary blank lines

2015-07-30 Thread Tony Cho
This patch removes the following warnings reported by checkpatch.pl. - Blank lines aren't necessary after an open brace '{' - Blank lines aren't necessary before a close brace '}' Signed-off-by: Tony Cho --- V2: This patch doesn't have a difference with th

[PATCH V2 3/5] staging: wilc1000: remove warnings on missing blank line

2015-07-30 Thread Tony Cho
This patch removes the warnings reported by checkpatch.pl on missing a blank line after declarations. Signed-off-by: Tony Cho --- V2: This patch doesn't have a difference with the first one. However, it's resent becasue not accepted with the 9th patch. --- drivers/staging/wilc1000/w

[PATCH V2 1/5] staging: wilc1000: #ifdef conditionals cover entire functions

2015-07-30 Thread Tony Cho
This patch lets preprocessor conditionals (#ifdef) related to WILC_SDIO_IRQ_GPIO to compile out the entire functions. Compiling out the entire functions is preferred rather than portions of functions or expressions becausue doing so makes code harder to read. Signed-off-by: Tony Cho --- V2

[PATCH V2 4/5] staging: wilc1000: remove errors on required space

2015-07-30 Thread Tony Cho
This patch removes the errors reported by checkpatch.pl, which is the space required before the open parenthesis '('. Signed-off-by: Tony Cho --- V2: This patch is not different with the first one. This patch includes warnings on no space required before the open parenthesis in th

[PATCH V2 5/5] staging: wilc1000: remove unused functions

2015-07-30 Thread Tony Cho
This patch removes the dead functions which are not used anymore. - sdio_check_bs - sdio_xfer_cnt Signed-off-by: Tony Cho --- V2: This patch doesn't have any difference with the first patch becasue this patch just removes unnecessary functions. However, it is resent because

Re: [PATCH V2 1/5] staging: wilc1000: #ifdef conditionals cover entire functions

2015-07-30 Thread tony . cho
On 2015년 07월 30일 20:56, Sudip Mukherjee wrote: On Thu, Jul 30, 2015 at 06:10:10PM +0900, Tony Cho wrote: This patch lets preprocessor conditionals (#ifdef) related to WILC_SDIO_IRQ_GPIO to compile out the entire functions. Compiling out the entire functions is preferred rather than portions

Re: [PATCH V2 1/5] staging: wilc1000: #ifdef conditionals cover entire functions

2015-07-30 Thread tony . cho
On 2015년 07월 30일 20:56, Sudip Mukherjee wrote: On Thu, Jul 30, 2015 at 06:10:10PM +0900, Tony Cho wrote: This patch lets preprocessor conditionals (#ifdef) related to WILC_SDIO_IRQ_GPIO to compile out the entire functions. Compiling out the entire functions is preferred rather than portions

Re: [PATCH 0/5] 2nd patch for illegal coding style

2015-07-30 Thread tony . cho
12:49, Greg KH wrote: On Thu, Jul 30, 2015 at 06:10:09PM +0900, Tony Cho wrote: This patch V2 includes only patches rejected. The rejected patch for the following, #ifdef conditionals cover entire functions, introduced new error on required space before the open parenthesis '('. This is

[PATCH 00/28] removing the compile warnings for 64-bit

2015-07-31 Thread Tony Cho
Hi Greg, This patch focuses on removing the compile warnings for 64bit incompatability. Most warnings are due to senseless type castings from driver handler to u32 and also to void pointers. They are removed by using the same handler with this series of patch. All warnings disappear. This patch is

[PATCH 01/28] staging: wilc1000: change type of driver handler in host interface msg

2015-07-31 Thread Tony Cho
as argument. Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c index 02aaf39..fb4194a 100644

[PATCH 03/28] staging: wilc1000: move structure WILC_WFIDrvHandle into wilc_wlan_if.h

2015-07-31 Thread Tony Cho
header file. Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.h | 3 --- drivers/staging/wilc1000/wilc_wlan_if.h | 4 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.h b/drivers/staging

[PATCH 04/28] staging: wilc1000: change type of argument 5 in SendConfigPkt

2015-07-31 Thread Tony Cho
. Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/coreconfigurator.c | 2 +- drivers/staging/wilc1000/coreconfigurator.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/coreconfigurator.c b/drivers/staging/wilc1000

[PATCH 05/28] staging: wilc1000: remove unnecessary type cast

2015-07-31 Thread Tony Cho
pointer to integer of different size [-Wpointer-to-int-cast] s32Error = SendConfigPkt(SET_CFG, &strWID, 1, true, (u32)pstrWFIDrv); Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 100 +++--- 1 file changed, 50 insert

[PATCH 02/28] staging: wilc1000: change void type of argument with WILC_WFIDrvHandle

2015-07-31 Thread Tony Cho
: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 74 +++ drivers/staging/wilc1000/host_interface.h | 2 +- 2 files changed, 38 insertions(+), 38 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c

[PATCH 06/28] staging: wilc1000: chage driver handler variable in SendConfigPkt

2015-07-31 Thread Tony Cho
From: glen lee This patch changes pstrWFIDrv with WILC_WFIDrvHandle type variable as a argument of SendConfigPkt in all associate code because the type of argument 5 in SendConfigPkt was changed with WILC_WFIDrvHandle type. Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging

[PATCH 07/28] staging: wilc1000: remove unused variables

2015-07-31 Thread Tony Cho
From: glen lee This patch removes unused variable pstrWFIDrv. Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 15 --- 1 file changed, 15 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging

[PATCH 15/28] staging: wilc1000: fix cast from pointer to integer warning

2015-07-31 Thread Tony Cho
ler = (u32)priv->hWILCWFIDrv; Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wilc1000/linux_wlan.c b/drivers/staging/wilc1000/linux_wlan.c index a808722..5dcb30c 100644

[PATCH 09/28] staging: wilc1000: change drvHandler type in wlan_cfg_set

2015-07-31 Thread Tony Cho
: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_wlan.c| 2 +- drivers/staging/wilc1000/wilc_wlan_if.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_wlan.c b/drivers/staging/wilc1000/wilc_wlan.c index 273073f..d5fc54b

[PATCH 11/28] staging: wilc1000: fix incompatible type in assignment warning

2015-07-31 Thread Tony Cho
; Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c index 66565f6..e9cd77d 100644 --- a

[PATCH 16/28] staging: wilc1000: change type of variable

2015-07-31 Thread Tony Cho
From: glen lee This patch changes gu8FlushedJoinReqDrvHandler of u32 type with WILC_WFIDrvHandle type. gu8FlushedJoinReqDrvHandler is used to hold driver handler for comparison but the type mismatches with it. Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000

[PATCH 10/28] staging: wilc1000: change driver handle variable

2015-07-31 Thread Tony Cho
From: glen lee This patch change (u32)pstrWFIDrv with priv->hWILCWFIDrv as a argument of wlan_cfg_set since the parameter drvHandler of uint32_t type in wlan_cfg_set was changed with WILC_WFIDrvHandle. Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wla

[PATCH 12/28] staging: wilc1000: change type of driver handler in tstrInterfaceInfo

2015-07-31 Thread Tony Cho
From: glen lee This patch changes drvHandler of uint32_t type with WILC_WFIDrvHandle type since drvHandler is used to pass WILC_WFIDrvHandle type to other functions. Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_wfi_netdevice.h | 2 +- 1 file changed, 1

[PATCH 14/28] staging: wilc1000: remove unnecessary type cast in host_int_set_wfi_drv_handler

2015-07-31 Thread Tony Cho
From: glen lee This patch removes unnecessary type cast since the u32 type parameter in host_int_set_wfi_drv_handler was changed with WILC_WFIDrvHandle. Therefore, No need to cast u32. Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 4

[PATCH 13/28] staging: wilc1000: change type of argument in host_int_set_wfi_drv_handler

2015-07-31 Thread Tony Cho
-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 2 +- drivers/staging/wilc1000/host_interface.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c index

[PATCH 08/28] staging: wilc1000: change drvHandler type in wlan_cfg_get

2015-07-31 Thread Tony Cho
: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/wilc_wlan.c| 2 +- drivers/staging/wilc1000/wilc_wlan_if.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_wlan.c b/drivers/staging/wilc1000/wilc_wlan.c index def72fd..273073f

[PATCH 20/28] staging: wilc1000: remove unnecessary type case

2015-07-31 Thread Tony Cho
From: glen lee This patch removes (u32) type cast which makes build warning since the u32Address of u32 type was changed with WILC_WFIDrvHandle. Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH 21/28] staging: wilc1000: remove unwanted type cast

2015-07-31 Thread Tony Cho
From: glen lee This patch removes (u32) type cast which makes build warning because the parameter of u32 type in the function host_int_set_wfi_drv_handler was changed with WILC_WFIDrvHandle. Not necessary to cast type. Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging

[PATCH 25/28] staging: wilc1000: change cast type from pointer to uintptr_t

2015-07-31 Thread Tony Cho
lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan.c | 2 +- drivers/staging/wilc1000/wilc_wlan.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/linux_wlan.c b/drivers/staging/wilc1000/linux_wlan.c index ec80849..14c2071 100644

[PATCH 19/28] staging: wilc1000: change type of variable in tstrHostIfSetDrvHandler

2015-07-31 Thread Tony Cho
From: glen lee This patch changes u32Address of u32 type with WILC_WFIDrvHandle type in the structure tstrHostIfSetDrvHandler because u32Address is used to hold driver handler. Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.h | 2 +- 1 file

[PATCH 26/28] staging: wilc1000: remove unused variable

2015-07-31 Thread Tony Cho
From: glen lee This patch removes unused variable pstrWFIDrv. Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000

[PATCH 24/28] staging: wilc1000: fix cast from pointer to integer warning

2015-07-31 Thread Tony Cho
to integer of different size [-Wpointer-to-int-cast] skb_reserve(skb, (unsigned int)skb->data & 0x3); Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/w

[PATCH 18/28] staging: wilc1000: change pstrWFIDrv with drvHandler

2015-07-31 Thread Tony Cho
From: glen lee As type of gu8FlushedJoinReqDrvHandler was changed with WILC_WFIDrvHandle, change right value with drvHandler to match type. Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH 17/28] staging: wilc1000: fix comparison between different type warning

2015-07-31 Thread Tony Cho
] if (gu8FlushedInfoElemAsoc != NULL && gu8FlushedJoinReqDrvHandler == (u32)drvHandler) { Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/

[PATCH 23/28] staging: wilc1000: change type of parameter in wilc_wlan_cfg_commit

2015-07-31 Thread Tony Cho
From: glen lee This patch changes drvHandler of uinit32_t type with WILC_WFIDrvHandle type to match driver handle type since wilc_wlan_cfg_commit gets drvHandler of WILC_WFIDrvHandle type as argument now. Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000

[PATCH 22/28] staging: wilc1000: fix passing argument from incompatible type warning

2015-07-31 Thread Tony Cho
ler->u32Address as argument as it is. Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c

[PATCH 28/28] staging: wilc1000: change data type of variable

2015-07-31 Thread Tony Cho
:6878:15: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] pstrWFIDrv = (tstrWILC_WFIDrv *)drvHandler; Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff

[PATCH 27/28] staging: wilc1000: fix casting build warning

2015-07-31 Thread Tony Cho
ULL) Signed-off-by: glen lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c index c5db385..b36f98c 100644 --- a/driv

Re: [PATCH 25/28] staging: wilc1000: change cast type from pointer to uintptr_t

2015-08-03 Thread tony . cho
05:50, Greg KH wrote: On Fri, Jul 31, 2015 at 04:38:33PM +0900, Tony Cho wrote: From: glen lee To use drvHandler as integer operation, cast type to uintptr_t. Here is the build warning. drivers/staging/wilc1000/wilc_wlan.c:1871:23: warning: cast from pointer to integer of different size [-Wpoint

[PATCH 01/32] staging: wilc1000: remove PLL_WORKAROUND

2015-09-30 Thread Tony Cho
This patch removes unnecessary definition, PLL_WORKAROUND from the Makefile. There is no use throughout the driver. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wilc1000/Makefile b/drivers

[PATCH 02/32] staging: wilc1000: remove AGING_ALG

2015-09-30 Thread Tony Cho
This patch removes AGING_ALG from the Makefile because it is always in use. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/Makefile | 1 - drivers/staging/wilc1000/coreconfigurator.h | 2 -- 2 files changed, 3 deletions(-) diff --git a/drivers/staging/wilc1000/Makefile b

[PATCH 04/32] staging: wilc1000: rename strHostIFconnectAttr

2015-09-30 Thread Tony Cho
This patch renames strHostIFconnectAttr to con_info to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 32 +++ 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/drivers/staging/wilc1000

[PATCH 03/32] staging: wilc1000: rename strHostIFscanAttr

2015-09-30 Thread Tony Cho
This patch renames strHostIFscanAttr to scan_info in order to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/drivers/staging/wilc1000

[PATCH 05/32] staging: wilc1000: rename strRcvdNetworkInfo

2015-09-30 Thread Tony Cho
This patch renames strRcvdNetworkInfo to net_info to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers

[PATCH 06/32] staging: wilc1000: rename strRcvdGnrlAsyncInfo

2015-09-30 Thread Tony Cho
This patch renames strRcvdGnrlAsyncInfo to async_info to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers

[PATCH 07/32] staging: wilc1000: rename strHostIFkeyAttr

2015-09-30 Thread Tony Cho
This patch renames strHostIFkeyAttr to key_info to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 100 +++--- 1 file changed, 50 insertions(+), 50 deletions(-) diff --git a/drivers/staging/wilc1000

[PATCH 08/32] staging: wilc1000: rename strHostIFCfgParamAttr

2015-09-30 Thread Tony Cho
This patch renames strHostIFCfgParamAttr to cfg_info to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging

[PATCH 10/32] staging: wilc1000: rename struct set_channel

2015-09-30 Thread Tony Cho
This patch renames struct set_channel to channel_attr to keep the naming convention such as scan_attr and connect_attr. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/wilc1000

[PATCH 13/32] staging: wilc1000: rename struct set_beacon

2015-09-30 Thread Tony Cho
This patch renames struct set_beacon to beacon_attr to keep the naming convention as scan_attr, connect_attr and channel_attr. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging

[PATCH 14/32] staging: wilc1000: rename strHostIFSetBeacon

2015-09-30 Thread Tony Cho
This patch renames strHostIFSetBeacon to beacon_info to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging

[PATCH 11/32] staging: wilc1000: rename strHostIFSetChan

2015-09-30 Thread Tony Cho
This patch renames strHostIFSetChan to channel_info to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging

[PATCH 12/32] staging: wilc1000: remove struct del_beacon

2015-09-30 Thread Tony Cho
This patch removes struct del_beacon which is not necessary. This patch also changes Handle_DelBeacon function by removing 2nd parameter, struct del_beacon because it is not used inside. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 24 +++- 1 file

[PATCH 17/32] staging: wilc1000: rename strEditStaParam

2015-09-30 Thread Tony Cho
This patch renames strEditStaParam to edit_sta_info to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging

[PATCH 09/32] staging: wilc1000: remove struct get_channel

2015-09-30 Thread Tony Cho
This patch removes struct get_channel from the driver because it is not used anywhere. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 15 --- 1 file changed, 15 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000

[PATCH 16/32] staging: wilc1000: rename strDelStaParam

2015-09-30 Thread Tony Cho
This patch renames strDelStaParam to del_sta_info to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging

[PATCH 15/32] staging: wilc1000: rename strAddStaParam

2015-09-30 Thread Tony Cho
This patch renames strAddStaParam to add_sta_info to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging

[PATCH 18/32] staging: wilc1000: remove struct timer_cb

2015-09-30 Thread Tony Cho
This patch removes struct timer_cb which is not used. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 15 --- 1 file changed, 15 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c index da8e589

[PATCH 20/32] staging: wilc1000: rename strHostIfStaInactiveT

2015-09-30 Thread Tony Cho
This patch renames strHostIfStaInactiveT to mac_info to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging

[PATCH 19/32] staging: wilc1000: rename strPowerMgmtparam

2015-09-30 Thread Tony Cho
This patch renames strPowerMgmtparam to pwr_mgmt_info to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers

[PATCH 21/32] staging: wilc1000: rename strHostIfSetDrvHandler

2015-09-30 Thread Tony Cho
This patch renames strHostIfSetDrvHandler to drv to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging

[PATCH 22/32] staging: wilc1000: rename strHostIfSetIP

2015-09-30 Thread Tony Cho
This patch renames strHostIfSetIP to ip_info to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers

[PATCH 23/32] staging: wilc1000: rename strHostIfSetMulti

2015-09-30 Thread Tony Cho
This patch renames strHostIfSetMulti to multicast_info to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers

[PATCH 25/32] staging: wilc1000: rename strHostIfSetMacAddress

2015-09-30 Thread Tony Cho
This patch renames strHostIfSetMacAddress to set_mac_info to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers

[PATCH 30/32] staging: wilc1000: rename pUserData

2015-09-30 Thread Tony Cho
This patch renames pUserData to data to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000

[PATCH 26/32] staging: wilc1000: rename strHostIfGetMacAddress

2015-09-30 Thread Tony Cho
This patch renames strHostIfGetMacAddress to get_mac_info to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers

[PATCH 31/32] staging: wilc1000: rename strHostIFDelAllSta

2015-09-30 Thread Tony Cho
This patch renames strHostIFDelAllSta to del_all_sta_info to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers

[PATCH 24/32] staging: wilc1000: rename strHostIfSetOperationMode

2015-09-30 Thread Tony Cho
This patch renames strHostIfSetOperationMode to mode to avoid CamelCase naming convention. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging

  1   2   3   4   5   6   7   >