[RFC PATCH] staging: wilc1000: scan_while_connected can be static

2015-10-14 Thread kbuild test robot
Signed-off-by: Fengguang Wu --- 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 a09db2c..0e0d1db 100644 --- a/drivers/staging/wilc1000/host_interface.c +++ b/drive

Re: [PATCH 19/40] staging: wilc1000: rename variable gbScanWhileConnected

2015-10-14 Thread kbuild test robot
Hi Leo, [auto build test WARNING on staging/staging-next -- if it's inappropriate base, please suggest rules for selecting the more suitable base] url: https://github.com/0day-ci/linux/commits/Tony-Cho/staging-wilc1000-rename-ssidLen-of-struct-join_bss_param/20151015-123159 reproduce:

Re: [PATCH 17/40] staging: wilc1000: rename variable g_hPeriodicRSSI

2015-10-14 Thread kbuild test robot
Hi Leo, [auto build test WARNING on staging/staging-next -- if it's inappropriate base, please suggest rules for selecting the more suitable base] url: https://github.com/0day-ci/linux/commits/Tony-Cho/staging-wilc1000-rename-ssidLen-of-struct-join_bss_param/20151015-123159 reproduce:

[RFC PATCH] staging: wilc1000: periodic_rssi can be static

2015-10-14 Thread kbuild test robot
Signed-off-by: Fengguang Wu --- 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 3c1110c..b29d7b186 100644 --- a/drivers/staging/wilc1000/host_interface.c +++ b/dri

Re: [PATCH 16/40] staging: wilc1000: rename variable hSemHostIntDeinit

2015-10-14 Thread kbuild test robot
Hi Leo, [auto build test WARNING on staging/staging-next -- if it's inappropriate base, please suggest rules for selecting the more suitable base] url: https://github.com/0day-ci/linux/commits/Tony-Cho/staging-wilc1000-rename-ssidLen-of-struct-join_bss_param/20151015-123159 reproduce:

[RFC PATCH] staging: wilc1000: hif_sema_deinit can be static

2015-10-14 Thread kbuild test robot
Signed-off-by: Fengguang Wu --- 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 acb8379d..aa5c9d7 100644 --- a/drivers/staging/wilc1000/host_interface.c +++ b/driv

[RFC PATCH] staging: wilc1000: hif_sema_driver can be static

2015-10-14 Thread kbuild test robot
Signed-off-by: Fengguang Wu --- 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 b4f8c3a..32c73b1 100644 --- a/drivers/staging/wilc1000/host_interface.c +++ b/drive

Re: [PATCH 14/40] staging: wilc1000: rename variable hSemDeinitDrvHandle

2015-10-14 Thread kbuild test robot
Hi Leo, [auto build test WARNING on staging/staging-next -- if it's inappropriate base, please suggest rules for selecting the more suitable base] url: https://github.com/0day-ci/linux/commits/Tony-Cho/staging-wilc1000-rename-ssidLen-of-struct-join_bss_param/20151015-123159 reproduce:

[PATCH 40/40] staging: wilc1000: fixes comparison to NULL could be written

2015-10-14 Thread Tony Cho
From: Leo Kim This patch fixes the checks reported by checkpatch.pl for comparison to NULL could be written. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 222 -- 1 file changed, 115 insertions(+), 107 deletions(-)

[PATCH 39/40] staging: wilc1000: fixes alignment should match open parenthesis

2015-10-14 Thread Tony Cho
From: Leo Kim This patch fixes the checks reported by checkpatch.pl for alignment should match open parenthesis. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 155 +++--- 1 file changed, 79 insertions(+), 76 deletions(-)

[PATCH 38/40] staging: wilc1000: fixes unnecessary variable replacement

2015-10-14 Thread Tony Cho
From: Leo Kim This patch removes an unnecessary variable use for the errors and returns errors directly without a result variable. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 28 +--- 1 file changed, 9 insertions(+), 1

[PATCH 37/40] staging: wilc1000: move variable FALSE_FRMWR_CHANNEL

2015-10-14 Thread Tony Cho
From: Leo Kim This patch move variable 'FALSE_FRMWR_CHANNEL' to local define position. Signed-off-by: Leo Kim 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/

[PATCH 36/40] staging: wilc1000: move variable clients_count

2015-10-14 Thread Tony Cho
From: Leo Kim This patch move static variable clients_count to local variable position. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c

[PATCH 35/40] staging: wilc1000: rename variable s32Error

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable s32Error to result to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 784 ++ 1 file changed, 371 insertions(+), 413 deletions(-) diff --git

[PATCH 30/40] staging: wilc1000: rename variable gu8Flushed11iMode

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable gu8Flushed11iMode to mode_11i to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/

[PATCH 31/40] staging: wilc1000: rename variable gu8FlushedAuthType

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable gu8FlushedAuthType to auth_type to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/stagin

[PATCH 34/40] staging: wilc1000: rename variable gu8FlushedJoinReqDrvHandler

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable gu8FlushedJoinReqDrvHandler to join_req_drv to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) di

[PATCH 28/40] staging: wilc1000: rename variable gu8FlushedJoinReq

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable gu8FlushedJoinReq to join_req to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff

[PATCH 33/40] staging: wilc1000: rename variable gu32FlushedInfoElemAsocSize

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable gu32FlushedInfoElemAsocSize to info_element_size to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff

[PATCH 29/40] staging: wilc1000: rename variable gu8FlushedInfoElemAsoc

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable gu8FlushedInfoElemAsoc to info_element to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 29 - 1 file changed, 16 insertions(+), 13 deletion

[PATCH 32/40] staging: wilc1000: rename variable gu32FlushedJoinReqSize

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable gu32FlushedJoinReqSize to join_req_size to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/d

[PATCH 25/40] staging: wilc1000: rename variable gu32InactiveTime

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable gu32InactiveTime to inactive_time to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/s

[PATCH 27/40] staging: wilc1000: remove unused variable gu32WidConnRstHack

2015-10-14 Thread Tony Cho
From: Leo Kim This patch removes unused the variable gu32WidConnRstHack. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host

[PATCH 19/40] staging: wilc1000: rename variable gbScanWhileConnected

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable gbScanWhileConnected to scan_while_connected to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 19 +-- 1 file changed, 9 insertions(+), 10 deletions(-)

[PATCH 26/40] staging: wilc1000: rename variable gu8DelBcn

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable gu8DelBcn to del_beacon to avoid CamelCase naming convention. Signed-off-by: Leo Kim 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

[PATCH 20/40] staging: wilc1000: rename variable gs8Rssi

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable gs8Rssi to rssi to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/staging/wilc1000/hos

[PATCH 21/40] staging: wilc1000: rename variable gs8lnkspd

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable gs8lnkspd to link_speed. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/s

[PATCH 23/40] staging: wilc1000: rename variable gs8SetIP

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable gs8SetIP to set_ip to avoid CamelCase naming convention. Signed-off-by: Leo Kim 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/

[PATCH 22/40] staging: wilc1000: rename variable gu8Chnl

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable gu8Chnl to ch_no to avoid CamelCase naming convention. Signed-off-by: Leo Kim 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

[PATCH 24/40] staging: wilc1000: rename variable gs8GetIP

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable gs8GetIP to get_ip to avoid CamelCase naming convention. Signed-off-by: Leo Kim 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/

[PATCH 17/40] staging: wilc1000: rename variable g_hPeriodicRSSI

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable g_hPeriodicRSSI to periodic_rssi to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 20 1 file changed, 8 insertions(+), 12 deletions(-) diff --git

[PATCH 13/40] staging: wilc1000: rename variable hSemHostIFthrdEnd

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable hSemHostIFthrdEnd to hif_sema_thread to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/driver

[PATCH 18/40] staging: wilc1000: rename variable gapu8RcvdAssocResp

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable gapu8RcvdAssocResp to rcv_assoc_resp to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers

[PATCH 15/40] staging: wilc1000: rename variable hWaitResponse

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable hWaitResponse to hif_sema_wait_response to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(

[PATCH 14/40] staging: wilc1000: rename variable hSemDeinitDrvHandle

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable hSemDeinitDrvHandle to hif_sema_driver to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/

[PATCH 16/40] staging: wilc1000: rename variable hSemHostIntDeinit

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable hSemHostIntDeinit to hif_sema_deinit to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git

[PATCH 08/40] staging: wilc1000: rename au8Interval of struct join_bss_param

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames au8Interval of struct join_bss_param to interval to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/dri

[PATCH 09/40] staging: wilc1000: rename au8StartTime of struct join_bss_param

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames au8StartTime of struct join_bss_param to start_time to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git

[PATCH 12/40] staging: wilc1000: rename variable gMsgQHostIF

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable gMsgQHostIF to hif_msg_q to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 109 +++--- 1 file changed, 53 insertions(+), 56 deletions(-) diff -

[PATCH 11/40] staging: wilc1000: rename variable HostIFthreadHandler

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames variable HostIFthreadHandler to hif_thread_handler to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a

[PATCH 10/40] staging: wilc1000: remove unused variable gWFiDrvHandle

2015-10-14 Thread Tony Cho
From: Leo Kim This patch removes unused the variable gWFiDrvHandle. Signed-off-by: Leo Kim 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/host_interf

[PATCH 07/40] staging: wilc1000: rename au8Duration of struct join_bss_param

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames au8Duration of struct join_bss_param to duration to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/dri

[PATCH 02/40] staging: wilc1000: rename u8NoaEnbaled of struct join_bss_param

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames u8NoaEnbaled of struct join_bss_param to noa_enabled to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/

[PATCH 04/40] staging: wilc1000: rename u8CtWindow of struct join_bss_param

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames u8CtWindow of struct join_bss_param to ct_window to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/driver

[PATCH 01/40] staging: wilc1000: rename ssidLen of struct join_bss_param

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames ssidLen of struct join_bss_param to ssid_len to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/stag

[PATCH 06/40] staging: wilc1000: rename u8Index of struct join_bss_param

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames u8Index of struct join_bss_param to idx to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging

[PATCH 05/40] staging: wilc1000: rename u8Count of struct join_bss_param

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames u8Count of struct join_bss_param to cnt to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging

[PATCH 03/40] staging: wilc1000: rename u8OppEnable of struct join_bss_param

2015-10-14 Thread Tony Cho
From: Leo Kim This patch renames u8OppEnable of struct join_bss_param to opp_enabled to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --

Re: [PATCH] staging: octeon: fixed few coding style warnings

2015-10-14 Thread Arnd Bergmann
On Wednesday 14 October 2015 14:04:00 David Daney wrote: > On 10/14/2015 01:58 PM, Arnd Bergmann wrote: > > On Wednesday 14 October 2015 08:47:44 David Daney wrote: > >> On 10/14/2015 07:06 AM, Sakshi Bansal wrote: > >>> Fixed allignment issues and line over 80 characters > >> > >> Use spell checki

Re: [PATCH] staging: octeon: fixed few coding style warnings

2015-10-14 Thread David Daney
On 10/14/2015 01:58 PM, Arnd Bergmann wrote: On Wednesday 14 October 2015 08:47:44 David Daney wrote: On 10/14/2015 07:06 AM, Sakshi Bansal wrote: Fixed allignment issues and line over 80 characters Use spell checking on 'allignment' But that is not the main problem with the patch... You a

Re: [PATCH] staging: octeon: fixed few coding style warnings

2015-10-14 Thread Arnd Bergmann
On Wednesday 14 October 2015 08:47:44 David Daney wrote: > On 10/14/2015 07:06 AM, Sakshi Bansal wrote: > > Fixed allignment issues and line over 80 characters > > Use spell checking on 'allignment' > > But that is not the main problem with the patch... > > > You are changing things other than

[PATCH 2/4] staging: fsl-mc: refactored mc_send_command()

2015-10-14 Thread J. German Rivera
Moved wait logic in mc_send_command() to its own function --- drivers/staging/fsl-mc/bus/mc-sys.c | 50 - 1 file changed, 38 insertions(+), 12 deletions(-) diff --git a/drivers/staging/fsl-mc/bus/mc-sys.c b/drivers/staging/fsl-mc/bus/mc-sys.c index 2c5f109..c4

[PATCH 0/4] staging: fsl-mc: MC command serialization

2015-10-14 Thread J. German Rivera
This patch series depends on the patch series posted at http://www.mail-archive.com/linux-kernel@vger.kernel.org/msg996780.html This patch series addresses the following item from the TODO list for the MC bus driver to exit staging: * Management Complex (MC) command serialization. Locking mechani

[PATCH 3/4] staging: fsl-mc:Added support for atomic portals

2015-10-14 Thread J. German Rivera
Refactored mc_send_command() to support two flavors of polling: - preemptible (for non-atomic portals), which was already supported. It calls usleep_range() between polling iterations. - non-preemptible (for atomic portals), which is needed when mc_send_command() is called with interrupts disab

[PATCH 1/4] staging: fsl-mc: changed timeout units for MC cmd completion

2015-10-14 Thread J. German Rivera
Changed units for the timeout to wait for completion of MC command, from jiffies to milliseconds. --- drivers/staging/fsl-mc/bus/mc-sys.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/fsl-mc/bus/mc-sys.c b/drivers/staging/fsl-mc/bus/mc-sys.c index 07848

[PATCH 4/4] staging: fsl-mc: Added serialization to mc_send_command()

2015-10-14 Thread J. German Rivera
When the same portal is used to call mc_send_command() from two different threads or a thread and an interrupt handler, serialization is required, as the MC only supports one outstanding command per MC portal. Thus, a new command should not be sent to the MC until the last command sent has been res

[PATCH v2 09/12] staging: fsl-mc: refactored error exit in allocator probe/remove

2015-10-14 Thread J. German Rivera
Replaced error gotos with direct returns in fsl_mc_allocator_probe() and fsl_mc_allocator_remove(), since the only error handling done in those functions is to exit. Signed-off-by: J. German Rivera --- CHANGE HISTORY Changes in v2: none drivers/staging/fsl-mc/bus/mc-allocator.c | 23 +-

[PATCH v2 04/12] staging: fsl-mc: Changed dev_info() calls to dev_dbg()

2015-10-14 Thread J. German Rivera
Changed dev_info() calls to dev_dbg() in fsl_mc_allocator_probe/fsl_mc_allocator_remove, as they are useful only for debugging. --- CHANGE HISTORY Changes in v2: none drivers/staging/fsl-mc/bus/mc-allocator.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/sta

[PATCH v2 02/12] staging: fsl-mc: fsl_mc_io object refactoring

2015-10-14 Thread J. German Rivera
Each fsl_mc_io object is associated with an fsl_mc_device object of type "dpmcp" representing the MC portal associated with the fsl_mc_io object. Before, we were representing this association with an fsl_mc_resource pointer. To enhance code clarity, it is more straight forward to use an fsl_mc_devi

[PATCH v2 12/12] staging: fsl-mc: Added missing initializer in fsl_mc_bus_driver

2015-10-14 Thread J. German Rivera
owner needs to be initialized as THIS_MOUDLE. --- CHANGE HISTORY Changes in v2: none drivers/staging/fsl-mc/bus/mc-bus.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/fsl-mc/bus/mc-bus.c b/drivers/staging/fsl-mc/bus/mc-bus.c index 4ac3d07..84db55b 100644 --- a/drivers/stag

[PATCH v2 10/12] staging: fsl-mc: Fixed WARN_ON() in fsl_mc_resource_pool_remove_device

2015-10-14 Thread J. German Rivera
Check that resource is not NULL before de-referencing it. --- CHANGE HISTORY Changes in v2: none drivers/staging/fsl-mc/bus/mc-allocator.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fsl-mc/bus/mc-allocator.c b/drivers/staging/fsl-mc/bus/mc-allocator.c in

[PATCH v2 11/12] staging: fsl-mc: fixed bug in uninitialized root dprc irq count

2015-10-14 Thread J. German Rivera
When initializing the object attributes for the root dprc, the irq_count was uninitialized. Initialize it to 1. --- CHANGE HISTORY Changes in v2: none drivers/staging/fsl-mc/bus/mc-bus.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/fsl-mc/bus/mc-bus.c b/drivers/staging/fs

[PATCH v2 00/12] staging: fsl-mc: Cleanup and bug fixes

2015-10-14 Thread J. German Rivera
This patch series includes the following code cleanup and bug fixes for the fsl-mc bus driver: Patch 1: Naming cleanup in fsl_mc-portal_allocate Patch 2: fsl_mc_io object refactoring Patch 3: dpmcp opening/closing refactoring Patch 4: Changed dev_info() calls to dev_dbg() Patch 5: Changed types of

[PATCH v2 03/12] staging: fsl-mc: dpmcp opening/closing refactoring

2015-10-14 Thread J. German Rivera
Before, we were opening and closing a mc_io's dpmcp object in fsl_mc_portal_reset(), since that was the only function that was calling dpmcp MC operations. However, it is better for maintainability to open the dpmcp object when it gets associated with an mc_io object, and close it when this associa

[PATCH v2 08/12] staging: fsl-mc: Fixed bug in fsl_mc_allocator_remove

2015-10-14 Thread J. German Rivera
Call fsl_mc_resource_pool_remove_device() only if mc_dev->resource is not NULL. --- CHANGE HISTORY Changes in v2: none drivers/staging/fsl-mc/bus/mc-allocator.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/staging/fsl-mc/bus/mc-allocator.c b/drivers/stagin

[PATCH v2 06/12] staging: fsl-mc: Removed unused DPMCP macros

2015-10-14 Thread J. German Rivera
The macros were a left-over from a previous implementation of the dpmcp APIs and are no longer used. --- CHANGE HISTORY Changes in v2: none drivers/staging/fsl-mc/bus/dpmcp-cmd.h | 79 -- 1 file changed, 79 deletions(-) diff --git a/drivers/staging/fsl-mc/bus/dpm

[PATCH v2 05/12] staging_fsl-mc: Changed types of flags, portal size in

2015-10-14 Thread J. German Rivera
Changed these two fields from 32-bit integers to 16-bit integers in struct fsl_mc_io, as 32 bits is too much for these fields. This change does not affect other components since fsl_mc_io is an opaque type. --- CHANGE HISTORY Changes in v2: none drivers/staging/fsl-mc/include/mc-sys.h | 4 ++--

[PATCH v2 07/12] staging: fsl-mc: Fixed alignment of copyright comment

2015-10-14 Thread J. German Rivera
Whitespace cleanup-- add missing spaces in column 1 of copyright --- CHANGE HISTORY Changes in v2: none drivers/staging/fsl-mc/include/dpcon-cmd.h | 60 +++--- 1 file changed, 30 insertions(+), 30 deletions(-) diff --git a/drivers/staging/fsl-mc/include/dpcon-cmd.h b/dr

[PATCH v2 01/12] staging: fsl-mc: Naming cleanup in fsl_mc-portal_allocate

2015-10-14 Thread J. German Rivera
mc_adev is a local variable for the allocated dpmcp object. Renamed mc_adev as dpmcp_dev for clarity. --- CHANGE HISTORY Changes in v2: none drivers/staging/fsl-mc/bus/mc-allocator.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/fsl-mc/bus/mc-

Re: [PATCH] staging: octeon: fixed few coding style warnings

2015-10-14 Thread Aaro Koskinen
Hi, On Wed, Oct 14, 2015 at 07:36:37PM +0530, Sakshi Bansal wrote: > Fixed allignment issues and line over 80 characters You are also doing many other things which are not mention in the commit log. > /* > - printk_ratelimited("Port %d

[PATCH] drivers:staging:gdm72xx: Fix coding style issues

2015-10-14 Thread Janis Gengeris
Fixes coding style warnings as reported by checkpatch.pl. Signed-off-by: Janis Gengeris --- drivers/staging/gdm72xx/gdm_wimax.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/gdm72xx/gdm_wimax.c b/drivers/staging/gdm72xx/gdm_wimax.c index 6e8dbaf..69f00c

[PATCH RESEND 1/4] Staging: rtl8712: Fix warning prefer eth_broadcast_addr() over memset()

2015-10-14 Thread Punit Vara
This patch is to the rtl871x_ioctl_linux.c that fixes up following warning by checkpatch.pl: -Prefer eth_broadcast_addr() over memset() Signed-off-by: Punit Vara --- drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/stagi

[PATCH v2 4/4] Staging: rtl8712: fix warning for placing constant on the right side of test

2015-10-14 Thread Punit Vara
Put constant on the right side of a test.Problem found using checkpatch: Warning:Comparisons should place the constant on the right side of the test Signed-off-by: Punit Vara --- drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 16 1 file changed, 8 insertions(+), 8 deletions(-

[PATCH v2 2/4] Staging: rtl8712: Use ether_addr_equal() over memcmp()

2015-10-14 Thread Punit Vara
This patch is to the rtl871x_ioctl_linux.c file that fixes up following warning reported by checkpatch.pl : -Prefer ether_addr_equal() or ether_addr_equal_unaligned() over memcmp() bssid and pnetwork->network.MacAddress both are 6 byte array which aligned with u16 Signed-off-by: Punit Vara ---

[PATCH RESEND 3/4] Staging: rtl8712: Coding style warnings fix for block comments

2015-10-14 Thread Punit Vara
This patch is to the rtl871x_ioctl_linux.c file that fixes up following warnings reported by checkpatch.pl : -Block comments use * on subsequent lines -Block comments use a trailing */ on a separate line Signed-off-by: Punit Vara --- drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 26 ++

[PATCH RESEND 0/4] Staging: rtl8712: Fix coding style warnings

2015-10-14 Thread Punit Vara
This set of patch fixes following warnings reported by checkpatch.pl: -Prefer eth_broadcast_addr() over memset() -Prefer ether_addr_equal() or ether_addr_equal_unaligned() over memcmp() -Block comments use * on subsequent lines -Block comments use a trailing */ on a separate line -Comparisons shou

Re: [PATCH] staging: nvec: fixed few coding style warnings

2015-10-14 Thread Dan Carpenter
On Wed, Oct 14, 2015 at 07:38:22PM +0530, Sakshi Bansal wrote: > Fixed allignmnet issues and block comments usage > Split it apart by type of fix. > @@ -617,7 +618,7 @@ static irqreturn_t nvec_interrupt(int irq, void *dev) > } else { > nvec->rx = nvec_msg_allo

Re: [PATCH] staging: octeon: fixed few coding style warnings

2015-10-14 Thread David Daney
On 10/14/2015 07:06 AM, Sakshi Bansal wrote: Fixed allignment issues and line over 80 characters Use spell checking on 'allignment' But that is not the main problem with the patch... You are changing things other than white space and comment formatting, can you tell us on which platforms th

Re: [PATCH] staging: octeon: fixed few coding style warnings

2015-10-14 Thread David Daney
On 10/14/2015 08:51 AM, Sakshi Bansal wrote: I am testing it on Fedora 21. Does your Fedora 21 platform use this driver in any way? Does it even build the driver (produce any .o files for any of the files you changed)? No, I didn't think so. You have to have a way to test the patch (or at

Re: [PATCH] staging: octeon: fixed few coding style warnings

2015-10-14 Thread David Daney
On 10/14/2015 09:04 AM, Sakshi Bansal wrote: Well I did not test the patch Never send patches to code that you don't at lease attempt to compile. since the changes were mostly coding style related or would not affect the code. As for the null check, if the original check is valid I can remov

I am Miss Gita Arora from New Delhi India.

2015-10-14 Thread
Dear Friend. I am Miss Gita Arora from New Delhi India I am 29Years Old Female, i am working in Reserve Bank of India New Delhi Branch Auditing and accounting section staff please do not feel bad, I viewed your Profile online while searching for a reliable person i request your consent a

[PATCH] staging: nvec: fixed few coding style warnings

2015-10-14 Thread Sakshi Bansal
Fixed allignmnet issues and block comments usage Signed-off-by: Sakshi Bansal --- drivers/staging/nvec/README | 2 +- drivers/staging/nvec/nvec.c | 41 --- drivers/staging/nvec/nvec_paz00.c | 1 - drivers/staging/nvec/nvec_power.c | 6 -- 4

[PATCH] staging: octeon: fixed few coding style warnings

2015-10-14 Thread Sakshi Bansal
Fixed allignment issues and line over 80 characters Signed-off-by: Sakshi Bansal --- drivers/staging/octeon/ethernet-defines.h | 2 +- drivers/staging/octeon/ethernet-mdio.c| 2 +- drivers/staging/octeon/ethernet-mem.c | 4 ++-- drivers/staging/octeon/ethernet-rgmii.c | 7 +++

[PATCH] staging: comedi: fixed few coding style warnings

2015-10-14 Thread Sakshi Bansal
Fixed block comments usage of * on subsequent lines Signed-off-by: Sakshi Bansal --- drivers/staging/comedi/comedi.h| 181 +++-- drivers/staging/comedi/comedi_pcmcia.h | 3 +- drivers/staging/comedi/comedilib.h | 26 ++--- 3 files changed, 120 insertion

[PATCH] staging: netlogic: xlr_net.h: fixedcoding stly warnings

2015-10-14 Thread Sakshi Bansal
Fixed block comments usage of * on subsequent lines Signed-off-by: Sakshi Bansal --- drivers/staging/netlogic/xlr_net.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/netlogic/xlr_net.h b/drivers/staging/netlogic/xlr_net.h index 2f65ec5..7ae8874 100644 ---

[PATCH] staging: unisys: visorinput: no need to track open count

2015-10-14 Thread Benjamin Romer
From: Tim Sell evdev_open_device() and evdev_close_device() (in evdev.c) already track the open count under lock, and will only call visorinput_open() on the first open and visorinput_close() on the last close. So this patch removes the unnessary logic from visorinput, and the now-unused counter

Re: [PATCH 00/45] staging: comedi: cb_pcidas: cleanup driver

2015-10-14 Thread Ian Abbott
On 14/10/15 01:47, H Hartley Sweeten wrote: Following is the big cleanup of the comedi cb_pcidas driver. H Hartley Sweeten (45): staging: comedi: cb_pcidas: tidy up multi-line comments staging: comedi: cb_pcidas: remove NUM_CHANNELS_* defines staging: comedi: cb_pcidas: refactor 'ai_bit

[PATCH 12/16] staging: wilc1000: move clean up codes into wl_wlan_cleanup function

2015-10-14 Thread Tony Cho
From: Glen Lee This patch moves clean up codes from exit_wilc_driver into the wl_wlan_cleanup newly introduced in this patch. In addition, it is called by linux_sdio_remove function. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan.c | 87 +++

[PATCH 09/16] staging: wilc1000: rename aBSSID in the struct wilc_vif

2015-10-14 Thread Tony Cho
This patch renames aBSSID in the struct wilc_vif to the bssid. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan.c | 14 +++--- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 6 +++--- drivers/staging/wilc1000/wilc_wfi_netdevice.h | 2 +- 3 files chan

[PATCH 10/16] staging: wilc1000: rename drvHandler in the struct wilc_vif

2015-10-14 Thread Tony Cho
This patch renames drvHandler in the struct wilc_vif to the hif_drv. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan.c | 2 +- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 24 +++ drivers/staging/wilc1000/wilc_wfi_netdevice.h | 2 +- 3

[PATCH 16/16] staging: wilc1000: assign pointer of wl to sdio device data

2015-10-14 Thread Tony Cho
From: Glen Lee This patch assigns wl pointer to sdio device data. The global variable wl will be removed finally. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan.c | 3 ++- drivers/staging/wilc1000/linux_wlan_sdio.c| 5 +++-- drivers/staging

[PATCH 15/16] staging: wilc1000: introduce struct wilc_sdio for sdio data

2015-10-14 Thread Tony Cho
From: Glen Lee This patch introduces struct wilc_sdio for sdio driver data. The wilc_sdio is allocated and set as driver data when the sdio is probed and deallocated when the sdio is removed. Signed-off-by: Glen Lee Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan_sdio.c | 16 +

[PATCH 13/16] staging: wilc1000: rename wilc_firmware in the struct wilc

2015-10-14 Thread Tony Cho
This patch renames wilc_firmware in the struct wilc to the firmware. In addition, null assignments to the wl->firmware after release_firmware are removed because it is not necessary. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan.c | 15 +++ drivers/staging/w

[PATCH 14/16] staging: wilc1000: remove COMPLEMENT_BOOT

2015-10-14 Thread Tony Cho
This patch removes a preprocessor definition, COMPLEMENT_BOOT which is not used anymore. This is just workaround to avoid weird issue, which is that 11b core is not ready after the power is givin to the chip. However, this issue happened only in the unstable hardware and no more seen. In addition,

[PATCH 11/16] staging: wilc1000: rename wilc_netdev in the struct wilc_vif

2015-10-14 Thread Tony Cho
This patch renames wilc_netdev in the struct wilc_vif to the ndev. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan.c | 45 +++ drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 34 - drivers/staging/wilc1000/wilc_wfi_netdevice.h

[PATCH 02/16] staging: wilc1000: remove typedef from the linux_wlan_t

2015-10-14 Thread Tony Cho
This patch removes typedef from the struct linux_wlan_t and renames it to the wilc. In addition, all of linux_wlan_t is replaced with struct wilc and memory allocation style is changed with preferred form as well like the following: p = kmalloc(sizeof(*p), ...) where "struct wilc" is used

[PATCH 08/16] staging: wilc1000: rename aSrcAddress in the struct wilc_vif

2015-10-14 Thread Tony Cho
This patch renames aSrcAddress in the struct wilc_vif to the src_addr. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan.c | 4 ++-- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 8 drivers/staging/wilc1000/wilc_wfi_netdevice.h | 2 +- 3 files changed

[PATCH 07/16] staging: wilc1000: rename strInterfaceInfo in the struct wilc

2015-10-14 Thread Tony Cho
This patch renames strInterfaceInfo in the struct wilc to the vif. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan.c | 74 +++ drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 74 +++ drivers/staging/wilc1000/wilc_wfi_netdevi

[PATCH 01/16] staging: wilc1000: remove extern declarations of g_linux_wlan

2015-10-14 Thread Tony Cho
This patch removes extern declaration of g_linux_wlan from the followings because it is declared as extern in the wilc_wfi_netdevice.h file. - linux_mon.c - linux_wlan_sdio.c - wilc_wfi_cfgoperations.c Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_mon.c | 2 -- drivers

[PATCH 03/16] staging: wilc1000: rename g_linux_wlan to wl

2015-10-14 Thread Tony Cho
This patch renames g_linux_wlan to wl. Signed-off-by: Tony Cho --- drivers/staging/wilc1000/linux_wlan.c | 289 +++--- drivers/staging/wilc1000/linux_wlan_sdio.c| 4 +- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 114 - drivers/staging/wilc10

  1   2   >