[PATCH net-next 3/9] rxrpc: The client call state must be changed before attachment to conn

2016-09-04 Thread David Howells
We must set the client call state to RXRPC_CALL_CLIENT_SEND_REQUEST before attaching the call to the connection struct, not after, as it's liable to receive errors and conn aborts as soon as the assignment is made - and these will cause its state to be changed outside of the initiating thread's

Re: [PATCH linux v3 3/9] xen: introduce xen_vcpu_id mapping

2016-09-04 Thread Boris Ostrovsky
On 09/02/2016 11:29 AM, Julien Grall wrote: > >> diff --git a/arch/x86/xen/enlighten.c b/arch/x86/xen/enlighten.c >> index 0f87db2..c833912 100644 >> --- a/arch/x86/xen/enlighten.c >> +++ b/arch/x86/xen/enlighten.c >> @@ -1795,6 +1806,12 @@ static void __init init_hvm_pv_info(void) >> >>

[PATCH net-next 4/9] rxrpc: Randomise epoch and starting client conn ID values

2016-09-04 Thread David Howells
Create a random epoch value rather than a time-based one on startup and set the top bit to indicate that this is the case. Also create a random starting client connection ID value. This will be incremented from here as new client connections are created. Signed-off-by: David Howells

[PATCH net-next 4/9] rxrpc: Randomise epoch and starting client conn ID values

2016-09-04 Thread David Howells
Create a random epoch value rather than a time-based one on startup and set the top bit to indicate that this is the case. Also create a random starting client connection ID value. This will be incremented from here as new client connections are created. Signed-off-by: David Howells ---

[PATCH net-next 7/9] fs/afs/rxrpc: Remove deprecated create_singlethread_workqueue

2016-09-04 Thread David Howells
From: Bhaktipriya Shridhar The workqueue "afs_async_calls" queues work item >async_work per afs_call. Since there could be multiple calls and since these calls can be run concurrently, alloc_workqueue has been used to replace the deprecated create_singlethread_workqueue

[PATCH net-next 8/9] fs/afs/callback: Remove deprecated create_singlethread_workqueue

2016-09-04 Thread David Howells
From: Bhaktipriya Shridhar The workqueue "afs_callback_update_worker" queues multiple work items viz >cb_broken_work, >cb_break_work which require strict execution ordering. Hence, an ordered dedicated workqueue has been used. Since the workqueue is being used on a

[PATCH net-next 7/9] fs/afs/rxrpc: Remove deprecated create_singlethread_workqueue

2016-09-04 Thread David Howells
From: Bhaktipriya Shridhar The workqueue "afs_async_calls" queues work item >async_work per afs_call. Since there could be multiple calls and since these calls can be run concurrently, alloc_workqueue has been used to replace the deprecated create_singlethread_workqueue instance. The

[PATCH net-next 8/9] fs/afs/callback: Remove deprecated create_singlethread_workqueue

2016-09-04 Thread David Howells
From: Bhaktipriya Shridhar The workqueue "afs_callback_update_worker" queues multiple work items viz >cb_broken_work, >cb_break_work which require strict execution ordering. Hence, an ordered dedicated workqueue has been used. Since the workqueue is being used on a memory reclaim path,

[PATCH net-next 5/9] rxrpc: Don't change the epoch

2016-09-04 Thread David Howells
It seems the local epoch should only be changed on boot, so remove the code that changes it for client connections. Signed-off-by: David Howells --- net/rxrpc/conn_client.c | 32 1 file changed, 8 insertions(+), 24 deletions(-) diff

[PATCH net-next 5/9] rxrpc: Don't change the epoch

2016-09-04 Thread David Howells
It seems the local epoch should only be changed on boot, so remove the code that changes it for client connections. Signed-off-by: David Howells --- net/rxrpc/conn_client.c | 32 1 file changed, 8 insertions(+), 24 deletions(-) diff --git

Re: [PATCH v3 0/4] fs/afs: Remove deprecated create_singlethread_workqueue

2016-09-04 Thread David Howells
Bhaktipriya Shridhar wrote: > This patch set removes deprecated create_singlethread_workqueue > usages in fs/afs. Applied. David

Re: [PATCH v3 0/4] fs/afs: Remove deprecated create_singlethread_workqueue

2016-09-04 Thread David Howells
Bhaktipriya Shridhar wrote: > This patch set removes deprecated create_singlethread_workqueue > usages in fs/afs. Applied. David

[PATCH net-next 9/9] fs/afs/flock: Remove deprecated create_singlethread_workqueue

2016-09-04 Thread David Howells
From: Bhaktipriya Shridhar The workqueue "afs_lock_manager" queues work item >lock_work, per vnode. Since there can be multiple vnodes and since their work items can be executed concurrently, alloc_workqueue has been used to replace the deprecated

[PATCH net-next 1/3] rxrpc: Split sendmsg from packet transmission code

2016-09-04 Thread David Howells
Split the sendmsg code from the packet transmission code (mostly to be found in output.c). Signed-off-by: David Howells --- net/rxrpc/Makefile |1 net/rxrpc/ar-internal.h |9 - net/rxrpc/misc.c|5 net/rxrpc/output.c | 630

