[PATCH 12/15] staging/lustre: Add spaces preferred around that '{+, -, *, /, |, <<

From: Emoly Liu This patch fixes all checkpatch occurences of "CHECK: spaces preferred around that '{+,-,*,/,|,<<,>>,&}' (ctx:VxV)" in Lustre code. Signed-off-by: Emoly Liu Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lu_object.h | 3 +- .../lustre/lustre/include/lus

[PATCH 14/15] staging/lustre: Make alignment match open parenthesis

From: Emoly Liu This patch fixes most of checkpatch occurences of "CHECK: Alignment should match open parenthesis" in Lustre code. Signed-off-by: Emoly Liu Signed-off-by: Oleg Drokin --- .../staging/lustre/include/linux/libcfs/libcfs.h | 6 +- .../lustre/include/linux/libcfs/libcfs_debug.

[PATCH 09/15] lustre: pass iov_iter to ->lnd_recv()

From: Al Viro Signed-off-by: Al Viro --- .../staging/lustre/include/linux/lnet/lib-types.h | 5 +--- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.h| 3 +- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c | 32 -- .../staging/lustre/lnet/klnds/socklnd/socklnd.h|

[PATCH 06/15] ksocknal_lib_send_kiov(): sendmsg doesn't bugger iovec...

From: Al Viro Signed-off-by: Al Viro --- .../lustre/lnet/klnds/socklnd/socklnd_lib.c| 29 -- 1 file changed, 5 insertions(+), 24 deletions(-) diff --git a/drivers/staging/lustre/lnet/klnds/socklnd/socklnd_lib.c b/drivers/staging/lustre/lnet/klnds/socklnd/socklnd_li

[PATCH 15/15] staging/lustre: Remove unused cp_error from struct cl_page

cp_error member is not really set anywhere, so kill it and the only printing user of it too. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/cl_object.h | 2 -- drivers/staging/lustre/lustre/obdclass/cl_page.c | 4 ++-- 2 files changed, 2 insertions(+), 4 deletions(-) diff

[PATCH 11/15] staging/lustre: Always return EEXIST on mkdir for existing names

if the name already exists, but we don't have write permissions in the parent, force talking to the MDS to determine what more sensical error code to return. This also happens to fix matlab and other such programs that assume that EEXIST is the only valid error code for mkdir of an existing directo

[PATCH 05/15] ksocknal_lib_send_iov(): sendmsg doesn't bugger iovec...

From: Al Viro Signed-off-by: Al Viro --- .../lustre/lnet/klnds/socklnd/socklnd_lib.c| 38 ++ 1 file changed, 10 insertions(+), 28 deletions(-) diff --git a/drivers/staging/lustre/lnet/klnds/socklnd/socklnd_lib.c b/drivers/staging/lustre/lnet/klnds/socklnd/socklnd_l

[PATCH 13/15] staging/lustre: Fix unnecessary parentheses around variables

From: Emoly Liu This patch fixes all checkpatch occurences of "CHECK: Unnecessary parentheses around xxx" in Lustre code. Signed-off-by: Emoly Liu Signed-off-by: Oleg Drokin --- .../staging/lustre/lustre/include/lustre/lustre_user.h | 2 +- drivers/staging/lustre/lustre/lov/lov_obd.c

[PATCH 03/15] lustre: don't reinvent struct bio_vec

From: Al Viro Signed-off-by: Al Viro --- drivers/staging/lustre/include/linux/lnet/types.h | 16 + .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c | 10 +-- .../staging/lustre/lnet/klnds/socklnd/socklnd_cb.c | 16 ++--- .../lustre/lnet/klnds/socklnd/socklnd_lib.c| 42 +--

[PATCH 01/15] lustre: switch lnet_sock_write() to sock_sendmsg()

From: Al Viro Signed-off-by: Al Viro --- drivers/staging/lustre/lnet/lnet/lib-socket.c | 21 +++-- 1 file changed, 7 insertions(+), 14 deletions(-) diff --git a/drivers/staging/lustre/lnet/lnet/lib-socket.c b/drivers/staging/lustre/lnet/lnet/lib-socket.c index 891fd59..4e6dd51

[PATCH 10/15] lustre: introduce lnet_copy_{k, }iov2iter(), kill lnet_copy_{k, }iov2{k, }iov()

