[PATCH 5/5] staging: wilc1000: move send_config_pkt to wilc_wlan.c

send_config_pkt function and it's related structure wid_type is moved to wilc_wlan.c because wilc_wlan.c handle message transport. the coreconfigurator is all about frames, so it will be frame.[ch] later. Signed-off-by: Glen Lee --- drivers/staging/wilc1000/coreconfigurator.c | 64 --

[PATCH 3/5] staging: wilc1000: send_config_pkt: return linux error number

Use proper linux error number instead of -1. Signed-off-by: Glen Lee --- drivers/staging/wilc1000/coreconfigurator.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/coreconfigurator.c b/drivers/staging/wilc1000/coreconfigurator.c index 637e8ca..1

[PATCH 4/5] staging: wilc1000: send_config_pkt: remove unnecessary blank line

This patch remove unnecessary blank line which is reported by checkpatch.pl Signed-off-by: Glen Lee --- drivers/staging/wilc1000/coreconfigurator.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/wilc1000/coreconfigurator.c b/drivers/staging/wilc1000/coreconfigurator.c index

[PATCH 2/5] staging: wilc1000: send_config_pkt: use netdev print

This patch use netdev_xxx print format instead of custom print api and printk. Signed-off-by: Glen Lee --- drivers/staging/wilc1000/coreconfigurator.c | 13 - 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/drivers/staging/wilc1000/coreconfigurator.c b/drivers/staging/

[PATCH 1/5] staging: wilc1000: change enum variable name with lower case

This patch changes WID_TYPE with wid_type which is preferred style. Signed-off-by: Glen Lee --- drivers/staging/wilc1000/coreconfigurator.h | 2 +- drivers/staging/wilc1000/wilc_wlan_if.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/coreconf

[PATCH] Staging: rtl8188eu: fix space prohibited before that ','

This patch fixes the following checkpatch.pl error: fix space prohibited before that ',' Signed-off-by: Abdul Hussain --- drivers/staging/rtl8188eu/core/rtw_xmit.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_xmit.c b/drivers/stag

Re.

I have an important discussion with you, please reply me here: paul.g.al...@usa.com ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH 00/19] staging/wilc1000 cleanups

On Friday 23 October 2015 16:51:13 Tony Cho wrote: > Hi Arnd, > First of all, I would like to say "thank you" for your efforts and > contributions. > We are updating the driver because new revision came up and making new > patches > to make it stable and elegant as Linux driver. In these days, we

[PATCH v6 6/7] drivers:hv: Define the channel type of Hyper-V PCI Express pass-through

From: Jake Oshins This defines the channel type for PCI front-ends in Hyper-V VMs. Signed-off-by: Jake Oshins --- include/linux/hyperv.h | 11 +++ 1 file changed, 11 insertions(+) diff --git a/include/linux/hyperv.h b/include/linux/hyperv.h index ea0a0e3..5587899 100644 --- a/include/

[PATCH v6 5/7] PCI: irqdomain: Look up IRQ domain by fwnode_handle

From: Jake Oshins This patch adds a second way of finding an IRQ domain associated with a root PCI bus. After looking to see if one can be found through the OF tree, it attempts to look up the IRQ domain through an fwnode_handle stored in the pci_sysdata struct. Signed-off-by: Jake Oshins ---

[PATCH v6 7/7] PCI: hv: New paravirtual PCI front-end for Hyper-V VMs

From: Jake Oshins This patch introduces a new driver which exposes a root PCI bus whenever a PCI Express device is passed through to a guest VM under Hyper-V. The device can be single- or multi-function. The interrupts for the devices are managed by an IRQ domain, implemented within the driver.

[PATCH v6 2/7] drivers:hv: Export hv_do_hypercall()

From: Jake Oshins This patch exposes the function that hv_vmbus.ko uses to make hypercalls. This is necessary for retargeting an interrupt when it is given a new affinity and vector. Signed-off-by: Jake Oshins --- drivers/hv/hv.c | 20 ++-- drivers/hv/hyperv_vmbus.h |

[PATCH v6 4/7] PCI: Add fwnode_handle to pci_sysdata