[PATCH net-next 9/9] fs/afs/flock: Remove deprecated create_singlethread_workqueue

2016-09-04 Thread David Howells
From: Bhaktipriya Shridhar The workqueue "afs_lock_manager" queues work item >lock_work, per vnode. Since there can be multiple vnodes and since their work items can be executed concurrently, alloc_workqueue has been used to replace the deprecated create_singlethread_workqueue instance. The

[PATCH net-next 1/3] rxrpc: Split sendmsg from packet transmission code

2016-09-04 Thread David Howells
Split the sendmsg code from the packet transmission code (mostly to be found in output.c). Signed-off-by: David Howells --- net/rxrpc/Makefile |1 net/rxrpc/ar-internal.h |9 - net/rxrpc/misc.c|5 net/rxrpc/output.c | 630

[PATCH net-next 3/3] rxrpc Move enum rxrpc_command to sendmsg.c

2016-09-04 Thread David Howells
Move enum rxrpc_command to sendmsg.c as it's now only used in that file. Signed-off-by: David Howells --- net/rxrpc/ar-internal.h |7 --- net/rxrpc/sendmsg.c |7 +++ 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/net/rxrpc/ar-internal.h

[PATCH net-next 0/3] rxrpc: Split output code from sendmsg code

2016-09-04 Thread David Howells
rxrpc-rewrite-20160904-2 David --- David Howells (3): rxrpc: Split sendmsg from packet transmission code rxrpc: Rearrange net/rxrpc/sendmsg.c rxrpc Move enum rxrpc_command to sendmsg.c net/rxrpc/Makefile |1 net/rxrpc/ar-internal.h | 16 - net/rxrpc/misc.c

[PATCH net-next 2/3] rxrpc: Rearrange net/rxrpc/sendmsg.c

2016-09-04 Thread David Howells
Rearrange net/rxrpc/sendmsg.c to be in a more logical order. This makes it easier to follow and eliminates forward declarations. Signed-off-by: David Howells --- net/rxrpc/sendmsg.c | 1006 +-- 1 file changed, 501

[PATCH net-next 3/3] rxrpc Move enum rxrpc_command to sendmsg.c

2016-09-04 Thread David Howells
Move enum rxrpc_command to sendmsg.c as it's now only used in that file. Signed-off-by: David Howells --- net/rxrpc/ar-internal.h |7 --- net/rxrpc/sendmsg.c |7 +++ 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/net/rxrpc/ar-internal.h

[PATCH net-next 0/3] rxrpc: Split output code from sendmsg code

2016-09-04 Thread David Howells
rxrpc-rewrite-20160904-2 David --- David Howells (3): rxrpc: Split sendmsg from packet transmission code rxrpc: Rearrange net/rxrpc/sendmsg.c rxrpc Move enum rxrpc_command to sendmsg.c net/rxrpc/Makefile |1 net/rxrpc/ar-internal.h | 16 - net/rxrpc/misc.c

[PATCH net-next 2/3] rxrpc: Rearrange net/rxrpc/sendmsg.c

2016-09-04 Thread David Howells
Rearrange net/rxrpc/sendmsg.c to be in a more logical order. This makes it easier to follow and eliminates forward declarations. Signed-off-by: David Howells --- net/rxrpc/sendmsg.c | 1006 +-- 1 file changed, 501 insertions(+), 505 deletions(-)

[PATCH net-next 0/9] rxrpc: Small fixes

2016-09-04 Thread David Howells
/git/dhowells/linux-fs.git rxrpc-rewrite-20160904-1 David --- Arnd Bergmann (1): rxrpc: fix undefined behavior in rxrpc_mark_call_released Bhaktipriya Shridhar (4): fs/afs/vlocation: Remove deprecated create_singlethread_workqueue fs/afs/rxrpc: Remove deprecated

[PATCH net-next 0/9] rxrpc: Small fixes

2016-09-04 Thread David Howells
/git/dhowells/linux-fs.git rxrpc-rewrite-20160904-1 David --- Arnd Bergmann (1): rxrpc: fix undefined behavior in rxrpc_mark_call_released Bhaktipriya Shridhar (4): fs/afs/vlocation: Remove deprecated create_singlethread_workqueue fs/afs/rxrpc: Remove deprecated

[PATCH net-next 1/9] rxrpc: fix undefined behavior in rxrpc_mark_call_released

2016-09-04 Thread David Howells
From: Arnd Bergmann gcc -Wmaybe-initialized correctly points out a newly introduced bug through which we can end up calling rxrpc_queue_call() for a dead connection: net/rxrpc/call_object.c: In function 'rxrpc_mark_call_released': net/rxrpc/call_object.c:600:5: error: 'sched' may

[PATCH net-next 1/9] rxrpc: fix undefined behavior in rxrpc_mark_call_released

2016-09-04 Thread David Howells
From: Arnd Bergmann gcc -Wmaybe-initialized correctly points out a newly introduced bug through which we can end up calling rxrpc_queue_call() for a dead connection: net/rxrpc/call_object.c: In function 'rxrpc_mark_call_released': net/rxrpc/call_object.c:600:5: error: 'sched' may be used

