Re: [PATCH] staging: lustre: avoid going through unlock/lock overhead

2017-09-07 Thread Cihangir Akturk
On Thu, Sep 07, 2017 at 02:33:49PM +0200, Greg KH wrote: > On Thu, Sep 07, 2017 at 01:57:42PM +0300, Cihangir Akturk wrote: > > Unlocking a spin lock and then immediately locking without doing > > anything useful in between buys us nothing, except wasting CPU cycles. > > No

[PATCH] staging: lustre: avoid going through unlock/lock overhead

2017-09-07 Thread Cihangir Akturk
/o2iblnd.o After: text data bssdechex filename 70095 2356 4108 76559 12b0f drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.o Signed-off-by: Cihangir Akturk --- drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c | 18 -- 1 file changed, 8 insertions(+), 10 deletions

[PATCH] staging: pi433: fix interrupt handler signatures

2017-08-25 Thread Cihangir Akturk
Remove "struct pt_regs *" parameter from interrupt handlers, since it is no longer passed to interrupt handlers. Also, convert return types to irqreturn_t. Additionally, move DIO_irq_handler variable into the setup_GPIO function, as it's not used outside of this function. Signed-o

Re: [PATCH] staging: lustre: fix structure size for ARM OABI

2017-08-21 Thread Cihangir Akturk
On Sun, Aug 20, 2017 at 03:52:26AM +0100, James Simmons wrote: > > > On Aug 17, 2017, at 10:26, Greg KH wrote: > > > > > > On Wed, Aug 16, 2017 at 05:44:15PM +0300, Cihangir Akturk wrote: > > >> When building the kernel for the ARM architecture witho

[PATCH] staging: lustre: fix structure size for ARM OABI

2017-08-16 Thread Cihangir Akturk
this issue. Otherwise we get the following error: drivers/staging/lustre/lustre/lov/lov_pack.c:352:2: note: in expansion of macro ‘BUILD_BUG_ON’ BUILD_BUG_ON(sizeof(lum) != sizeof(struct lov_mds_md_v3)); Signed-off-by: Cihangir Akturk --- drivers/staging/lustre/lustre/include/lustre/lustre_user.h | 2

Re: [PATCH] [media] media: imx: depends on V4L2 sub-device userspace API

2017-08-14 Thread Cihangir Akturk
On Mon, Aug 14, 2017 at 10:57:46AM -0700, Steve Longerbeam wrote: > Hi Akturk, this has already been fixed, see Sorry. Apparently, I missed to update my tree, somehow. > > 4560cb4a0c ("media: imx: add VIDEO_V4L2_SUBDEV_API dependency"). > > Steve > > On 08/14

[PATCH] [media] media: imx: depends on V4L2 sub-device userspace API

2017-08-14 Thread Cihangir Akturk
-off-by: Cihangir Akturk --- drivers/staging/media/imx/Kconfig | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/media/imx/Kconfig b/drivers/staging/media/imx/Kconfig index 7eff50b..d8c3890 100644 --- a/drivers/staging/media/imx/Kconfig +++ b/drivers/staging/media

[PATCH] [media] media: imx: use setup_timer

2017-08-13 Thread Cihangir Akturk
Use setup_timer function instead of initializing timer with the function and data fields. Generated by: scripts/coccinelle/api/setup_timer.cocci. Signed-off-by: Cihangir Akturk --- drivers/staging/media/imx/imx-ic-prpencvf.c | 5 ++--- drivers/staging/media/imx/imx-media-csi.c | 5 ++--- 2

Re: [PATCH v3 00/28] DRM API Conversions

2017-08-11 Thread Cihangir Akturk
On Fri, Aug 11, 2017 at 02:24:19PM +, Deucher, Alexander wrote: > > -Original Message- > > From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf > > Of Cihangir Akturk > > Sent: Friday, August 11, 2017 8:33 AM > > Cc: de...@d

[PATCH v3 28/28] drm: vboxvideo: switch to drm_*_get(), drm_*_put() helpers

2017-08-11 Thread Cihangir Akturk
functions to use the new APIs. Generated by: scripts/coccinelle/api/drm-get-put.cocci Signed-off-by: Cihangir Akturk --- drivers/staging/vboxvideo/vbox_fb.c | 2 +- drivers/staging/vboxvideo/vbox_main.c | 8 drivers/staging/vboxvideo/vbox_mode.c | 2 +- 3 files changed, 6 insertions(+), 6

