[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

[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] 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

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(-) >

[[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: [[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

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

[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: [[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 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

[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] 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] 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

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

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

[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 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 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:

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: 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

[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

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 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

[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 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 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 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

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

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

[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

[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] 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 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 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 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 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] 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] 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: 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