[PATCH net-next 2/9] rxrpc: Fix uninitialised variable warning

2016-09-04 Thread David Howells
Fix the following uninitialised variable warning: ../net/rxrpc/call_event.c: In function 'rxrpc_process_call': ../net/rxrpc/call_event.c:879:58: warning: 'error' may be used uninitialized in this function [-Wmaybe-uninitialized] _debug("post net error %d", error);

[PATCH net-next 2/9] rxrpc: Fix uninitialised variable warning

2016-09-04 Thread David Howells
Fix the following uninitialised variable warning: ../net/rxrpc/call_event.c: In function 'rxrpc_process_call': ../net/rxrpc/call_event.c:879:58: warning: 'error' may be used uninitialized in this function [-Wmaybe-uninitialized] _debug("post net error %d", error);

RE: [PATCH v5 4/8] char: rpmb: provide a user space interface

2016-09-04 Thread Winkler, Tomas
\ > Subject: Re: [PATCH v5 4/8] char: rpmb: provide a user space interface > > On Sun 2016-09-04 11:35:33, Winkler, Tomas wrote: > > > > > On Thu, Sep 01, 2016 at 08:05:26PM +, Winkler, Tomas wrote: > > > > > > > > > > > > > > On Sun, Aug 07, 2016 at 09:44:03AM +, Winkler, Tomas wrote: >

RE: [PATCH v5 4/8] char: rpmb: provide a user space interface

2016-09-04 Thread Winkler, Tomas
\ > Subject: Re: [PATCH v5 4/8] char: rpmb: provide a user space interface > > On Sun 2016-09-04 11:35:33, Winkler, Tomas wrote: > > > > > On Thu, Sep 01, 2016 at 08:05:26PM +, Winkler, Tomas wrote: > > > > > > > > > > > > > > On Sun, Aug 07, 2016 at 09:44:03AM +, Winkler, Tomas wrote: >

Re: [PATCH v2 1/6] rtl8723au: remove declaration of unimplemented functions

2016-09-04 Thread Luca Ceresoli
Dear Kalle, thanks for your feedback. On 02/09/2016 18:18, Kalle Valo wrote: > Luca Ceresoli writes: > >> These functions have been declared without any implementation since >> the first commit (364e30ebd2dbaccba430c603da03e68746eb932a) and there >> has been no mention

Re: [PATCH v2 1/6] rtl8723au: remove declaration of unimplemented functions

2016-09-04 Thread Luca Ceresoli
Dear Kalle, thanks for your feedback. On 02/09/2016 18:18, Kalle Valo wrote: > Luca Ceresoli writes: > >> These functions have been declared without any implementation since >> the first commit (364e30ebd2dbaccba430c603da03e68746eb932a) and there >> has been no mention of them in following

Re: [PATCH 2/3] ARM: pxa: lubbock: add pcmcia clock

2016-09-04 Thread Russell King - ARM Linux
On Sun, Sep 04, 2016 at 09:08:27PM +0100, Russell King - ARM Linux wrote: > On Sun, Sep 04, 2016 at 08:59:46PM +0200, Robert Jarzmik wrote: > > Add the clock provided to the PCMCIA block so that sa_pcmcia_add() > > doesn't end up on error while probing "1800" device. > > I don't think this is

Re: [PATCH 2/3] ARM: pxa: lubbock: add pcmcia clock

2016-09-04 Thread Russell King - ARM Linux
On Sun, Sep 04, 2016 at 09:08:27PM +0100, Russell King - ARM Linux wrote: > On Sun, Sep 04, 2016 at 08:59:46PM +0200, Robert Jarzmik wrote: > > Add the clock provided to the PCMCIA block so that sa_pcmcia_add() > > doesn't end up on error while probing "1800" device. > > I don't think this is

arch/ia64/kernel/entry.S:621: Error: Operand 2 of `adds' should be a 14-bit integer (-8192-8191)

2016-09-04 Thread kbuild test robot
Hi Will, FYI, the error/warning still remains. tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 9ca581b50dab6103183396852cc08e440fcda18e commit: da48d094ce5d7c7dcdad9011648a81c42fd1c2ef Kconfig: remove HAVE_LATENCYTOP_SUPPORT date: 8 months ago

arch/ia64/kernel/entry.S:621: Error: Operand 2 of `adds' should be a 14-bit integer (-8192-8191)

2016-09-04 Thread kbuild test robot
Hi Will, FYI, the error/warning still remains. tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 9ca581b50dab6103183396852cc08e440fcda18e commit: da48d094ce5d7c7dcdad9011648a81c42fd1c2ef Kconfig: remove HAVE_LATENCYTOP_SUPPORT date: 8 months ago

Re: [PATCH] qed: Remove OOM messages

2016-09-04 Thread David Miller
From: Joe Perches Date: Fri, 2 Sep 2016 10:48:47 -0700 > These messages are unnecessary as OOM allocation failures already do > a dump_stack() giving more or less the same information. > > $ size drivers/net/ethernet/qlogic/qed/built-in.o* (defconfig x86-64) >text

