[PATCH] Staging: lustre: lnet: code style fix

2017-03-24 Thread Sergiy Redko
Fixed code style error found by checkpatch by adding a space after a comma in function parameter list. Signed-off-by: Sergiy Redko --- drivers/staging/lustre/lnet/lnet/router.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lnet/lnet/router.c b/driver

[PATCH] staging: media: atomisp: compress return logic

2017-03-24 Thread Arushi Singhal
Simplify function returns by merging assignment and return. Signed-off-by: Arushi Singhal --- drivers/staging/media/atomisp/pci/atomisp2/atomisp_v4l2.c | 8 ++-- .../media/atomisp/pci/atomisp2/css2400/runtime/bufq/src/bufq.c| 7 ++- 2 files changed, 4 insertions(+), 11 deleti

[PATCH] staging: greybus: compress return logic

2017-03-24 Thread Arushi Singhal
Simplify function returns by merging assignment and return. Signed-off-by: Arushi Singhal --- drivers/staging/greybus/loopback.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/staging/greybus/loopback.c b/drivers/staging/greybus/loopback.c index aaf29a5fac83..08

[PATCH net-next 1/2] netvsc: Fix a bug in sub-channel handling

2017-03-24 Thread kys
From: K. Y. Srinivasan All netvsc channels are handled via NAPI. Setup the "read mode" correctly for the netvsc sub-channels. Signed-off-by: K. Y. Srinivasan --- drivers/net/hyperv/rndis_filter.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/drivers/net/hyperv/rn

[PATCH net-next 0/2] netvsc: Fix miscellaneous issues

2017-03-24 Thread kys
From: K. Y. Srinivasan Fix miscellaneous issues. K. Y. Srinivasan (2): netvsc: Fix a bug in sub-channel handling netvsc: Properly initialize the return value drivers/net/hyperv/netvsc_drv.c |2 +- drivers/net/hyperv/rndis_filter.c |5 + 2 files changed, 6 insertions(+), 1 del

[PATCH net-next 2/2] netvsc: Properly initialize the return value

2017-03-24 Thread kys
From: K. Y. Srinivasan Initialize the return value correctly. Signed-off-by: K. Y. Srinivasan --- drivers/net/hyperv/netvsc_drv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/net/hyperv/netvsc_drv.c b/drivers/net/hyperv/netvsc_drv.c index eb7ae79..f830bbb 1

[PATCH v3] staging: media: davinci_vpfe: Replace a bit shift.

2017-03-24 Thread Arushi Singhal
This patch replaces bit shifting on 1 with the BIT(x) macro. This was done with coccinelle: @@ constant c; @@ -1 << c +BIT(c) Signed-off-by: Arushi Singhal --- changes in v3 - change the subject. - remove extra parenthesis. drivers/staging/media/davinci_vpfe/dm365_ipipe.c | 2 +- drivers/

[PATCH] Clear sparse warning: different address space

2017-03-24 Thread Guillermo O. Freschi
Made tagged `index` as `__user` to remove warning from `obd_iocontrol` call. Signed-off-by: Guillermo O. Freschi --- drivers/staging/lustre/lustre/llite/dir.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/llite/dir.c b/drivers/staging/lustre/l

[PATCH] Staging: atomisp - octal permissions, style fix

2017-03-24 Thread Derek Robson
Changed permissions to octal style Found using checkpatch Signed-off-by: Derek Robson --- drivers/staging/media/atomisp/pci/atomisp2/atomisp_drvfs.c | 9 +++-- drivers/staging/media/atomisp/pci/atomisp2/hmm/hmm.c | 8 2 files changed, 7 insertions(+), 10 deletions(-) diff --g

[PATCH] Staging: vt6655 - block comments style fix

2017-03-24 Thread Derek Robson
Fixed style of block comments Found using checkpatch Signed-off-by: Derek Robson --- drivers/staging/vt6655/rf.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/vt6655/rf.h b/drivers/staging/vt6655/rf.h index b6e853784a26..37600093cab2 100644 --- a/drivers/sta

Re: [PATCH] staging: lustre: Remove redundant code