[PATCH v3 00/28] DRM API Conversions

2017-08-11 Thread Cihangir Akturk
ry Reding Date: Tue Feb 28 15:46:42 2017 +0100 drm: Introduce drm_property_blob_{get,put}() Cihangir Akturk (28): drm/amdgpu: switch to drm_*_get(), drm_*_put() helpers drm: mali-dp: switch to drm_*_get(), drm_*_put() helpers drm/armada: switch to drm_*_get(), drm_*_put() helpers drm/as

[PATCH v2 29/29] staging: vboxvideo: switch to drm_*_get(), drm_*_put() helpers

2017-08-10 Thread Cihangir Akturk
functions to use the new APIs. Generated by: scripts/coccinelle/api/drm-get-put.cocci Signed-off-by: Cihangir Akturk --- drivers/staging/vboxvideo/vbox_fb.c | 2 +- drivers/staging/vboxvideo/vbox_main.c | 8 drivers/staging/vboxvideo/vbox_mode.c | 2 +- 3 files changed, 6 insertions(+), 6

[PATCH v2 00/29] DRM API Conversions

2017-08-10 Thread Cihangir Akturk
m_object_{get,put}() commit 6472e5090be7c78749a3c279b4faae87ab835c40 Author: Thierry Reding Date: Tue Feb 28 15:46:42 2017 +0100 drm: Introduce drm_property_blob_{get,put}() Cihangir Akturk (29): drm/amdgpu: switch to drm_*_get(), drm_*_put() helpers drm: mali-dp: switch to drm_*_get(),

[PATCH] staging: lustre: mgc: fix potential use after free in error path

2017-08-07 Thread Cihangir Akturk
7f8c drivers/staging/lustre/lustre/mgc/mgc_request.o after: textdata bss dec hex filename 26092 22564208 325567f2c drivers/staging/lustre/lustre/mgc/mgc_request.o Signed-off-by: Cihangir Akturk --- drivers/staging/lustre/lustre/mgc/mgc_request.c | 16 +++--

Re: [PATCH] staging: lustre: grab the cld->cld_lock mutex unconditionally

2017-08-04 Thread Cihangir Akturk
On Thu, Aug 03, 2017 at 08:31:15PM +0300, Cihangir Akturk wrote: > On Thu, Aug 03, 2017 at 05:52:44PM +0100, James Simmons wrote: > > > > > Instead of using the locked variable as a helper to determine the state > > > of the mutex cld->cld_lock, expand the sco

Re: [PATCH] staging: lustre: grab the cld->cld_lock mutex unconditionally

2017-08-03 Thread Cihangir Akturk
eported by coccinelle: > > > > drivers/staging/lustre/lustre/mgc/mgc_request.c:359:2-12: second lock on > > line 365 > > > > Signed-off-by: Cihangir Akturk > > --- > > drivers/staging/lustre/lustre/mgc/mgc_request.c | 13 - > > 1 file

[PATCH 29/29] staging: vboxvideo: switch to drm_*{get,put} helpers

2017-08-03 Thread Cihangir Akturk
drm_*_reference() and drm_*_unreference() functions are just compatibility alias for drm_*_get() and drm_*_put() adn should not be used by new code. So convert all users of compatibility functions to use the new APIs. Signed-off-by: Cihangir Akturk --- drivers/staging/vboxvideo/vbox_fb.c | 2

Re: [PATCH] staging: vboxvideo: use drm_gem_object_put instead of drm_gem_object_unreference

2017-08-02 Thread Cihangir Akturk
On Wed, Aug 02, 2017 at 02:38:50PM +0200, Daniel Vetter wrote: > On Wed, Aug 2, 2017 at 1:46 AM, Cihangir Akturk wrote: > > drm_gem_object_unreference is a compatibility alias for drm_gem_object_put > > so should > > not be used by new code. > > > > This

[PATCH] staging: fsl-mc: fix resource_size.cocci warnings

2017-08-02 Thread Cihangir Akturk
Use resource_size() function on resource objects instead of explicit computation. This patch fixes the following coccinelle warning: drivers/staging/fsl-mc/bus/dprc-driver.c:620:35-38: ERROR: Missing resource_size with mc_dev -> regions Signed-off-by: Cihangir Akturk --- drivers/staging/