Re: [PATCH] qed: Remove OOM messages

2016-09-04 Thread David Miller
From: Joe Perches Date: Fri, 2 Sep 2016 10:48:47 -0700 > These messages are unnecessary as OOM allocation failures already do > a dump_stack() giving more or less the same information. > > $ size drivers/net/ethernet/qlogic/qed/built-in.o* (defconfig x86-64) >text data bss

[PATCH] x86-ksysfs: Use kmalloc_array() in create_setup_data_nodes()

2016-09-04 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 4 Sep 2016 22:15:09 +0200 A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "kmalloc_array". This issue was detected by

[PATCH] x86-ksysfs: Use kmalloc_array() in create_setup_data_nodes()

2016-09-04 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 4 Sep 2016 22:15:09 +0200 A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "kmalloc_array". This issue was detected by using the Coccinelle software.

Re: [PATCH v5 4/8] char: rpmb: provide a user space interface

2016-09-04 Thread Pavel Machek
On Sun 2016-09-04 11:35:33, Winkler, Tomas wrote: > > > On Thu, Sep 01, 2016 at 08:05:26PM +, Winkler, Tomas wrote: > > > > > > > > > > > On Sun, Aug 07, 2016 at 09:44:03AM +, Winkler, Tomas wrote: > > > > > > > > > > > > On Mon 2016-07-18 23:27:49, Tomas Winkler wrote: > > > > > > > The

Re: [PATCH v5 4/8] char: rpmb: provide a user space interface

2016-09-04 Thread Pavel Machek
On Sun 2016-09-04 11:35:33, Winkler, Tomas wrote: > > > On Thu, Sep 01, 2016 at 08:05:26PM +, Winkler, Tomas wrote: > > > > > > > > > > > On Sun, Aug 07, 2016 at 09:44:03AM +, Winkler, Tomas wrote: > > > > > > > > > > > > On Mon 2016-07-18 23:27:49, Tomas Winkler wrote: > > > > > > > The

[PATCH 2/2] v4l2-ctrls: add generic H.264 decoder codec settings structure

2016-09-04 Thread Randy Li
The generic decoder settings for H.264. It is modified from the VA-API. Adding the extra data required by the Rockchip. Signed-off-by: Randy Li --- include/uapi/linux/videodev2.h | 103 + 1 file changed, 103 insertions(+) diff --git

[PATCH RFC 0/2] add the generic H.264 decoder settings controls

2016-09-04 Thread Randy Li
This is not done yet. The rockchip VA-API driver[1] still need a third part library to pre-parse the nalu data. Maybe after the third part library free version[2] had done, it would be clear that we else filed we may need. Those structures comes from VA-API SPCE. But still not enough to driver a

[PATCH RFC 0/2] add the generic H.264 decoder settings controls

2016-09-04 Thread Randy Li
This is not done yet. The rockchip VA-API driver[1] still need a third part library to pre-parse the nalu data. Maybe after the third part library free version[2] had done, it would be clear that we else filed we may need. Those structures comes from VA-API SPCE. But still not enough to driver a

[PATCH 2/2] v4l2-ctrls: add generic H.264 decoder codec settings structure

2016-09-04 Thread Randy Li
The generic decoder settings for H.264. It is modified from the VA-API. Adding the extra data required by the Rockchip. Signed-off-by: Randy Li --- include/uapi/linux/videodev2.h | 103 + 1 file changed, 103 insertions(+) diff --git

[PATCH 1/2] [media] v4l2-ctrls: add H.264 decoder settings controls

2016-09-04 Thread Randy Li
These two controls would be used to set the H.264 codec settings for decoder. Signed-off-by: Randy Li --- drivers/media/v4l2-core/v4l2-ctrls.c | 2 ++ include/uapi/linux/v4l2-controls.h | 2 ++ 2 files changed, 4 insertions(+) diff --git

[PATCH 1/2] [media] v4l2-ctrls: add H.264 decoder settings controls

2016-09-04 Thread Randy Li
These two controls would be used to set the H.264 codec settings for decoder. Signed-off-by: Randy Li --- drivers/media/v4l2-core/v4l2-ctrls.c | 2 ++ include/uapi/linux/v4l2-controls.h | 2 ++ 2 files changed, 4 insertions(+) diff --git a/drivers/media/v4l2-core/v4l2-ctrls.c

Re: [PATCH] tpm: move struct tpm_class_ops to drivers/char/tpm/tpm.h

2016-09-04 Thread Jason Gunthorpe
On Sat, Sep 03, 2016 at 09:26:05AM +0300, Jarkko Sakkinen wrote: > > > > OK, how would one get the chip instance? Most subsystems have a get function that returns a kref'd pointer. For TPM all we really need today is a 'get_default_tpm_for_ns' kind of function. > This still doesn't explain why

Re: [PATCH] tpm: move struct tpm_class_ops to drivers/char/tpm/tpm.h

2016-09-04 Thread Jason Gunthorpe
On Sat, Sep 03, 2016 at 09:26:05AM +0300, Jarkko Sakkinen wrote: > > > > OK, how would one get the chip instance? Most subsystems have a get function that returns a kref'd pointer. For TPM all we really need today is a 'get_default_tpm_for_ns' kind of function. > This still doesn't explain why

