On Fri, Feb 3, 2017 at 3:42 PM, Dan Carpenter <dan.carpen...@oracle.com> wrote:
> On Sat, Dec 03, 2016 at 03:14:26PM +0530, Tabrez khan wrote:
>> Add parentheses around variable x for the readability purpose.
>>
>
> It's not really about readability...
>
> regards,
Remove unnecessary braces from single line if statement.
This warning is found using checkpatch.pl.
Signed-off-by: Tabrez khan <khan.tabre...@gmail.com>
---
drivers/staging/lustre/lustre/obdecho/echo_client.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/s
Space required before assignment operator,
to make close to kernel coding style.
This error is found using checkpatch.pl.
Signed-off-by: Tabrez khan <khan.tabre...@gmail.com>
---
drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Space required before assignment operator,
to make close to kernel coding style.
This error is found using checkpatch.pl.
Signed-off-by: Tabrez khan <khan.tabre...@gmail.com>
---
drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Remove unnecessary braces from single line if statement.
This warning is found using checkpatch.pl.
Signed-off-by: Tabrez khan <khan.tabre...@gmail.com>
---
drivers/staging/lustre/lustre/osc/lproc_osc.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/s
Remove unnecessary braces from single-line if statement.
This warning is found using checkpatch.pl.
Signed-off-by: Tabrez khan <khan.tabre...@gmail.com>
---
drivers/staging/lustre/lustre/ptlrpc/import.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/s
Remove unnecessary braces {} from single line if statement.
This warning is found using checkpatch.pl.
Signed-off-by: Tabrez khan <khan.tabre...@gmail.com>
---
drivers/staging/lustre/lustre/ptlrpc/import.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/s
Remove unnecessary braces {} for single while statement.
This warning is found using checkpatch.pl.
Signed-off-by: Tabrez khan <khan.tabre...@gmail.com>
---
drivers/staging/lustre/lustre/obdclass/cl_io.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/s
On Thu, Dec 15, 2016 at 7:08 AM, Greg KH <gre...@linuxfoundation.org> wrote:
> On Thu, Dec 15, 2016 at 07:03:52AM +0530, Tabrez khan wrote:
>> Remove unnecessary braces {} for single if statement block.
>> This warning is found using checkpatch.pl.
>>
>> Signe
Remove unnecessary braces {} for single if statement block.
This warning is found using checkpatch.pl.
Signed-off-by: Tabrez khan <khan.tabre...@gmail.com>
---
drivers/staging/lustre/lustre/osc/osc_cache.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/s
Remove unnecessary braces {} for single if statement block.
This warning is found using checkpatch.pl.
Signed-off-by: Tabrez khan <khan.tabre...@gmail.com>
---
drivers/staging/lustre/lustre/osc/osc_cache.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/s
From: Tabrez khan <khan.tabre...@gmail.com>
Remove blank lines.
Signed-off-by: Tabrez khan <khan.tabre...@gmail.com>
---
drivers/staging/i4l/act2000/module.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/i4l/act2000/module.c
b/drivers/staging/i4l/act2000/mo
Remove braces {} for single if statement block.
Signed-off-by: Tabrez khan <khan.tabre...@gmail.com>
---
drivers/staging/i4l/act2000/module.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/i4l/act2000/module.c
b/drivers/staging/i4l/act2000/module.c
Add parentheses around variable x for the readability purpose.
This warning was found using checkpatch.pl.
Signed-off-by: Tabrez khan <khan.tabre...@gmail.com>
---
drivers/staging/media/bcm2048/radio-bcm2048.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/s
14 matches
Mail list logo