On 2015년 09월 14일 13:25, Greg KH wrote:
On Mon, Sep 14, 2015 at 10:29:26AM +0900, Tony Cho wrote:
On 2015년 09월 12일 11:35, Greg KH wrote:
On Fri, Sep 11, 2015 at 12:04:19PM +0900, Tony Cho wrote:
From: Glen Lee
This patch removes a unused enum tenuFrameClass.
Signed-off-by: Glen Lee
Hi Greg,
I am sorry but please ignore this patch in this time.
It's my mistake to include this patch from my queue into the series of 7
patches.
Thanks,
Tony.
On 2015년 09월 11일 16:52, Tony Cho wrote:
This patch removes the preprocessor definition, RX_BH_TYPE and use
This patch aims to remove the wilc_errorsupport.h which is unnecessary in the
wilc driver. This patch replaces the error types defined by wilc driver with
kernel generic error types and removes macros which handles the errors.
Leo Kim (7):
staging: wilc1000: remove definition WILC_IS_ERR
stagi
From: Leo Kim
This patch removes the definition WILC_IS_ERR which is not used anymore.
Signed-off-by: Leo Kim
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/wilc_errorsupport.h | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wilc1000
: Tony Cho
---
drivers/staging/wilc1000/wilc_errorsupport.h | 17 -
1 file changed, 17 deletions(-)
diff --git a/drivers/staging/wilc1000/wilc_errorsupport.h
b/drivers/staging/wilc1000/wilc_errorsupport.h
index 99ded3d..054ed76 100644
--- a/drivers/staging/wilc1000
and
ERRORHANDLER in some of functions.
Signed-off-by: Leo Kim
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/coreconfigurator.c | 6 +-
drivers/staging/wilc1000/host_interface.c | 919 +-
drivers/staging/wilc1000/wilc_errorsupport.h | 5 -
drivers
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 3 ++-
drivers/staging/wilc1000/wilc_errorsupport.h | 7 ---
drivers/staging/wilc1000/wilc_msgqueue.c | 5 -
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 24 ---
4
From: Leo Kim
This patch removes the definition WILC_NULLCHECK which is not used anymore.
Signed-off-by: Leo Kim
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 8 ++--
drivers/staging/wilc1000/wilc_errorsupport.h | 6 --
drivers/staging
From: Leo Kim
This patch removes the file, wilc_errorsupport which is not used anymore.
Remove #include "wilc_errorsupport.h" from the source code.
Signed-off-by: Leo Kim
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/wilc_errorsupport.h | 15 ---
drivers/stagin
error types.
Signed-off-by: Leo Kim
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/coreconfigurator.c | 24 +--
drivers/staging/wilc1000/host_interface.c | 238 +++---
drivers/staging/wilc1000/linux_mon.c | 14 +-
drivers/staging/wilc1000
From: Glen Lee
g_num_total_switches is never used so just delete it.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/coreconfigurator.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/wilc1000/coreconfigurator.h
b/drivers/staging/wilc1000
From: Glen Lee
The define MAX_PACKET_BUFF_SIZE and STATUS_MSG_LEN is not used in the driver.
Delete two defines.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/coreconfigurator.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/staging/wilc1000
From: Glen Lee
SendRawpacket is declared but not implemented. It is not used also.
So just delete it.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/coreconfigurator.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/wilc1000/coreconfigurator.h
From: Glen Lee
g_flushing_in_progress is never used so just delete it.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
b/drivers/staging
On 2015년 09월 15일 15:27, Sudip Mukherjee wrote:
On Tue, Sep 15, 2015 at 11:53:34AM +0900, Tony Cho wrote:
This patch aims to remove the wilc_errorsupport.h which is unnecessary in the
wilc driver. This patch replaces the error types defined by wilc driver with
kernel generic error types and
On 2015년 09월 15일 16:13, Sudip Mukherjee wrote:
On Tue, Sep 15, 2015 at 02:33:54PM +0900, Tony Cho wrote:
From: Glen Lee
g_num_total_switches is never used so just delete it.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
you only deleted from the .h file but it still remains in
Hi Greg,
If it's already in your tree and can be applied alone, no problem to me. I will
later send remaining series of it.
Thanks,
Tony.
On 2015년 09월 15일 22:31, Greg KH wrote:
On Tue, Sep 15, 2015 at 10:17:41AM +0900, Tony Cho wrote:
Hi Greg,
I am sorry but please ignore this pat
On 2015년 09월 15일 22:46, Greg KH wrote:
On Tue, Sep 15, 2015 at 05:26:35PM +0900, Tony Cho wrote:
On 2015년 09월 15일 16:13, Sudip Mukherjee wrote:
On Tue, Sep 15, 2015 at 02:33:54PM +0900, Tony Cho wrote:
From: Glen Lee
g_num_total_switches is never used so just delete it.
Signed-off-by
: wilc1000: delete function declaration SendRawPacket
staging: wilc1000: coreconfigurator.h: remove unused defines
staging: wilc1000: remove unused variable g_flushing_in_progress
Tony Cho (1):
staging: wilc1000: remove PANDA_BOARD
drivers/staging/wilc1000/coreconfigurator.c | 165
From: Glen Lee
The define PHY_802_11n is useless because wilc1000 support 802.11n as always.
Remove PHY_802_11n and it related code.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/coreconfigurator.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers
From: Glen Lee
gstrConfigPktInfo, SemHandleSendPkt and SemHandlePktResp are never used in
the drivers. So just delete the variables and it's related code.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/coreconfigurator.c | 11 ---
1 file change
From: Glen Lee
Function NetworkInfoReceived, GnrlAsyncInfoReceived and
host_int_ScanCompleteReceived are declared in the header file already.
Just removes the declaration and comments.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/coreconfigurator.c | 8
From: Glen Lee
Variable g_num_total_switches is never used so just remove it.
gastrWIDs is also not used because g_num_total_switches is deleted.
Remove gastrWIDs also.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/coreconfigurator.c | 127
From: Glen Lee
SendRawpacket is declared but not implemented. It is not used also.
So just delete it.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/coreconfigurator.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/wilc1000/coreconfigurator.h
This patch removes one of the platform dependencies, PANDA_BOARD from the
driver codes. The device tree will provide the platform dependencies in
the future commits.
Signed-off-by: Tony Cho
Signed-off-by: Glen Lee
---
drivers/staging/wilc1000/linux_wlan.c | 4
drivers/staging
From: Glen Lee
wilc1000 driver does not support five Ghz band. Remove FIVE_GHZ_BAND and
it's related code.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/coreconfigurator.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/drivers/staging/wil
From: Glen Lee
The define MAX_PACKET_BUFF_SIZE and STATUS_MSG_LEN is not used in the driver.
Delete two defines.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/coreconfigurator.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/staging/wilc1000
From: Glen Lee
g_flushing_in_progress is never used so just delete it.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
b/drivers/staging
From: Glen Lee
This patch removes follwing unused defines.
MAX_CFG_PKTLEN
MSG_HEADER_LEN
QUERY_MSG_TYPE
WRITE_MSG_TYPE
RESP_MSG_TYPE
WRITE_RESP_SUCCESS
INVALID
MAC_ADDR_LEN
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/coreconfigurator.c | 8
1 file
From: Leo Kim
This patch replaces WILC_NULLCHECK with the plain statements and then removes
the macro WILC_NULLCHECK in wilc_errorsupport.h which is not used anymore.
Signed-off-by: Leo Kim
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 7 +--
drivers
From: Leo Kim
This patch removes the macro WILC_ERRORCHECK which is not used
anymore by replacing it with the plain statements.
This patch also removes the WILC_CATCH macros from some of functions not
to make the build warnings.
Signed-off-by: Leo Kim
Signed-off-by: Tony Cho
---
drivers
ERRORHANDLER from some of functions.
Signed-off-by: Leo Kim
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/coreconfigurator.c | 4 +-
drivers/staging/wilc1000/host_interface.c | 869 +-
drivers/staging/wilc1000/wilc_errorsupport.h | 5 -
drivers/staging
mon.h file.
Signed-off-by: Leo Kim
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/wilc_errorsupport.h | 15 ---
drivers/staging/wilc1000/wilc_msgqueue.c | 1 +
drivers/staging/wilc1000/wilc_msgqueue.h | 1 -
drivers/staging/wilc1000/wilc_oswrapper.h| 3 ---
4 files
: Tony Cho
---
drivers/staging/wilc1000/wilc_errorsupport.h | 17 -
1 file changed, 17 deletions(-)
diff --git a/drivers/staging/wilc1000/wilc_errorsupport.h
b/drivers/staging/wilc1000/wilc_errorsupport.h
index 99ded3d..054ed76 100644
--- a/drivers/staging/wilc1000
This series of patch is resent in total 6 patches as version 2. This version
fixes the warnings reported by checkpatch.pl not to make additional warnings.
The concern for the memory leak issue is checked again in this version.
The error types provided by wilc driver are being replaced by the ones
.
Signed-off-by: Leo Kim
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/coreconfigurator.c | 24 +--
drivers/staging/wilc1000/host_interface.c | 238 +++---
drivers/staging/wilc1000/linux_mon.c | 14 +-
drivers/staging/wilc1000/linux_wlan.c
From: Glen Lee
linux_wlan_rxq_task is not used in the driver. Just remove it.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/linux_wlan.c | 26 --
1 file changed, 26 deletions(-)
diff --git a/drivers/staging/wilc1000/linux_wlan.c
b
From: Glen Lee
WILC_P2P is always used in the driver. So delete define WILC_P2P and ifdef line.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/coreconfigurator.c | 2 --
drivers/staging/wilc1000/coreconfigurator.h | 2 --
drivers/staging/wilc1000
From: Glen Lee
This driver use WILC_AP_EXTERNAL_MLME define always. Delete define
WILC_AP_EXTERNAL_MLME and ifdef line.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 30 +++
drivers/staging/wilc1000/linux_mon.c
From: Glen Lee
TCP_ENHANCEMENTS is always in use. Remove define TCP_ENHANCEMENTS, ifdef line,
ifndef line and codes inside ifndef.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/linux_wlan.c | 36 ---
drivers/staging/wilc1000
wlan_rxq_task which was deleted in a previous patch.
So delete variable rxq_event, rxq_wait_event, rxq_wait and it's related codes.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/linux_wlan.c | 9 -
drivers/staging/wilc1000/wilc_wfi_netdevice.h | 1
From: Glen Lee
The rxq_thread never runs since it's kthread_run code was delete in a previous
patch. Remove kthread_run and it's related codes.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/linux_wlan.c | 6 --
drivers/stagin
From: Glen Lee
rxq_thread_started is initiallized but never used in the driver. Remove
the variable and init code line.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/linux_wlan.c | 1 -
drivers/staging/wilc1000/wilc_wfi_netdevice.h | 1 -
2 files
From: Glen Lee
The function pointer wlan_handle_rx_que is not called anywhere. Delete
wlan_handle_rx_que and it's assignment code.
But the function wilc_wlan_handle_rxq in assignment code is used. So leave it.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wil
This series of patch is the version 2 for the RX_BH_TYPE patch before sent.
Glen Lee (2):
staging: wilc1000: remove define RX_BH_TYPE
staging: wilc1000: remove do-nothing ifdef statement
drivers/staging/wilc1000/linux_wlan.c | 149 +-
drivers/staging/wilc1000/
: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/linux_wlan.c | 139 --
drivers/staging/wilc1000/linux_wlan_common.h | 3 -
drivers/staging/wilc1000/wilc_wfi_netdevice.h | 15 ---
drivers/staging/wilc1000/wilc_wlan.c | 3 -
4 files
From: Glen Lee
This patch removes do-nothing ifdef, else statement and add ifdef WILC_SDIO
for disable_sdio_interrupt().
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/linux_wlan.c | 10 ++
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a
From: Leo Kim
This patch modify null check routine.
- Null check error non return. (Handle_RcvdGnrlAsyncInfo)
Signed-off-by: Leo Kim
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/staging
From: Leo Kim
This patch removes the warnings reported by checkpatch.pl
for line over 80 characters.
Signed-off-by: Leo Kim
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/wilc_wlan_if.h | 650 ++--
1 file changed, 374 insertions(+), 276 deletions(-)
diff
From: Leo Kim
This patch removes the warnings reported by checkpatch.pl
for using multiple blank lines.
Signed-off-by: Leo Kim
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/wilc_wlan_if.h | 12
1 file changed, 12 deletions(-)
diff --git a/drivers/staging/wilc1000
From: Leo Kim
This patch fix alignment of some defines in wilc_wlan_if.h.
Signed-off-by: Leo Kim
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/wilc_wlan_if.h | 580
1 file changed, 290 insertions(+), 290 deletions(-)
diff --git a/drivers/staging
On 2015년 09월 17일 18:06, Nicolas Ferre wrote:
Le 17/09/2015 10:50, Tony Cho a écrit :
From: Leo Kim
This patch modify null check routine.
- Null check error non return. (Handle_RcvdGnrlAsyncInfo)
It doesn't parse...
Is it fixing a bug? What were the consequences without the return?
-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging/wilc1000/host_interface.c
index 6fdf392..151e8c4 100644
--- a/drivers/staging/wilc1000
From: Leo Kim
This patch removes the warnings reported by checkpatch.pl
for using multiple blank lines.
Signed-off-by: Leo Kim
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/wilc_wlan_if.h | 12
1 file changed, 12 deletions(-)
diff --git a/drivers/staging/wilc1000
The previous series of 4 patches include a commit message ambiguous and
not enough to explain why the patch is made and needed. So, this version 2
includes the modification on the commit message of the patch for a null check
fix.
Leo Kim (4):
staging: wilc1000: fix null check routine
staging:
From: Leo Kim
This patch removes the warnings reported by checkpatch.pl
for line over 80 characters.
Signed-off-by: Leo Kim
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/wilc_wlan_if.h | 650 ++--
1 file changed, 374 insertions(+), 276 deletions(-)
diff
From: Leo Kim
This patch fix alignment of some defines in wilc_wlan_if.h.
Signed-off-by: Leo Kim
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/wilc_wlan_if.h | 580
1 file changed, 290 insertions(+), 290 deletions(-)
diff --git a/drivers/staging
This patch gets rid of typedef from the union and renames tuniHostIFmsgBody
with simply message_body because the Linux coding style does not recommend
camelcase notation and typedef for structure, enmu and also union.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 8
This patch changes the camelcase notation, uniHostIFmsgBody which is a member
variable of structure host_if_msg to body in order to comply with the Linux
coding style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 296 +++---
1 file changed, 148
The Linux kernel coding style does not recommend to use typedef for the
structure. This patch removes typedef from the host_if_msg structure.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 294 +++---
1 file changed, 147 insertions(+), 147
This patch changes the name, strHostIFmsg simply to msg because the
camelcase notation is not recommended in Linux.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 794 +++---
1 file changed, 397 insertions(+), 397 deletions(-)
diff --git a
This patch removes typedef from the struct tstrRcvdNetworkInfo and
renames it to rcvd_net_info in order to comply with the Linux coding
style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 7 ---
drivers/staging/wilc1000/host_interface.h | 6 +++---
2 files changed
This patch changes the name, u16MsgId simply to id to avoid the camelcase
notation.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 102 +++---
1 file changed, 51 insertions(+), 51 deletions(-)
diff --git a/drivers/staging/wilc1000
This patch removes typedef from the structure tstrHostIFscanAttr and
changes the camelcase name, tstrHostIFscanAttr to scan_attr in order to
comply with the Linux coding style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 14 +++---
1 file changed, 7
This patch removes typedef from the struct tstrHostIFconnectAttr and
renames it to connect_attr to comply with the Linux coding style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/drivers
This patch changes the name, tstrHostIFmsg to host_if_msg not to use
camelcase notation in the Linux.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 294 +++---
1 file changed, 147 insertions(+), 147 deletions(-)
diff --git a/drivers/staging
This patch removes typedef from the struct tstrHostIFGetChan and renames
it to get_channel in order to comply with the Linux coding style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers
This patch removes typedef from the struct tstrRcvdGnrlAsyncInfo and
renames it to rcvd_async_info in order to comply with the Linux coding
style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff
This patch removes typedef from the struct tstrHostIFkeyAttr and renames
it to key_attr in oder to comply with the Linux coding style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/drivers
This patch removes typedef from the struct tstrHostIFCfgParamAttr and
renames it to cfg_param_attr in order to comply with the Linux coding
style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff
This patch removes typedef from the struct tstrHostIFDelBeacon and
renames it to del_beacon in order to comply with the Linux coding style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a
This patch removes typedef from the struct tstrWILC_AddStaParam and
renames it to add_sta_param in oder to comply with the Linux coding
style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 37 +--
drivers/staging/wilc1000/host_interface.h
This patch removes typedef from the tstrTimerCb and renames it to
timer_cb in order to comply with the Linux coding style.
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
This patch removes typedef from the struct tstrHostIFSetChan and ranames
it to set_channel in order to comply with the Linux coding style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a
This patch removes typedef from the struct tstrHostIFSetBeacon and
renames it to set_beacon in order to comply with the Linux coding style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 15 ---
1 file changed, 8 insertions(+), 7 deletions(-)
diff --git a
This patch removes typedef from the struct tstrHostIfBASessionInfo and
renames it to ba_session_info in order to comply with the Linux coding
style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 17 ++---
drivers/staging/wilc1000/host_interface.h | 5
This patch removes typedef from the struct tstrHostIfSetMacAddress and
renames it set_mac_addr in order to comply with the Linux coding style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 5 +++--
drivers/staging/wilc1000/host_interface.h | 5 ++---
2 files changed, 5
This patch removes typedef from the struct tstrHostIfSetDrvHandler and
renames it to drv_handler in order to comply with the Linux coding
style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 7 ---
drivers/staging/wilc1000/host_interface.h | 4 ++--
2 files changed
This patch removes typedef from the struct tstrHostIfGetMacAddress and
renames it get_mac_addr in order to comply with the Linux coding style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 5 +++--
drivers/staging/wilc1000/host_interface.h | 5 ++---
2 files changed, 5
This patch removes typedef from the struct tstrHostIfSetOperationMode
and renames it to op_mode in order to comply with the Linux coding
style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 5 +++--
drivers/staging/wilc1000/host_interface.h | 4 ++--
2 files changed, 5
This patch removes typedef from the struct tstrHostIFSetMulti and
renames it to set_multicast in order to comply with the Linux coding
style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 17 +
1 file changed, 9 insertions(+), 8 deletions(-)
diff --git
This patch removes typedef from the struct tstrHostIfStaInactiveT and
renames it to sta_inactive_t in oder to comply with the Linux coding
style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git
This patch removes typedef from the struct tstrHostIFSetIPAddr and
renames it to set_ip_addr in order to comply with the Linux coding
style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/drivers
This patch removes typedef from the struct tstrHostIfPowerMgmtParam and
renames it to power_mgmt_param in order to comply with the Linux coding
style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff
This patch removes typedef from the struct tstrHostIFDelSta and renames
it del_sta in order to comply with the Linux coding style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/drivers
This patch removes typedef from the struct tstrHostIfRemainOnChan and
renames it to remain_ch in order to comply with the Linux coding style
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 8 +---
drivers/staging/wilc1000/host_interface.h | 6 +++---
2 files changed
This patch removes typedef from the struct tstrHostIfRegisterFrame and
renames it to reg_frame in order to comply with the Linux coding style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 5 +++--
drivers/staging/wilc1000/host_interface.h | 7 ++-
2 files changed
This patch removes the global variable, u32Intialized which is not
necessary from the host_interface.c file.
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
The comments in union message_body are not essential and cross 80 ending
line, so they are deleted and also a dead code commented out in the union
is deleted.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 27 +--
1 file changed, 13 insertions
The global variable msgQ_created is removed because it is not necessary.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging/wilc1000/host_interface.c
index
This patch removes typedef from the struct tstrHostIFDelAllSta and
renames it to del_all_sta in order to comply with the Linux coding
style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a
On 2015년 09월 19일 11:50, Greg KH wrote:
On Thu, Sep 17, 2015 at 07:02:17PM +0900, Tony Cho wrote:
From: Leo Kim
This patch removes the potential faults which may happen when unexpectedly
getting access to invalid pointer. The pointer of pstrWFIDrv is unlikely
to be invalid. However, it is
On 2015년 09월 19일 12:00, Greg KH wrote:
On Fri, Sep 18, 2015 at 06:11:05PM +0900, Tony Cho wrote:
This patch changes the name, strHostIFmsg simply to msg because the
camelcase notation is not recommended in Linux.
Signed-off-by: Tony Cho
Because of the other patches I didn't accept,
This is the version 2 for the series of the patch that mostly removes the
typedef from the structures and renames its names to comply with the Linux
coding style.
The version 2 is rebased again onto the staging-testing branch to synchronize
with the mainstream.
Tony Cho (34
This patch changes the name, tstrHostIFmsg to host_if_msg not to use
camelcase notation in the Linux.
Signed-off-by: Tony Cho
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/wilc1000/host_interface.c | 294 +++---
1 file changed, 147 insertions(+), 147 deletions
This patch changes the name, strHostIFmsg simply to msg because the
camelcase notation is not recommended in Linux.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 794 +++---
1 file changed, 397 insertions(+), 397 deletions(-)
diff --git a
This patch gets rid of typedef from the union and renames tuniHostIFmsgBody
with simply message_body because the Linux coding style does not recommend
camelcase notation and typedef for structure, enmu and also union.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 8
This patch changes the name, u16MsgId simply to id to avoid the camelcase
notation.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 102 +++---
1 file changed, 51 insertions(+), 51 deletions(-)
diff --git a/drivers/staging/wilc1000
The Linux kernel coding style does not recommend to use typedef for the
structure. This patch removes typedef from the host_if_msg structure.
Signed-off-by: Tony Cho
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/wilc1000/host_interface.c | 294 +++---
1 file
This patch removes typedef from the struct tstrRcvdGnrlAsyncInfo and
renames it to rcvd_async_info in order to comply with the Linux coding
style.
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff
501 - 600 of 631 matches
Mail list logo