From: Al Viro Signed-off-by: Al Viro Signed-off-by: James Simmons Signed-off-by: Oleg Drokin --- .../staging/lustre/include/linux/lnet/lib-lnet.h | 57 +--- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c | 36 +-- drivers/staging/lustre/lnet/lnet/lib-move.c| 309 --

[PATCH 08/15] lustre: constify lib-move.c stuff

From: Al Viro Signed-off-by: Al Viro --- .../staging/lustre/include/linux/lnet/lib-lnet.h | 28 +++--- drivers/staging/lustre/lnet/lnet/lib-move.c| 20 2 files changed, 24 insertions(+), 24 deletions(-) diff --git a/drivers/staging/lustre/include/linu

[PATCH 07/15] lustre: ->kss_scratch... are unused now

From: Al Viro Signed-off-by: Al Viro --- drivers/staging/lustre/lnet/klnds/socklnd/socklnd.h | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/lustre/lnet/klnds/socklnd/socklnd.h b/drivers/staging/lustre/lnet/klnds/socklnd/socklnd.h index d5efb42..84a915c 100644 --- a/driv

[PATCH 02/15] lustre: simplify the living hell out of ksocknal_lib_recv_kiov()

From: Al Viro ... by using ITER_BVEC recvmsg Signed-off-by: Al Viro --- .../staging/lustre/lnet/klnds/socklnd/socklnd.h| 6 +- .../lustre/lnet/klnds/socklnd/socklnd_lib.c| 98 ++ 2 files changed, 12 insertions(+), 92 deletions(-) diff --git a/drivers/staging/l

[PATCH 04/15] ksocknal_lib_recv_iov(): recvmsg doesn't bugger iovec anymore...

From: Al Viro Signed-off-by: Al Viro --- .../lustre/lnet/klnds/socklnd/socklnd_lib.c| 22 +- 1 file changed, 5 insertions(+), 17 deletions(-) diff --git a/drivers/staging/lustre/lnet/klnds/socklnd/socklnd_lib.c b/drivers/staging/lustre/lnet/klnds/socklnd/socklnd_li

[PATCH 00/15] Lustre cleanups.

Here are some more Lustre cleanups. The biggest of the bunch being Al Viro's work to convert lnet to sendmsg and iovec iterators. Also some style fixes. Please consider. Al Viro (10): lustre: switch lnet_sock_write() to sock_sendmsg() lustre: simplify the living hell out of ksocknal_lib_recv

[PATCH v18 net-next 1/1] hv_sock: introduce Hyper-V Sockets

Hyper-V Sockets (hv_sock) supplies a byte-stream based communication mechanism between the host and the guest. It's somewhat like TCP over VMBus, but the transportation layer (VMBus) is much simpler than IP. With Hyper-V Sockets, applications between the host and the guest can talk to each other d

[PATCH v18 net-next 0/1] introduce Hyper-V VM Sockets(hv_sock)

Hyper-V Sockets (hv_sock) supplies a byte-stream based communication mechanism between the host and the guest. It's somewhat like TCP over VMBus, but the transportation layer (VMBus) is much simpler than IP. With Hyper-V Sockets, applications between the host and the guest can talk to each other d

Re: [PATCH v4 4/7] staging: speakup: replace spk_strlwr() with strlcpytolower()

Markus Mayer writes: > After introducing generic strltolower() and strtolower(), spk_strlwr() > is no longer needed. > > Signed-off-by: Markus Mayer > Acked-by: Samuel Thibault Acked-by: Chris Brannon Haven't looked at much kernel code in a while, but this looks good. -- Chris _

[PATCH v4 4/7] staging: speakup: replace spk_strlwr() with strlcpytolower()

After introducing generic strltolower() and strtolower(), spk_strlwr() is no longer needed. Signed-off-by: Markus Mayer Acked-by: Samuel Thibault --- drivers/staging/speakup/kobjects.c| 3 +-- drivers/staging/speakup/main.c| 3 ++- drivers/staging/speakup/speakup.h | 1 - dri

[PATCH v4 1/7] lib: string: add functions to case-convert strings

Add a collection of generic functions to convert strings to lowercase or uppercase. Changing the case of a string (with or without copying it first) seems to be a recurring requirement in the kernel that is currently being solved by several duplicated implementations doing the same thing. This cha

