[PATCH V2 2/8] Drivers: scsi: storvsc: Set cmd_per_lun to reflect value supported by the Host

Set cmd_per_lun to reflect value supported by the Host. In this version of the patch I have addressed comments from Christoph Hellwig h...@infradead.org Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c |2 +- 1 files changed, 1

Re: [PATCH 1/2] staging: rtl8723au: core: rtw_ap.c - removed NULL pointer check before kfree()

On Thursday 10 July 2014 11:25 AM, Anil Belur wrote: From: Anil Belur ask...@gmail.com - as kfree() internally check for NULL, additional check it not required. Sorry - please ignore this patch set, they are already fixed by someone else recently.

[PATCH v7 16/33] net: gdm_lte - set name assign type

Devices are named according to the scheme lteXpdnY, where Y is the nic type, and X is the normal enumeration, so the scheme is NET_NAME_ENUM. Signed-off-by: Tom Gundersen t...@jklm.no Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Won Kang wkan...@gmail.com Cc: Rashika Kheria

[PATCH v2 02/60] staging: ced1401: remove typedef for DMADESC

Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/usb1401.c |5 +++-- drivers/staging/ced1401/usb1401.h |6 +++--- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/staging/ced1401/usb1401.c b/drivers/staging/ced1401/usb1401.c index

[PATCH v2 00/60] staging: ced1401: rename camel case function and vars

This patchset renames all camel case and Hungarian notation names in ced1401. It fixes as well some checkpatch warnings and errors. I've not modified any logic in the driver. Please note that I've tryed to not add any checkpatch warning/error. If some patches give warnings or errors they were

[PATCH v2 04/60] staging: ced1401: rename members of struct circ_blk

Rename members with CamelCase and Hungarian notation Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 46 +++ drivers/staging/ced1401/usb1401.c | 74 ++--- drivers/staging/ced1401/usb1401.h |

[PATCH v2 06/60] staging: ced1401: rename members of struct dmadesc

Rename members with CamelCase and Hungarian notation Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 11 drivers/staging/ced1401/usb1401.c | 53 ++--- drivers/staging/ced1401/usb1401.h | 15 ++-

[PATCH v2 01/60] staging: ced1401: remove typedef for TRANSAREA

Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 21 - drivers/staging/ced1401/usb1401.c |9 ++--- drivers/staging/ced1401/usb1401.h |6 +++--- 3 files changed, 21 insertions(+), 15 deletions(-) diff --git

[PATCH v2 30/60] staging: ced1401: fix ced_quick_check()

Rename camel case arguments and locals in function ced_quick_check() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 36 +++- 1 file changed, 19 insertions(+), 17 deletions(-) diff --git

[PATCH v2 12/60] staging: ced1401: fix ced_stage_chunk()

Rename camel case arguments and locals in function ced_stage_chunk() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/usb1401.c | 53 ++--- 1 file changed, 31 insertions(+), 22 deletions(-) diff --git

[PATCH v2 18/60] staging: ced1401: fix ced_handle_esc()

Rename camel case arguments and locals in function ced_handle_esc() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/usb1401.c | 41 - 1 file changed, 22 insertions(+), 19 deletions(-) diff --git

[PATCH v2 15/60] staging: ced1401: fix ced_read_word()

Rename camel case arguments and locals in function ced_read_word() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/usb1401.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/drivers/staging/ced1401/usb1401.c

[PATCH v2 08/60] staging: ced1401: fix ced_writechar_callback()

Rename camel case locals in function ced_writechar_callback() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/usb1401.c | 36 ++-- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git

[PATCH v2 20/60] staging: ced1401: fix ced_allowi()

Rename camel case arguments and locals in function ced_allowi() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/usb1401.c | 47 - 1 file changed, 26 insertions(+), 21 deletions(-) diff --git

[PATCH v2 32/60] staging: ced1401: fix ced_get_string()

Rename camel case arguments and locals in function ced_get_string() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 58 ++--- drivers/staging/ced1401/usb1401.h |2 +- 2 files changed, 30 insertions(+), 30

[PATCH v2 27/60] staging: ced1401: fix ced_get_state()

Rename camel case arguments and locals in function ced_get_state() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 28 1 file changed, 16 insertions(+), 12 deletions(-) diff --git a/drivers/staging/ced1401/ced_ioc.c

[PATCH v2 19/60] staging: ced1401: fix ced_readchar_callback()

