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
/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
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
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
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
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
-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
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
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
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
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
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
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(),
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 +++--
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
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
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
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
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/
-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
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
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
.
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
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
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
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
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
: 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
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
: 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
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
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
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
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
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
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
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
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
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
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
40 matches
Mail list logo