[PATCH v4 0/7] lib: string: add functions to case-convert strings

This series introduces a family of generic string case conversion functions. This kind of functionality is needed in several places in the kernel. Right now, everybody seems to be implementing their own copy of this functionality. Based on the discussion of the previous version of this series[1] a

[PATCH net-next] hv_netvsc: Fix VF register on bonding devices

From: Haiyang Zhang Added a condition to avoid bonding devices with same MAC registering as VF. Signed-off-by: Haiyang Zhang Reviewed-by: K. Y. Srinivasan --- drivers/net/hyperv/netvsc_drv.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/hyperv/netvsc_

[PATCH 1/9] staging: slicoss: handle allocation failure in slic_init_adapter

The memory allocation in slic_init_adapter() can fail. Return an error in this case and unwind properly. Also make sure that the allocated memory is properly freed in case of an error in the calling probe() function. By doing this also replace the alloc() followed by memset to zero the memory with

[PATCH 4/9] staging: slicoss: start tx queue when interface is brought up

There is no reason to delay tx queue activation until a link is detected. So start the queue when the interface is brought up and stop it when the interface is brought down. Signed-off-by: Lino Sanfilippo --- drivers/staging/slicoss/slicoss.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletio

[PATCH 2/9] staging: slicoss: notifiy network stack about change in link state

Notify the network stack about link states via netif_carrier_[off|on](). Also set the link state off initially and when the interface is brought down. Signed-off-by: Lino Sanfilippo --- drivers/staging/slicoss/slicoss.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/staging/

[PATCH 3/9] staging: slicoss: ensure mapping of io-memory is uncached

From: Lino Sanfilippo Writes to registers should be done uncached for various reasons. Ensure this by replacing ioremap() with ioremap_nocache(). Signed-off-by: Lino Sanfilippo --- drivers/staging/slicoss/slicoss.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/sta

[PATCH 7/9] staging: slicoss: dont use a structure but offsets for register accesses

Use the new register accessors that use offsets instead of the slic_regs structure to read/write registers. Since not longer needed remove the structure completley. Signed-off-by: Lino Sanfilippo --- drivers/staging/slicoss/slic.h| 1 - drivers/staging/slicoss/slichw.h | 219 -

Slicoss fixes and improvements

Hi, this patch series tries to get the slicoss driver code closer to a state in which it is ready to be moved out of staging. Patch 1 is a resend of patch I sent a week ago and handles an allocation failure in slic_init_adapter(). Patch 2 fixes link state notification. Patch 3 turns the mapping

[PATCH 9/9] staging: slicoss: simplify struct for statistics

Merge several structures for statistics to one structure and remove unnecessary union nesting. Signed-off-by: Lino Sanfilippo --- drivers/staging/slicoss/slic.h| 18 + drivers/staging/slicoss/slichw.h | 79 --- drivers/staging/slicoss/slicoss.c |

[PATCH 8/9] staging: slicoss: improve implementation concerning the shared memory

Remove the volatile specifiers of struct slic_shmem. Furthermore store the bus addresses for the isr pointer, link status and statistics separately and access the upper and lower word of these addresses by means of [lower|upper]_32_bits(). By doing this take the endianness into account. Signed-off

[PATCH 5/9] staging: slicoss: introduce register accessors that use register offsets

Introduce accessor functions that read and write registers by using a register offset. This is in preparation to replace the register addressing by means of the slic_regs struct with an addressing by means of offsets. Signed-off-by: Lino Sanfilippo --- drivers/staging/slicoss/slic.h| 24 +++

[PATCH 6/9] staging: slicoss: avoid PCI write posting

Introduce the function slic_flush_write() which reads from the HOSTID register and can be used to avoid PCI write posting. Use the function at several critical places in the code. Signed-off-by: Lino Sanfilippo --- drivers/staging/slicoss/slic.h| 5 + drivers/staging/slicoss/slicoss.c |

Re: [PATCH 2/3] staging/rtl8192e: use s8 instead of char

Hi On 2016-07-22, Arnd Bergmann wrote: > On Friday, July 22, 2016 7:55:36 AM CEST Jes Sorensen wrote: > > Stefan Lippers-Hollmann writes: > > > On 2016-07-20, Arnd Bergmann wrote: > > >> On Wednesday, July 20, 2016 11:33:43 AM CEST Jes Sorensen wrote: > > >> > Arnd Bergmann writes: > >

