[PATCH] drivers: android: fix watermark when cma enabled

2014-08-05 Thread Zengtao (B)
From: zengtao prime.z...@hisilicon.com when cma is enabled, the watermark caculation will include the cma pages, while it can't use cma pages. this will cause oom while lowmemory killer don't work. Signed-off-by: zengtao prime.z...@hisilicon.com --- drivers/staging/android/lowmemorykiller.c |

[PATCH net-next] hyperv: Increase the buffer length for netvsc_channel_cb()

2014-08-05 Thread Haiyang Zhang
When the buffer is too small for a packet from VMBus, a bigger buffer will be allocated in netvsc_channel_cb() and retry reading the packet from VMBus. Increasing this buffer size will reduce the retry overhead. Signed-off-by: Haiyang Zhang haiya...@microsoft.com Reviewed-by: Dexuan Cui

Re: [PATCH 13/16] staging: comedi: cb_pcidas64: use comedi_pci_detach()

2014-08-05 Thread Ian Abbott
On 2014-08-04 23:55, H Hartley Sweeten wrote: Use comedi_pci_detach() to handle the boilerplate part of the (*detach) for this PCI driver. This also fixes a race condition where the interrupt handler is released before the interrupts are disabled in the hardware. Signed-off-by: H Hartley

Re: [PATCH 15/16] staging: comedi: gsc_hpdi: use comedi_pci_detach()

2014-08-05 Thread Ian Abbott
On 2014-08-04 23:55, H Hartley Sweeten wrote: Use comedi_pci_detach() to handle the boilerplate part of the (*detach) for this PCI driver. This also fixes a race condition where the interrupt handler is released before the interrupts are disabled in the hardware. Signed-off-by: H Hartley

Re: [PATCH v2 0/8] staging: iio: accel: Multiple coding style fixes

2014-08-05 Thread Dan Carpenter
Could you fold all the blank line patches into a single patch and do the other stuff in a different patch? regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org

Re: [PATCH 16/16] staging: comedi: rtd520: use comedi_pci_detach()

2014-08-05 Thread Ian Abbott
On 2014-08-04 23:55, H Hartley Sweeten wrote: Use comedi_pci_detach() to handle the boilerplate part of the (*detach) for this PCI driver. Remove the unnecessary read/write of the PLC_INTRCS_REG. The rtd_reset() function writes '0' to this register. Signed-off-by: H Hartley Sweeten

Re: [PATCH] drivers: android: fix watermark when cma enabled

2014-08-05 Thread Dan Carpenter
On Tue, Aug 05, 2014 at 09:43:48AM +, Zengtao (B) wrote: From: zengtao prime.z...@hisilicon.com This is for when you are sending patches on behalf of someone else. Just fix your from header. Use your real name. Zeng Tao? when cma is enabled, the watermark caculation will include the

Re: [PATCH 00/44] drivers/staging/bcm/Bcmchar.c variable name rename patches

2014-08-05 Thread Dan Carpenter
Combine all the patches which have the same subjects. regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH 07/10] staging: ozwpan: Use slab cache for oz_urb_link allocation

2014-08-05 Thread Dan Carpenter
On Mon, Aug 04, 2014 at 02:54:53PM +0200, Christoph Jaeger wrote: @@ -2341,6 +2289,11 @@ int oz_hcd_init(void) if (usb_disabled()) return -ENODEV; + + oz_urb_link_cache = KMEM_CACHE(oz_urb_link, 0); + if (unlikely(!oz_urb_link_cache)) + return

[PATCH 01/17] Staging: bcm: Bcmchar.c: Fixed indentation of function arguments

2014-08-05 Thread Matthias Beyer
Signed-off-by: Matthias Beyer m...@beyermatthias.de --- drivers/staging/bcm/Bcmchar.c | 56 --- 1 file changed, 31 insertions(+), 25 deletions(-) diff --git a/drivers/staging/bcm/Bcmchar.c b/drivers/staging/bcm/Bcmchar.c index c1e01f7..aff21d0 100644 ---

[PATCH 02/17] Staging: bcm: Bcmchar.c: Renamed variable Adapter - ad

2014-08-05 Thread Matthias Beyer
Renamed variable Adapter - ad in bcm_handle_nvm_read_cmd() handle_flash2x_adapter() bcm_char_open() bcm_char_release() bcm_char_read() bcm_char_ioctl_reg_read_private() bcm_char_ioctl_reg_write_private() bcm_char_ioctl_reg_write_private()

