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
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
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
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
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
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
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/
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
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
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
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
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
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
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
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
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(-)
__
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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(-)
>
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
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
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
39 matches
Mail list logo