The seq_printf return value, because it's frequently misused,
will eventually be converted to void.
See: commit 1f33c41c03da ("seq_file: Rename seq_overflow() to
seq_has_overflowed() and make public")
Convert these "return seq_printf(...)" uses to:
seq_printf(seq, ...);
retu
These functions don't need to return anything.
No caller uses the return value.
Miscellanea:
Integrate the individual strings to reduce object size ~100 bytes.
Signed-off-by: Joe Perches
---
drivers/staging/lustre/include/linux/libcfs/libcfs_hash.h | 4 ++--
drivers/staging/lustre/lustre/libc
As Al Viro said:
we are getting well-meaning folks who try to check that return value,
again and again, getting it wrong every time. Typical idiocies:
* return some kind of error out of ->show() on overflows. Pointless
*and* wrong - only hard errors (== fail read(2) with that) should be
The seq_printf return value, because it's frequently misused,
will eventually be converted to void.
See: commit 1f33c41c03da ("seq_file: Rename seq_overflow() to
seq_has_overflowed() and make public")
Signed-off-by: Joe Perches
---
drivers/staging/i2o/i2o_proc.c | 18 --
1
The seq_printf return value, because it's frequently misused,
will eventually be converted to void.
See: commit 1f33c41c03da ("seq_file: Rename seq_overflow() to
seq_has_overflowed() and make public")
Convert these uses to:
seq_printf(seq, ...);
return 0;
Done via cocci sc
The seq_printf return value, because it's frequently misused,
will eventually be converted to void.
See: commit 1f33c41c03da ("seq_file: Rename seq_overflow() to
seq_has_overflowed() and make public")
Signed-off-by: Joe Perches
---
drivers/staging/rtl8192e/rtllib_module.c | 4
The seq_printf return value, because it's frequently misused,
will eventually be converted to void.
See: commit 1f33c41c03da ("seq_file: Rename seq_overflow() to
seq_has_overflowed() and make public")
Convert the remaining uses by hand.
Miscellanea:
o Convert fixed string output to seq_put
From: Matteo Semenzato
This patch fixes the following warnings:
space prohibited before that close parenthesis ')'
space prohibited after that open parenthesis '('
Signed-off-by: Matteo Semenzato
---
drivers/staging/fbtft/fb_bd663474.c | 100 ++--
1 file changed
Fixes quoted string split accross lines checkpatch.pl warning.
Signed-off-by: Boran Car
---
drivers/staging/media/lirc/lirc_zilog.c | 11 ---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/media/lirc/lirc_zilog.c
b/drivers/staging/media/lirc/lirc_zilog.c
i
On Sat, Feb 21, 2015 at 07:38:57PM +, Boran Car wrote:
> This fixes quoted string split accross lines.
That's not an issue for this field, sorry.
And your subject didn't say for what part of the kernel this is needed.
thanks,
greg k-h
___
devel ma
This fixes quoted string split accross lines.
Signed-off-by: Boran Car
---
drivers/staging/media/lirc/lirc_zilog.c | 11 ---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/media/lirc/lirc_zilog.c
b/drivers/staging/media/lirc/lirc_zilog.c
index e16627c..344
This patch fixes the checkpatch.pl warning:
WARNING: braces {} are not necessary for single statement blocks
Signed-off-by: Edward Lipinsky
---
drivers/staging/lustre/lnet/lnet/lib-eq.c |3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/lustre/lnet/lnet/lib
On Fri, Feb 20, 2015 at 11:34:09PM +0200, Andrey Utkin wrote:
> See below how sparse output changed with these changes.
> In few words:
> - fixed printf specifiers for size_t;
> - trying to fix address space specifiers issues, not sure what's correct
> approach, ASKING FOR COMMENTS AND HELP;
Send
On Sat, Feb 21, 2015 at 04:01:46PM +0100, Matteo Semenzato wrote:
> From: Matt
>
> This patch fixes the following warnings:
> space prohibited before that close parenthesis ')'
> space prohibited after that open parenthesis '('
> Signed-off-by: Matteo Semenzato
Your "From:" line doesn't match y
On 21/02/15 18:11, Jonathan Cameron wrote:
> On 08/02/15 18:22, Rickard Strandqvist wrote:
>> Change struct iio_prtc_trigger_info frequency
>> type from int to unsigned int.
>> Since it is always treated as such in the driver
>> so they type should probably reflect this.
>>
>> Signed-off-by: Rickar
On 08/02/15 18:22, Rickard Strandqvist wrote:
> Change struct iio_prtc_trigger_info frequency
> type from int to unsigned int.
> Since it is always treated as such in the driver
> so they type should probably reflect this.
>
> Signed-off-by: Rickard Strandqvist
The only argument against this is t
From: Matt
This patch fixes the following warnings:
space prohibited before that close parenthesis ')'
space prohibited after that open parenthesis '('
Signed-off-by: Matteo Semenzato
---
drivers/staging/fbtft/fb_bd663474.c | 100 ++--
1 file changed, 50 insertio
I find the Kconfig dependencies of the various Unisys drivers rather
confusing. Due to the dependencies, you must select the drivers one by
one to see the following ones. So if you are looking for a specific
driver, it is not visible by default. And if you don't know exactly
what you need, it's eve
18 matches
Mail list logo