[PATCH 2/5] unisys: use simpler kthread_ API

of the should_stop and a call to KILL can be replaced with kthread_stop function. Cc: Ken Cox j...@redhat.com Cc: Benjamin Romer benjamin.ro...@unisys.com Signed-off-by: Devendra Naga devendra.a...@gmail.com --- drivers/staging/unisys/uislib/uisthread.c | 20 +++- 1 file changed, 7

[PATCH 3/5] unisys: use kthread_should_stop in the thread

convert the users of should_stop variable into kthread_should_stop() API. Cc: Ken Cox j...@redhat.com Cc: Benjamin Romer benjamin.ro...@unisys.com Signed-off-by: Devendra Naga devendra.a...@gmail.com --- drivers/staging/unisys/virthba/virthba.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[RFC PATCH 0/5] unisys: kthread cleanup

should_stop variable. The 5th patch removes the variables should_stop and KILL as they are no longer required. All patches applies on next-20150120 cleanly. All the patches are compile tested on X86_64 allmodconfig. Devendra Naga (5): unisys: replace kthread_create and wake_up_process

[PATCH 1/5] unisys: replace kthread_create and wake_up_process with kthread_run

kthread_run calls kthread_create and if the thread is created it then calls wake_up_process on the corresponding returned task struct. So the code can be simplified by calling just kthread_run. Cc: Ken Cox j...@redhat.com Cc: Benjamin Romer benjamin.ro...@unisys.com Signed-off-by: Devendra Naga

[PATCH 4/5] unisys: use kthread_should_stop API in the lib thread

convert the users of should_stop into using kthread_should_stop API. Cc: Ken Cox j...@redhat.com Cc: Benjamin Romer benjamin.ro...@unisys.com Signed-off-by: Devendra Naga devendra.a...@gmail.com --- drivers/staging/unisys/uislib/uislib.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions

[PATCH 5/5] unisys: remove the thread variable and API

remove the should_stop variable and KILL API as they are no longer required Cc: Ken Cox j...@redhat.com Cc: Benjamin Romer benjamin.ro...@unisys.com Signed-off-by: Devendra Naga devendra.a...@gmail.com --- drivers/staging/unisys/include/uisthread.h | 1 - drivers/staging/unisys/uislib

[RFC PATCH 2/5] unisys: use simpler kthread_ API

of the should_stop and a call to KILL can be replaced with kthread_stop function. Cc: Ken Cox j...@redhat.com Cc: Benjamin Romer benjamin.ro...@unisys.com Signed-off-by: Devendra Naga devendra.a...@gmail.com --- drivers/staging/unisys/uislib/uisthread.c | 20 +++- 1 file changed, 7

[RFC PATCH 1/5] unisys: replace kthread_create and wake_up_process with kthread_run

kthread_run calls kthread_create and if the thread is created it then calls wake_up_process on the corresponding returned task struct. So the code can be simplified by calling just kthread_run. Cc: Ken Cox j...@redhat.com Cc: Benjamin Romer benjamin.ro...@unisys.com Signed-off-by: Devendra Naga

[RFC PATCH 5/5] unisys: remove the thread variable and API

remove the should_stop variable and KILL API as they are no longer required Cc: Ken Cox j...@redhat.com Cc: Benjamin Romer benjamin.ro...@unisys.com Signed-off-by: Devendra Naga devendra.a...@gmail.com --- drivers/staging/unisys/include/uisthread.h | 1 - drivers/staging/unisys/uislib

[RFC PATCH 3/5] unisys: use kthread_should_stop in the thread

convert the users of should_stop variable into kthread_should_stop() API. Cc: Ken Cox j...@redhat.com Cc: Benjamin Romer benjamin.ro...@unisys.com Signed-off-by: Devendra Naga devendra.a...@gmail.com --- drivers/staging/unisys/virthba/virthba.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[RFC PATCH 0/5] unisys: kthread cleanup

should_stop variable. The 5th patch removes the variables should_stop and KILL as they are no longer required. All patches applies on next-20150120 cleanly. All the patches are compile tested on X86_64 allmodconfig. Cc: Ken Cox j...@redhat.com Cc: Benjamin Romer benjamin.ro...@unisys.com Devendra

[PATCH] cptm1217: check if interrupts are masked at probe

the function cp_tm1217_mask_interrupt can return failure. added the check and the failure path. Cc: Ramesh Agarwal ramesh.agar...@intel.com Signed-off-by: Devendra Naga devendra.a...@gmail.com --- build tested only on x86_64. config is allmodconfig. drivers/staging/cptm1217/clearpad_tm1217.c