Rename camel case arguments and locals in function ced_readchar_callback() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/usb1401.c | 63 ++--- 1 file changed, 37 insertions(+), 26 deletions(-) diff --git

[PATCH v2 16/60] staging: ced1401: fix ced_read_huff()

Rename camel case arguments and locals in function ced_read_huff() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/usb1401.c | 45 - 1 file changed, 24 insertions(+), 21 deletions(-) diff --git

[PATCH v2 21/60] staging: ced1401: fix ced_ioctl()

Rename camel case arguments and locals in function ced_ioctl() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/usb1401.c | 66 + 1 file changed, 38 insertions(+), 28 deletions(-) diff --git

[PATCH v2 10/60] staging: ced1401: fix ced_copy_user_space()

Rename camel case arguments and locals in function ced_copy_user_space() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/usb1401.c | 66 + 1 file changed, 37 insertions(+), 29 deletions(-) diff --git

[PATCH v2 09/60] staging: ced1401: fix ced_send_chars()

Rename camel case arguments and locals in function ced_send_chars() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/usb1401.c | 44 ++--- 1 file changed, 22 insertions(+), 22 deletions(-) diff --git

[PATCH v2 25/60] staging: ced1401: fix ced_send_string()

Rename camel case arguments and locals in function ced_send_string() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 15 --- drivers/staging/ced1401/usb1401.h |2 +- 2 files changed, 9 insertions(+), 8 deletions(-) diff --git

[PATCH v2 31/60] staging: ced1401: fix ced_get_char()

Rename camel case arguments and locals in function ced_get_char() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/ced1401/ced_ioc.c

[PATCH v2 35/60] staging: ced1401: fix ced_get_out_buf_space()

Rename camel case arguments and locals in function ced_get_out_buf_space() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/drivers/staging/ced1401/ced_ioc.c

[PATCH v2 17/60] staging: ced1401: fix ced_read_dma_info()

Rename camel case arguments and locals in function ced_read_dma_info() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/usb1401.c | 90 - 1 file changed, 48 insertions(+), 42 deletions(-) diff --git

[PATCH v2 13/60] staging: ced1401: fix ced_read_write_mem()

Rename camel case arguments and locals in function ced_read_write_mem() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/usb1401.c | 86 + drivers/staging/ced1401/usb1401.h |5 ++- 2 files changed, 52 insertions(+), 39

[PATCH v2 05/60] staging: ced1401: rename members of struct transarea

Rename members with CamelCase and Hungarian notation Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 118 drivers/staging/ced1401/usb1401.c | 137 +++--

[PATCH v2 29/60] staging: ced1401: fix ced_is_1401()

Rename camel case arguments and locals in function ced_is_1401() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 48 + 1 file changed, 27 insertions(+), 21 deletions(-) diff --git

[PATCH v2 28/60] staging: ced1401: fix ced_in_self_test()

Rename camel case arguments and locals in function ced_in_self_test() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/ced1401/ced_ioc.c

[PATCH v2 24/60] staging: ced1401: fix ced_put_chars()

Rename camel case arguments and locals in function ced_put_chars() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/staging/ced1401/ced_ioc.c

[PATCH v2 34/60] staging: ced1401: fix ced_line_count()

Rename camel case arguments and locals in function ced_line_count() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 22 -- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/drivers/staging/ced1401/ced_ioc.c

[PATCH v2 14/60] staging: ced1401: fix ced_read_char()

Rename camel case arguments and locals in function ced_read_char() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/usb1401.c | 23 --- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/drivers/staging/ced1401/usb1401.c

[PATCH v2 36/60] staging: ced1401: fix ced_clear_area()

Rename camel case arguments and locals in function ced_clear_area() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 90 - drivers/staging/ced1401/usb1401.h |2 +- 2 files changed, 51 insertions(+), 41

[PATCH v2 26/60] staging: ced1401: fix ced_send_char()

Rename camel case arguments and locals in function ced_send_char() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/ced1401/ced_ioc.c

[PATCH v2 22/60] staging: ced1401: fix ced_disconnect()

Rename camel case locals in function ced_disconnect() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/usb1401.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/ced1401/usb1401.c b/drivers/staging/ced1401/usb1401.c

[PATCH v2 33/60] staging: ced1401: fix ced_stat_1401()

