Re: [PATCH] dma-buf, drm, ion: Propagate error code from dma_buf_start_cpu_access()

2016-03-20 Thread Sumit Semwal
On 19 March 2016 at 15:39, Daniel Vetter wrote: > On Fri, Mar 18, 2016 at 08:02:39PM +, Chris Wilson wrote: >> Drivers, especially i915.ko, can fail during the initial migration of a >> dma-buf for CPU access. However, the error code from the driver was not >> being propagated back to ioctl an

Re: [PATCH v3] staging: netlogic: Fixed alignment of parentheseis checkpatch warning

2016-03-20 Thread Valdis . Kletnieks
On Sat, 19 Mar 2016 19:22:09 -0700, Joe Perches said: > On Sun, 2016-03-20 at 07:48 +0530, Parth Sane wrote: > > Hi, > > Thanks for pointing out that the changes have been done. Nevertheless > > this was a good learning exercise. How do I check which changes have > > already been done? > > Use this

Re: [PATCH 0/5] staging: rtl8712: Fixed Multiple FSF address checkpatch warnings

2016-03-20 Thread Joshua Clayton
On Sunday, March 20, 2016 11:12:32 PM Parth Sane wrote: > > Fixed Multiple FSF address checkpatch warnings to conform to kernel coding > > style. > > > > Parth Sane (5): > > staging: rtl8712: Fixed FSF address warning in basic_types.h > > staging: rtl8712: Fixed FSF address warning in drv_types

[PATCH] Staging: wlan-ng: removed "goto " instructions where this is not necessary.

2016-03-20 Thread Claudiu Beznea
This patch removes "goto " instructions which do only a return. In this way, aditional instructions were removed. Signed-off-by: Claudiu Beznea --- drivers/staging/wlan-ng/cfg80211.c | 112 + 1 file changed, 39 insertions(+), 73 deletions(-) diff --git a/driv

[PATCH] rtl8192u: fix typo in debug message

2016-03-20 Thread Nik Nyby
This fixes a mis-spelled word in a few debug statements. Signed-off-by: Nik Nyby --- drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c | 2 +- drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8192u/iee

RE: [PATCH 1/1] scsi: storvsc: Support manual scan of FC hosts on Hyper-V

2016-03-20 Thread KY Srinivasan
> -Original Message- > From: James Bottomley [mailto:james.bottom...@hansenpartnership.com] > Sent: Friday, March 18, 2016 3:41 PM > To: KY Srinivasan ; Martin K. Petersen > > Cc: Christoph Hellwig ; gre...@linuxfoundation.org; > linux-ker...@vger.kernel.org; de...@linuxdriverproject.org

Re: [PATCH 0/5] staging: rtl8712: Fixed Multiple FSF address checkpatch warnings

2016-03-20 Thread Parth Sane
> Fixed Multiple FSF address checkpatch warnings to conform to kernel coding > style. > > Parth Sane (5): > staging: rtl8712: Fixed FSF address warning in basic_types.h > staging: rtl8712: Fixed FSF address warning in drv_types.h > staging: rtl9712: Fixed FSF address warning in ethernet.h > s

Re: [PATCH 0/5] staging: rtl8712: Fixed Multiple FSF address checkpatch warnings

2016-03-20 Thread Roger H Newell
On Sun, Mar 20, 2016 at 1:28 PM, Parth Sane wrote: > *** BLURB HERE *** > > Parth Sane (5): > staging: rtl8712: Fixed FSF address warning in basic_types.h > staging: rtl8712: Fixed FSF address warning in drv_types.h > staging: rtl9712: Fixed FSF address warning in ethernet.h > staging: rtl

[PATCH] dma-buf, drm, ion: Propagate error code from dma_buf_start_cpu_access()

2016-03-20 Thread Chris Wilson
Drivers, especially i915.ko, can fail during the initial migration of a dma-buf for CPU access. However, the error code from the driver was not being propagated back to ioctl and so userspace was blissfully ignorant of the failure. Rendering corruption ensues. Whilst fixing the ioctl to return the

Re: [PATCH] staging: rtl8712: Fixed Multiple Parenthesis Alignment

2016-03-20 Thread Parth Sane
> On 20-Mar-2016, at 10:03 PM, Roger H Newell wrote: > > > On Mar 20, 2016 1:21 PM, "Parth Sane" wrote: > > > > > > > On 20-Mar-2016, at 9:18 PM, Joe Perches wrote: > > > > > > On Sun, 2016-03-20 at 14:33 +, Parth Sane wrote: > > >> Fixed multiple parenthesis alignment warnings. > > >> Si

Re: [PATCH] staging: rtl8712: Removed copyright notice

2016-03-20 Thread Joe Perches
On Sun, 2016-03-20 at 03:32 +, Parth Sane wrote: > Removed copyright notice as per checkpatch check warning. This is not a good patch subject nor commit message. Removing copyright is a serious thing. You are not doing that. You are removing a paragraph about an organization's address that i