[PATCH] staging: vboxvideo: use drm_gem_object_put instead of drm_gem_object_unreference

2017-08-01 Thread Cihangir Akturk
-by: Cihangir Akturk --- drivers/staging/vboxvideo/vbox_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/vboxvideo/vbox_main.c b/drivers/staging/vboxvideo/vbox_main.c index d0c6ec7..1addeac 100644 --- a/drivers/staging/vboxvideo/vbox_main.c +++ b/drivers

[PATCH] staging: lustre: grab the cld->cld_lock mutex unconditionally

2017-08-01 Thread Cihangir Akturk
uest.o Additionally silences the following warning reported by coccinelle: drivers/staging/lustre/lustre/mgc/mgc_request.c:359:2-12: second lock on line 365 Signed-off-by: Cihangir Akturk --- drivers/staging/lustre/lustre/mgc/mgc_request.c | 13 - 1 file changed, 4 insertions(+), 9 del

[PATCH] staging: vboxvideo: select DRM_TTM

2017-08-01 Thread Cihangir Akturk
vboxvideo code uses various ttm_* functions, so it needs "drivers/gpu/drm/ttm/*" to be built. In order to build these objects select DRM_TTM. This commit fixes "ttm_*" undefined build errors. Signed-off-by: Cihangir Akturk --- drivers/staging/vboxvideo/Kconfig | 1

[PATCH] staging: lustre: split error handling code into multiple labels

2016-04-09 Thread Cihangir Akturk
. Additionally silences the following coccinelle warning: drivers/staging/lustre/lustre/obdecho/echo_client.c:762:22-27: ERROR: ed is NULL but dereferenced. Signed-off-by: Cihangir Akturk --- .../staging/lustre/lustre/obdecho/echo_client.c| 54 -- 1 file changed, 20 insertions

[PATCH] staging: fsl-mc: get rid of mutex_locked variables

2016-04-09 Thread Cihangir Akturk
lock on line 204 drivers/staging/fsl-mc/bus/mc-allocator.c:89:1-7: preceding lock on line 57 drivers/staging/fsl-mc/bus/mc-allocator.c:157:1-7: preceding lock on line 124 Signed-off-by: Cihangir Akturk --- drivers/staging/fsl-mc/bus/mc-allocator.c | 61 --- 1 file

[PATCH 2/2] staging: fsl-mc: fix incorrect type passed to dev_err macros

2016-03-14 Thread Cihangir Akturk
dev_err macros expect const struct device ** as its second argument, but here the argument we are passing is of typ struct device **. This patch fixes this error. Signed-off-by: Cihangir Akturk --- drivers/staging/fsl-mc/bus/mc-bus.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions

[PATCH 1/2] staging: fsl-mc: fix incorrect type passed to dev_dbg macros

2016-03-14 Thread Cihangir Akturk
dev_dbg macros expect const struct device ** as its second argument but here the argument we are passing is of type struct device ** this patch fixes this error. Signed-off-by: Cihangir Akturk --- drivers/staging/fsl-mc/bus/mc-sys.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions

[PATCH 0/2] fix incorrect type passed to dev_* macros

2016-03-14 Thread Cihangir Akturk
Following patches fixes the incorrect pointer type passed to dev_* macros. Cihangir Akturk (2): staging: fsl-mc: fix incorrect type passed to dev_dbg macros staging: fsl-mc: fix incorrect type passed to dev_err macros drivers/staging/fsl-mc/bus/mc-bus.c | 4 ++-- drivers/staging/fsl-mc/bus

[PATCH] drivers: staging: rtl8723au: remove unneeded null test

2016-03-04 Thread Cihangir Akturk
: Cihangir Akturk --- drivers/staging/rtl8723au/core/rtw_mlme.c | 13 ++--- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_mlme.c b/drivers/staging/rtl8723au/core/rtw_mlme.c index 3adda55..a786fc4 100644 --- a/drivers/staging/rtl8723au/core

Re: drivers: staging: rtl8723au: remove unneeded null test

2016-03-04 Thread Cihangir Akturk
On Fri, Mar 04, 2016 at 03:08:19PM +0200, Cihangir Akturk wrote: > null test on pnetwork removed, because the iterator variable > list_for_each_entry_safe cannot be null. > > This commit fixes the following error reported by coccinelle: > > drivers/staging/rtl8723au/core/rt