Rename camel case arguments and locals in function ced_stat_1401() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/ced1401/ced_ioc.c

[PATCH v2 23/60] staging: ced1401: fix comment

Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/ced1401/ced_ioc.c b/drivers/staging/ced1401/ced_ioc.c index 60a671e..1b7ed4f 100644 ---

[PATCH v2 49/60] staging: ced1401: fix ced_dbg_cmd()

Rename camel case arguments and locals in function ced_dbg_cmd() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/staging/ced1401/ced_ioc.c

[PATCH v2 37/60] staging: ced1401: fix ced_set_area()

Rename camel case arguments and locals in function ced_set_area() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 109 +++-- 1 file changed, 57 insertions(+), 52 deletions(-) diff --git

[PATCH v2 41/60] staging: ced1401: fix ced_wait_event()

Rename camel case arguments and locals in function ced_wait_event() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 58 - drivers/staging/ced1401/usb1401.h |2 +- 2 files changed, 33 insertions(+), 27

[PATCH v2 46/60] staging: ced1401: fix ced_check_self_test()

Rename camel case arguments and locals in function ced_check_self_test() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 40 +++-- drivers/staging/ced1401/usb1401.h |3 ++- 2 files changed, 23 insertions(+), 20

[PATCH v2 42/60] staging: ced1401: fix ced_test_event()

Rename camel case arguments and locals in function ced_test_event() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 20 +++- drivers/staging/ced1401/usb1401.h |2 +- 2 files changed, 12 insertions(+), 10 deletions(-) diff

[PATCH v2 43/60] staging: ced1401: fix ced_get_transfer()

Rename camel case arguments and locals in function ced_get_transfer() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 18 ++ drivers/staging/ced1401/usb1401.h |2 +- 2 files changed, 11 insertions(+), 9 deletions(-) diff

[PATCH v2 50/60] staging: ced1401: fix ced_dbg_peek/poke()

Rename camel case arguments and locals in function ced_dbg_peek() and ced_dbg_poke() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 44 ++--- drivers/staging/ced1401/usb1401.h |4 ++-- 2 files changed, 24

[PATCH v2 45/60] staging: ced1401: fix ced_start_self_test()

Rename camel case arguments and locals in function ced_start_self_test() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/drivers/staging/ced1401/ced_ioc.c

[PATCH v2 56/60] staging: ced1401: fix ced_free_circ_block()

Rename camel case arguments and locals in function ced_free_circ_block() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 103 - drivers/staging/ced1401/usb1401.h |2 +- 2 files changed, 58 insertions(+), 47

[PATCH v2 39/60] staging: ced1401: fix ced_unset_transfer()

Rename camel case arguments and locals in function ced_unset_transfer() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 10 +- drivers/staging/ced1401/usb1401.h |2 +- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git

[PATCH v2 48/60] staging: ced1401: fix ced_transfer_flags()

Rename camel case arguments and locals in function ced_transfer_flags() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers/staging/ced1401/ced_ioc.c

[PATCH v2 40/60] staging: ced1401: fix ced_set_event()

Rename camel case arguments and locals in function ced_set_event() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 33 - drivers/staging/ced1401/usb1401.h |3 ++- 2 files changed, 22 insertions(+), 14

[PATCH v2 53/60] staging: ced1401: fix ced_dbg_stop_loop()

Rename camel case arguments and locals in function ced_dbg_stop_loop() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/ced1401/ced_ioc.c

[PATCH v2 38/60] staging: ced1401: fix ced_set_transfer()

Rename camel case arguments and locals in function ced_set_transfer() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 19 +++ drivers/staging/ced1401/usb1401.h |3 ++- 2 files changed, 13 insertions(+), 9 deletions(-) diff

[PATCH v2 47/60] staging: ced1401: fix ced_type_of_1401()

Rename camel case arguments and locals in function ced_type_of_1401() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/staging/ced1401/ced_ioc.c

[PATCH v2 44/60] staging: ced1401: fix ced_state_of_1401()

Rename camel case arguments and locals in function ced_state_of_1401() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/ced1401/ced_ioc.c

[PATCH v2 54/60] staging: ced1401: fix ced_set_circular()

Rename camel case arguments and locals in function ced_set_circular() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 25 ++--- drivers/staging/ced1401/usb1401.h |3 ++- 2 files changed, 16 insertions(+), 12 deletions(-)