[PATCH 5/5] staging: rtl9712: Fixed FSF address warning in ieee80211.c

2016-03-20 Thread Parth Sane
Signed-off-by: Parth Sane --- drivers/staging/rtl8712/ieee80211.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/rtl8712/ieee80211.c b/drivers/staging/rtl8712/ieee80211.c index d13b4d5..8918654 100644 --- a/drivers/staging/rtl8712/ieee80211.c +++ b/drivers/staging/rtl8712

[PATCH 4/5] staging: rtl9712: Fixed FSF address warning in hal_init.c

2016-03-20 Thread Parth Sane
Signed-off-by: Parth Sane --- drivers/staging/rtl8712/hal_init.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/rtl8712/hal_init.c b/drivers/staging/rtl8712/hal_init.c index 17d2b5f..afa7fe78 100644 --- a/drivers/staging/rtl8712/hal_init.c +++ b/drivers/staging/rtl8712/ha

[PATCH 3/5] staging: rtl9712: Fixed FSF address warning in ethernet.h

2016-03-20 Thread Parth Sane
Signed-off-by: Parth Sane --- drivers/staging/rtl8712/ethernet.h | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/rtl8712/ethernet.h b/drivers/staging/rtl8712/ethernet.h index fad173f..039da36 100644 --- a/drivers/staging/rtl8712/ethernet.h +++ b/drivers/staging/rtl8712/eth

[PATCH 2/5] staging: rtl8712: Fixed FSF address warning in drv_types.h

2016-03-20 Thread Parth Sane
Signed-off-by: Parth Sane --- drivers/staging/rtl8712/drv_types.h | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/rtl8712/drv_types.h b/drivers/staging/rtl8712/drv_types.h index 29e47e1..ae79047 100644 --- a/drivers/staging/rtl8712/drv_types.h +++ b/drivers/staging/rtl8712

[PATCH 0/5] staging: rtl8712: Fixed Multiple FSF address checkpatch warnings

2016-03-20 Thread Parth Sane
*** BLURB HERE *** Parth Sane (5): staging: rtl8712: Fixed FSF address warning in basic_types.h staging: rtl8712: Fixed FSF address warning in drv_types.h staging: rtl9712: Fixed FSF address warning in ethernet.h staging: rtl9712: Fixed FSF address warning in hal_init.c staging: rtl9712:

[PATCH 1/5] staging: rtl8712: Fixed FSF address warning in basic_types.h

2016-03-20 Thread Parth Sane
Signed-off-by: Parth Sane --- drivers/staging/rtl8712/basic_types.h | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/rtl8712/basic_types.h b/drivers/staging/rtl8712/basic_types.h index 7561bed..f5c0231 100644 --- a/drivers/staging/rtl8712/basic_types.h +++ b/drivers/staging

Re: [PATCH] staging: rtl8712: Fixed Multiple Parenthesis Alignment

2016-03-20 Thread Parth Sane
> On 20-Mar-2016, at 9:18 PM, Joe Perches wrote: > > On Sun, 2016-03-20 at 14:33 +, Parth Sane wrote: >> Fixed multiple parenthesis alignment warnings. >> Signed-off-by: Parth Sane > > Please always add a blank line after your changelog and > before your "Signed-off-by:" line > >> diff --

Re: [PATCH] staging: rtl8712: Fixed Multiple Parenthesis Alignment

2016-03-20 Thread Joe Perches
On Sun, 2016-03-20 at 14:33 +, Parth Sane wrote: > Fixed multiple parenthesis alignment warnings. > Signed-off-by: Parth Sane Please always add a blank line after your changelog and before your "Signed-off-by:" line > diff --git a/drivers/staging/rtl8712/hal_init.c > b/drivers/staging/rtl87

Re: [PATCH] staging: rtl8712: Removed FSF address warning

2016-03-20 Thread Parth Sane
> On 20-Mar-2016, at 7:56 PM, Larry Finger wrote: > > On 03/20/2016 08:59 AM, Parth Sane wrote: >> Removed checkpatch warning caused by FSF address block >> Signed-off-by: Parth Sane >> --- >> drivers/staging/rtl8712/hal_init.c | 4 >> 1 file changed, 4 deletions(-) > > Now you have at l

[PATCH] staging: rtl8712: Fixed Multiple Parenthesis Alignment

2016-03-20 Thread Parth Sane
Fixed multiple parenthesis alignment warnings. Signed-off-by: Parth Sane --- drivers/staging/rtl8712/hal_init.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8712/hal_init.c b/drivers/staging/rtl8712/hal_init.c index 225ef37..afa7fe78 100644 --- a/

Re: [PATCH] staging: rtl8712: Removed FSF address warning

2016-03-20 Thread Larry Finger
On 03/20/2016 08:59 AM, Parth Sane wrote: Removed checkpatch warning caused by FSF address block Signed-off-by: Parth Sane --- drivers/staging/rtl8712/hal_init.c | 4 1 file changed, 4 deletions(-) Now you have at least 3 different patches all with the same subject! How is the maintai

