[PATCH] staging: rtl8723au: fix up coding style warnings reported by checkpatch.pl.

2015-08-02 Thread Ting-Chih Hsiao
remove spaces at the start of a line align enum variable with other parameters Signed-off-by: Ting-Chih Hsiao s894...@hotmail.com --- drivers/staging/rtl8723au/hal/odm_RegConfig8723A.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH 2/2] iw: remove android-nl.c with unneeded workaround

2015-08-02 Thread Arik Nemtsov
On Fri, Jul 31, 2015 at 7:01 PM, enh e...@google.com wrote: no, because this is meant for the platform build system rather than the NDK. although the NDK has a concept of target API level, the platform only has current. Don't you have PLATFORM_VERSION?

[PATCH] Staging: wilc1000 :Insert blank line after declaration

2015-08-02 Thread Shraddha Barke
This patch fixes the following checkpatch.pl warning: WARNING: Missing a blank line after declarations Signed-off-by: Shraddha Barke shraddha.6...@gmail.com --- drivers/staging/wilc1000/coreconfigurator.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH] staging: vt6655: vnt_bss_info_changed check conf-beacon_rate is not NULL

2015-08-02 Thread Malcolm Priestley
conf-beacon_rate can be NULL on association. So check conf-beacon_rate BSS_CHANGED_BEACON_INFO needs to flagged in changed as the beacon_rate will appear later. Signed-off-by: Malcolm Priestley tvbox...@gmail.com Cc: sta...@vger.kernel.org # v3.19+ --- drivers/staging/vt6655/device_main.c | 5

Re: [PATCH] bcma: populate bus DT subnodes as platform_device-s

2015-08-02 Thread Hauke Mehrtens
On 06/28/2015 05:17 PM, Rafał Miłecki wrote: Our bus should allow defining children nodes as we may want to specify devices attached to the bus. This is required e.g. to specify NAND or ChipCommon cores and use bus's address and IRQ mappings. Signed-off-by: Rafał Miłecki zaj...@gmail.com

[PATCH -4.2] rtlwifi: rtl8723be: Add module parameter for MSI interrupts

2015-08-02 Thread Larry Finger
The driver code allows for the disabling of MSI interrupts; however the module_parm line was missed and the option fails to show with modinfo. Signed-off-by: Larry Finger larry.fin...@lwfinger.net Cc: Stable sta...@vger.kernel.org [3.15+] --- Kalle, This one-liner should be applied as soon as

Re: [PATCH v2] bcma: switch GPIO portions to use GPIOLIB_IRQCHIP

2015-08-02 Thread Hauke Mehrtens
On 08/02/2015 08:00 PM, Hauke Mehrtens wrote: From: Linus Walleij linus.wall...@linaro.org This switches the BCMA GPIO driver to use GPIOLIB_IRQCHIP to handle its interrupts instead of rolling its own copy of the irqdomain handling etc. Signed-off-by: Linus Walleij

drivers: staging: wilc1000: Fix a few style erros

2015-08-02 Thread Chandra S Gorentla
These patches fixes a few style errors reported by checkpatch.pl. PATCH 3/3 should be applied after PATCH 2/3. -- To unsubscribe from this list: send the line unsubscribe linux-wireless in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: [PATCH] staging: rtl8723au: fix coding style warnings reported by checkpatch.pl.

2015-08-02 Thread Greg KH
On Sun, Aug 02, 2015 at 01:00:58PM +0800, Ting-Chih Hsiao wrote: From 8c86ae927a38693f587826f51fca5d374e756768 Mon Sep 17 00:00:00 2001 From: Ting-Chih Hsiao s894...@hotmail.com Date: Sun, 2 Aug 2015 12:32:19 +0800 Why is this information in the body of your email? And please use

[PATCH v2] bcma: switch GPIO portions to use GPIOLIB_IRQCHIP

2015-08-02 Thread Hauke Mehrtens
From: Linus Walleij linus.wall...@linaro.org This switches the BCMA GPIO driver to use GPIOLIB_IRQCHIP to handle its interrupts instead of rolling its own copy of the irqdomain handling etc. Signed-off-by: Linus Walleij linus.wall...@linaro.org Signed-off-by: Hauke Mehrtens ha...@hauke-m.de ---

[PATCH 3/3] drivers: staging: wilc1000: remove space after '('

2015-08-02 Thread Chandra S Gorentla
The character ' ' is removed after the character '('. This fixes the checkpatch.pl error - space prohibited after that open parenthesis '('. Signed-off-by: Chandra S Gorentla csgoren...@gmail.com --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH 2/3] drivers: staging: wilc1000: move { to previous line

2015-08-02 Thread Chandra S Gorentla
Moved the opening brace of code blocks to the previous lines. This fixes the checkpatch.pl error - that open brace { should be on the previous line. Signed-off-by: Chandra S Gorentla csgoren...@gmail.com --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 9 +++-- 1 file changed, 3

[PATCH 1/3] drivers: staging: wilc1000: use 'void' for no arguments functions

2015-08-02 Thread Chandra S Gorentla
Added 'void' keyword in the paranthesis of function definitions, when there are no arguments to the functions. This fixes the checkpatch.pl error - Bad function definition 'function()' should probably be function(void). Signed-off-by: Chandra S Gorentla csgoren...@gmail.com ---

Re: [PATCH 2/2] iw: remove android-nl.c with unneeded workaround

2015-08-02 Thread enh
On Sat, Aug 1, 2015 at 11:57 PM, Arik Nemtsov a...@wizery.com wrote: On Fri, Jul 31, 2015 at 7:01 PM, enh e...@google.com wrote: no, because this is meant for the platform build system rather than the NDK. although the NDK has a concept of target API level, the platform only has current.

[PATCH] bcma: fix access to host_pdev for PCIe devices

2015-08-02 Thread Hauke Mehrtens
bus-host_pdev is part of a union so bus-host_pdev != NULL is probably also true for PCIe devices, because there it accesses bus-host_pci. If we access the dev member at the offset defined in struct platform_device in struct pci_dev instead we probably get something else. This patch adds a new