[PATCH v2 52/60] staging: ced1401: fix ced_dbg_get_data()

Rename camel case arguments and locals in function ced_dbg_get_data() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 19 ++- drivers/staging/ced1401/usb1401.h |2 +- 2 files changed, 11 insertions(+), 10 deletions(-) diff

[PATCH v2 57/60] staging: ced1401: usb1401.c fix checkpatch errors

Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/usb1401.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/ced1401/usb1401.c b/drivers/staging/ced1401/usb1401.c index 26e5f1b..a3d1ed6 100644 ---

[PATCH v2 58/60] staging: ced1401: usb1401.h fix checkpatch errors

Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/usb1401.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/ced1401/usb1401.h b/drivers/staging/ced1401/usb1401.h index 8e800c3..da4d90c 100644 ---

[PATCH v2 60/60] staging: ced1401: usb1401.c fix checkpatch warnings

Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/usb1401.c | 263 +++-- 1 file changed, 167 insertions(+), 96 deletions(-) diff --git a/drivers/staging/ced1401/usb1401.c b/drivers/staging/ced1401/usb1401.c index

[PATCH v2 55/60] staging: ced1401: fix ced_get_circ_block()

Rename camel case arguments and locals in function ced_get_circ_block() Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 34 +- drivers/staging/ced1401/usb1401.h |2 +- 2 files changed, 18 insertions(+), 18

Re: [PATCH 4/8] Drivers: scsi: storvsc: Filter WRITE_SAME_16

On Wed, Jul 09, 2014 at 10:36:26PM +, KY Srinivasan wrote: Ok; I am concerned about older kernels that do not have no_write_same flag. I suppose I can work directly with these Distros and give them a choice: either implement the no_write_same flag or filter the command in our driver. I

Re: [PATCH 6/8] Drivers: scsi: storvsc: Implement an abort handler

On Wed, Jul 09, 2014 at 06:51:38PM +, KY Srinivasan wrote: On Azure, we sometimes have unbounded I/O latencies and some distributions (such as SLES12) based on recent kernels are invoking the Abort Handler. Any kernel will invoke the abort handler if present, and then escalate to the

Re: [PATCH 7/8] drivers: scsi: storvsc: Set srb_flags in all cases

On Wed, Jul 09, 2014 at 06:40:09PM +, KY Srinivasan wrote: On Tue, Jul 08, 2014 at 05:46:51PM -0700, K. Y. Srinivasan wrote: Correctly set SRB flags for all valid I/O directions. Some IHV drivers on the Windows host require this. What are IHV drivers? If the target is a SAN

Re: [PATCH 7/8] drivers: scsi: storvsc: Set srb_flags in all cases

default: vm_srb-data_in = UNKNOWN_TYPE; - vm_srb-win8_extension.srb_flags = 0; + vm_srb-win8_extension.srb_flags |= (SRB_FLAGS_DATA_IN | + SRB_FLAGS_DATA_OUT); This would usually be a command that

Re: [PATCH V2 2/8] Drivers: scsi: storvsc: Set cmd_per_lun to reflect value supported by the Host

- .cmd_per_lun = 1, + .cmd_per_lun = 255, .can_queue =STORVSC_MAX_IO_REQUESTS*STORVSC_MAX_TARGETS, slave_configure immediately adjusts this down to STORVSC_MAX_IO_REQUESTS (250), any reson to start out with the magic 255 here?

Re: [PATCH 6/8] Drivers: scsi: storvsc: Implement an abort handler

On Wed, Jul 9, 2014 at 8:51 PM, KY Srinivasan k...@microsoft.com wrote: -Original Message- From: Christoph Hellwig [mailto:h...@infradead.org] Sent: Wednesday, July 9, 2014 1:44 AM To: KY Srinivasan Cc: linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; oher...@suse.com;

[PATCH] Staging: panel: fixed frivilous else statement warning

Fixed one instance of the following checkpatch.pl warning in panel.c: WARNING: else is not generally useful after a break or return Signed-off-by: Vincent Heuken m...@vincentheuken.com --- drivers/staging/panel/panel.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git

[PATCH 1/2] staging: comedi: vmk80xx: remove MODULE_VERSION() from in-kernel module

Signed-off-by: Ian Abbott abbo...@mev.co.uk --- drivers/staging/comedi/drivers/vmk80xx.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/comedi/drivers/vmk80xx.c b/drivers/staging/comedi/drivers/vmk80xx.c index 0adf3cf..831c3b7 100644 ---