[PATCH] staging: rtl8712: Removed FSF address warning

2016-03-20 Thread Parth Sane
Removed checkpatch warning caused due to FSF address paragraph Signed-off-by: Parth Sane --- drivers/staging/rtl8712/mlme_linux.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/rtl8712/mlme_linux.c b/drivers/staging/rtl8712/mlme_linux.c index e4e4bde..f124d72 100644 --- a

[PATCH] staging: rtl8712: Removed FSF address warning

2016-03-20 Thread Parth Sane
Removed checkpatch warning caused due to FSF address paragraph Signed-off-by: Parth Sane --- drivers/staging/rtl8712/ieee80211.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/rtl8712/ieee80211.c b/drivers/staging/rtl8712/ieee80211.c index d13b4d5..8918654 100644 --- a/dr

[PATCH] staging: rtl8712: Removed FSF address warning

2016-03-20 Thread Parth Sane
Removed checkpatch warning caused by FSF address block Signed-off-by: Parth Sane --- drivers/staging/rtl8712/hal_init.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/rtl8712/hal_init.c b/drivers/staging/rtl8712/hal_init.c index 8008efe..225ef37 100644 --- a/drivers/stagi

[PATCH] staging: rtl8712: Removed FSF address warning

2016-03-20 Thread Parth Sane
Removed checkpatch warning caused by FSF address paragraph. Signed-off-by: Parth Sane --- drivers/staging/rtl8712/ethernet.h | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/rtl8712/ethernet.h b/drivers/staging/rtl8712/ethernet.h index fad173f..039da36 100644 --- a/driver

[PATCH] staging: comedi: dt282x: tidy up register bit defines

2016-03-20 Thread H Hartley Sweeten
Arnd Bergmann pointed out that gcc-6 warns about passing negative signed integer into swab16() due to the macro expansion of 'outw'. It appears that the register map constants are causing the warnings. Actually, it might just be the (1 << 15) ones... Convert all the constants as suggested by chec

[PATCH v2] staging: rtl8712: Removed warning about FSFs mailing address

2016-03-20 Thread Parth Sane
Removed FSF address notice which causes checkpatch warning. Signed-off-by: Parth Sane --- drivers/staging/rtl8712/drv_types.h | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/rtl8712/drv_types.h b/drivers/staging/rtl8712/drv_types.h index 29e47e1..ae79047 100644 --- a/dri

[PATCH 20/42] staging: comedi: ni_660x: Prefer kernel type 'u64' over 'uint64_t'

2016-03-20 Thread H Hartley Sweeten
Fix the checkpatch.pl issues. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/ni_660x.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/comedi/drivers/ni_660x.c b/drivers/staging/comedi/drivers

[PATCH 18/42] staging: comedi: ni_660x: remove ni_660x_num_counters()

2016-03-20 Thread H Hartley Sweeten
This inline function is only used by the (*auto_attach). Remove it and just use a local variable for the calculation. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/ni_660x.c | 16 +--- 1 file changed, 5 insertions(+), 11 d

RE: [PATCH] staging: comedi: ni_mio_common: fix the ni_write[blw]() functions

2016-03-20 Thread Hartley Sweeten
On Wednesday, March 16, 2016 3:22 AM, Ian Abbott wrote: > On 15/03/16 21:48, H Hartley Sweeten wrote: >> Memory mapped io (dev->mmio) should not also be writing to the ioport >> (dev->iobase) registers. Add the missing 'else' to these functions. >> >> Signed-off-by: H Hartley Sweeten >> Cc: Ian Ab

Re: [PATCH] Drivers: hv: vmbus: handle various crash scenarios

2016-03-20 Thread Vitaly Kuznetsov
Radim Krcmar writes: > 2016-03-18 13:33+0100, Vitaly Kuznetsov: >> Kdump keeps biting. Turns out CHANNELMSG_UNLOAD_RESPONSE is always >> delivered to CPU0 regardless of what CPU we're sending CHANNELMSG_UNLOAD >> from. vmbus_wait_for_unload() doesn't account for the fact that in case >> we're cra

Re: [PATCH v3] staging: netlogic: Fixed alignment of parentheseis checkpatch warning

2016-03-20 Thread Joe Perches
On Sun, 2016-03-20 at 02:03 +, Parth Sane wrote: > Fixed alignment of parenthesis checkpatch warning to fit coding > standards. This isn't the same code block as your previous patch. Previously you were modifying the block starting at line 120, this is starting at 168 and it doesn't actually

[PATCH 2/9] rtlwifi: btcoexist: Fix Smatch warning

2016-03-20 Thread Larry Finger
Smatch reports the following: CHECK drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtc8723b2ant.c drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtc8723b2ant.c:3078 btc8723b2ant_run_coexist_mechanism() warn: inconsistent indenting Signed-off-by: Larry Finger --- drivers/net/wireless/