[PATCH 03/17] Staging: bcm: Bcmchar.c: Renamed variable pReadData - read_data

2014-08-05 Thread Matthias Beyer
Renamed variable pReadData - read_data in bcm_handle_nvm_read_cmd() handle_flash2x_adapter() Signed-off-by: Matthias Beyer m...@beyermatthias.de --- drivers/staging/bcm/Bcmchar.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git

[PATCH 05/17] Staging: bcm: Bcmchar.c: Renamed variable Status - status

2014-08-05 Thread Matthias Beyer
Renamed variable Status - status in bcm_handle_nvm_read_cmd() handle_flash2x_adapter() bcm_char_ioctl_reg_read_private() bcm_char_ioctl_reg_write_private() bcm_char_ioctl_reg_write_private() bcm_char_ioctl_eeprom_reg_read() bcm_char_ioctl_gpio_set_request()

[PATCH 04/17] Staging: bcm: Bcmchar.c: Renamed variable stNVMReadWrite - nvm_rw

2014-08-05 Thread Matthias Beyer
Renamed variable stNVMReadWrite - nvm_rw in bcm_handle_nvm_read_cmd() handle_flash2x_adapter() Signed-off-by: Matthias Beyer m...@beyermatthias.de --- drivers/staging/bcm/Bcmchar.c | 17 - 1 file changed, 8 insertions(+), 9 deletions(-) diff --git

[PATCH 00/17] drivers/staging/bcm/Bcmchar.c variable name rename patches

2014-08-05 Thread Matthias Beyer
Hi, as Dan suggested, I squashed the patches with the same subjects. Please note, as always: As I do not have the appropriate hardware, I'm not able to test these patches. I compiled them at least. Kind regards, Matthias Beyer Matthias Beyer (17): Staging: bcm: Bcmchar.c: Fixed indentation

[PATCH 06/17] Staging: bcm: Bcmchar.c: Renamed variable ulDSDMagicNumInUsrBuff - dsd_magic_num_in_usr_buff

2014-08-05 Thread Matthias Beyer
Renamed variable ulDSDMagicNumInUsrBuff - dsd_magic_num_in_usr_buff in handle_flash2x_adapter(). Signed-off-by: Matthias Beyer m...@beyermatthias.de --- drivers/staging/bcm/Bcmchar.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/bcm/Bcmchar.c

[PATCH 07/17] Staging: bcm: Bcmchar.c: Renamed variable pTarang - tarang

2014-08-05 Thread Matthias Beyer
Renamed variable pTarang - tarang in bcm_char_open() bcm_char_release() bcm_char_read() Signed-off-by: Matthias Beyer m...@beyermatthias.de --- drivers/staging/bcm/Bcmchar.c | 46 +-- 1 file changed, 23 insertions(+), 23 deletions(-) diff

[PATCH 08/17] Staging: bcm: Bcmchar.c: Renamed variable PktLen - pkt_len

2014-08-05 Thread Matthias Beyer
Renamed variable PktLen - pkt_len in bcm_char_read(). Signed-off-by: Matthias Beyer m...@beyermatthias.de --- drivers/staging/bcm/Bcmchar.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/bcm/Bcmchar.c b/drivers/staging/bcm/Bcmchar.c index

[PATCH 09/17] Staging: bcm: Bcmchar.c: Renamed variable Packet - packet

2014-08-05 Thread Matthias Beyer
Renamed variable Packet - packet in bcm_char_read(). Signed-off-by: Matthias Beyer m...@beyermatthias.de --- drivers/staging/bcm/Bcmchar.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/bcm/Bcmchar.c b/drivers/staging/bcm/Bcmchar.c index

[PATCH 11/17] Staging: bcm: Bcmchar.c: Renamed variable IoBuffer - io_buff

2014-08-05 Thread Matthias Beyer
Renamed variable IoBuffer - io_buff in bcm_char_ioctl_reg_read_private() bcm_char_ioctl_reg_write_private() bcm_char_ioctl_reg_write_private() bcm_char_ioctl_eeprom_reg_read() bcm_char_ioctl_gpio_set_request() Signed-off-by: Matthias Beyer m...@beyermatthias.de ---

[PATCH 16/17] Staging: bcm: Bcmchar.c: Renamed variable uiBit - bit