[PATCH 2/2] staging: comedi: ni_daq_700: remove MODULE_VERSION() and commentary

We don't module version numbers for in-kernel drivers. The kernel version number is sufficient information. Signed-off-by: Ian Abbott abbo...@mev.co.uk --- drivers/staging/comedi/drivers/ni_daq_700.c | 7 --- 1 file changed, 7 deletions(-) diff --git

[PATCH 0/2] staging: comedi: remove MODULE_VERSION()

A couple of comedi drivers use MODULE_VERSION(), which is inappropriate for in-kernel drivers. 1) staging: comedi: vmk80xx: remove MODULE_VERSION() from in-kernel module 2) staging: comedi: ni_daq_700: remove MODULE_VERSION() and commentary drivers/staging/comedi/drivers/ni_daq_700.c | 7

[PATCH v4 0/4] staging: rtl8192u: checkpatch fixes

From: sanjeev sharma sanjeev_sha...@mentor.com This version of patch incorporated Joe Greg KH review comment. sanjeev sharma (4): staging: rtl8192u: Remove useless return statement staging: rtl8192u: Removed unnecessary whitespace tab warning staging: rtl8192u: Fixed switch and case

[PATCH v4 3/4] staging: rtl8192u: Fixed switch and case indentation error

From: sanjeev sharma sanjeev_sha...@mentor.com This patch will fix switch and case indentation Error reported by checkpatch script. Signed-off-by: Sanjeev Sharma sanjeev_sha...@mentor.com --- drivers/staging/rtl8192u/r819xU_phy.c | 12 ++-- 1 file changed, 6 insertions(+), 6

[PATCH v4 1/4] staging: rtl8192u: Remove useless return statement

From: sanjeev sharma sanjeev_sha...@mentor.com This patch will remove unneeded return statements in code. Signed-off-by: Sanjeev Sharma sanjeev_sha...@mentor.com --- drivers/staging/rtl8192u/r819xU_phy.c | 13 - 1 file changed, 13 deletions(-) diff --git

[PATCH v4 2/4] staging: rtl8192u: Removed unnecessary whitespace tab warning

From: sanjeev sharma sanjeev_sha...@mentor.com This patch will remove whitespace and tabs reported by checkpatch.pl script. Signed-off-by: Sanjeev Sharma sanjeev_sha...@mentor.com --- drivers/staging/rtl8192u/r819xU_phy.c | 23 --- 1 file changed, 12 insertions(+), 11

[PATCH v4 4/4] staging: rtl8192u: Fixed too long line

From: sanjeev sharma sanjeev_sha...@mentor.com This patch will fix too long lines warning reported by checkpatch.pl. Signed-off-by: Sanjeev Sharma sanjeev_sha...@mentor.com --- drivers/staging/rtl8192u/r819xU_phy.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git

RE: [PATCH v3 4/4] staging: rtl8192u: Fixed too long lines

Hi Joe, I have incorporated your review comment in version v4. Regards Sanjeev Sharma -Original Message- From: Joe Perches [mailto:j...@perches.com] Sent: Wednesday, July 09, 2014 6:27 PM To: sanjeev sharma Cc: gre...@linuxfoundation.org; rmf...@gmail.com; peter.se...@gmail.com;

[PATCH 2/2] staging: unisys: added virtpci info entry

This patch adds the virtpci debugfs directory and the info entry inside of it. Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/virtpci/virtpci.c | 123 ++- 1 file changed, 119

Re: [PATCH 2/2] staging: unisys: added virtpci info entry

On Thu, Jul 10, 2014 at 10:34:14AM -0400, Erik Arfvidson wrote: This patch adds the virtpci debugfs directory and the info entry inside of it. Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com 2/2? Where is patch 1/2? Please

Re: Anybody working on tidspbridge?

On Thu, Jul 10, 2014 at 12:11 PM, Greg KH gre...@linuxfoundation.org wrote: On Thu, Jul 10, 2014 at 08:54:18AM +0300, Ivaylo Dimitrov wrote: On 9.07.2014 10:07, Tony Lindgren wrote: * Suman Anna s-a...@ti.com [140708 11:40]: Hi Peter, On 07/08/2014 09:36 AM, Greg KH wrote: On