Re: [PATCH 1/1] ARM: dts: sun8i: Add dts file for the NanoPi NEO SBC

2016-09-04 Thread Rask Ingemann Lambertsen
On Thu, Sep 01, 2016 at 01:48:54PM +1000, ja...@innovum.com.au wrote: > +#include You don't seem to use anything from that header file. > +#include > + > +/ { > + model = "FriendlyARM NanoPi NEO"; > + compatible = "friendlyarm,nanopi-neo", "allwinner,sun8i-h3"; It is good practice to

Re: [PATCH 1/1] ARM: dts: sun8i: Add dts file for the NanoPi NEO SBC

2016-09-04 Thread Rask Ingemann Lambertsen
On Thu, Sep 01, 2016 at 01:48:54PM +1000, ja...@innovum.com.au wrote: > +#include You don't seem to use anything from that header file. > +#include > + > +/ { > + model = "FriendlyARM NanoPi NEO"; > + compatible = "friendlyarm,nanopi-neo", "allwinner,sun8i-h3"; It is good practice to

Re: [linux-sunxi] [PATCH 3/7] drm/sun4i: Add SAT and DRC drivers

2016-09-04 Thread Peter Korsgaard
> "Maxime" == Maxime Ripard writes: Hi, > The A33 pipeline also has some new components called SAT and DRC. Even > though their exact features and programming model is not known (or > documented), they need to be clocked for the pipeline to carry the

Re: [linux-sunxi] [PATCH 3/7] drm/sun4i: Add SAT and DRC drivers

2016-09-04 Thread Peter Korsgaard
> "Maxime" == Maxime Ripard writes: Hi, > The A33 pipeline also has some new components called SAT and DRC. Even > though their exact features and programming model is not known (or > documented), they need to be clocked for the pipeline to carry the video > signal all the way. > Add

Re: [PATCH 2/3] ARM: pxa: lubbock: add pcmcia clock

2016-09-04 Thread Russell King - ARM Linux
On Sun, Sep 04, 2016 at 08:59:46PM +0200, Robert Jarzmik wrote: > Add the clock provided to the PCMCIA block so that sa_pcmcia_add() > doesn't end up on error while probing "1800" device. I don't think this is correct - SA is not really the PCMCIA controller - it's a load of logic which

Re: [PATCH 2/3] ARM: pxa: lubbock: add pcmcia clock

2016-09-04 Thread Russell King - ARM Linux
On Sun, Sep 04, 2016 at 08:59:46PM +0200, Robert Jarzmik wrote: > Add the clock provided to the PCMCIA block so that sa_pcmcia_add() > doesn't end up on error while probing "1800" device. I don't think this is correct - SA is not really the PCMCIA controller - it's a load of logic which

[PATCH] x86/hpet: Use kmalloc_array() in hpet_enable()

2016-09-04 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 4 Sep 2016 21:30:10 +0200 A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "kmalloc_array". This issue was detected by

[PATCH] x86/hpet: Use kmalloc_array() in hpet_enable()

2016-09-04 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 4 Sep 2016 21:30:10 +0200 A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "kmalloc_array". This issue was detected by using the Coccinelle software.

Re: [PATCH 2/4] sparc: bpf_jit: Move four assignments in bpf_jit_compile()

2016-09-04 Thread Bjørn Mork
Julian Calaby writes: > Otherwise this change is useless churn - you're making the code more > complicated, longer and harder to read for practically no benefit. He has been doing that for years now. And wasted maintainer resources along the way, discussing all the

Re: [PATCH 2/4] sparc: bpf_jit: Move four assignments in bpf_jit_compile()

2016-09-04 Thread Bjørn Mork
Julian Calaby writes: > Otherwise this change is useless churn - you're making the code more > complicated, longer and harder to read for practically no benefit. He has been doing that for years now. And wasted maintainer resources along the way, discussing all the pointless churn. There are

[PATCH 5/5] drivers: staging: rtl8723au: core: Fix indentation

2016-09-04 Thread Matthias Beyer
Signed-off-by: Matthias Beyer Signed-off-by: Manuel Messner --- drivers/staging/rtl8723au/core/rtw_mlme.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git

[PATCH 5/5] drivers: staging: rtl8723au: core: Fix indentation

2016-09-04 Thread Matthias Beyer
Signed-off-by: Matthias Beyer Signed-off-by: Manuel Messner --- drivers/staging/rtl8723au/core/rtw_mlme.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_mlme.c b/drivers/staging/rtl8723au/core/rtw_mlme.c index

[PATCH 4/5] drivers: staging: rtl8723au: core: Fix "space prohibited" warning

2016-09-04 Thread Matthias Beyer
This patch fixes the "space prohibited between function name and open parenthesis '('" warning from checkpatch.pl Signed-off-by: Matthias Beyer Signed-off-by: Manuel Messner --- drivers/staging/rtl8723au/core/rtw_mlme.c | 10

[PATCH 4/5] drivers: staging: rtl8723au: core: Fix "space prohibited" warning