Re: [PATCH 2/3] staging/rtl8192e: use s8 instead of char

On Friday, July 22, 2016 7:55:36 AM CEST Jes Sorensen wrote: > Stefan Lippers-Hollmann writes: > > Hi > > > > On 2016-07-20, Arnd Bergmann wrote: > >> On Wednesday, July 20, 2016 11:33:43 AM CEST Jes Sorensen wrote: > >> > Arnd Bergmann writes: > >> > > On Wednesday, July 20, 2016 7:25:19 AM CE

Re: [PATCH] staging: comedi: drivers: replace le16_to_cpu() with usb_endpoint_maxp()

On 22/07/16 16:29, Cheah Kok Cheong wrote: Use macro introduced in commit 939f325f4a0f ("usb: add usb_endpoint_maxp() macro") Signed-off-by: Cheah Kok Cheong --- drivers/staging/comedi/drivers/dt9812.c | 4 ++-- drivers/staging/comedi/drivers/ni_usb6501.c | 4 ++-- drivers/staging/comedi

[PATCH] staging: comedi: drivers: replace le16_to_cpu() with usb_endpoint_maxp()

Use macro introduced in commit 939f325f4a0f ("usb: add usb_endpoint_maxp() macro") Signed-off-by: Cheah Kok Cheong --- drivers/staging/comedi/drivers/dt9812.c | 4 ++-- drivers/staging/comedi/drivers/ni_usb6501.c | 4 ++-- drivers/staging/comedi/drivers/vmk80xx.c| 12 ++-- 3 fi

Re: [PATCH 30/58] staging: lustre: create striped directory

Hi, [auto build test ERROR on staging/staging-testing] [also build test ERROR on next-20160722] [cannot apply to v4.7-rc7] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/James-Simmons/staging

Re: [PATCH 2/3] staging/rtl8192e: use s8 instead of char

Stefan Lippers-Hollmann writes: > Hi > > On 2016-07-20, Arnd Bergmann wrote: >> On Wednesday, July 20, 2016 11:33:43 AM CEST Jes Sorensen wrote: >> > Arnd Bergmann writes: >> > > On Wednesday, July 20, 2016 7:25:19 AM CEST Jes Sorensen wrote: >> > >> Arnd Bergmann writes: > [...] >> Yes, I w

Re: [PATCH] [media] tw686x-kh: Delete an unnecessary check before the function call "video_unregister_device"

SF Markus Elfring writes: > The video_unregister_device() function tests whether its argument is NULL > and then returns immediately. Thus the test around the call is not needed. > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring > --- > drivers/stag

[PATCH 32/32] Staging/octeon-usb/octeon-hcd.c: Compression of lines for immediate return This patch compresses two lines in to a single line in file octeon-hcd.c

if immediate return statement is found. It also removes variable bytes_written as it is no longer needed. It is done using script Coccinelle. And coccinelle uses following semantic patch for this compression function: @@ expression ret; identifier f; @@ -ret = +return f(...); -return ret;

Re: [PATCH] Staging: media: timblogiw: File cleanup.

This driver will be removed in 4.8, so I won't take this patch. Regards, Hans On 07/20/2016 08:47 PM, Jeremiah Goerdt wrote: > Cleaned up checkpatch.pl warnings and checks. > > Signed-off-by: Jeremiah Goerdt > --- > drivers/staging/media/timb/timblogiw.c | 134 > -

Re: staging: ks7010: Return directly after a failed kmalloc()

On Fri, Jul 22, 2016 at 09:36:53AM +0200, SF Markus Elfring wrote: > >> I guess that further clarification might be needed for affected > >> implementation details. > > > > That's OK, too. > > > > Acked-by: Wolfram Sang > > Does this acknowledgement include also the acceptance for > the suggest

Re: staging: ks7010: Return directly after a failed kmalloc()

>> I guess that further clarification might be needed for affected >> implementation details. > > That's OK, too. > > Acked-by: Wolfram Sang Does this acknowledgement include also the acceptance for the suggested change around calls of the functions "sdio_claim_host" and "sdio_release_host" wit