[PATCH] android: binder: Remove deprecated create_singlethread_workqueue

2016-08-13 Thread Bhaktipriya Shridhar
_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. Signed-off-by: B

[PATCH] Drivers: hv: connection: Remove create_workqueue

2016-06-18 Thread Bhaktipriya Shridhar
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 <bhaktipriy...@gmail.com> --- drivers/hv/connection.c | 3 ++- 1 fil

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

2016-02-27 Thread Bhaktipriya Shridhar
not be be 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 <bhaktipriy...@gmail.com> --- drivers/staging/octeon/ethe

[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 <bhaktipriy...@gmail.com> --- drivers/staging/lustre/lustre/obdclass/linux/linux-module.c | 3 +-- 1 file chan

[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 <bhaktipriy...@gmail.com> --- drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 10 -- 1 file changed, 10 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 <bhaktipriy...@gmail.com> --- Changes in v2: - Removed the unnecessary blank lines. drivers/staging/rtl8723a

[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 <bhaktipriy...@gmail.com> --- drivers/staging/comedi/drivers/dt2801.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/d

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

2016-01-09 Thread Bhaktipriya Shridhar
Replace all occurences of (1<<x) by BIT(x) in the file ad7150.c to get rid of checkpatch.pl "CHECK" output "Prefer using the BIT macro" Signed-off-by: Bhaktipriya Shridhar <bhaktipriy...@gmail.com> --- drivers/staging/iio/cdc/ad7150.c | 6 +++--- 1 file chang

[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 <bhaktipriy...@gmail.com> --- drivers/staging/lustre/lustre/obdclass/llog_swab.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/s

[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 <bhaktipriy...@gmail.com> --- drivers/staging/iio/accel/sca3000_ring.c | 2 +- 1 file changed, 1 insertion(+), 1 deletio

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

2015-12-13 Thread Bhaktipriya Shridhar
(*rx_p == NULL) { Signed-off-by: Bhaktipriya Shridhar <bhaktipriy...@gmail.com> --- 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(+

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

2015-12-13 Thread Bhaktipriya Shridhar
(*rx_p == NULL) { Signed-off-by: Bhaktipriya Shridhar <bhaktipriy...@gmail.com> --- 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/stagi

[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 <bhaktipriy...@gmail.com> --- drivers/staging/unisys/visorbus/visorchipset.c | 2 +- 1 file chan

[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 bhaktipriy...@gmail.com --- drivers/staging/dgnc/dgnc_mgmt.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/dgnc

[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 bhaktipriy...@gmail.com --- drivers/staging/dgnc/dgnc_mgmt.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/dgnc

[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: 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 bhaktipriy...@gmail.com --- drivers/staging/dgnc/dgnc_neo.c | 175 ++-- 1 file changed, 115 insertions(+), 60 deletions