Re: [PATCH resend] staging, lustre: fix a sparse error

2014-10-12 Thread WANG Chao
On 10/12/14 at 08:17pm, Al Viro wrote: > On Fri, Oct 10, 2014 at 11:21:16AM +0800, WANG Chao wrote: > > > I think __user annotation is for no dereferencing in kernel space. In > > this case, I think it's fine to override this error by __force. Because > > they're pointers with identical target typ

Re: [PATCH 1/2] staging: rtl8188eu: Fix coding style space missing problems

2014-10-12 Thread hejianet
Thanks for the suggestion, I will update it later On Thu, 09 Oct 2014 07:35:59 -0700 from j...@perches.com wrote: > git ls-files "drivers/staging/rtl8188eu/*.[ch]" | \ > xargs ./scripts/checkpatch.pl -f --fix-inplace --strict --types=SPACING ___ deve

[PATCH] staging: wlan-ng: Renamed prism2_fragmenthreshold in prism2mib.c

2014-10-12 Thread Erich Cordoba Malibran
From: Erich Cordoba Malibran Renamed function to fix style issues. Signed-off-by: Erich Cordoba Malibran --- drivers/staging/wlan-ng/prism2mib.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/drivers/staging/wlan-ng/prism2mib.c b/drivers/stagi

Re: [PATCH] staging: wlan-ng: Renamed prism2_fragmenthreshold in prism2mib.c

2014-10-12 Thread Greg KH
On Sun, Oct 12, 2014 at 09:29:32PM -0500, Erich Cordoba Malibran wrote: > From: Erich Cordoba Malibran > > Renamed function to fix style issues. Why is changing a name, to a misspelled one, a "style issue fix"? > > Signed-off-by: Erich Cordoba Malibran > --- > drivers/staging/wlan-ng/prism2

Re: [PATCH] staging: dgap: re-arrange functions for removing forward declarations.

2014-10-12 Thread Greg KH
On Mon, Oct 13, 2014 at 11:34:25AM +0900, Daeseok Youn wrote: > Re-arrange the functions for removing forward declarations. > > Signed-off-by: Daeseok Youn > --- > This patch has too many changes for re-arranging the functions. > So I wonder that I should break this up into smaller patches. Are

Re: [PATCH] Staging: wlan-ng: prism2 simple style modifications

2014-10-12 Thread Greg KH
On Sun, Oct 12, 2014 at 11:44:07AM -0500, Erich Cordoba Malibran wrote: > From: Erich Cordoba > > prism2fw.c > prism2sta.c > Fixed style warning reported by checkpatch.pl > prism2mib.c > Renamed function prism2mib_framenthreshold to > meet style guidelines. You are doing multip

Re: [PATCH resend] staging, lustre: fix a sparse error

2014-10-12 Thread Al Viro
On Fri, Oct 10, 2014 at 11:21:16AM +0800, WANG Chao wrote: > I think __user annotation is for no dereferencing in kernel space. In > this case, I think it's fine to override this error by __force. Because > they're pointers with identical target types. Umm... The real question seems to be whethe

[PATCH 0/2] staging: rtl8192u: Fix sparse warnings

2014-10-12 Thread Koray Gulcu
The first patch fixes some endianness warnings detected by sparse. The second patch amends lines with more than 80 characters in the firs patch. Koray Gulcu (2): staging:rtl8192u: Fix sparse warnings of r8182U_core staging:rtl8192u: Fix checkpatch.pl warnings drivers/staging/rtl8192u/r8192U

[PATCH 2/2] staging: rtl8192u: Fix checkpatch.pl warnings

2014-10-12 Thread Koray Gulcu
This patch fixes "line over 80 characters" warnings found in the first patch by breaking u4bAcParam's calculation down into a few steps. Signed-off-by: Koray Gulcu --- drivers/staging/rtl8192u/r8192U_core.c | 18 -- 1 file changed, 12 insertions(+), 6 deletions(-) diff --git

[PATCH 1/2] staging: rtl8192u: Fix sparse warnings of r8182U_core