From: Jake Oshins This patch adds an fwnode_handle to struct pci_sysdata, which is used by the next patch in the series when trying to locate an IRQ domain associated with a root PCI bus. Signed-off-by: Jake Oshins --- arch/x86/include/asm/pci.h | 13 + include/asm-generic/pci.h |

[PATCH v6 3/7] PCI: Make it possible to implement a PCI MSI IRQ Domain in a module.

From: Jake Oshins The Linux kernel already has the concpet of IRQ domain, whereing a component can expose a set of IRQs which are managed by a particular interrupt controller chip or other subsystem. The PCI driver exposes the notion of an IRQ domain for Message-Signaled Interrupts (MSI) from PCI

[PATCH v6 0/7] PCI: hv: New paravirtual PCI front-end for Hyper-V VMs

From: Jake Oshins This version of this patch series incorporates feedback from Andy Shevchenko. First, export functions that allow correlating Hyper-V virtual processors and Linux cpus, along with the means for invoking a hypercall that targets interrupts at chosen vectors on specfic cpus. Seco

[PATCH v6 1/7] drivers:hv: Export a function that maps Linux CPU num onto Hyper-V proc num

From: Jake Oshins This patch exposes the mapping between Linux CPU number and Hyper-V virtual processor number. This is necessary because the hypervisor needs to know which virtual processor to target when making a mapping in the Interrupt Redirection Table in the I/O MMU. Signed-off-by: Jake Os

[PATCH] drivers:staging:gdm72xx Fix spaces preferred around that ...

This fixes all messages about spacing preferred from checkpatch.pl Signed-off-by: Bogicevic Sasa --- drivers/staging/gdm72xx/netlink_k.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/gdm72xx/netlink_k.c b/drivers/staging/gdm72xx/netlink_k.c index b7e5

[PATCH] drivers:staging:gdm72xx Fix no space is necessary after a cast

This fixes message no space is necessary after a cast Signed-off-by: Bogicevic Sasa --- drivers/staging/gdm72xx/netlink_k.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/gdm72xx/netlink_k.c b/drivers/staging/gdm72xx/netlink_k.c index f3cdaa6..b7e5e70 100644

[PATCH] drivers:staging:gdm72xx Fix spaces preferred around that ...

This fixes all spacing violation messages from checkpatch.pl Signed-off-by: Bogicevic Sasa --- drivers/staging/gdm72xx/gdm_wimax.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/gdm72xx/gdm_wimax.c b/drivers/staging/gdm72xx/gdm_wimax.c ind

RE: [PATCH 1/1] drivers/hv: correct tsc page sequence invalid value

> -Original Message- > From: Denis V. Lunev [mailto:d...@openvz.org] > Sent: Monday, November 2, 2015 3:34 AM > Cc: rka...@virtuozzo.com; de...@linuxdriverproject.org; linux- > ker...@vger.kernel.org; Andrey Smetanin ; KY > Srinivasan ; Haiyang Zhang > ; Vitaly Kuznetsov ; > Denis V. Lune

[PATCH v2 6/7] staging: lustre: remove white space in hash.c

Cleanup all the unneeded white space in hash.c. Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/libcfs/hash.c | 342 ++- 1 files changed, 177 insertions(+), 165 deletions(-) diff --git a/drivers/staging/lustre/lustre/libcfs/hash.c b/drivers/staging/lustr

[PATCH v2 5/7] staging: lustre: handle NULL comparisons correctly for libcfs_hash.h

Remove all direct NULL comparisons in libcfs_hash.h. Signed-off-by: James Simmons --- .../lustre/include/linux/libcfs/libcfs_hash.h |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/include/linux/libcfs/libcfs_hash.h b/drivers/staging/lustre

[PATCH v2 0/7] staging: lustre: second series for libcfs hash code cleanup

This patch covers more style cleanup series for the libcfs hash code. Mostly removal of white spaces and resolving the checkpath issues in libcfs_hash.h. James Simmons (7): staging: lustre: remove white space in libcfs_hash.h staging: lustre: remove obsolete comment in libcfs_hash.h staging:

[PATCH v2 4/7] staging: lustre: use proper comment blocks for libcfs_hash.h