2016-09-04 Thread Matthias Beyer
This patch fixes the "space prohibited between function name and open parenthesis '('" warning from checkpatch.pl Signed-off-by: Matthias Beyer Signed-off-by: Manuel Messner --- drivers/staging/rtl8723au/core/rtw_mlme.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff

[PATCH 3/5] drivers: staging: rtl8723au: core: Refactor pointless branching

2016-09-04 Thread Matthias Beyer
As ether_addr_equal returns a bool, we can simply use the return value to set the variable. Signed-off-by: Matthias Beyer Signed-off-by: Manuel Messner --- drivers/staging/rtl8723au/core/rtw_mlme.c | 7 ++- 1 file changed, 2

[PATCH 3/5] drivers: staging: rtl8723au: core: Refactor pointless branching

2016-09-04 Thread Matthias Beyer
As ether_addr_equal returns a bool, we can simply use the return value to set the variable. Signed-off-by: Matthias Beyer Signed-off-by: Manuel Messner --- drivers/staging/rtl8723au/core/rtw_mlme.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git

[PATCH 2/5] drivers: staging: rtl8723au: core: simplify if-break-else

2016-09-04 Thread Matthias Beyer
As the if statement breaks/continues the loop, the else block is not useful and can be omitted. Signed-off-by: Matthias Beyer Signed-off-by: Manuel Messner --- drivers/staging/rtl8723au/core/rtw_mlme.c | 25

[PATCH 2/5] drivers: staging: rtl8723au: core: simplify if-break-else

2016-09-04 Thread Matthias Beyer
As the if statement breaks/continues the loop, the else block is not useful and can be omitted. Signed-off-by: Matthias Beyer Signed-off-by: Manuel Messner --- drivers/staging/rtl8723au/core/rtw_mlme.c | 25 +++-- 1 file changed, 11 insertions(+), 14 deletions(-) diff

[PATCH 1/5] drivers: staging: rtl8723au: core: Fix checkpatch.pl errors

2016-09-04 Thread Matthias Beyer
This patch fixes the ERRORs which are reported from the checkpatch.pl script for this file. Signed-off-by: Matthias Beyer --- drivers/staging/rtl8723au/core/rtw_mlme.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH 1/5] drivers: staging: rtl8723au: core: Fix checkpatch.pl errors

2016-09-04 Thread Matthias Beyer
This patch fixes the ERRORs which are reported from the checkpatch.pl script for this file. Signed-off-by: Matthias Beyer --- drivers/staging/rtl8723au/core/rtw_mlme.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_mlme.c

[PATCH 0/5] Some cleanup patches for drivers/staging/rtl8723au/core/rtw_mlme.c

2016-09-04 Thread Matthias Beyer
This patchset fixes some errors and warnings reported by checkpatch.pl. Matthias Beyer (5): drivers: staging: rtl8723au: core: Fix checkpatch.pl errors drivers: staging: rtl8723au: core: simplify if-break-else drivers: staging: rtl8723au: core: Refactor pointless branching drivers:

[PATCH 0/5] Some cleanup patches for drivers/staging/rtl8723au/core/rtw_mlme.c

2016-09-04 Thread Matthias Beyer
This patchset fixes some errors and warnings reported by checkpatch.pl. Matthias Beyer (5): drivers: staging: rtl8723au: core: Fix checkpatch.pl errors drivers: staging: rtl8723au: core: simplify if-break-else drivers: staging: rtl8723au: core: Refactor pointless branching drivers:

Re: [PATCH] x86/amd_nb: Use kmalloc_array() in amd_cache_gart()

2016-09-04 Thread Joe Perches
On Sun, 2016-09-04 at 21:08 +0200, SF Markus Elfring wrote: > * A multiplication for the size determination of a memory allocation >   indicated that an array data structure should be processed. >   Thus use the corresponding function "kmalloc_array". > >   This issue was detected by using the

Re: [PATCH] x86/amd_nb: Use kmalloc_array() in amd_cache_gart()

2016-09-04 Thread Joe Perches
On Sun, 2016-09-04 at 21:08 +0200, SF Markus Elfring wrote: > * A multiplication for the size determination of a memory allocation >   indicated that an array data structure should be processed. >   Thus use the corresponding function "kmalloc_array". > >   This issue was detected by using the

[PATCH] x86/amd_nb: Use kmalloc_array() in amd_cache_gart()

2016-09-04 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 4 Sep 2016 21:01:47 +0200 * A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "kmalloc_array". This issue was

[PATCH] x86/amd_nb: Use kmalloc_array() in amd_cache_gart()

2016-09-04 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 4 Sep 2016 21:01:47 +0200 * A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "kmalloc_array". This issue was detected by using the Coccinelle

Re: [PATCH] MAINTAINERS: Add maintainer entry for Ingenic JZ4780 NAND driver

2016-09-04 Thread Boris Brezillon
On Tue, 30 Aug 2016 15:22:55 +0100 Harvey Hunt wrote: Applied. Thanks for doing that. Boris > Signed-off-by: Harvey Hunt > Cc: Zubair Lutfullah Kakakhel > Cc: linux-...@lists.infradead.org > Cc:

