On Thursday 11 June 2015 05:24 PM, Sudip Mukherjee wrote:
> On Thu, Jun 11, 2015 at 09:51:15AM +, Abdul, Hussain (H.) wrote:
>> This patch removes unwnated true from boolean tests.
> did u mean "unwanted"?
>
> and for all your four patches the email header From: line is not matching
> the Signe
On Thursday 11 June 2015 08:06 PM, gre...@linuxfoundation.org wrote:
> On Thu, Jun 11, 2015 at 10:14:49AM +, Abdul, Hussain (H.) wrote:
>> This patch add IRQF_ONESHOT flag in threaded IRQs request without a primary
>> handler.
>>
>> Signed-off-by: Abdul Hussain
>> ---
>> drivers/staging/ste_
change own data type(WILC_Char) to common data type(s8)
Signed-off-by: Dean Lee
---
drivers/staging/wilc1000/coreconfigurator.c | 24 +++
drivers/staging/wilc1000/coreconfigurator.h | 4 +-
drivers/staging/wilc1000/host_interface.c | 88 +++
drivers/s
On Thu, Jun 11, 2015 at 06:39:09PM +0530, Sudip Mukherjee wrote:
> These variables were being set but not used afterwards.
>
> --- a/drivers/staging/rtl8188eu/core/rtw_sta_mgt.c
> +++ b/drivers/staging/rtl8188eu/core/rtw_sta_mgt.c
> @@ -147,18 +147,14 @@ inline struct sta_info
> *rtw_get_stainfo
change own data type(WILC_BOOL) to common data type(bool)
but that's contain true/false value. so change with them.
Signed-off-by: Dean Lee
---
Change
remove WILC_BOOL structure.
---
drivers/staging/wilc1000/coreconfigurator.c | 26 ++--
drivers/staging/wilc1000/coreconfigurator.h
On 2015년 06월 12일 13:56, Dean Lee wrote:
change own data type(WILC_BOOL) to common data type(bool)
but that's contain true/false value. so change with them.
Signed-off-by: Dean Lee
---
drivers/staging/wilc1000/coreconfigurator.c | 16 +--
drivers/staging/wilc1000/coreconfigurator.h
change own data type(WILC_BOOL) to common data type(bool)
but that's contain true/false value. so change with them.
Signed-off-by: Dean Lee
---
drivers/staging/wilc1000/coreconfigurator.c | 16 +--
drivers/staging/wilc1000/coreconfigurator.h | 4 +-
drivers/staging/wilc1000/fifo_b
Signed-off-by: Luca Ceresoli
Cc: Greg Kroah-Hartman
Cc: Larry Finger
---
drivers/staging/rtl8188eu/core/rtw_ap.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8188eu/core/rtw_ap.c
b/drivers/staging/rtl8188eu/core/rtw_ap.c
index 2b09972..581af88 100644
-
is_valid_p2p_probereq is initialized to false and never modified. Code
depending on it is dead code. Remove all of it.
Signed-off-by: Luca Ceresoli
Cc: Greg Kroah-Hartman
Cc: Larry Finger
---
drivers/staging/rtl8188eu/core/rtw_mlme_ext.c | 6 --
1 file changed, 6 deletions(-)
diff --git a
These are internal functions. Remove their declaration in rtw_mlme_ext.h
and make them static.
Signed-off-by: Luca Ceresoli
Cc: Greg Kroah-Hartman
Cc: Larry Finger
---
drivers/staging/rtl8188eu/core/rtw_mlme_ext.c| 57 +++-
drivers/staging/rtl8188eu/include/rtw_mlme_ext
The is_valid_p2p_probereq is never referenced in the function body.
Signed-off-by: Luca Ceresoli
Cc: Greg Kroah-Hartman
Cc: Larry Finger
---
drivers/staging/rtl8188eu/core/rtw_mlme_ext.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/rtl8188eu/core/rtw
These are internal functions. Remove their declaration in rtw_mlme_ext.h
and make them static.
Signed-off-by: Luca Ceresoli
Cc: Greg Kroah-Hartman
Cc: Larry Finger
---
drivers/staging/rtl8188eu/core/rtw_mlme_ext.c| 60
drivers/staging/rtl8188eu/include/rtw_mlme_ext
The is_valid_p2p_probereq is never referenced in the function body.
Signed-off-by: Luca Ceresoli
Cc: Greg Kroah-Hartman
Cc: Jes Sorensen
Cc: Larry Finger
---
drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/drivers/stagi
Handle all the style issues reported by checkpatch.pl.
Remove general white spaces, spaces in function calls,
alignments etc.
Signed-off-by: James Simmons
---
drivers/staging/lustre/include/linux/lnet/api.h| 32 +-
.../staging/lustre/include/linux/lnet/lib-lnet.h | 36 +-
.../staging/l
For the functions LNetInit and LNetFini move away from
camel case to lnet_init and lnet_fini.
Signed-off-by: James Simmons
---
drivers/staging/lustre/include/linux/lnet/api.h|3 ---
.../staging/lustre/include/linux/lnet/lib-lnet.h |3 +++
.../staging/lustre/include/linux/lnet/lib-t
From: John L. Hammond
Originally socklnd_lib-linux.h contained linux specific
wrappers and defines but since the linux kernel is the
only supported platform now we can merge what little
remains in the header into socklnd.h. This is broken
out of the original patch 12932 that was merged to the
Int
Currently the lnet headers used by user land contain various internal
LNet structures that are only used by kernel space. Move the user land
structures to headers used by user land. The kernel structures are
relocated to headers that are never exposed to user land.
Signed-off-by: James Simmons
--
Point to the right place for GNU license. Update Intel copyright.
Signed-off-by: James Simmons
---
drivers/staging/lustre/include/linux/lnet/api.h| 13 +
.../staging/lustre/include/linux/lnet/lib-lnet.h | 10 +++---
.../staging/lustre/include/linux/lnet/lib-types.h |
Remove ralnd, ptllnd, mxlnd, qswlnd drivers. They are no
longer supported and have not even been buildable
for a long time.
Signed-off-by: James Simmons
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6209
Reviewed-on: http://review.whamcloud.com/13663
Reviewed-by: Isaac Huang
Reviewed-by: D
From: John L. Hammond
Remove the linux specific headers from lnet/include/lnet/linux/,
moving whatever was worthwhile from them to their parent headers or
elsewhere.
Signed-off-by: John L. Hammond
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2675
Reviewed-on: http://review.whamcloud.com/
Currently several special macros LNet NID macros exist
in libcfs.h and libcfs_private.h. Move those macros
out to the lnet header types.h. The new lnet header
nidstr.h contains LNet NID string data that can be
used by user land LNet utilities and the LNet kernel
drivers.
Signed-off-by: James Simmo
From: John L. Hammond
With the move to support only the linux kernel their is
no need to keep "linux" in the socklnd source file names.
This is broken out of the original patch 12932 that was
merged to the Intel/OpenSFS branch.
Signed-off-by: John L. Hammond
Intel-bug-id: https://jira.hpdd.inte
Currently the lnet headers used for user land applications
contain various kernel definations. This is due to the
fact libcfs contains kernel wrappers for user land which
will be going away. This patch sorted the header data
so all kernel containing structures are moved out of
headers that user lan
Actually it doesn't leak. But it will Oops.
regards,
dan carpenter
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
On Thu, Jun 11, 2015 at 09:13:50AM -0700, gre...@linuxfoundation.org wrote:
> On Thu, Jun 11, 2015 at 07:48:43AM +, Gujulan Elango, Hari Prasath (H.)
> wrote:
> > From: Hari Prasath Gujulan Elango
> >
> > This patch removes code that is unused for any functionality in the
> > ioctl SIOCDEVPR
On Thu, Jun 11, 2015 at 04:22:21PM +, ja...@microsoft.com wrote:
> From: Jake Oshins
>
> This patch does some slight refactoring on the hv_vmbus driver and its
> dependents and then introduces a new paravirtual front-end driver for PCI
> which supports PCI Express devices passed through to a
On Thu, Jun 11, 2015 at 04:22:27PM +, ja...@microsoft.com wrote:
> From: Jake Oshins
>
> drivers:pci:hv: Update MAINTAINERS
Huh?
>
> drivers:pci:hv: New paravirtual PCI front-end for Hyper-V VMs
What is this odd format?
> Signed-off-by: Jake Oshins
No blank line?
Come on, you all know
On Thu, Jun 11, 2015 at 04:22:23PM +, ja...@microsoft.com wrote:
> From: Jake Oshins
>
> Signed-off-by: Jake Oshins
No changelog body text at all? That's a sure way to get a patch
instantly deleted from my queue...
___
devel mailing list
de...@l
From: Jake Oshins
drivers:pci:hv: Update MAINTAINERS
drivers:pci:hv: New paravirtual PCI front-end for Hyper-V VMs
Signed-off-by: Jake Oshins
---
MAINTAINERS|1 +
drivers/pci/Kconfig|9 +
drivers/pci/host/Makefile |1 +
drivers/pci/host/hv_pcifr
From: Jake Oshins
Signed-off-by: Jake Oshins
---
drivers/hv/vmbus_drv.c | 105 ++--
drivers/video/fbdev/hyperv_fb.c | 2 +-
include/linux/hyperv.h | 2 +-
3 files changed, 92 insertions(+), 17 deletions(-)
diff --git a/drivers/hv/vmbus_
From: Jake Oshins
Signed-off-by: Jake Oshins
---
drivers/hv/vmbus_drv.c | 87 -
drivers/video/fbdev/hyperv_fb.c | 46 ++
include/linux/hyperv.h | 7 +++-
3 files changed, 112 insertions(+), 28 deletions(-)
diff --gi
From: Jake Oshins
Signed-off-by: Jake Oshins
---
arch/x86/include/asm/mshyperv.h | 2 ++
arch/x86/kernel/cpu/mshyperv.c | 30 ++
2 files changed, 32 insertions(+)
diff --git a/arch/x86/include/asm/mshyperv.h b/arch/x86/include/asm/mshyperv.h
index c163215..bf7789e
From: Jake Oshins
Signed-off-by: Jake Oshins
---
drivers/hv/vmbus_drv.c | 16
include/linux/hyperv.h | 2 ++
2 files changed, 18 insertions(+)
diff --git a/drivers/hv/vmbus_drv.c b/drivers/hv/vmbus_drv.c
index 067a469..65e4eb6 100644
--- a/drivers/hv/vmbus_drv.c
+++ b/drivers
From: Jake Oshins
Signed-off-by: Jake Oshins
---
include/linux/hyperv.h | 11 +++
1 file changed, 11 insertions(+)
diff --git a/include/linux/hyperv.h b/include/linux/hyperv.h
index 6fec42d..1903238 100644
--- a/include/linux/hyperv.h
+++ b/include/linux/hyperv.h
@@ -1144,6 +1144,17 @@
From: Jake Oshins
This patch does some slight refactoring on the hv_vmbus driver and its
dependents and then introduces a new paravirtual front-end driver for PCI
which supports PCI Express devices passed through to a Linux guest running
in a Hyper-V VM.
To support this front-end, several new me
On Thu, Jun 11, 2015 at 02:22:33PM +, Ravindran, Madhusudhanan (M.) wrote:
> From: Madhusudhanan Ravindran
>
> pass wlan irq gpio number in run time using DT as
> mentioned in the TODO list.
>
> Signed-off-by: Madhusudhanan Ravindran
> --
> * did not test this code on real hardware.
On Thu, Jun 11, 2015 at 01:31:38PM +, Gujulan Elango, Hari Prasath (H.)
wrote:
> From: Hari Prasath Gujulan Elango
>
> This patch removes the timer related wrapper functions
> WILC_TimerCreate(),WILC_TimerStart(),WILC_TimerStop() and
> WILC_TimerDestroy() and uses kernel API's directly.In th
On Thu, Jun 11, 2015 at 07:48:43AM +, Gujulan Elango, Hari Prasath (H.)
wrote:
> From: Hari Prasath Gujulan Elango
>
> This patch removes code that is unused for any functionality in the
> ioctl SIOCDEVPRIVATE + 1.But since it is mentioned in the TODO
> list that it needs to be replaced with
On Thu, Jun 11, 2015 at 01:52:28PM +0900, Dean.lee wrote:
>
>
> On 2015년 06월 11일 12:50, Greg KH wrote:
> >On Thu, Jun 11, 2015 at 12:44:24PM +0900, Dean Lee wrote:
> >>This driver has odd message in print string.
> >>So this patch removes the data type.
> >>
> >>Signed-off-by: Dean Lee
> >>---
>
On Thu, Jun 11, 2015 at 02:33:39PM +0900, Johnny Kim wrote:
>
> On 2015년 06월 11일 09:40, Greg KH wrote:
> >On Wed, Jun 10, 2015 at 05:06:44PM +0900, Johnny Kim wrote:
> >>-static int wilc_wlan_cfg_commit(int type, uint32_t drvHandler)
> >>+static int wilc_wlan_cfg_commit(int type, size_t drvHandler
On Thu, Jun 11, 2015 at 10:14:49AM +, Abdul, Hussain (H.) wrote:
> This patch add IRQF_ONESHOT flag in threaded IRQs request without a primary
> handler.
>
> Signed-off-by: Abdul Hussain
> ---
> drivers/staging/ste_rmi4/synaptics_i2c_rmi4.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deleti
From: Madhusudhanan Ravindran
pass wlan irq gpio number in run time using DT as
mentioned in the TODO list.
Signed-off-by: Madhusudhanan Ravindran
--
* did not test this code on real hardware.
---
drivers/staging/wilc1000/linux_wlan.c | 28
drivers/st
From: Hari Prasath Gujulan Elango
This patch removes the timer related wrapper functions
WILC_TimerCreate(),WILC_TimerStart(),WILC_TimerStop() and
WILC_TimerDestroy() and uses kernel API's directly.In the process,the
unused variables,files are also deleted.
Signed-off-by: Hari Prasath Gujulan El
On Thu, Jun 11, 2015 at 12:35:48PM +, Gujulan Elango, Hari Prasath (H.)
wrote:
> From: Hari Prasath Gujulan Elango
>
> This patch removes the timer related wrapper functions
> WILC_TimerCreate(),WILC_TimerStart(),WILC_TimerStop() and
> WILC_TimerDestroy() and uses kernel API's directly.In th
These variables were being set but not used afterwards.
Signed-off-by: Sudip Mukherjee
---
drivers/staging/rtl8188eu/core/rtw_mlme.c| 3 ---
drivers/staging/rtl8188eu/core/rtw_sta_mgt.c | 6 +-
drivers/staging/rtl8188eu/hal/phy.c | 10 ++
drivers/staging/rt
The CONFIGs, the enum and the function declaration was not being
used anywhere.
Signed-off-by: Sudip Mukherjee
---
drivers/staging/rtl8188eu/include/odm.h | 8
drivers/staging/rtl8188eu/include/odm_HWConfig.h | 3 ---
2 files changed, 11 deletions(-)
diff --git a/drivers/stagi
From: Harisangam Sharvari
This patch was detected with the help of coccinelle tool.
The redundant comparisons of bool variables are removed in r8192U_dm.c.
Signed-off-by: Harisangam Sharvari
---
drivers/staging/rtl8192u/r8192U_dm.c | 12 ++--
1 file changed, 6 insertions(+), 6 delet
From: Hari Prasath Gujulan Elango
This patch removes the timer related wrapper functions
WILC_TimerCreate(),WILC_TimerStart(),WILC_TimerStop() and
WILC_TimerDestroy() and uses kernel API's directly.In the process,the
unused variables,files are also deleted.
Signed-off-by: Hari Prasath Gujulan El
Replace uses of OBD_ALLOC_LARGE by libcfs_kvzalloc and OBD_FREE_LARGE by
kvfree.
The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)
//
@@
expression ptr,size;
@@
- OBD_ALLOC_LARGE(ptr,size)
+ ptr = libcfs_kvzalloc(size, GFP_NOFS)
@@
expression ptr,size;
@@
-
Replace uses of OBD_ALLOC_LARGE by libcfs_kvzalloc and OBD_FREE_LARGE by
kvfree.
The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)
//
@@
expression ptr,size;
@@
- OBD_ALLOC_LARGE(ptr,size)
+ ptr = libcfs_kvzalloc(size, GFP_NOFS)
@@
expression ptr,size;
@@
-
Replace uses of OBD_ALLOC_LARGE by libcfs_kvzalloc and OBD_FREE_LARGE by
kvfree.
The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)
//
@@
expression ptr,size;
@@
- OBD_ALLOC_LARGE(ptr,size)
+ ptr = libcfs_kvzalloc(size, GFP_NOFS)
@@
expression ptr,size;
@@
-
Replace uses of OBD_ALLOC_LARGE by libcfs_kvzalloc and OBD_FREE_LARGE by
kvfree.
The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)
//
@@
expression ptr,size;
@@
- OBD_ALLOC_LARGE(ptr,size)
+ ptr = libcfs_kvzalloc(size, GFP_NOFS)
@@
expression ptr,size;
@@
-
Replace uses of OBD_ALLOC_LARGE by libcfs_kvzalloc and OBD_FREE_LARGE by
kvfree.
The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)
//
@@
expression ptr,size;
@@
- OBD_ALLOC_LARGE(ptr,size)
+ ptr = libcfs_kvzalloc(size, GFP_NOFS)
@@
expression ptr,size;
@@
-
Replace uses of OBD_ALLOC_LARGE by libcfs_kvzalloc and OBD_FREE_LARGE by
kvfree.
The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)
//
@@
expression ptr,size;
@@
- OBD_ALLOC_LARGE(ptr,size)
+ ptr = libcfs_kvzalloc(size, GFP_NOFS)
@@
expression ptr,size;
@@
-
Replace uses of OBD_ALLOC_LARGE by libcfs_kvzalloc and OBD_FREE_LARGE by
kvfree.
The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)
//
@@
expression ptr,size;
@@
- OBD_ALLOC_LARGE(ptr,size)
+ ptr = libcfs_kvzalloc(size, GFP_NOFS)
@@
expression ptr,size;
@@
-
Replace uses of OBD_ALLOC_LARGE by libcfs_kvzalloc and OBD_FREE_LARGE by
kvfree.
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
On Thu, Jun 11, 2015 at 09:51:15AM +, Abdul, Hussain (H.) wrote:
> This patch removes unwnated true from boolean tests.
did u mean "unwanted"?
and for all your four patches the email header From: line is not matching
the Signed-off-by: line. Please add an extra From: line like others of
your c
On Thu, Jun 11, 2015 at 02:46:21PM +0530, Harisangam Sharvari wrote:
> From: Harisangam Sharvari
>
> This patch was detected with the help of coccinelle tool.
> The unnecessary comparisons of bool variables are removed.
>
> Signed-off-by: Harisangam Sharvari
> ---
> drivers/staging/rtl8192u/r
This patch add IRQF_ONESHOT flag in threaded IRQs request without a primary
handler.
Signed-off-by: Abdul Hussain
---
drivers/staging/ste_rmi4/synaptics_i2c_rmi4.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/ste_rmi4/synaptics_i2c_rmi4.c
b/drivers/stagin
This patch remove unneeded variable used to store return value.
Signed-off-by: Abdul Hussain
---
drivers/staging/unisys/visorbus/visorbus_main.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/visorbus/visorbus_main.c
b/drivers/staging/unisys/visorbu
This patch removes unwnated true from boolean tests.
Signed-off-by: Abdul Hussain
---
drivers/staging/vt6655/rxtx.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/vt6655/rxtx.c b/drivers/staging/vt6655/rxtx.c
index 33c4aa4..534338c 100644
--- a/driver
This patch remove true and false from boolean tests.
Signed-off-by: Abdul Hussain
---
drivers/staging/vt6656/card.c | 2 +-
drivers/staging/vt6656/main_usb.c | 2 +-
drivers/staging/vt6656/rxtx.c | 2 +-
drivers/staging/vt6656/usbpipe.c | 2 +-
4 files changed, 4 insertions(+), 4 deleti
From: Harisangam Sharvari
This patch was detected with the help of coccinelle tool.
The unnecessary comparisons of bool variables are removed.
Signed-off-by: Harisangam Sharvari
---
drivers/staging/rtl8192u/r8192U_dm.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff
On Mon, Jun 08, 2015 at 04:01:57PM -0700, Tolga Ceylan wrote:
> In dir.c and llite_lib.c, sparse reports multiple warnings messages
> due to different address spaces. This patch resolves these warnings
> by adding the tag __user for username addresses.
>
> Signed-off-by: Tolga Ceylan
> ---
> dri
On Wed, Jun 10, 2015 at 5:48 PM, Greg Kroah-Hartman
wrote:
>
> Are you sure all of these are correct? The kernel/user api for lustre
> is a complex beast, and just casting away the pointer types isn't
> usually the proper thing to do in order to resolve the issues here.
>
> thanks,
>
> greg k-h
From: Hari Prasath Gujulan Elango
This patch removes code that is unused for any functionality in the
ioctl SIOCDEVPRIVATE + 1.But since it is mentioned in the TODO
list that it needs to be replaced with generic API functions,the
debug print alone is left as it is.
Signed-off-by: Hari Prasath Gu
From: Hari Prasath Gujulan Elango
This patch replaces the kmalloc followed by copy_from_user by the
wrapper routine memdup_user.
Signed-off-by: Hari Prasath Gujulan Elango
---
drivers/staging/wilc1000/linux_wlan.c | 12 +++-
1 file changed, 3 insertions(+), 9 deletions(-)
diff --git a
67 matches
Mail list logo
