[PATCH] android: binder: Remove deprecated create_singlethread_workqueue

2016-08-13 Thread Bhaktipriya Shridhar
ystem_wq allows multiple work items to overlap executions even on the same CPU; however, a per-cpu workqueue doesn't have any CPU locality or global ordering guarantee unless the target CPU is explicitly specified and thus the increase of local concurrency shouldn't make any difference. S

[PATCH] Drivers: hv: connection: Remove create_workqueue

2016-06-18 Thread Bhaktipriya Shridhar
een set to guarantee forward progress under memory pressure, which is a requirement in this case. Since there are only a fixed number of work items, explicit concurrency limit is unnecessary here. Signed-off-by: Bhaktipriya Shridhar --- drivers/hv/connection.c | 3 ++- 1 file changed, 2 insertion

[PATCH v3] staging: octeon: Convert create_singlethread_workqueue()

2016-02-27 Thread Bhaktipriya Shridhar
ordered. Hence, concurrency can be increased by switching to system_wq. All work items are sync canceled in cvm_oct_remove() so it is guaranteed that no work is in flight by the time exit path runs. Signed-off-by: Bhaktipriya Shridhar --- drivers/staging/octeon/ethernet.c| 21

[PATCH] staging: lusture: obdclass: Remove unnecessary NULL check

2016-02-18 Thread Bhaktipriya Shridhar
NULL check before the debugfs_remove_recursive function is not needed. This was detected using scripts/coccinelle/free/ifnullfree.cocci Signed-off-by: Bhaktipriya Shridhar --- drivers/staging/lustre/lustre/obdclass/linux/linux-module.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions

[PATCH v2] staging: rtl8723au: Fixes unnecessary return warning

2016-01-29 Thread Bhaktipriya Shridhar
This patch fixes checkpatch.pl warning in rtw_mlme_ext.c file. WARNING: void function return statements are not generally useful Signed-off-by: Bhaktipriya Shridhar --- Changes in v2: - Removed the unnecessary blank lines. drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 20

[PATCH] staging: rtl8723au: Fixes unnecessary return warning

2016-01-29 Thread Bhaktipriya Shridhar
This patch fixes checkpatch.pl warning in rtw_mlme_ext.c file. WARNING: void function return statements are not generally useful Signed-off-by: Bhaktipriya Shridhar --- drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/staging

[PATCH] staging: comedi: dt2801: Prefer using the BIT macro

2016-01-11 Thread Bhaktipriya Shridhar
As suggested by checkpatch.pl, this patch replaces bit shifting on 1 with the BIT(x) macro. Signed-off-by: Bhaktipriya Shridhar --- drivers/staging/comedi/drivers/dt2801.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/comedi/drivers

[PATCH] staging: iio: cdc: Prefer using the BIT macro

2016-01-09 Thread Bhaktipriya Shridhar
Replace all occurences of (1< --- drivers/staging/iio/cdc/ad7150.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/iio/cdc/ad7150.c b/drivers/staging/iio/cdc/ad7150.c index e8d0ff2..0b934f7 100644 --- a/drivers/staging/iio/cdc/ad7150.c +++ b/drivers/stagin

[PATCH] staging: lusture: obdclass: Remove return statement in void function

2015-12-31 Thread Bhaktipriya Shridhar
Fix the following checkpatch.pl warning: WARNING: void function return statements are not generally useful Signed-off-by: Bhaktipriya Shridhar --- drivers/staging/lustre/lustre/obdclass/llog_swab.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/obdclass

[PATCH v3] Staging: iio: accel: Fixed NULL comparison style

2015-12-13 Thread Bhaktipriya Shridhar
rx_p == NULL) { Signed-off-by: Bhaktipriya Shridhar --- Changes in v3: - Replaced "if (!rx_p)" with "if (!*rx_p)" - Changes are made on the original code drivers/staging/iio/accel/sca3000_ring.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/d

[PATCH v2] Staging: iio: accel: Fixed NULL comparison style

2015-12-13 Thread Bhaktipriya Shridhar
rx_p == NULL) { Signed-off-by: Bhaktipriya Shridhar --- Changes in v2: -Replaced "if (!rx_p)" with "if (!*rx_p)" drivers/staging/iio/accel/sca3000_ring.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/iio/accel/sca3000_ring.c b/driv

[PATCH] Staging: iio: accel: Fixed NULL comparison style

2015-12-13 Thread Bhaktipriya Shridhar
This patch fixes checkpatch.pl check: CHECK: Comparison to NULL could be written "!rx_p" + if (*rx_p == NULL) { Signed-off-by: Bhaktipriya Shridhar --- drivers/staging/iio/accel/sca3000_ring.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging

[PATCH] staging: unisys: visorchipset.c fixed spacing around operator

2015-12-09 Thread Bhaktipriya Shridhar
This patch fixes checkpatch.pl warning for visorchipset.c CHECK: spaces preferred around that '*' (ctx:VxV) +#define MAX_CONTROLVM_PAYLOAD_BYTES (1024*128) Signed-off-by: Bhaktipriya Shridhar --- drivers/staging/unisys/visorbus/visorchipset.c | 2 +- 1 file changed, 1 insertion(+),

[PATCH] Staging: dgnc: Fixed line over 80 characters in dgnc_mgmt.c

2015-08-09 Thread Bhaktipriya Shridhar
Fixed coding style issue "warning line over 80 characters" detected by checkpatch.pl in dgnc_mgmt.c Signed-off-by: Bhaktipriya Shridhar --- drivers/staging/dgnc/dgnc_mgmt.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/dgnc/dgnc_mgmt.c b/drive

[PATCH] Staging: dgnc: Fixed line over 80 characters in dgnc_mgmt.c

2015-08-09 Thread Bhaktipriya Shridhar
Fixed coding style issue "warning line over 80 characters" detected by checkpatch.pl in dgnc_mgmt.c Signed-off-by: Bhaktipriya Shridhar --- drivers/staging/dgnc/dgnc_mgmt.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/dgnc/dgnc_mgmt.c b/drive

[PATCH] Staging: dgnc: Fixed line over 80 characters in dgnc_mgmt.c

2015-08-09 Thread Bhaktipriya Shridhar
Fixed coding style issue "warning line over 80 characters" detected by checkpatch.pl in dgnc_mgmt.c --- drivers/staging/dgnc/dgnc_mgmt.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/dgnc/dgnc_mgmt.c b/drivers/staging/dgnc/dgnc_mgmt.c index b13318a..e8fd3da

[PATCH] staging: dgnc: dgnc_neo.c: Style and spelling fixes

2015-07-28 Thread Bhaktipriya Shridhar
- Removed the comment "Which I dont care about" (1 place) 2.Fixes code lines that are over 80-characters by -breaking up long math lines -breaking up long function calls 2.Spelling errors in comments. -tho -> though -baudrate -> baud rate

[PATCH] Staging: dgnc: dgnc_neo.c: Fix line over 80 characters

2015-07-28 Thread Bhaktipriya Shridhar
This patch fixes checkpatch.pl warnings in the file dgnc/dgnc_neo.c WARNING : line over 80 characters Signed-off-by: Bhaktipriya Shridhar --- drivers/staging/dgnc/dgnc_neo.c | 175 ++-- 1 file changed, 115 insertions(+), 60 deletions(-) diff --git a/drivers