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
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:
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:
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
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:
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
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
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:
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(-)
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(-)
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
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/
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
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
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/
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
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
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
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
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
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
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
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
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(-)
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
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
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
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/
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
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/
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
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
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
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(
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/
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
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
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
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 -
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
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
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
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/
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
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
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
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
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 --
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
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
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
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
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
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
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
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
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 +-
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
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
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
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
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
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
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
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
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
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 ++--
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
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-
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
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
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
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(-
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
---
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 ++
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
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
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
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
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
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
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
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 +++
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
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
---
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
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
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 +++
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
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
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
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 +
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
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,
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
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
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
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
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
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 - 100 of 106 matches
Mail list logo