2014-10-12 Thread Koray Gulcu
This patch fixes the following endianness warnings found by sparse running with CF=-D__CHECK_ENDIAN__: drivers/staging/rtl8192u/r8192U_core.c:1794:10: warning: incorrect type in initializer (different base types) drivers/staging/rtl8192u/r8192U_core.c:1794:10:expected restricted __le16 drive

[PATCH] Staging: wlan-ng: prism2 simple style modifications

2014-10-12 Thread Erich Cordoba Malibran
From: Erich Cordoba prism2fw.c prism2sta.c Fixed style warning reported by checkpatch.pl prism2mib.c Renamed function prism2mib_framenthreshold to meet style guidelines. Signed-off-by: Erich Cordoba Malibran --- drivers/staging/wlan-ng/prism2fw.c | 10 +++--- drive

Re: [PATCH] staging: quickstart: remove driver

2014-10-12 Thread Angelo Arrifano
Hi all, I cannot maintain the driver anymore since my only machine with support to the "quickstart" hardware died quite some time ago. I would be happy if someone else could continue maintaining the driver though. :) Best regards, - Angelo Arrifano On Thu, 2014-07-24 at 21:58 +0300, Kristina Mar

[PATCH] staging: lustre: lustre: ptlrpc: sec.c: Replacing strncat with strlcat to avoid overwrite size

2014-10-12 Thread Rickard Strandqvist
Changed from using strncat with strlcat to avoid overwrite the max size. Take the opportunity to change a snprint to strlcpy. Signed-off-by: Rickard Strandqvist --- drivers/staging/lustre/lustre/ptlrpc/sec.c |5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/stag

哪些因素与价格没有关系

2014-10-12 Thread 陈翊雪
83陈翊雪l.xls Description: Binary data ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: comedi: drivers: comedi_bond.c: Changed from using strncat to strlcat

2014-10-12 Thread Rickard Strandqvist
Changed from using strncat to strlcat to simplify the code Signed-off-by: Rickard Strandqvist --- drivers/staging/comedi/drivers/comedi_bond.c |6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/comedi/drivers/comedi_bond.c b/drivers/staging/comedi/drive

[PATCH v2] Staging: lustre: llite: fix some sparse warnings about userspace pointer

2014-10-12 Thread Anh Le
Add __user macro to the function declarations that accept userspace pointers as arguments. Signed-off-by: Anh Le --- v2: Fixed checkpatch's parenthesis alignment warnings, noted by Sudip Mukherjee . .../staging/lustre/lustre/include/lprocfs_status.h | 6 +-- drivers/staging/lustre/lustre/llit

Re: [PATCH 11/15] staging: rtl8723au: Remove a stack of write only counters

2014-10-12 Thread Jes Sorensen
Greg KH writes: > On Sat, Oct 11, 2014 at 05:32:49AM -0700, Greg KH wrote: >> On Fri, Oct 10, 2014 at 09:41:34PM +0200, jes.soren...@redhat.com wrote: >> > From: Jes Sorensen >> > >> > We don't do anything with these, so get rid of them >> > >> > Signed-off-by: Jes Sorensen >> > --- >> > driv

Re: [PATCH 00/15] staging: rtl8723au: Fixes and cleanups

2014-10-12 Thread Jes Sorensen
Greg KH writes: > On Fri, Oct 10, 2014 at 09:41:23PM +0200, jes.soren...@redhat.com wrote: >> From: Jes Sorensen >> >> Hi, >> >> Please find attached a couple of fixes and cleanups for the rtl8723au >> driver. This includes a fix for the mac_addr alignment reported by Dan >> Carpenter and two p

[PATCH 1/1] Octeon: Staging Driver: Remove coding warnings

2014-10-12 Thread Akshay Sarode
Checkpatch coding warnings of 'Missing a blank line after declaration' Signed-off-by: Akshay Sarode --- drivers/staging/octeon/ethernet-tx.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/drivers/staging/octeon/ethernet-tx.c b/drivers/staging/octeon/ethernet-tx.c index 4e54d85