The script checkpatch.pl reported problems with the style of the comment blocks. This patch resolves those problems. Signed-off-by: James Simmons --- .../lustre/include/linux/libcfs/libcfs_hash.h | 21 +-- 1 files changed, 14 insertions(+), 7 deletions(-) diff --git a/dri

[PATCH v2 1/7] staging: lustre: remove white space in libcfs_hash.h

Cleanup all the unneeded white space in libcfs_hash.h. Signed-off-by: James Simmons --- .../lustre/include/linux/libcfs/libcfs_hash.h | 135 ++-- 1 files changed, 70 insertions(+), 65 deletions(-) diff --git a/drivers/staging/lustre/include/linux/libcfs/libcfs_hash.h b/dr

[PATCH v2 3/7] staging: lustre: move linux hash.h header to start of libcfs_hash.h

Minor style cleanup to put hash.h header to the top of the libcfs_hash.h file. Signed-off-by: James Simmons --- .../lustre/include/linux/libcfs/libcfs_hash.h |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/include/linux/libcfs/libcfs_hash.

[PATCH v2 7/7] staging: lustre: place linux header first in hash.c

Always place linux headers first in libcfs header files. This avoid can potential build issues if any changes to a libcfs header land that starts using a linux header definition. Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/libcfs/hash.c |2 +- 1 files changed, 1 insertions

[PATCH v2 2/7] staging: lustre: remove obsolete comment in libcfs_hash.h

Remove comment hash_long which was removed long ago. Signed-off-by: James Simmons --- .../lustre/include/linux/libcfs/libcfs_hash.h |7 --- 1 files changed, 0 insertions(+), 7 deletions(-) diff --git a/drivers/staging/lustre/include/linux/libcfs/libcfs_hash.h b/drivers/staging/lus

[PATCH 2/3] staging: rtl8188eu: while loop replaced by for loop

Here is more suitable for loop. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/core/rtw_mlme.c | 9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_mlme.c b/drivers/staging/rtl8188eu/core/rtw_mlme.c index c1b82f7..5099c5a 1006

[PATCH 1/3] staging: rtl8188eu: do .. while (0) loop replaced by while (...) loop

It is a simple and clear representation of this loop. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/core/rtw_cmd.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_cmd.c b/drivers/staging/rtl8188eu/core/rtw_cmd.c index 9b

[PATCH 3/3] staging: rtl8188eu: 'infinite' loop removed

The body of this loop is executed only once, so it can be removed. In this loop no keyword 'continue', only 'break' at the end. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/hal/rtl8188eu_xmit.c | 34 -- 1 file changed, 15 insertions(+), 19 deletions(-) diff

[PATCH] drivers:staging:gdm72xx Fix two spacing issues

This fixes two "spaces preferred around ..." messages from checkpatch.pl Signed-off-by: Bogicevic Sasa --- drivers/staging/gdm72xx/gdm_sdio.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/gdm72xx/gdm_sdio.c b/drivers/staging/gdm72xx/gdm_sdio.c index b05

Re: [lustre-devel] [PATCH 1/3] staging: lustre: checkpatch cleanups for nidstring.c

Yeah. That is often the fastest way to fix all the checkpatch warnings. Checkpatch warnings are pretty mechanical. Just send like 100 patches at a time until everything is fixed. Don't overthink. Say your patch breaks the alignment then you have to fix that, but otherwise only fix one thing at

Re: [PATCH 2/2] comedi: drivers: Fix - BIT macro used coding style issue

On 02/11/15 14:25, ranjithec...@gmail.com wrote: From: Ranjith BIT macro is used for defining BIT location instead of shifting operator - coding style issue Signed-off-by: Ranjith --- drivers/staging/comedi/drivers/addi_apci_1032.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [PATCH 1/2] comedi: drivers: Fix - BIT macro used coding style issue

On 02/11/15 14:25, ranjithec...@gmail.com wrote: From: Ranjith BIT macro is used for defining bit location instead of shifting operator - coding style issue Signed-off-by: Ranjith T --- drivers/staging/comedi/drivers/addi_apci_1032.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletio

[PATCH 2/2] comedi: drivers: Fix - BIT macro used coding style issue