[PATCH V2 2/2] iio: meter: ade7759: add error handling in _reset and _stop_device

/staging/iio/meter/ade7759.c:309:6: warning: ‘val’ may be used uninitialized in this function [-Wmaybe-uninitialized] Signed-off-by: Devendra Naga devendra.a...@gmail.com --- Hello, Please see if this is the error message you wanted. Build tested on next-20141231 with randconfig sent to iio ml

[PATCH V2 1/2] iio: meter: ade7754: add error handling in _reset and _stop_device

/staging/iio/meter/ade7754.c:368:6: warning: ‘val’ may be used uninitialized in this function [-Wmaybe-uninitialized] Signed-off-by: Devendra Naga devendra.a...@gmail.com --- Hello, Please see if this is the error message you wanted. Build tested on next-20141231 with randconfig sent to iio ml

[PATCH] staging: mt29f_spinand: check for the return value of spinand_read_status

The spinan_read_status can fail, check for the return value and fail if the spinand_read_status fails. Cc: Kamlakant Patel kamlakant.pa...@broadcom.com Signed-off-by: Devendra Naga devendra.a...@gmail.com --- compile tested only with make allmodconfig (platform X86_64 ) drivers/staging

[PATCH] staging: clocking-wizard: check for a valid clk_name pointer

The clk_name gets the return value from kasprintf (part of which does the allocation of the returned buffer). So check the return pointer before using it for clk_register_fixed_factor. Cc: Soren Brinkmann soren.brinkm...@xilinx.com Signed-off-by: Devendra Naga devendra.a...@gmail.com --- I have

[PATCH] staging: rtl8192e: add missing tasklet_kill on remove path

The powersave tasklet is created in rtllib_softmac_init and it is not removed while unloading the module. Cc: Larry Finger larry.fin...@lwfinger.net Signed-off-by: Devendra Naga devendra.a...@gmail.com --- Compile tested only on top the linux-next for : 20141121 drivers/staging/rtl8192e

[PATCH 2/3] staging: gs_fpgpaboot: fix compiler warning with make W=1

drivers/staging/gs_fpgaboot/gs_fpga.o LD drivers/staging/gs_fpgaboot/built-in.o by removing the size variable. Signed-off-by: Devendra Naga devendrana...@gmail.com --- drivers/staging/gs_fpgaboot/gs_fpgaboot.c |2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging

[PATCH 1/3] staging: gs_fpgaboot: cleanup in gs_fpgaboot_init function

the error path is cleanup to return the correct (function call return value) error code. Signed-off-by: Devendra Naga devendrana...@gmail.com --- drivers/staging/gs_fpgaboot/gs_fpgaboot.c | 15 +-- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/drivers/staging

[PATCH 3/3] staging: gs_fpgaboot: fix a compiler warning with make W=2

declaration is here [-Wshadow] by renaming file function argument of gs_load_image with fw_file. Signed-off-by: Devendra Naga devendrana...@gmail.com --- drivers/staging/gs_fpgaboot/gs_fpgaboot.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/gs_fpgaboot

Re: [RFC 1/2] staging: ion: shrink page-pool by page unit

On Fri, Oct 24, 2014 at 5:01 AM, Gioh Kim gioh@lge.com wrote: 2014-10-24 오전 8:16, Gioh Kim 쓴 글: 2014-10-24 오전 5:36, Devendra Naga 쓴 글: Hello, On Thu, Oct 23, 2014 at 6:22 PM, Gioh Kim gioh@lge.com wrote: sys_heap = container_of(heap, struct ion_system_heap, heap

[PATCH v2] Fix error handling in dgnc_start()

to register_chrdev. Signed-off-by: Devendra Naga devendrana...@gmail.com --- Changes since v2: (added solutions for Dan's comments) 1. removed error message print 2. added error handling in device_create 3. added goto err_device_destroy when dgnc_tty_preinit fails drivers/staging/dgnc/dgnc_driver.c

Re: [PATCH] staging: dgnc: Fix a possible crash when the class_create fails

Hello Dan, On Wed, Oct 15, 2014 at 2:47 PM, Dan Carpenter dan.carpen...@oracle.com wrote: On Mon, Oct 13, 2014 at 10:26:54PM +0530, devendrana...@gmail.com wrote: From: Devendra Naga devendrana...@gmail.com Just fix your email client, so the from header is correct. These From headers