Re: [PATCH] list: Fix order of arguments for hlist_add_after(_rcu)

2014-06-07 Thread Andrew Morton
On Fri, 6 Jun 2014 10:22:51 -0700 Paul E. McKenney paul...@linux.vnet.ibm.com wrote: On Fri, Jun 06, 2014 at 03:56:52PM +, David Laight wrote: From: Behalf Of Ken Helias All other add functions for lists have the new item as first argument and the position where it is added as

Re: [PATCH 2/5] staging/iio: IIO_SIMPLE_DUMMY_BUFFER neds IIO_BUFFER

2014-06-07 Thread Jonathan Cameron
On 05/06/14 21:48, Arnd Bergmann wrote: An obviously missing 'select' statement, without this we get a link error Signed-off-by: Arnd Bergmann a...@arndb.de Cc: Jonathan Cameron ji...@kernel.org Cc: linux-...@vger.kernel.org Applied to the fixes-togreg branch of iio.git Thanks, Jonathan ---

Re: [PATCH v2 1/4] dma-mapping: Add devm_ interface for dma_map_single()

2014-06-07 Thread Eli Billauer
Hello Shuah, We agree that the streaming API was originally *intended* for short map-unmap DMA sessions, and that dma_alloc_noncoherent() was the *intended* API for those who want to hold the DMA during a device's lifetime. We also agree that on some platforms, DMA mappings are precious,

[PATCH] Staging: comedi: check the return value of kobject_set_name

2014-06-07 Thread Anton Protopopov
Added a check of the return value of the kobject_set_name function. Signed-off-by: Anton Protopopov a.s.protopo...@gmail.com --- drivers/staging/comedi/comedi_fops.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/drivers/staging/comedi/comedi_fops.c

[PATCHv3 2/3] list: Fix order of arguments for hlist_add_after(_rcu)

2014-06-07 Thread Ken Helias
From: Ken Helias kenhel...@firemail.de All other add functions for lists have the new item as first argument and the position where it is added as second argument. This was changed for no good reason in this function and makes using it unnecessary confusing. The name was changed to let old code

Re: [PATCH v2 0/5] staging: ft1000: ft1000-usb: ft1000_debug.c: Fix style errors and warnings.

2014-06-07 Thread Greg KH
On Fri, Jun 06, 2014 at 08:02:13PM -0700, Thomas Wood wrote: Changes since v1: * Made single patch into a patch set. * Added better commit messages. Is this better, or do I still have to split up my first patch? At first glance, it looks fine, I'll queue this up after 3.16-rc1 is out (in

[PATCH linux-next] staging: r8192ee: Adapt flush function prototype

2014-06-07 Thread Vincent Stehlé
Commit 77be2c54c5bd 'mac80211: add vif to flush call' modifies the flush operation prototype. Update r8192ee function accordingly. This fixes the following compilation warnings: drivers/staging/rtl8192ee/core.c: At top level: drivers/staging/rtl8192ee/core.c:1599:2: warning: initialization

Re: [PATCH] staging: bcm: Fix out of bounds access in CmHosts.c

2014-06-07 Thread Dan Carpenter
On Wed, Jun 04, 2014 at 07:04:30AM +0900, Masanari Iida wrote: An array u8IPv6FlowLable[] is defined as 3 in cntrl_SignalingInterface.h. But in CmHosts.c, the kernel accessed to clsRule.u8IPv6FlowLable[5]. It's only used in debug code which is ifdefed out. It's a bit dangerous to change the

Re: [PATCH] staging: tidspbridge: pmgr: dspapi.c: Cleaning up uninitialized variable

2014-06-07 Thread Dan Carpenter
On Wed, Jun 04, 2014 at 12:23:39AM +0200, Rickard Strandqvist wrote: There is a risk that the variables will be used without being initialized. Has also improved error handling, after an email proposal from Dan Carpenter. Signed-off-by: Rickard Strandqvist

Re: [PATCH linux-next] staging: r8192ee: Adapt flush function prototype

2014-06-07 Thread Larry Finger
On 06/07/2014 12:39 PM, Vincent Stehlé wrote: Commit 77be2c54c5bd 'mac80211: add vif to flush call' modifies the flush operation prototype. Update r8192ee function accordingly. This fixes the following compilation warnings: drivers/staging/rtl8192ee/core.c: At top level:

Re: [PATCH] staging: tidspbridge: pmgr: dspapi.c: Cleaning up uninitialized variable

2014-06-07 Thread Rickard Strandqvist
Hi Sure, no problem! Glad I could help, and additionally with more than just a two-line fix this time :-) Best regards Rickard Strandqvist 2014-06-07 21:44 GMT+02:00 Dan Carpenter dan.carpen...@oracle.com: On Wed, Jun 04, 2014 at 12:23:39AM +0200, Rickard Strandqvist wrote: There is a risk

[PATCH] staging: rtl8821ae: rtl8821ae: hw.c: Cleaning up if statement that always evaluates to false

2014-06-07 Thread Rickard Strandqvist
I find a logical error in an if statement '(X 0xfc) == 0x3' is always false After pointing this out, Larry Finger informed what would be the correct one. '(X 0x3) == 0x3' Signed-off-by: Rickard Strandqvist rickard_strandqv...@spectrumdigital.se --- drivers/staging/rtl8821ae/rtl8821ae/hw.c |