From: Ranjith BIT macro is used for defining BIT location instead of shifting operator - coding style issue Signed-off-by: Ranjith --- drivers/staging/comedi/drivers/addi_apci_1032.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/comedi/drivers/addi_apci

[PATCH 1/2] comedi: drivers: Fix - BIT macro used coding style issue

From: Ranjith BIT macro is used for defining bit location instead of shifting operator - coding style issue Signed-off-by: Ranjith T --- drivers/staging/comedi/drivers/addi_apci_1032.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/comedi/drivers/ad

[PATCH 14/17] staging: most: rename DIM_ServiceChannel to dim_service_channel

This patch renames DIM_ServiceChannel to dim_service_channel to avoid camelcase found by checkpatch. CHECK: Avoid CamelCase: FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:857: Signed-off-by: Chaehyun Lim --- drivers/staging/most/hdm-dim2/dim2_hal.c | 2 +- drivers/staging/most/hdm-dim2/dim2_h

[PATCH 12/17] staging: most: rename DIM_DestroyChannel to dim_destroy_channel

This patch renames DIM_DestroyChannel to dim_destroy_channel to avoid camelcase found by checkpatch. CHECK: Avoid CamelCase: FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:806: Signed-off-by: Chaehyun Lim --- drivers/staging/most/hdm-dim2/dim2_hal.c | 2 +- drivers/staging/most/hdm-dim2/dim2_h

[PATCH 15/17] staging: most: fix argument name of DIM_GetChannelState

The second argument name of DIM_GetChannelState declaration changes from dim_ch_state_ptr to state_ptr. The DIM_GetChannelState declaration and definition has same argument name as state_ptr. Signed-off-by: Chaehyun Lim --- drivers/staging/most/hdm-dim2/dim2_hal.h | 2 +- 1 file changed, 1 inser

[PATCH 08/17] staging: most: rename DIM_InitControl to dim_init_control

This patch renames DIM_InitControl to dim_init_control to avoid camelcase found by checkpatch. CHECK: Avoid CamelCase: FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:742: Signed-off-by: Chaehyun Lim --- drivers/staging/most/hdm-dim2/dim2_hal.c | 4 ++-- drivers/staging/most/hdm-dim2/dim2_hal.h

[PATCH 13/17] staging: most: rename DIM_ServiceIrq to dim_service_irq

This patch renames DIM_ServiceIrq to dim_service_irq to avoid camelcase found by checkpatch. CHECK: Avoid CamelCase: FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:819: Signed-off-by: Chaehyun Lim --- drivers/staging/most/hdm-dim2/dim2_hal.c | 2 +- drivers/staging/most/hdm-dim2/dim2_hal.h | 2

[PATCH 16/17] staging: most: rename DIM_GetChannelState to dim_get_channel_state

This patch renames DIM_GetChannelState to dim_get_channel_state to avoid camelcase found by checkpatch. CHECK: Avoid CamelCase: FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:865: Signed-off-by: Chaehyun Lim --- drivers/staging/most/hdm-dim2/dim2_hal.c | 4 ++-- drivers/staging/most/hdm-dim2/d

[PATCH 17/17] staging: most: rename DIM_EnqueueBuffer to dim_enqueue_buffer

This patch renames DIM_EnqueueBuffer to dim_enqueue_buffer to avoid camelcase found by checkpatch. CHECK: Avoid CamelCase: FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:877: Signed-off-by: Chaehyun Lim --- drivers/staging/most/hdm-dim2/dim2_hal.c | 3 ++- drivers/staging/most/hdm-dim2/dim2_ha

[PATCH 11/17] staging: most: rename DIM_InitSync to dim_init_sync

This patch renames DIM_InitSync to dim_init_sync to avoid camelcase found by checkpatch. CHECK: Avoid CamelCase: FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:781: Signed-off-by: Chaehyun Lim --- drivers/staging/most/hdm-dim2/dim2_hal.c | 4 ++-- drivers/staging/most/hdm-dim2/dim2_hal.h | 4 +

[PATCH 10/17] staging: most: rename DIM_InitIsoc to dim_init_isoc