2014-08-05 Thread Matthias Beyer
Renamed variable uiBit - bit in bcm_char_ioctl_gpio_set_request(). Signed-off-by: Matthias Beyer m...@beyermatthias.de --- drivers/staging/bcm/Bcmchar.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/bcm/Bcmchar.c b/drivers/staging/bcm/Bcmchar.c

[PATCH 15/17] Staging: bcm: Bcmchar.c: Renamed variable ucResetValue - reset_val

2014-08-05 Thread Matthias Beyer
Renamed variable ucResetValue - reset_val in bcm_char_ioctl_gpio_set_request(). Signed-off-by: Matthias Beyer m...@beyermatthias.de --- drivers/staging/bcm/Bcmchar.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/bcm/Bcmchar.c

[PATCH 12/17] Staging: bcm: Bcmchar.c: Renamed variable Bufflen - buff_len

2014-08-05 Thread Matthias Beyer
Renamed variable Bufflen - buff_len in bcm_char_ioctl_reg_read_private(). Signed-off-by: Matthias Beyer m...@beyermatthias.de --- drivers/staging/bcm/Bcmchar.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/bcm/Bcmchar.c

[PATCH 14/17] Staging: bcm: Bcmchar.c: Renamed variable uiTempVar - tmp

2014-08-05 Thread Matthias Beyer
Renamed variable uiTempVar - tmp in bcm_char_ioctl_reg_write_private() bcm_char_ioctl_eeprom_reg_read() Signed-off-by: Matthias Beyer m...@beyermatthias.de --- drivers/staging/bcm/Bcmchar.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git

[PATCH 13/17] Staging: bcm: Bcmchar.c: Renamed variable sWrmBuffer - wrm_buff

2014-08-05 Thread Matthias Beyer
Renamed variable sWrmBuffer - wrm_buff in bcm_char_ioctl_reg_write_private() bcm_char_ioctl_eeprom_reg_read() Signed-off-by: Matthias Beyer m...@beyermatthias.de --- drivers/staging/bcm/Bcmchar.c | 32 1 file changed, 16 insertions(+), 16 deletions(-)

[PATCH 10/17] Staging: bcm: Bcmchar.c: Renamed variable sRdmBuffer - rdm_buff

2014-08-05 Thread Matthias Beyer
Renamed variable sRdmBuffer - rdm_buff in bcm_char_ioctl_reg_read_private() bcm_char_ioctl_reg_write_private() Signed-off-by: Matthias Beyer m...@beyermatthias.de --- drivers/staging/bcm/Bcmchar.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff

[PATCH 17/17] Staging: bcm: Bcmchar.c: Renamed variable uiOperation - operation

2014-08-05 Thread Matthias Beyer
Renamed variable uiOperation - operation in bcm_char_ioctl_gpio_set_request(). Signed-off-by: Matthias Beyer m...@beyermatthias.de --- drivers/staging/bcm/Bcmchar.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/bcm/Bcmchar.c

Re: [PATCH] staging: comedi: add NI USB-6501 initial support

2014-08-05 Thread Ian Abbott
On 2014-08-05 15:22, Luca Ellero wrote: This is a preliminary version, some features are not implemented yet: GPIO: works counter: doesn't work Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- This is a preliminary version of the NI USB-6501 driver. Every

RE: [PATCH 13/16] staging: comedi: cb_pcidas64: use comedi_pci_detach()

2014-08-05 Thread Hartley Sweeten
On Tuesday, August 05, 2014 2:59 AM, Ian Abbott wrote: On 2014-08-04 23:55, H Hartley Sweeten wrote: Use comedi_pci_detach() to handle the boilerplate part of the (*detach) for this PCI driver. This also fixes a race condition where the interrupt handler is released before the interrupts are

[PATCH 1/1] staging: lustre: Remove circular dependency on header

2014-08-05 Thread Pranith Kumar
The following patch fixes a build error on sparc32. I think it should go to stable 3.16. Remove a circular dependency on atomic.h header file which leads to compilation failure on sparc32 as reported here: http://kisskb.ellerman.id.au/kisskb/buildresult/11340509/ The specific dependency is as

[PATCH] staging: rtl8188eu: Fix static symbol sparse warning

2014-08-05 Thread Miguel Oliveira
From: Miguel cmro...@gmail.com Fix sparse warning: drivers/staging/rtl8188eu/core/rtw_cmd.c:52:5: warning: symbol '_rtw_enqueue_cmd' was not declared. Should it be static? by making it static since it's only used once. Signed-off-by: Miguel Oliveira cmro...@gmail.com ---