Re: Anybody working on tidspbridge?

On Thu, Jul 10, 2014 at 08:54:18AM +0300, Ivaylo Dimitrov wrote: On 9.07.2014 10:07, Tony Lindgren wrote: * Suman Anna s-a...@ti.com [140708 11:40]: Hi Peter, On 07/08/2014 09:36 AM, Greg KH wrote: On Tue, Jul 08, 2014 at 03:03:58PM +0200, Peter Meerwald wrote: Hello, Given the

[PATCH] staging: unisys: added virtpci info entry

This patch adds the virtpci debugfs directory and the info entry inside of it. Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com --- drivers/staging/unisys/virtpci/virtpci.c | 123 ++- 1 file changed, 119 insertions(+), 4 deletions(-) diff --git

[PATCH] staging: r8821ae: avoid leaking format string

This make sure a format string cannot leak into the work queue name nor the printk buffer. Signed-off-by: Kees Cook keesc...@chromium.org --- This is the third time this set of macros has entered the kernel: drivers/net/wireless/rtlwifi/btcoexist/halbtcoutsrc.h

Re: [PATCH] staging: r8821ae: avoid leaking format string

On Thu, Jul 10, 2014 at 01:16:11PM -0700, Kees Cook wrote: This make sure a format string cannot leak into the work queue name nor the printk buffer. Signed-off-by: Kees Cook keesc...@chromium.org --- This is the third time this set of macros has entered the kernel:

Re: [PATCH] staging: unisys: added virtpci info entry