2017-03-24 Thread Dilger, Andreas
On Mar 24, 2017, at 07:44, Greg Kroah-Hartman wrote: > > On Fri, Mar 24, 2017 at 05:09:53PM +0530, Arushi Singhal wrote: >> Remove the code which do not have any value. >> >> Signed-off-by: Arushi Singhal >> --- >> drivers/staging/lustre/lnet/lnet/net_fault.c | 5 - >> 1 file changed, 5 del

Re: [Outreachy kernel] [PATCH v2] staging: media: davinci_vpfe:Replace a bit shift.

2017-03-24 Thread Julia Lawall
There should be a spac after every colon in the subject. Please pay attention to these small details, so you don't have to send the same patch over and over. julia On Fri, 24 Mar 2017, Arushi Singhal wrote: > This patch replaces bit shifting on 1 with the BIT(x) macro. > This was done with cocc

[PATCH 2/2] pci-hyperv: Fix an atomic bug

2017-03-24 Thread kys
From: K. Y. Srinivasan The memory allocation here needs to be non-blocking. Fix the issue. Signed-off-by: K. Y. Srinivasan Cc: --- drivers/pci/host/pci-hyperv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/pci/host/pci-hyperv.c b/drivers/pci/host/pci-hyper

[PATCH 1/2] PCI: hv: Fix a bug in specifying CPU affinity

2017-03-24 Thread kys
From: K. Y. Srinivasan When we have 32 or more CPUs in the affinity mask, we should use a special constant to specify that to the host. Fix this issue. Signed-off-by: K. Y. Srinivasan Cc: --- drivers/pci/host/pci-hyperv.c | 10 +++--- 1 files changed, 7 insertions(+), 3 deletions(-) di

[PATCH 1/2] pci-hyperv: Fix a bug in specifying CPU affinity

2017-03-24 Thread kys
From: K. Y. Srinivasan When we have 32 or more CPUs in the affinity mask, we should use a special constant to specify that to the host. Fix this issue. Signed-off-by: K. Y. Srinivasan Cc: --- drivers/pci/host/pci-hyperv.c | 11 --- 1 files changed, 8 insertions(+), 3 deletions(-) d

[PATCH 0/2] pci-hyperv: Some miscellaneous fixes

2017-03-24 Thread kys
From: K. Y. Srinivasan Some miscellaneous fixes. K. Y. Srinivasan (2): pci-hyperv: Fix a bug in specifying CPU affinity pci-hyperv: Fix an atomic bug drivers/pci/host/pci-hyperv.c | 13 + 1 files changed, 9 insertions(+), 4 deletions(-) __

[PATCH v3] HV: properly delay KVP packets when negotiation is in progress

2017-03-24 Thread Long Li
From: Long Li The host may send multiple negotiation packets (due to timeout) before the KVP user-mode daemon is connected. We need to defer processing those packets until the daemon is negotiated and connected. It's okay for guest to respond to all negotiation packets. In addition, the host ma

Re: [RFC PATCH v2 15/32] x86: Add support for changing memory encryption attribute in early boot

2017-03-24 Thread Borislav Petkov
On Thu, Mar 02, 2017 at 10:15:28AM -0500, Brijesh Singh wrote: > Some KVM-specific custom MSRs shares the guest physical address with > hypervisor. When SEV is active, the shared physical address must be mapped > with encryption attribute cleared so that both hypervsior and guest can > access the d

[PATCH v2] staging: media: davinci_vpfe:Replace a bit shift.

2017-03-24 Thread Arushi Singhal
This patch replaces bit shifting on 1 with the BIT(x) macro. This was done with coccinelle: @@ constant c; @@ -1 << c +BIT(c) Signed-off-by: Arushi Singhal --- changes in v2 -Remove unnecessary parenthesis. drivers/staging/media/davinci_vpfe/dm365_ipipe.c | 2 +- drivers/staging/media/davi

[PATCH v2] staging: media: omap4iss: Replace a bit shift by a use of BIT.

2017-03-24 Thread Arushi Singhal
This patch replaces bit shifting on 1 with the BIT(x) macro. This was done with coccinelle: @@ constant c; @@ -1 << c +BIT(c) Signed-off-by: Arushi Singhal --- changes in v2 -Remove unnecessary parenthesis drivers/staging/media/omap4iss/iss_csi2.c| 2 +- drivers/staging/media/omap4iss/iss

