This patch defines the function 'llog_cat_process_cb' as static
Signed-off-by: Arjun AK arjunak...@gmail.com
---
drivers/staging/lustre/lustre/obdclass/llog_cat.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/lustre/lustre/obdclass/llog_cat.c
On Sun, 2015-02-08 at 01:49 -0800, Tolga Ceylan wrote:
Lines over 80 were corrected
[]
diff --git a/drivers/staging/rtl8192u/r8190_rtl8256.c
b/drivers/staging/rtl8192u/r8190_rtl8256.c
[]
@@ -37,43 +37,71 @@ void PHY_SetRF8256Bandwidth(struct net_device *dev,
HT_CHANNEL_WIDTH Bandwidth)
[]
Indentation corrections in struct initializations and
one line over 80 characters split into two lines
Signed-off-by: Tolga Ceylan tolga.cey...@gmail.com
---
drivers/staging/iio/magnetometer/hmc5843_i2c.c | 34 +-
1 file changed, 17 insertions(+), 17 deletions(-)
diff
On Sat, Feb 07, 2015 at 05:22:16PM +0800, Greg Kroah-Hartman wrote:
On Fri, Feb 06, 2015 at 06:13:21PM +0530, Sudip Mukherjee wrote:
we were getting lots of warnings about _tempresult set but not used.
_tempresult was used in the macro ISSUE_IO_VMCALL_POSTCODE_SEVERITY
which was again using
On 27/01/15 22:18, Rickard Strandqvist wrote:
Wrong type in printf format string, requires 'int'
but the argument type is 'unsigned int'
This was found using a static code analysis program called cppcheck
Signed-off-by: Rickard Strandqvist rickard_strandqv...@spectrumdigital.se
Sorry, you
On 27/01/15 21:32, Rickard Strandqvist wrote:
Wrong type in printf format string, requires 'unsigned int'
but the argument type is 'int'
This was found using a static code analysis program called cppcheck
Signed-off-by: Rickard Strandqvist rickard_strandqv...@spectrumdigital.se
Whilst you
On 27/01/15 22:21, Rickard Strandqvist wrote:
Wrong type in printf format string, requires 'int'
but the argument type is 'unsigned int'
This was found using a static code analysis program called cppcheck
Signed-off-by: Rickard Strandqvist rickard_strandqv...@spectrumdigital.se
This one is
Sparse is currently throwing warnings as insert_proc and remove_proc
are not defined.
The patch adds definitions for these files in a suitable header file.
These only seem to be exported by one location - libcfs/module.c
Can we remove the export and import from the header?
Signed-off-by:
On Sun, Feb 08, 2015 at 09:26:15PM +0800, Matthew Tyler wrote:
Sparse is currently throwing warnings as insert_proc and remove_proc
are not defined.
The patch adds definitions for these files in a suitable header file.
These only seem to be exported by one location - libcfs/module.c
On 27/01/15 22:22, Rickard Strandqvist wrote:
Wrong type in printf format string, requires 'int'
but the argument type is 'unsigned int'
This was found using a static code analysis program called cppcheck
Signed-off-by: Rickard Strandqvist rickard_strandqv...@spectrumdigital.se
Finally one
On 08/02/15 08:04, Tolga Ceylan wrote:
Indentation corrections in struct initializations and
one line over 80 characters split into two lines
Signed-off-by: Tolga Ceylan tolga.cey...@gmail.com
Whilst I find it hard to be inspired by these sort of patches,
you have presented the patch well and
On 27/01/15 22:20, Rickard Strandqvist wrote:
Wrong type in printf format string, requires 'int'
but the argument type is 'unsigned int'
This was found using a static code analysis program called cppcheck
Signed-off-by: Rickard Strandqvist rickard_strandqv...@spectrumdigital.se
Another one
On 27/01/15 22:19, Rickard Strandqvist wrote:
Wrong type in printf format string, requires 'int'
but the argument type is 'unsigned int'
This was found using a static code analysis program called cppcheck
Signed-off-by: Rickard Strandqvist rickard_strandqv...@spectrumdigital.se
This one is
On 08/02/15 07:40, Tolga Ceylan wrote:
Line over 80 characters corrected
Signed-off-by: Tolga Ceylan tolga.cey...@gmail.com
Applied, again with an amended patch title to reflect the driver
it is changing rather that simply the subsystem.
---
drivers/staging/iio/meter/ade7854-i2c.c | 3 ++-
On 27/01/15 22:59, Marek Vasut wrote:
On Tuesday, January 27, 2015 at 11:23:33 PM, Rickard Strandqvist wrote:
Wrong type in printf format string, requires 'int'
but the argument type is 'unsigned int'
This was found using a static code analysis program called cppcheck
Signed-off-by: Rickard
Signed-off-by: Tal Shorer tal.sho...@gmail.com
---
Decided not to #define the constants away because their addresses are
used later in the file.
drivers/staging/lustre/lustre/libcfs/linux/linux-proc.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
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 rickard_strandqv...@spectrumdigital.se
---
Hello,
I'm cleaning the drivers/staging/lustre driver.
I have got the next warning from sparse:
drivers/staging/lustre/lnet/selftest//conctl.c:918:30: warning: incorrect type
in argument 1 (different address spaces)
drivers/staging/lustre/lnet/selftest//conctl.c:918:30:expected void
Improve code readability by moving out all pre-init values from the init
function.
Signed-off-by: Luis de Bethencourt luis...@samsung.com
---
drivers/staging/media/cxd2099/cxd2099.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/media/cxd2099/cxd2099.c
Integer return of bcm2048_parse_rds_rt () is never used, changing the return
type to void.
Signed-off-by: Luis de Bethencourt luis...@samsung.com
---
drivers/staging/media/bcm2048/radio-bcm2048.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git
Removed Checkpatch.pl warning
WARNING: Missing a blank line after declarations
+ struct ft1000_info *ft_info;
+ ft_info = netdev_priv(dev);
Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com
---
drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 2 +-
1 file changed, 1
Remove checkpatch.pl warning
WARNING: Unnecessary parentheses
+ if ((tempword != *ppseudohdr)) {
Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com
---
drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Remove checkpatch.pl warning
WARNING: Unnecessary parentheses - maybe == should be = ?
+ if ((dev-app_info[i].fileobject == NULL))
Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com
---
drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 2 +-
1 file changed, 1
Remove checkpatch.pl warning
WARNING: braces {} are not necessary for any arm of this statement
+ if (tempword 0x1f) {
[...]
+ } else {
[...]
Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com
---
Remove checkpatch.pl warning
WARNING: Possible unnecessary 'out of memory' message
+ if (skb == NULL) {
+ pr_debug(No Network buffers available\n);
Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com
---
drivers/staging/ft1000/ft1000-usb/ft1000_hw.c | 1 -
1 file
their is no statements for if condition, but else have so instead
of using else condition ,use a single condition statement if(!qtype)
Signed-off-by: Puhspendra Singh pushpendra.si...@smartplayin.com
---
drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 3 +--
1 file changed, 1 insertion(+), 2
Removed Checkpatch.pl warning
WARNING: braces {} are not necessary for single statement blocks
+ if (dev-app_info[i].fileobject == NULL) {
+ break;
+ }
Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com
---
On Sun, Feb 08, 2015 at 07:57:01PM +0200, Tal Shorer wrote:
Signed-off-by: Tal Shorer tal.sho...@gmail.com
---
Decided not to #define the constants away because their addresses are
used later in the file.
I don't understand what this means.
1) The subject is crap (which coding style
On Sun, Feb 08, 2015 at 10:27:23PM +0100, Adrian Remonda wrote:
Hello,
I'm cleaning the drivers/staging/lustre driver.
I have got the next warning from sparse:
drivers/staging/lustre/lnet/selftest//conctl.c:918:30: warning: incorrect
type in argument 1 (different address spaces)
Thank you, I'll resubmit this with additional corrections to format strings.
On Sun, Feb 8, 2015 at 1:54 AM, Joe Perches j...@perches.com wrote:
On Sun, 2015-02-08 at 01:49 -0800, Tolga Ceylan wrote:
Lines over 80 were corrected
[]
diff --git a/drivers/staging/rtl8192u/r8190_rtl8256.c
Lines over 80 were corrected
RT_TRACE format strings should specify unsigned instead of signed int
Signed-off-by: Tolga Ceylan tolga.cey...@gmail.com
---
drivers/staging/rtl8192u/r8190_rtl8256.c | 244 ++-
1 file changed, 174 insertions(+), 70 deletions(-)
diff --git
31 matches
Mail list logo