On Thu, Jul 10, 2014 at 02:45:12PM -0400, Erik Arfvidson wrote: + str_pos += snprintf(vbuf + str_pos, len - str_pos, \n); + if (copy_to_user(buf, vbuf, str_pos)) { The length checking here still isn't correct. snprintf() returns the number of bytes which would have been printed if

Re: [PATCH] staging: unisys: added virtpci info entry

On Thu, Jul 10, 2014 at 02:45:12PM -0400, Erik Arfvidson wrote: This patch adds the virtpci debugfs directory and the info entry inside of it. Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com --- drivers/staging/unisys/virtpci/virtpci.c | 123 ++- 1

Re: [PATCH v2 58/60] staging: ced1401: usb1401.h fix checkpatch errors

On Thu, Jul 10, 2014 at 11:04:14AM +0200, Luca Ellero wrote: Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com Again, which one? ___ devel mailing list de...@linuxdriverproject.org

Re: [PATCH v2 57/60] staging: ced1401: usb1401.c fix checkpatch errors

On Thu, Jul 10, 2014 at 11:04:13AM +0200, Luca Ellero wrote: Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com Which checkpatch warnings? Please be specific. ___ devel mailing list de...@linuxdriverproject.org

Re: [PATCH v2 59/60] staging: ced1401: ced_ioc.c fix checkpatch warnings

On Thu, Jul 10, 2014 at 11:04:15AM +0200, Luca Ellero wrote: Signed-off-by: Luca Ellero luca.ell...@brickedbrain.com --- drivers/staging/ced1401/ced_ioc.c | 212 +++-- 1 file changed, 131 insertions(+), 81 deletions(-) That's a lot of changes all in one

RE: [PATCH V2 2/8] Drivers: scsi: storvsc: Set cmd_per_lun to reflect value supported by the Host

-Original Message- From: Christoph Hellwig [mailto:h...@infradead.org] Sent: Thursday, July 10, 2014 3:25 AM To: KY Srinivasan Cc: linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; oher...@suse.com; jbottom...@parallels.com; h...@infradead.org; jasow...@redhat.com;

Re: [PATCH v4 4/4] staging: rtl8192u: Fixed too long line

On Thu, Jul 10, 2014 at 05:54:57PM +0530, sanjeev sharma wrote: From: sanjeev sharma sanjeev_sha...@mentor.com This patch will fix too long lines warning reported by checkpatch.pl. Signed-off-by: Sanjeev Sharma sanjeev_sha...@mentor.com --- drivers/staging/rtl8192u/r819xU_phy.c | 9

Re: [PATCH 4/8] Drivers: scsi: storvsc: Filter WRITE_SAME_16

On Thu, 2014-07-10 at 21:02 +, KY Srinivasan wrote: -Original Message- From: James Bottomley [mailto:jbottom...@parallels.com] Sent: Wednesday, July 9, 2014 3:27 PM To: KY Srinivasan Cc: linux-ker...@vger.kernel.org; m...@mkp.net; h...@infradead.org;

RE: [PATCH 7/8] drivers: scsi: storvsc: Set srb_flags in all cases

-Original Message- From: Christoph Hellwig [mailto:h...@infradead.org] Sent: Thursday, July 10, 2014 3:17 AM To: KY Srinivasan Cc: Christoph Hellwig; linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; oher...@suse.com; jbottom...@parallels.com; jasow...@redhat.com;

RE: [PATCH 6/8] Drivers: scsi: storvsc: Implement an abort handler

-Original Message- From: Christoph Hellwig [mailto:h...@infradead.org] Sent: Thursday, July 10, 2014 3:13 AM To: KY Srinivasan Cc: Christoph Hellwig; linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; oher...@suse.com; jbottom...@parallels.com; jasow...@redhat.com;

[PATCH] staging: gdm724x: fix leak at failure path in init_usb()

If an allocation in init_usb() failed, it returns without deallocation of already allocated resources. The patch fix it and replaces GFP_ATOMIC by GFP_KERNEL in alloc_tx_sdu_struct() and alloc_rx_struct() as long as they are called from probe only. Found by Linux Driver Verification project

Re: [PATCH] [media] staging/solo6x10: SOLO6X10 should select BITREVERSE

On Sun, 6 Jul 2014 10:58:41 +0200 Geert Uytterhoeven ge...@linux-m68k.org wrote: If CONFIG_SOLO6X10=y, but CONFIG_BITREVERSE=m: drivers/built-in.o: In function `solo_osd_print': (.text+0x1c7a1f): undefined reference to `byte_rev_table' make: *** [vmlinux] Error 1 Reported-by:

[PATCH 0/4] staging: comedi: addi_apci_1564: small fixes in apci1564_auto_attach() and apci1564_detach()

This is a small patchset containing a handful of fixes to the ADDI-DATA APCI1564 driver that I would like to get out of the way before I forget to take care of them. From here, I will move to start fixing the digital input/timer/counters/watchdog functionality of the board. Chase Southwood (4):

[PATCH 1/4] staging: comedi: addi_apci_1564: remove len_chanlist from di and do subdevices

This value is only needed for subdevices that support async commands. The comedi core will default the value to 1 when it is not initialized. Signed-off-by: Chase Southwood chase.southw...@gmail.com Cc: Ian Abbott abbo...@mev.co.uk Cc: H Hartley Sweeten hswee...@visionengravers.com ---

[PATCH 2/4] staging: comedi: addi_apci_1564: remove unnecessary dev-board_name initialization

The dev-board_name is now initialized by the comedi core before calling the(*attach) or (*auto_attach) function in a driver. As long as the driver does no additional probing, it's no longer necessary initialize the board_name. Signed-off-by: Chase Southwood chase.southw...@gmail.com Cc: Ian

[PATCH 3/4] staging: comedi: addi_apci_1564: remove null check of devpriv in apci1564_detach()

There is no need to test whether devpriv is null in this function. The check looks left over and we can just remove it. Signed-off-by: Chase Southwood chase.southw...@gmail.com Cc: Ian Abbott abbo...@mev.co.uk Cc: H Hartley Sweeten hswee...@visionengravers.com ---

[PATCH 4/4] staging: comedi: addi_apci_1564: fix s-maxdata assignment in do subdevice init.

s-maxdata for the do subdevice should be 1, however currently it is being set to 0x. Fix this. Signed-off-by: Chase Southwood chase.southw...@gmail.com Cc: Ian Abbott abbo...@mev.co.uk Cc: H Hartley Sweeten hswee...@visionengravers.com --- This patch is at the end because I'm somewhat

Re: [BISECTED][REGRESSION] Loading Hyper-V network drivers is racy in 3.14+ on Hyper-V 2012 R2

On Mon, Jul 07, 2014 at 07:13:41PM +0100, Sitsofe Wheeler wrote: On Mon, Jul 07, 2014 at 04:54:20PM +, Haiyang Zhang wrote: -Original Message- From: Sitsofe Wheeler [mailto:sits...@gmail.com] Sent: Sunday, July 6, 2014 4:18 PM To: Haiyang Zhang Cc: KY Srinivasan;

  1   2   >