Re: [PATCH] MAINTAINERS: Add maintainer entry for Ingenic JZ4780 NAND driver

2016-09-04 Thread Boris Brezillon
On Tue, 30 Aug 2016 15:22:55 +0100 Harvey Hunt wrote: Applied. Thanks for doing that. Boris > Signed-off-by: Harvey Hunt > Cc: Zubair Lutfullah Kakakhel > Cc: linux-...@lists.infradead.org > Cc: linux-kernel@vger.kernel.org > --- > MAINTAINERS | 6 ++ > 1 file changed, 6 insertions(+)

Re: [PATCH v2 1/3] mtd: introduce the mtd_pairing_scheme concept

2016-09-04 Thread Boris Brezillon
On Thu, 1 Sep 2016 11:15:24 -0700 Brian Norris wrote: > Hi, > > I've had this on my plate to respond to for a while now, and I haven't > brought myself to actually care that much about the choice. So I'll > respond now to keep from leaving you hanging, but I'm not

Re: [PATCH v2 1/3] mtd: introduce the mtd_pairing_scheme concept

2016-09-04 Thread Boris Brezillon
On Thu, 1 Sep 2016 11:15:24 -0700 Brian Norris wrote: > Hi, > > I've had this on my plate to respond to for a while now, and I haven't > brought myself to actually care that much about the choice. So I'll > respond now to keep from leaving you hanging, but I'm not sure I'm that > helpful :( No

[PATCH 2/3] ARM: pxa: lubbock: add pcmcia clock

2016-09-04 Thread Robert Jarzmik
Add the clock provided to the PCMCIA block so that sa_pcmcia_add() doesn't end up on error while probing "1800" device. Signed-off-by: Robert Jarzmik --- arch/arm/mach-pxa/lubbock.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/mach-pxa/lubbock.c

[PATCH 1/3] ARM: pxa: pxa_cplds: fix interrupt handling

2016-09-04 Thread Robert Jarzmik
Since its initial commit, the driver is buggy for multiple interrupts handling. The translation from the former lubbock.c file was not complete, and might stall all interrupt handling when multiple interrupts occur. This is especially true when inside the interrupt handler and if a new interrupt

[PATCH 2/3] ARM: pxa: lubbock: add pcmcia clock

2016-09-04 Thread Robert Jarzmik
Add the clock provided to the PCMCIA block so that sa_pcmcia_add() doesn't end up on error while probing "1800" device. Signed-off-by: Robert Jarzmik --- arch/arm/mach-pxa/lubbock.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/mach-pxa/lubbock.c b/arch/arm/mach-pxa/lubbock.c

[PATCH 1/3] ARM: pxa: pxa_cplds: fix interrupt handling

2016-09-04 Thread Robert Jarzmik
Since its initial commit, the driver is buggy for multiple interrupts handling. The translation from the former lubbock.c file was not complete, and might stall all interrupt handling when multiple interrupts occur. This is especially true when inside the interrupt handler and if a new interrupt

[PATCH 3/3] [DO_NOT_REVIEW] sa1111: left up to Russell King

2016-09-04 Thread Robert Jarzmik
In order to be able to cope with ordering problems, especially when the interrupt descriptor is allocated only after the sa is probed, add a workaround to "wait" for the interrupt chip to be available. Signed-off-by: Robert Jarzmik --- arch/arm/common/sa.c | 4

[PATCH 3/3] [DO_NOT_REVIEW] sa1111: left up to Russell King

2016-09-04 Thread Robert Jarzmik
In order to be able to cope with ordering problems, especially when the interrupt descriptor is allocated only after the sa is probed, add a workaround to "wait" for the interrupt chip to be available. Signed-off-by: Robert Jarzmik --- arch/arm/common/sa.c | 4 1 file changed, 4

[PATCH 1/1] drm/i915/dsi: silence a warning about uninitialized return value

2016-09-04 Thread Nicolas Iooss
When building the kernel with clang and some warning flags, the compiler reports that the return value of dcs_get_backlight() may be uninitialized: drivers/gpu/drm/i915/intel_dsi_dcs_backlight.c:53:2: error: variable 'data' is used uninitialized whenever 'for' loop exits because its

[PATCH 1/1] drm/i915/dsi: silence a warning about uninitialized return value

2016-09-04 Thread Nicolas Iooss
When building the kernel with clang and some warning flags, the compiler reports that the return value of dcs_get_backlight() may be uninitialized: drivers/gpu/drm/i915/intel_dsi_dcs_backlight.c:53:2: error: variable 'data' is used uninitialized whenever 'for' loop exits because its

Re: [PATCH] net: lantiq_etop: Remove unused 'i' variable

2016-09-04 Thread David Miller
From: Paul Burton Date: Fri, 2 Sep 2016 15:26:54 +0100 > Commit e7f4dc3536a4 ("mdio: Move allocation of interrupts into core") > removed the only use of the 'i' variable from ltq_etop_mdio_init() but > left the variable declaration behind, leading to the following

Re: [PATCH] net: lantiq_etop: Remove unused 'i' variable