This patch renames DIM_InitIsoc to dim_init_isoc to avoid camelcase found by checkpatch. CHECK: Avoid CamelCase: FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:756: Signed-off-by: Chaehyun Lim --- drivers/staging/most/hdm-dim2/dim2_hal.c | 4 ++-- drivers/staging/most/hdm-dim2/dim2_hal.h | 4 +

[PATCH 09/17] staging: most: rename DIM_InitAsync to dim_init_async

This patch renames DIM_InitAsync to dim_init_async to avoid camelcase found by checkpatch. CHECK: Avoid CamelCase: FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:749: Signed-off-by: Chaehyun Lim --- drivers/staging/most/hdm-dim2/dim2_hal.c | 4 ++-- drivers/staging/most/hdm-dim2/dim2_hal.h | 4

[PATCH 07/17] staging: most: rename DIM_NormSyncBufferSize to dim_norm_sync_buffer_size

This patch renames DIM_NormSyncBufferSize to dim_norm_sync_buffer_size to avoid camelcase found by checkpatch. CHECK: Avoid CamelCase: FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:734: Signed-off-by: Chaehyun Lim --- drivers/staging/most/hdm-dim2/dim2_hal.c | 2 +- drivers/staging/most/hdm-d

[PATCH 06/17] staging: most: rename DIM_NormIsocBufferSize to dim_norm_isoc_buffer_size

This patch renames DIM_NormIsocBufferSize to dim_norm_isoc_buffer_size to avoid camelcase found by checkpatch. CHECK: Avoid CamelCase: FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:720: Signed-off-by: Chaehyun Lim --- drivers/staging/most/hdm-dim2/dim2_hal.c | 2 +- drivers/staging/most/hdm-d

[PATCH 03/17] staging: most: rename DIMCB_IoWrite to dimcb_io_write

This patch renames DIMCB_IoWrite to dimcb_io_write to avoid camelcase found by checkpatch. CHECK: Avoid CamelCase: FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:154: Signed-off-by: Chaehyun Lim --- drivers/staging/most/hdm-dim2/dim2_hal.c | 76 drivers/staging

[PATCH 04/17] staging: most: rename DIMCB_IoRead to dimcb_io_read

This patch renames DIMCB_IoRead to dimcb_io_read to avoid camelcase found by checkpatch. CHECK: Avoid CamelCase: FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:157: Signed-off-by: Chaehyun Lim --- drivers/staging/most/hdm-dim2/dim2_hal.c | 16 drivers/staging/most/hdm-dim2/dim

[PATCH 05/17] staging: most: rename DIM_NormCtrlAsyncBufferSize to dim_norm_ctrl_async_buffer_size

This patch renames DIM_NormCtrlAsyncBufferSize to dim_norm_ctrl_async_buffer_size to avoid camelcase found by checkpatch CHECK: Avoid CamelCase: FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:709: Signed-off-by: Chaehyun Lim --- drivers/staging/most/hdm-dim2/dim2_hal.c | 2 +- drivers/staging/

[PATCH 01/17] staging: most: rename DIM_GetLockState to dim_get_lock_state

This patch renames DIM_GetLockState to dim_get_lock_state to avoid camelcase found by checkpatch. CHECK: Avoid CamelCase: FILE: drivers/staging/most/hdm-dim2/dim2_hdm.c:131: Signed-off-by: Chaehyun Lim --- drivers/staging/most/hdm-dim2/dim2_hal.c | 2 +- drivers/staging/most/hdm-dim2/dim2_hal.

[PATCH 02/17] staging: most: rename DIMCB_OnError to dimcb_on_error

This patch renames DIMCB_OnError to dimcb_on_error to avoid camelcase found by checkpatch. CHECK: Avoid CamelCase: FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:77: Signed-off-by: Chaehyun Lim --- drivers/staging/most/hdm-dim2/dim2_hal.c | 2 +- drivers/staging/most/hdm-dim2/dim2_hal.h | 2 +-

Re: [PATCH] Staging: rtl8188eu: checkpatch cleanup for core/rtw_xmit.c

On Mon, Nov 02, 2015 at 07:00:14AM +, Abdul Hussain wrote: > This patch will remove the error reported by checkpatch.pl script > You have to say what error was fixed and the subject is very vague as well. regards, dan carpenter ___ devel mailing l