Re: [PATCH] staging: rtl8188eu: Fix static symbol sparse warning

2014-08-05 Thread Fabio Estevam
On Tue, Aug 5, 2014 at 3:49 PM, Miguel Oliveira cmro...@gmail.com wrote: From: Miguel cmro...@gmail.com Please use your full name in the From field. ___ devel mailing list de...@linuxdriverproject.org

Re: [PATCH 13/16] staging: comedi: cb_pcidas64: use comedi_pci_detach()

2014-08-05 Thread Ian Abbott
On 2014-08-05 17:12, Hartley Sweeten wrote: On Tuesday, August 05, 2014 2:59 AM, Ian Abbott wrote: On 2014-08-04 23:55, H Hartley Sweeten wrote: Use comedi_pci_detach() to handle the boilerplate part of the (*detach) for this PCI driver. This also fixes a race condition where the interrupt

Re: [PATCH] staging: rtl8188eu: Fix static symbol sparse warning

2014-08-05 Thread Dan Carpenter
On Tue, Aug 05, 2014 at 07:49:54PM +0100, Miguel Oliveira wrote: From: Miguel cmro...@gmail.com Leave this line out, git should get this from your email unless you are sending on someone else's behalf. Also this name is wrong and your email header has the correct full name. regards, dan

Re: [PATCH] staging: rtl8188eu: Fix static symbol sparse warning

2014-08-05 Thread Miguel Oliveira
It was a git miss configuration in a new host, that was fixed. Sorry for the inconvenient. Regards, Miguel Oliveira 2014-08-05 18:56 GMT+02:00 Dan Carpenter dan.carpen...@oracle.com: On Tue, Aug 05, 2014 at 07:49:54PM +0100, Miguel Oliveira wrote: From: Miguel cmro...@gmail.com Leave this

[PATCH] staging: rtl8188eu: HalPhyRf_8188e: Fix static symbol sparse warning

2014-08-05 Thread Miguel Oliveira
Fix sparse warning: drivers/staging/rtl8188eu/hal/HalPhyRf_8188e.c:25:4: warning: symbol 'ODM_GetRightChnlPlaceforIQK' was not declared. Should it be static? by making it static since it's only used once. Signed-off-by: Miguel Oliveira cmro...@gmail.com ---

RE: [PATCH 13/16] staging: comedi: cb_pcidas64: use comedi_pci_detach()

2014-08-05 Thread Hartley Sweeten
On Tuesday, August 05, 2014 9:54 AM, Ian Abbott wrote: On 2014-08-05 17:12, Hartley Sweeten wrote: On Tuesday, August 05, 2014 2:59 AM, Ian Abbott wrote: It still has a race condition between the interrupt handler and iounmap(devpriv-main_iobase) though. (Even though you've disabled PLX

[PATCH 02/14] staging: unisys: fix formatting in timskmod.h

2014-08-05 Thread Benjamin Romer
Fix all whitespace formatting issues in timskmod.h. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/include/timskmod.h | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/unisys/include/timskmod.h b/drivers/staging/unisys/include/timskmod.h

[PATCH 11/14] staging: unisys: remove do{} while(0) in macros in channel.h

2014-08-05 Thread Benjamin Romer
The CHANNEL_*_MISMATCH error message macros should not be inside of do blocks. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- .../staging/unisys/common-spar/include/channels/channel.h| 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git

[PATCH 04/14] staging: unisys: remove unused macros from timskmod.h

2014-08-05 Thread Benjamin Romer
Several macros in timskmod.h are unused. Remove them. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/include/timskmod.h | 14 -- 1 file changed, 14 deletions(-) diff --git a/drivers/staging/unisys/include/timskmod.h

[PATCH 05/14] staging: unisys: get rid of uiscmpxchg64

2014-08-05 Thread Benjamin Romer
Remove the uiscmpxchg64 macro from uisqueue.h and uisqueue.c. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/include/uisqueue.h | 15 --- drivers/staging/unisys/uislib/uisqueue.c | 6 ++ 2 files changed, 2 insertions(+), 19 deletions(-) diff

[PATCH 08/14] staging: unisys: fix spacing in iovmcall_gnuc.h

2014-08-05 Thread Benjamin Romer
There are spacing errors in the functions in iovmcall_gnuc. Fix these. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/common-spar/include/iovmcall_gnuc.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 10/14] staging: unisys: fix spacing in vbusdeviceinfo.h