2016-09-04 Thread David Miller
From: Paul Burton Date: Fri, 2 Sep 2016 15:26:54 +0100 > Commit e7f4dc3536a4 ("mdio: Move allocation of interrupts into core") > removed the only use of the 'i' variable from ltq_etop_mdio_init() but > left the variable declaration behind, leading to the following compiler > warning: > >

Re: [PATCH] net: ti: cpmac: Fix compiler warning due to type confusion

2016-09-04 Thread David Miller
From: Paul Burton Date: Fri, 2 Sep 2016 15:22:48 +0100 > cpmac_start_xmit() used the max() macro on skb->len (an unsigned int) > and ETH_ZLEN (a signed int literal). This led to the following compiler > warning: > > In file included from include/linux/list.h:8:0, >

Re: [PATCH] net: ti: cpmac: Fix compiler warning due to type confusion

2016-09-04 Thread David Miller
From: Paul Burton Date: Fri, 2 Sep 2016 15:22:48 +0100 > cpmac_start_xmit() used the max() macro on skb->len (an unsigned int) > and ETH_ZLEN (a signed int literal). This led to the following compiler > warning: > > In file included from include/linux/list.h:8:0, >from

Re: [PATCH 1/3] sched/cputime: Improve scalability of times()/clock_gettime() on 32 bit cpus

2016-09-04 Thread Giovanni Gherdovich
On Thu, 2016-09-01 at 12:29 +0200, Peter Zijlstra wrote: > On Thu, Sep 01, 2016 at 12:07:34PM +0200, Stanislaw Gruszka wrote: > >  > > On Thu, Sep 01, 2016 at 11:49:06AM +0200, Peter Zijlstra wrote: > > >  > > > You're now making rather hot paths slower to benefit a rather > > > slow path, that

Re: [PATCH 1/3] sched/cputime: Improve scalability of times()/clock_gettime() on 32 bit cpus

2016-09-04 Thread Giovanni Gherdovich
On Thu, 2016-09-01 at 12:29 +0200, Peter Zijlstra wrote: > On Thu, Sep 01, 2016 at 12:07:34PM +0200, Stanislaw Gruszka wrote: > >  > > On Thu, Sep 01, 2016 at 11:49:06AM +0200, Peter Zijlstra wrote: > > >  > > > You're now making rather hot paths slower to benefit a rather > > > slow path, that

Re: [Patch v5 0/2] net: ethernet: xilinx: mac addr and mips

2016-09-04 Thread David Miller
From: Zubair Lutfullah Kakakhel Date: Fri, 2 Sep 2016 12:39:24 +0100 > A couple of simple patches to generate the random mac address > if none is found. And enabling the driver for mips. > > Based on v4.8-rc4. > > These were part of a larger series but that series

Re: [Patch v5 0/2] net: ethernet: xilinx: mac addr and mips

2016-09-04 Thread David Miller
From: Zubair Lutfullah Kakakhel Date: Fri, 2 Sep 2016 12:39:24 +0100 > A couple of simple patches to generate the random mac address > if none is found. And enabling the driver for mips. > > Based on v4.8-rc4. > > These were part of a larger series but that series is growing > wildly.

[PATCH] KVM: nVMX: pass valid guest linear-address to the L1

2016-09-04 Thread Jan Dakinevich
If EPT support is exposed to L1 hypervisor, guest linear-address field of VMCS should contain GVA of L2, the access to which caused EPT violation. Signed-off-by: Jan Dakinevich --- arch/x86/kvm/vmx.c | 3 +++ 1 file changed, 3 insertions(+) diff --git

[PATCH] KVM: nVMX: pass valid guest linear-address to the L1

2016-09-04 Thread Jan Dakinevich
If EPT support is exposed to L1 hypervisor, guest linear-address field of VMCS should contain GVA of L2, the access to which caused EPT violation. Signed-off-by: Jan Dakinevich --- arch/x86/kvm/vmx.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/x86/kvm/vmx.c b/arch/x86/kvm/vmx.c

Re: [PATCH] drm/tegra: Expose color key and plane blending controls to userspace

2016-09-04 Thread Dmitry Osipenko
On 02.09.2016 18:51, Thierry Reding wrote: > On Fri, Sep 02, 2016 at 05:32:19PM +0200, Thierry Reding wrote: >> On Fri, Sep 02, 2016 at 12:33:42PM +0300, Dmitry Osipenko wrote: >>> Chromakey is a simple way of video overlay overlap implementation. This >>> patch adds 2 new IOCTL's: first - sets

Re: [PATCH] drm/tegra: Expose color key and plane blending controls to userspace

2016-09-04 Thread Dmitry Osipenko
On 02.09.2016 18:51, Thierry Reding wrote: > On Fri, Sep 02, 2016 at 05:32:19PM +0200, Thierry Reding wrote: >> On Fri, Sep 02, 2016 at 12:33:42PM +0300, Dmitry Osipenko wrote: >>> Chromakey is a simple way of video overlay overlap implementation. This >>> patch adds 2 new IOCTL's: first - sets

<    1   2   3   4   5   6   >