RE:

2017-03-24 Thread WebStar Loan Firm
Are you in need of a loan for business / personal loan? Apply now by email, note, this offer is for serious minded people Only: web.loanf...@gmail.com ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/l

[PATCH v2] staging: media: atomisp:Replace a bit shift.

2017-03-24 Thread Arushi Singhal
This patch replaces bit shifting on 1 with the BIT(x) macro. This was done with coccinelle: @@ constant c; @@ -1 << c +BIT(c) Signed-off-by: Arushi Singhal --- changes in v2 -remove the unnecessary parenthesis. drivers/staging/media/atomisp/pci/atomisp2/atomisp_cmd.c | 12 ++-- drive

Re: [PATCH 1/2 v5] pci-hyperv: properly handle pci bus remove

2017-03-24 Thread Bjorn Helgaas
On Thu, Mar 23, 2017 at 02:58:10PM -0700, Long Li wrote: > From: Long Li > > hv_pci_devices_present is called in hv_pci_remove when we remove a PCI > device from host (e.g. by disabling SRIOV on a device). In hv_pci_remove, > the bus is already removed before the call, so we don't need to rescan

[PATCH] staging: iio: Remove extra Parenthesis.

2017-03-24 Thread Arushi Singhal
Remove the extra parenthesis remove the checkpatch issue. Signed-off-by: Arushi Singhal --- drivers/staging/iio/cdc/ad7746.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c index 033a41fd9bee

Re: [PATCH] staging: media: atomisp: remove ifdef around HMM_BO_ION

