This changes the remaining printk statements with the log level of KERN_INFO
to use the more appropriate netdev_info for print messages related to code
using or part of the networking subsystem. Further more this was found by
running checkpatch.pl on the file, rtllib_tx.c.
Signed-off-by: Nicholas
TIF_MEMDIE should not be set on a process if it does not have a valid
->mm, and this is protected by task_lock().
If TIF_MEMDIE gets set after the mm has detached, and the process fails to
exit, then the oom killer will defer forever waiting for it to exit.
Make sure that the mm is still valid
I focused on function declarations (placed return type on the same line
as function name), and spaces between function name and parenthesis
Signed-off-by: Guillaume Matheron
---
.../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c| 445 +
1 file changed, 186 insertions(+), 259
I fixed lines over 80 characters and unnecessary returns
Signed-off-by: Guillaume Matheron
---
.../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c| 53 +-
1 file changed, 31 insertions(+), 22 deletions(-)
diff --git a/drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c
b/dr
On Thu, Apr 02, 2015 at 06:24:21PM +0300, Dan Carpenter wrote:
> Still mangled. Read Documentation/email-clients.txt. Send the patch
> to yourself. Apply it with git am. Then resend when that works.
Also, the students had to end up using 'sendmail' by hand, piping it the
patch and some odd com
On Thu, Apr 02, 2015 at 04:57:52PM +0200, vcaba...@clipper.ens.fr wrote:
> >From 654380760bfc5b35ed3d3d4a7d3eb126fe747959 Mon Sep 17 00:00:00 2001
> From: Vivien Cabannes
> Date: Thu, 2 Apr 2015 16:32:15 +0200
> Subject: [PATCH] Staging: lustre: api-ni.c: adding tabs because lines were
> too long
On Thu, Apr 02, 2015 at 06:24:21PM +0300, Dan Carpenter wrote:
> Still mangled. Read Documentation/email-clients.txt. Send the patch
> to yourself. Apply it with git am. Then resend when that works.
Dan, sorry about these, I just taught a class how to write a kernel
patch, and emails got all m
>From 654380760bfc5b35ed3d3d4a7d3eb126fe747959 Mon Sep 17 00:00:00 2001
From: Vivien Cabannes
Date: Thu, 2 Apr 2015 16:32:15 +0200
Subject: [PATCH] Staging: lustre: api-ni.c: adding tabs because lines were
too long.
Adding tabs because lines were too long.
---
drivers/staging/lustre/lnet/lnet/
Replace two occurrences of "+1" with simply "1".
Signed-off-by: Luca Wehrstedt
---
drivers/staging/lustre/lustre/lov/lov_io.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/lustre/lustre/lov/lov_io.c
b/drivers/staging/lustre/lustre/lov/lov_io.c
index
remove space between function names and open parentheses as reported by
checkpatch.pl.
Signed-off-by: Auguste Olivry
---
drivers/staging/lustre/lnet/selftest/ping_test.c | 30 +++---
1 file changed, 15 insertions(+), 15 deletions(-)
diff --git a/drivers/staging/lustre/lnet/sel
remove space between function names and open parentheses as reported by
checkpatch.pl.
Signed-off-by: Auguste Olivry
---
drivers/staging/lustre/lnet/selftest/ping_test.c | 30 +++---
1 file changed, 15 insertions(+), 15 deletions(-)
diff --git a/drivers/staging/lustre/lnet/sel
Fix your email client from header so it shows your name.
regards,
dan carpenter
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Also fix your email client from header so that it shows your name.
regards,
dan carpenter
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
On Thu, Apr 02, 2015 at 05:03:38PM +0200, vcaba...@clipper.ens.fr wrote:
> >From 654380760bfc5b35ed3d3d4a7d3eb126fe747959 Mon Sep 17 00:00:00 2001
> From: Vivien Cabannes
> Date: Thu, 2 Apr 2015 16:32:15 +0200
> Subject: [PATCH] Staging: lustre: api-ni.c: adding tabs because lines were
> too long
On Thu, Apr 02, 2015 at 04:32:32PM +0200, Gwendoline Chouasne-Guillon wrote:
> >From d4def87bcfa41640c5ba0e171036c0ee9da98c08 Mon Sep 17 00:00:00 2001
> From: Gwendoline Chouasne-Guillon
> Date: Thu, 2 Apr 2015 16:11:04 +0200
> Subject: [PATCH] Staging: lustre: o2iblnd_modparam.c: remove unneeded
Still mangled. Read Documentation/email-clients.txt. Send the patch
to yourself. Apply it with git am. Then resend when that works.
regards,
dan carpenter
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/ma
Thanks!
regards,
dan carpenter
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
>From f770478d61e80e66cb310b92529f968c12831206 Mon Sep 17 00:00:00 2001
From: Cyrille Ruggero
Date: Thu, 2 Apr 2015 16:09:14 +0200
Subject: [PATCH] Staging: lustre: dir.c: adding a blank after a declaration.
Adding a blank in this file after a declaration as reported by checkpatch.pl.
Signed-off
>From 654380760bfc5b35ed3d3d4a7d3eb126fe747959 Mon Sep 17 00:00:00 2001
From: Vivien Cabannes
Date: Thu, 2 Apr 2015 16:32:15 +0200
Subject: [PATCH] Staging: lustre: api-ni.c: adding tabs because lines were
too long.
Adding tabs because lines were too long.
---
drivers/staging/lustre/lnet/lnet/
>From f770478d61e80e66cb310b92529f968c12831206 Mon Sep 17 00:00:00 2001
From: Cyrille Ruggero
Date: Thu, 2 Apr 2015 16:09:14 +0200
Subject: [PATCH] Staging: lustre: dir.c: adding a blank after a declaration.
Adding a blank in this file after a declaration as reported by checkpatch.pl.
---
drive
>From d4def87bcfa41640c5ba0e171036c0ee9da98c08 Mon Sep 17 00:00:00 2001
From: Gwendoline Chouasne-Guillon
Date: Thu, 2 Apr 2015 16:11:04 +0200
Subject: [PATCH] Staging: lustre: o2iblnd_modparam.c: remove unneeded space
before the arguments of a function
Remove unneeded space before the arguments
>From f770478d61e80e66cb310b92529f968c12831206 Mon Sep 17 00:00:00 2001
From: Cyrille Ruggero
Date: Thu, 2 Apr 2015 16:09:14 +0200
Subject: [PATCH] Staging: lustre: dir.c: adding a blank after a declaration.
Adding a blank in this file after a declaration as reported by checkpatch.pl.
---
drive
The driver has to unregister from platform device when it's unloaded
Signed-off-by: Phong Tran
---
drivers/staging/android/ion/ion_test.c | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/android/ion/ion_test.c
b/drivers/staging/android/ion/ion_test.c
in
Add the remove() method for deregister from misc device
when it's unloaded.
Signed-off-by: Phong Tran
---
drivers/staging/android/ion/ion_test.c | 12
1 file changed, 12 insertions(+)
diff --git a/drivers/staging/android/ion/ion_test.c
b/drivers/staging/android/ion/ion_test.c
inde
Base on the file comment should define GPL v2 for ion test driver
Signed-off-by: Phong Tran
---
drivers/staging/android/ion/ion_test.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/android/ion/ion_test.c
b/drivers/staging/android/ion/ion_test.c
index 654acb5..3bc461c 10064
Hi Greg, Arve, Riley, Dan,
This patcheset fixes:
(1) Define the GPL V2 for ion_test driver
(2) Add the remove() platform driver callback to
deregister the misc device
(3) unregister platform device when driver is unloaded
Thank for Dan's reivew.
Changes from v1:
Imporve the coding st
Remove an unneeded return statement in this file as reported by
checkpatch.pl.
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/lustre/lnet/selftest/module.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/lustre/lnet/selftest/module.c
b/drivers/staging/lustre/lnet/selft
This patch fixes the sparse warning: "cast to restricted __le16" reported
for rtl8723au/hal/rtl8723au_xmit.c
Signed-off-by: Piotr Witoslawski
---
v2: Changing pointer type instead of casting, as suggested by Jes Sorensen
drivers/staging/rtl8723au/hal/rtl8723au_xmit.c | 2 +-
1 file changed, 1
On Tue, 31 Mar 2015, Drokin, Oleg wrote:
>
> On Mar 31, 2015, at 11:57 AM, gre...@linuxfoundation.org wrote:
>
> > On Tue, Mar 31, 2015 at 05:15:23PM +0200, Julia Lawall wrote:
> >> On Tue, 31 Mar 2015, Dhere, Chaitanya (C.) wrote:
> >>
> >>> This patch replaces kzalloc and copy_from_user with m
Dan Carpenter writes:
> This doesn't look right and it doesn't have a changelog explainly the
> weirdness.
>
... and the fix is as ugly as it gets!
If something like this is needed, creating a __le16 *ptr at the
beginnging of the function would be a lot better.
Jes
_
This doesn't look right and it doesn't have a changelog explainly the
weirdness.
regards,
dan carpenter
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Signed-off-by: Piotr Witoslawski
---
drivers/staging/rtl8723au/hal/rtl8723au_xmit.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8723au/hal/rtl8723au_xmit.c
b/drivers/staging/rtl8723au/hal/rtl8723au_xmit.c
index 1759487..256958e 100644
--- a/drivers/st
Hi,
You are receiving this email because we wish you to use our target email
marketing service.
Email marketing is one of the best marketing strategies of all time and has
helped many businesses
globally achieve their goals, double their profits and increase their
client base.
We have worked on
33 matches
Mail list logo
