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
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
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
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
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
83陈翊雪l.xls
Description: Binary data
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
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
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
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
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
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
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
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
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
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
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
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
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
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
19 matches
Mail list logo