2014-08-05 Thread Benjamin Romer
There was a missing line between declarations and code in vbusdeviceinfo.h. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/common-spar/include/vbusdeviceinfo.h | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 01/14] staging: unisys: get rid of unused VMMIO types

2014-08-05 Thread Benjamin Romer
Delete the unused common VMMIO types in timskmod.h. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/include/timskmod.h | 11 --- 1 file changed, 11 deletions(-) diff --git a/drivers/staging/unisys/include/timskmod.h

[PATCH 09/14] staging: unisys: clean up vmcall functions

2014-08-05 Thread Benjamin Romer
Clean up the format of the vmcall functions in iovmcall_gnuc.h. These functions are rewritten for clarity and to correct the indention, without changing any functionality. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- .../unisys/common-spar/include/iovmcall_gnuc.h | 26

[PATCH 14/14] staging: unisys: fix macros in iochannel.h

2014-08-05 Thread Benjamin Romer
Remove the do blocks around several macros and fix spacing and trailing semicolons. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- .../unisys/common-spar/include/channels/iochannel.h| 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git

[PATCH 03/14] staging: unisys: get rid of semaphore macros

2014-08-05 Thread Benjamin Romer
Remove all of the semaphore macros from timskmod.h and switch all uses of those types to the correct function names. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/include/timskmod.h | 28 -- drivers/staging/unisys/uislib/uislib.c

[PATCH 13/14] staging: unisys: clean up comments in controlvmchannel.h message packet

2014-08-05 Thread Benjamin Romer
The comments for the large union CONTROLVM_MESSAGE_PACKET were inconsistently placed and one extended past the 80 char limit. These are cleaned up. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- .../include/channels/controlvmchannel.h| 41 -- 1 file

[PATCH 06/14] staging: unisys: fix whitespace in uisutils.h

2014-08-05 Thread Benjamin Romer
Correct a couple of missing blank lines in uisutils.h. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/include/uisutils.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/unisys/include/uisutils.h

[PATCH 00/14] staging: unisys: common headers cleanup

2014-08-05 Thread Benjamin Romer
This patch series removes unused macros, replaces redundant macros, fixes spacing and other formatting issues, and eliminates checkpatch warnings from the header files in the include and common-spar directories, other than volatile and new typedef warnings. The volatile warnings will be addressed

[PATCH 12/14] staging: unisys: fix spacing in channel.h

2014-08-05 Thread Benjamin Romer
Two functions were missing space between declarations and code. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/common-spar/include/channels/channel.h | 2 ++ 1 file changed, 2 insertions(+) diff --git

[PATCH] staging: rtl8192u: remove unused function.

2014-08-05 Thread fernando . apesteguia
From: Fernando Apesteguia fernando.apesteg...@gmail.com Remove ComputeTxTime since it is not used. Signed-off-by: Fernando Apesteguia fernando.apesteg...@gmail.com --- drivers/staging/rtl8192u/r8192U_core.c | 22 -- 1 file changed, 22 deletions(-) diff --git

Re: [PATCH 1/1] staging: lustre: Remove circular dependency on header

2014-08-05 Thread Pranith Kumar
Please note, this also fixes the following build error on sparc64 http://kisskb.ellerman.id.au/kisskb/buildresult/11607883/ On Tue, Aug 5, 2014 at 12:27 PM, Pranith Kumar bobby.pr...@gmail.com wrote: The following patch fixes a build error on sparc32. I think it should go to stable 3.16.

Re: [PATCH net-next] hyperv: Increase the buffer length for netvsc_channel_cb()

2014-08-05 Thread David Miller
From: Haiyang Zhang thomas1...@gmail.com Date: Fri, 1 Aug 2014 15:55:30 + When the buffer is too small for a packet from VMBus, a bigger buffer will be allocated in netvsc_channel_cb() and retry reading the packet from VMBus. Increasing this buffer size will reduce the retry overhead.

RE: [PATCH net-next] hyperv: Increase the buffer length for netvsc_channel_cb()

2014-08-05 Thread Haiyang Zhang
-Original Message- From: David Miller [mailto:da...@davemloft.net] Sent: Tuesday, August 5, 2014 4:04 PM To: thomas1...@gmail.com Cc: net...@vger.kernel.org; Haiyang Zhang; KY Srinivasan; o...@aepfle.de; jasow...@redhat.com; linux-ker...@vger.kernel.org; driverdev-