drivers: staging: rtl8723au: remove unneeded null test

2016-03-04 Thread Cihangir Akturk
: Cihangir Akturk --- drivers/staging/rtl8723au/core/rtw_mlme.c | 13 ++--- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_mlme.c b/drivers/staging/rtl8723au/core/rtw_mlme.c index 3adda55..a786fc4 100644 --- a/drivers/staging/rtl8723au/core

[PATCH v2] staging: lustre: obdclass: Make structure declerations static const

2015-07-22 Thread Cihangir Akturk
seq_operations should normally be const WARNING: struct file_operations should normally be const Signed-off-by: Cihangir Akturk --- drivers/staging/lustre/lustre/obdclass/linux/linux-module.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/obdclass

[PATCH v2] lustre: obdclass: Miscellaneous fixes

2015-07-22 Thread Cihangir Akturk
Previously sent two patches merged into one single patch, as suggested by Dan Carpenter. Cihangir Akturk (1): staging: lustre: obdclass: Make structure declerations static const drivers/staging/lustre/lustre/obdclass/linux/linux-module.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions

[PATCH] staging: lustre: ldlm: Make function static.

2015-07-21 Thread Cihangir Akturk
target_send_reply_msg function is not referenced outside of ldlm_lib.c file, so make it static. Signed-off-by: Cihangir Akturk --- drivers/staging/lustre/lustre/ldlm/ldlm_lib.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_lib.c b

[PATCH 0/2] staging: lustre: obdclass: Miscellaneous fixes

2015-07-20 Thread Cihangir Akturk
This patchset contains sparse and checkpatch.pl fixes. Cihangir Akturk (2): staging: lustre: obdclass: Make variables static. staging: lustre: obdclass: Add const qualifier to structure declerations. drivers/staging/lustre/lustre/obdclass/linux/linux-module.c | 4 ++-- 1 file changed, 2

[PATCH 2/2] staging: lustre: obdclass: Add const qualifier to structure declerations.

2015-07-20 Thread Cihangir Akturk
This patch fixes the following sparse warnings: WARNING: struct seq_operations should normally be const WARNING: struct file_operations should normally be const Signed-off-by: Cihangir Akturk --- drivers/staging/lustre/lustre/obdclass/linux/linux-module.c | 4 ++-- 1 file changed, 2 insertions

[PATCH 1/2] staging: lustre: obdclass: Make variables static.

2015-07-20 Thread Cihangir Akturk
obd_device_list_sops and obd_device_list_fops are not referenced outside of linux-module.c, so make them static. Signed-off-by: Cihangir Akturk --- drivers/staging/lustre/lustre/obdclass/linux/linux-module.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging

[PATCH] staging: rtl8192u: Fix static decleration sparse warning.

2015-03-24 Thread Cihangir Akturk
The function 'ieee80211_check_auth_response' is used only in this file, so make it static. This patch fixes the following sparse warning. 'ieee80211_check_auth_response' was not declared. Should it be static? Signed-off-by: Cihangir Akturk --- drivers/stagin

Re: [PATCH] staging: gs_fpgaboot: add bus_width module parameter

2014-08-22 Thread Cihangir Akturk
On Fri, Aug 22, 2014 at 11:14:09AM -0500, Greg KH wrote: > On Fri, Aug 22, 2014 at 02:16:02PM +0300, Cihangir Akturk wrote: > > This parameter allows the program bus width to be specified at > > module insertion time. > > Why? Who would set this? Why is it a module op

[PATCH] staging: gs_fpgaboot: add bus_width module parameter

2014-08-22 Thread Cihangir Akturk
This parameter allows the program bus width to be specified at module insertion time. Signed-off-by: Cihangir Akturk --- drivers/staging/gs_fpgaboot/gs_fpgaboot.c |6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/staging/gs_fpgaboot/gs_fpgaboot.c b/drivers

[PATCH] staging: gdm724x: fix misplaced open braces

2014-07-26 Thread Cihangir Akturk
This patch fixes the following checkpatch.pl issues in gdm_usb.c: ERROR: that open brace { should be on the previous line Signed-off-by: Cihangir Akturk --- drivers/staging/gdm724x/gdm_usb.c | 28 +++- 1 file changed, 11 insertions(+), 17 deletions(-) diff --git a