2017-03-24 Thread Greg Kroah-Hartman
On Fri, Mar 24, 2017 at 02:20:24PM +0100, Arnd Bergmann wrote: > The revert reintroduced a build failure without CONFIG_ION: > > media/atomisp/pci/atomisp2/hmm/hmm.c:52:2: error: excess elements in array > initializer [-Werror] > media/atomisp/pci/atomisp2/hmm/hmm.c:52:2: note: (near initializati

Re: [PATCH] staging: lustre: Remove redundant code

2017-03-24 Thread Greg Kroah-Hartman
On Fri, Mar 24, 2017 at 05:09:53PM +0530, Arushi Singhal wrote: > Remove the code which do not have any value. > > Signed-off-by: Arushi Singhal > --- > drivers/staging/lustre/lnet/lnet/net_fault.c | 5 - > 1 file changed, 5 deletions(-) > > diff --git a/drivers/staging/lustre/lnet/lnet/net

[PATCH] staging: media: atomisp: remove ifdef around HMM_BO_ION

2017-03-24 Thread Arnd Bergmann
The revert reintroduced a build failure without CONFIG_ION: media/atomisp/pci/atomisp2/hmm/hmm.c:52:2: error: excess elements in array initializer [-Werror] media/atomisp/pci/atomisp2/hmm/hmm.c:52:2: note: (near initialization for 'hmm_bo_type_strings') We should really be able to build in any

[PATCH] staging: lustre: Remove redundant code

2017-03-24 Thread Arushi Singhal
Remove the code which do not have any value. Signed-off-by: Arushi Singhal --- drivers/staging/lustre/lnet/lnet/net_fault.c | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/lustre/lnet/lnet/net_fault.c b/drivers/staging/lustre/lnet/lnet/net_fault.c index 18183cbb9859..b60

[PATCH v2] staging: speakup: Remove the explicit NULL comparison

2017-03-24 Thread Arushi Singhal
Fixed coding style for null comparisons in speakup driver to be more consistant with the rest of the kernel coding style. Replaced 'x != NULL' with 'x' and 'x = NULL' with '!x'. Signed-off-by: Arushi Singhal --- changes in v2 - change the commit subject drivers/staging/speakup/fakekey.c | 2

[PATCH v3] staging: speakup: Align the code.

2017-03-24 Thread Arushi Singhal
Delete tabs and add spaces to align the code to fix the checkpatch issue: "CHECK: Alignment should match open parenthesis". Signed-off-by: Arushi Singhal --- changes in v3 - make the commit message more clear. drivers/staging/speakup/speakup_apollo.c | 2 +- drivers/staging/speakup/speakup_dec

Re: [[RESEND]PATCH staging/speakup v3 3/3] use speakup_allocate as per required context

2017-03-24 Thread Pranay Srivastava
On Fri, Mar 24, 2017 at 2:13 PM, Greg KH wrote: > On Fri, Mar 24, 2017 at 02:07:11PM +0530, Pranay Kr. Srivastava wrote: >> speakup_allocate used GFP_ATOMIC for allocations >> even while during initialization due to it's use >> in notifier call. > > Is that a problem? No that's the way it should

[PATCH] Staging: fbtft: Fix checkpatch warning

2017-03-24 Thread Anar Adilova
This patch fixes the checkpatch.pl warning: EXPORT_SYMBOL(foo); should immediately follow its function/variable. The EXPORT_SYMBOL statements are placed inside if blocks, after both function implementations. Signed-off-by: Anar Adilova --- drivers/staging/fbtft/fbtft-core.c | 6 -- 1 file

Re: [PATCH] staging: wilc1000: use kernel define byte order macros

2017-03-24 Thread Dan Carpenter
On Thu, Mar 23, 2017 at 04:15:06PM -0600, Robert Perry Hooker wrote: > Well, yes, all data is 'endian' one way or another, right? I guess the byte > order of the tx/rx_buffers is host-endian > (which could be big), or _maybe_ network-endian... The good news is this code is Open Source[tm] so we d

Re: [[RESEND]PATCH staging/speakup v3 3/3] use speakup_allocate as per required context

2017-03-24 Thread Greg KH
On Fri, Mar 24, 2017 at 02:07:11PM +0530, Pranay Kr. Srivastava wrote: > speakup_allocate used GFP_ATOMIC for allocations > even while during initialization due to it's use > in notifier call. Is that a problem? > Pass GFP_ flags as well to speakup_allocate depending > on the context it is called

[[RESEND]PATCH staging/speakup v3 3/3] use speakup_allocate as per required context

2017-03-24 Thread Pranay Kr. Srivastava
speakup_allocate used GFP_ATOMIC for allocations even while during initialization due to it's use in notifier call. Pass GFP_ flags as well to speakup_allocate depending on the context it is called in. Signed-off-by: Pranay Kr. Srivastava --- drivers/staging/speakup/main.c | 8 1 file

Re: [PATCH] drivers/staging/media: atomisp: Removing redundant information from dev_err

2017-03-24 Thread Dan Carpenter
On Fri, Mar 24, 2017 at 11:18:33AM +0530, Pushkar Jambhlekar wrote: > Removing hardcoded function name as code is already using __func__ > > Signed-off-by: Pushkar Jambhlekar > --- > drivers/staging/media/atomisp/pci/atomisp2/hmm/hmm_bo.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >

[PATCH] drivers/staging/wilc1000: Removing explicit function tracing using dev_dbg/info

2017-03-24 Thread Pushkar Jambhlekar
ftrace can be used to trace functions. Removing function tracing using dev_dbg/info Signed-off-by: Pushkar Jambhlekar --- drivers/staging/wilc1000/wilc_sdio.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_sdio.c b/drivers/staging/wilc1000/wilc_sdio.c inde

[PATCH] drivers/staging/wilc1000: Using __func__ instead of hardcoded function name

2017-03-24 Thread Pushkar Jambhlekar
dev_err: replacing hardcoded function name with '%s' and __func__ Signed-off-by: Pushkar Jambhlekar --- drivers/staging/wilc1000/wilc_sdio.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_sdio.c b/drivers/staging/wilc1000/wilc_sdio.c index

[PATCH v3] HV: properly delay KVP packets when negotiation is in progress

2017-03-24 Thread Long Li
From: Long Li The host may send multiple negotiation packets (due to timeout) before the KVP user-mode daemon is connected. We need to defer processing those packets until the daemon is negotiated and connected. It's okay for guest to respond to all negotiation packets. In addition, the host ma