Re: [PATCH 29/38] staging: wilc1000: remove goto from linux_wlan_start_firmware

On Mon, Nov 02, 2015 at 05:51:12PM +0900, Glen Lee wrote: > PRINT_D(INIT_DBG, "Waiting for Firmware to get ready ...\n"); > ret = linux_wlan_lock_timeout(&wilc->sync_event, 5000); > if (ret) { > PRINT_D(INIT_DBG, "Firmware start timed out"); > - goto _fai

[PATCH 1/1] drivers/hv: correct tsc page sequence invalid value

From: Andrey Smetanin Hypervisor Top Level Functional Specification v3/4 says that TSC page sequence value = -1(0x) is used to indicate that TSC page no longer reliable source of reference timer. Unfortunately, we found that Windows Hyper-V guest side implementation uses sequence value =

[PATCH 2/2] staging: rtl8188eu: add missing delay in polling loops.

Previously the code could exit with failure too early. Signed-off-by: Rémy Oudompheng --- drivers/staging/rtl8188eu/hal/rtl8188e_hal_init.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/hal/rtl8188e_hal_init.c b/drivers/staging/rtl8188eu/hal/rtl

[PATCH 1/2] staging: rtl8188eu: jiffies are unsigned long

Remove rtw_get_passing_time_ms function and adjust type of relevant variables. Signed-off-by: Rémy Oudompheng --- drivers/staging/rtl8188eu/core/rtw_efuse.c| 4 +-- drivers/staging/rtl8188eu/core/rtw_mlme.c | 13 --- drivers/staging/rtl8188eu/core/rtw_mlme_ext.c | 43 +++

Re: [PATCH 2/2] staging: rtl8188eu: add missing delay in polling loops.

2015-11-01 17:55 GMT+01:00 Larry Finger : > On 11/01/2015 04:30 AM, =?UTF-8?q?R=C3=A9my=20Oudompheng?= wrote: >> >> Previously the code could exit with failure too early. > > > Your mailer is broken. The above line is what shows as the return address. > > Larry > Sorry for that, I will resend the

Re: [PATCH v2 13/20] staging: rtl8188eu: operator = replaced by += in loop increment

On 10/28/2015 06:20 PM, Dan Carpenter wrote: On Wed, Oct 28, 2015 at 04:14:31PM +0530, Sudip Mukherjee wrote: On Wed, Oct 28, 2015 at 12:44:04PM +0300, Dan Carpenter wrote: On Wed, Oct 28, 2015 at 11:54:39AM +0530, Sudip Mukherjee wrote: On Tue, Oct 27, 2015 at 10:26:00PM +0700, Ivan Safonov w

[PATCH 33/38] staging: wilc1000: remove goto from mac_open

From: Leo Kim This patch removes goto from mac_open function. If address is invalid, goto handles deinit process and return result. So, just call deinit process and return the error value directly instead of goto statement. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wi

[PATCH 38/38] staging: wilc1000: rename pu8UdpBuffer of mac_xmit function

From: Leo Kim This patch rename pu8UdpBuffer variable of mac_xmit function to udp_buf to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/

[PATCH 34/38] staging: wilc1000: rename Set_machw_change_vir_if function

From: Leo Kim This patch rename Set_machw_change_vir_if function to set_machw_change_vir_if to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 2 +- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 6

[PATCH 35/38] staging: wilc1000: rename host_int_get_MacAddress function

From: Leo Kim This patch rename host_int_get_MacAddress function to hif_get_mac_address to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/host_interface.c | 2 +- drivers/staging/wilc1000/host_interface.h | 2 +- drivers/staging/

[PATCH 32/38] staging: wilc1000: remove goto from wlan_initialize_threads

From: Leo Kim This patch remove goto feature from wlan_initialize_threads function. Goto feature is 'wilc->close=0' & return result. So, remove goto feature and it was replaced with the return value directly, as well as removed unused ret variable. Also, execute 'wilc->close=0' before return. Si

[PATCH 30/38] staging: wilc1000: remove goto from linux_wlan_firmware_download

From: Leo Kim This patch remove goto feature from linux_wlan_firmware_download function. Goto feature is return result. So, remove goto functions and it was replaced with the return value directly. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 6 ++

[PATCH 37/38] staging: wilc1000: rename QueueCount of mac_xmit function

From: Leo Kim This patch rename QueueCount variable of mac_xmit function to queue_count to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a

[PATCH 31/38] staging: wilc1000: fixes missing a blank line after declarations

From: Leo Kim This patch fixes the warnings reported by checkpatch.pl for Missing a blank line after declarations. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/wilc1000/linux_wlan

[PATCH 36/38] staging: wilc1000: rename s32Error of mac_ioctl function

From: Leo Kim This patch rename s32Error variable of mac_ioctl function to ret to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/drive

[PATCH 28/38] staging: wilc1000: fixes braces {} should be used on all arms of this statement

From: Leo Kim This patch fixes the checks reported by checkpatch.pl for braces {} should be used on all arms of this statement. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/d

[PATCH 29/38] staging: wilc1000: remove goto from linux_wlan_start_firmware

From: Leo Kim This patch remove goto feature from linux_wlan_start_firmware function. Goto feature is return result. So, remove goto functions and it was replaced with the return value directly. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 5 ++---

[PATCH 27/38] staging: wilc1000: rename pBSSID of function linux_wlan_set_bssid

From: Leo Kim This patch renames pBSSID of function linux_wlan_set_bssid to bssid to avoid CamelCase naming convention. Also, prototype linux_wlan_set_bssid in wilc_wfi_cfgoperations.c is moved to wilc_wfi_netdevice.h. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc100

[PATCH 22/38] staging: wilc1000: remove do-nothing if condition case.

From: Leo Kim This patch removes do-nothing if condition case. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/staging/wilc1000/linux_wlan.c b/drivers/staging/wilc1000/linux_wlan.c index 72

[PATCH 23/38] staging: wilc1000: rename function GetIfHandler

From: Leo Kim This patch renames GetIfHandler function name to get_if_handler to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/

driverdev-devel@linuxdriverproject.org

From: Leo Kim This patch fixes add to spaces around that '&&' or '||'. Reported by checkpatch.pl for spaces required around that '&&' or '||' (ctx:VxE). Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 5 ++--- 1 file changed, 2 insertions(+), 3 dele

[PATCH 19/38] staging: wilc1000: remove extern function in c file and move it to header file.

From: Leo Kim This patch removes extern resolve_disconnect_aberration in c file and move it to proper header file. Rename argument also to match with declaration. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/host_interface.h | 2 +- drivers/staging/wilc1000/linu

[PATCH 26/38] staging: wilc1000: rename Bssid1 of function get_if_handler

From: Leo Kim This patch renames Bssid1 of function get_if_handler to bssid1 to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/st

[PATCH 18/38] staging: wilc1000: linux_wlan: remove unused defines

From: Leo Kim This patch removes unused defines from linux_wlan.c file. - NM73131 - PLAT_CLM9722 Two defines are support custom feature that don't used anymore. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 10 ++ 1 file changed, 2 insert

[PATCH 25/38] staging: wilc1000: rename Bssid of function get_if_handler

From: Leo Kim This patch renames Bssid of function get_if_handler to bssid to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/stag

[PATCH 20/38] staging: wilc1000: remove warnings line over 80 characters

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: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 40 +++ 1 file changed, 27 insertions(+), 13 deletions(-) diff --git

[PATCH 24/38] staging: wilc1000: rename pMacHeader of function get_if_handler

From: Leo Kim This patch renames pMacHeader of function get_if_handler to mac_header to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/

[PATCH 17/38] staging: wilc1000: linux_wlan.c: clean up comments

From: Leo Kim There are over-commenting in the linux_wlan.c file and most of them are not helpful to explain what the code does and generate 80 ending line over warnings. So, all of comments and commented codes are removed in this patch. Comment will be added if necessary with the preferred Linux

[PATCH 10/38] staging: wilc1000: linux_wlan: remove unused define CUSTOMER_PLATFORM

From: Leo Kim This patch remove unused define CUSTOMER_PLATFORM from linux_wlan.c. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 15 --- 1 file changed, 15 deletions(-) diff --git a/drivers/staging/wilc1000/linux_wlan.c b/drivers/stag

[PATCH 13/38] staging: wilc1000: rename pstrWFIDrv of function wilc_set_multicast_list

From: Leo Kim This patch renames pstrWFIDrv of function wilc_set_multicast_list to hif_drv to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --gi

[PATCH 12/38] staging: wilc1000: rename pstrWFIDrv of function linux_wlan_init_test_config

From: Leo Kim This patch renames pstrWFIDrv of function linux_wlan_init_test_config to hif_drv to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

[PATCH 16/38] staging: wilc1000: fixes blank lines aren't necessary brace

From: Leo Kim This patch fixes the checks reported by checkpatch.pl for Blank lines aren't necessary after an open brace '{' and Blank lines aren't necessary before a close brace '}'. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 21 ---

[PATCH 14/38] staging: wilc1000: rename pstrWFIDrv of function mac_close

From: Leo Kim This patch renames pstrWFIDrv of function mac_close to hif_drv to avoid CamelCase naming convention. And, some debug print modification that has been included name 'pstrWFIDrv'. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 8

[PATCH 11/38] staging: wilc1000: rename pstrWFIDrv of function dev_state_ev_handler

From: Leo Kim This patch renames pstrWFIDrv of function dev_state_ev_handler to hif_drv to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-)

[PATCH 09/38] staging: wilc1000: replace explicit NULL comparisons with !

From: Leo Kim This patch replace explicit NULL comparison with ! or unmark operator to simplify code. Reported by checkpatch.pl for comparison to NULL could be written "!XXX" or "XXX". Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 45 +

[PATCH 15/38] staging: wilc1000: rename pIP_Add_buff of function dev_state_ev_handler

From: Leo Kim This patch renames pIP_Add_buff of function dev_state_ev_handler to ip_addr_buf to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/linux_wlan.c | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-

[PATCH 05/38] staging: wilc1000: rename pstrHostIfGetMacAddress of fuction Handle_GetMacAddress

From: Leo Kim This patch renames pstrHostIfGetMacAddress of fuction Handle_GetMacAddress to get_mac_addr to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/host_interface.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

[PATCH 03/38] staging: wilc1000: remove unused parameter of fuction Handle_get_IPAddress

From: Leo Kim This patch removes parameter pu8IPAddr of fuction Handle_get_IPAddress because it is not used in the function. Remove argument in the function call also. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/host_interface.c | 4 ++-- 1 file changed, 2 inse

[PATCH 06/38] staging: wilc1000: rename strHostIFCfgParamAttr of fuction Handle_CfgParam

From: Leo Kim This patch renames strHostIFCfgParamAttr of fuction Handle_CfgParam to cfg_param_attr to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/host_interface.c | 154 -- 1 file changed, 83 inser

[PATCH 04/38] staging: wilc1000: rename pstrHostIfSetMacAddress of fuction Handle_SetMacAddress

From: Leo Kim This patch renames pstrHostIfSetMacAddress of fuction Handle_SetMacAddress to set_mac_addr to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/host_interface.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

[PATCH 08/38] staging: wilc1000: rename gau8MulticastMacAddrList variable

From: Leo Kim This patch renames gau8MulticastMacAddrList variable to multicast_mac_addr_list to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/host_interface.c | 5 +++-- drivers/staging/wilc1000/linux_wlan.c | 11 -

[PATCH 01/38] staging: wilc1000: rename pu8IPAddr of fuction Handle_set_IPAddress

From: Leo Kim This patch renames pu8IPAddr of fuction Handle_set_IPAddress to ip_addr to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/host_interface.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --

[PATCH 07/38] staging: wilc1000: remove return type of Handle_wait_msg_q_empty

From: Leo Kim This patch changes return type of Handle_wait_msg_q_empty from s32 with void because return value is not used. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/host_interface.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drive

[PATCH 02/38] staging: wilc1000: rename firmwareIPAddress of fuction Handle_set_IPAddress

From: Leo Kim This patch renames firmwareIPAddress of fuction Handle_set_IPAddress to firmware_ip_addr to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Glen Lee --- drivers/staging/wilc1000/host_interface.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)