[PATCH] staging: android: binder.c: rename binder_user_error()

2014-08-10 Thread Tair Rzayev
Rename binder_user_error() to shorter but no less readable binder_usr_err() which fixes 34 "line over 80 characters" checkpatch warnings Signed-off-by: Tair Rzayev --- drivers/staging/android/binder.c | 74 1 file changed, 37 insertions(+), 37

[PATCH] staging: android: binder.c: binder_ioctl() cleanup

2014-06-03 Thread Tair Rzayev
binder_ioctl() is quite huge and checkpatch dirty - mostly because of the amount of code for the BINDER_WRITE_READ and BINDER_SET_CONTEXT_MGR. Moved that code into the new binder_ioctl_write_read() and binder_ioctl_set_ctx_mgr() Signed-off-by: Tair Rzayev --- drivers/staging/android/binder.c

[PATCH] staging: wlan-ng: cfg80211.c: Fix sparse warnings

2014-05-31 Thread Tair Rzayev
Include necessary header files to avoid "should be static" sparse warning Signed-off-by: Tair Rzayev --- drivers/staging/wlan-ng/cfg80211.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/wlan-ng/cfg80211.c b/drivers/staging/wlan-ng/cfg80211.c index f76f95

[PATCH] staging: media: solo6x10: solo6x10-jpeg.h: Fix sparse warning

2014-05-31 Thread Tair Rzayev
Make jpeg_dqt[][] static Signed-off-by: Tair Rzayev --- drivers/staging/media/solo6x10/solo6x10-jpeg.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/solo6x10/solo6x10-jpeg.h b/drivers/staging/media/solo6x10/solo6x10-jpeg.h index c5218ce..9e41185

Re: [PATCH] staging: android: binder.c: Use more appropriate functions for euid retrieval

2014-05-31 Thread Tair Rzayev
On 2014.05.31. 22:52, Greg KH wrote: > What kernel tree did you make this against? It doesn't seem to match up > with what is in my staging-next branch to merge with Linus for the next > release, so please work against that one. Made against staging-next branch of your linux-next staging tree.

[PATCH] staging: android: ion: ion_chunk_heap.c: Fix checkpatch warning

2014-05-31 Thread Tair Rzayev
Fix the over 80 character line Signed-off-by: Tair Rzayev --- drivers/staging/android/ion/ion_chunk_heap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/android/ion/ion_chunk_heap.c b/drivers/staging/android/ion/ion_chunk_heap.c index 3f2c12b..9c3e49a

[PATCH] staging: android: binder.c: Use more appropriate functions for euid retrieval

2014-05-31 Thread Tair Rzayev
Instead of getting the reference to whole credential structure, use task_euid() and current_euid() to get it. Signed-off-by: Tair Rzayev --- drivers/staging/android/binder.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/android/binder.c b

[PATCH v2] staging: keucr: smilmain.c: cosmetic updates

2014-05-04 Thread Tair Rzayev
* Move Check_D_MediaFmt() implementation up in smilmain.c to keep all the non-static functions at the top of the file; * Remove unnecessary extern and newlines from "init.h"; Signed-off-by: Tair Rzayev --- drivers/staging/keucr/init.h | 4 drivers/staging/keucr/smilm

Re: [PATCH v2] staging: keucr: smilmain.c: Fix sparse warning

2014-05-03 Thread Tair Rzayev
On 2014.04.26. 01:24, Greg KH wrote: > On Mon, Apr 21, 2014 at 08:42:47PM +0300, Tair wrote: >> From: Tair Rzayev >> >> Fix "Should be static" sparse warning: >> * Add Check_D_MediaFmt() declaration to smil.h; >> * Move Check_D_MediaFmt() implementatio