[PATCH] drivers/staging/comedi/drivers/comedi_test.c: fixed coding style problems

2014-08-05 Thread Niklas Svensson
Fixed checkpatch.pl warnings Signed-off-by: Niklas Svensson n...@flawful.org --- drivers/staging/comedi/drivers/comedi_test.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/comedi/drivers/comedi_test.c b/drivers/staging/comedi/drivers/comedi_test.c

[PATCH v6 2/2] Staging: rts5208: Remove CONFIG_RTS5208_DEBUG option

2014-08-05 Thread Fabio Falzoi
CONFIG_RTS5208_DEBUG is no more needed, we rely on dynamic debug config options instead. Signed-off-by: Fabio Falzoi fabio.falzo...@gmail.com --- drivers/staging/rts5208/Kconfig | 7 --- drivers/staging/rts5208/sd.c| 5 + 2 files changed, 1 insertion(+), 11 deletions(-) diff --git

[PATCH v6 0/2] Staging: rts5208: Use standard debug features

2014-08-05 Thread Fabio Falzoi
Use standard debug features instead of relying on the custom configuration option CONFIG_RTS5208_DEBUG and a series of home grown macros. This new version contains the remaining two chunks of the v5 patchset: the first two chunks have already been merged. Changes in v6: * Use print_hex_dump_bytes

[PATCH v6 1/2] Staging: rts5208: Use dev_dbg and print_hex_dump_bytes to dump memory

2014-08-05 Thread Fabio Falzoi
Use dev_dbg with %*ph format specifier and print_hex_dump_bytes to dump memory instead of relying on custom macro. Signed-off-by: Fabio Falzoi fabio.falzo...@gmail.com --- drivers/staging/rts5208/ms.c| 5 +++-- drivers/staging/rts5208/rtsx_chip.c | 6 --

[PATCH] staging: et131x: Fix errors caused by phydev-addr accesses before initialisation

2014-08-05 Thread Mark Einon
Fix two reported bugs, caused by et131x_adapter-phydev-addr being accessed before it is initialised, by: - letting et131x_mii_write() take a phydev address, instead of using the one stored in adapter by default. This is so et131x_mdio_write() can use it's own addr value. - removing

[PATCH] Staging: comedi: comedidev.h: fix checkpatch code style issue

2014-08-05 Thread Dzonatans Melgalvis
Fixing checkpatch warning - blank line missing after declaration. Signed-off-by: Dzonatans Melgalvis dz.melgal...@gmail.com --- drivers/staging/comedi/comedidev.h | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/staging/comedi/comedidev.h b/drivers/staging/comedi/comedidev.h

Re: [PATCH] Staging: comedi: comedidev.h: fix checkpatch code style issue

2014-08-05 Thread Greg KH
On Wed, Aug 06, 2014 at 02:29:45AM +0300, Dzonatans Melgalvis wrote: Fixing checkpatch warning - blank line missing after declaration. Signed-off-by: Dzonatans Melgalvis dz.melgal...@gmail.com --- drivers/staging/comedi/comedidev.h | 6 ++ 1 file changed, 6 insertions(+) diff --git

[PATCH] staging: comedi: drivers.c: fix checkpatch code style issue

2014-08-05 Thread Dzonatans Melgalvis
Fixing checkpatch warning - blank line missing after declaration. Signed-off-by: Dzonatans Melgalvis dz.melgal...@gmail.com --- drivers/staging/comedi/drivers.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/comedi/drivers.c b/drivers/staging/comedi/drivers.c index

Re: [PATCH] Staging: comedi: comedidev.h: fix checkpatch code style issue

2014-08-05 Thread Joe Perches
On Wed, 2014-08-06 at 02:29 +0300, Dzonatans Melgalvis wrote: Fixing checkpatch warning - blank line missing after declaration. These are false positives from a previous version of checkpatch. Please use the version in -next instead. No errors are reported with that version. Signed-off-by:

Re: [PATCH] staging: comedi: drivers.c: fix checkpatch code style issue

2014-08-05 Thread Joe Perches
On Wed, 2014-08-06 at 03:53 +0300, Dzonatans Melgalvis wrote: Fixing checkpatch warning - blank line missing after declaration. [] diff --git a/drivers/staging/comedi/drivers.c b/drivers/staging/comedi/drivers.c [] @@ -440,6 +440,7 @@ static void *comedi_recognize(struct comedi_driver *driv,