This patch fixes checkpatch.pl warning.
WARNING: space prohibited between function name and open parenthesis '('
Signed-off-by: Alberto Pires de Oliveira Neto
---
drivers/staging/lustre/lustre/fld/fld_internal.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drive
This patch fixes checkpatch.pl warning.
WARNING: void function return statements are not generally useful
Signed-off-by: Alberto Pires de Oliveira Neto
---
drivers/staging/lustre/lustre/fld/fld_request.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/lustre/lustre/fld
This patch fixes checkpatch.pl warning.
WARNING: space required after that close brace '}'
Signed-off-by: Alberto Pires de Oliveira Neto
---
drivers/staging/lustre/lustre/fld/lproc_fld.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/lustre/
Changes since v1:
- Put '}' in the next line instead of just insert a space for lproc_fld.c.
Alberto Pires de Oliveira Neto (3):
staging: lustre: space prohibited between function name and open parenthesis
'('
staging: lustre: void function return statements are
:
> On Sun, 2015-03-08 at 17:43 -0300, Alberto Pires de Oliveira Neto wrote:
>> This patch fixes checkpatch.pl warning.
>> WARNING: space required after that close brace '}'
>>
>> Signed-off-by: Alberto Pires de Oliveira Neto
>> ---
>> drivers/stagin
ok, I'll give it a try.
On Sun, Mar 8, 2015 at 7:07 PM, Joe Perches wrote:
> On Sun, 2015-03-08 at 19:04 -0300, Alberto Pires de Oliveira Neto wrote:
>> It make sense. But I would also have move some dependencies as well.
>> As I'm still very new to the kernel, a
2015 at 5:53 PM, Joe Perches wrote:
> On Sun, 2015-03-08 at 17:43 -0300, Alberto Pires de Oliveira Neto wrote:
>> This patch fixes checkpatch.pl warning.
>> WARNING: space required after that close brace '}'
>>
>> Signed-off-by: Alberto Pires de Oliveira Neto
>
This patch fixes checkpatch.pl warning.
WARNING: space prohibited between function name and open parenthesis '('
Signed-off-by: Alberto Pires de Oliveira Neto
---
drivers/staging/lustre/lustre/fld/fld_internal.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drive
This patch fixes checkpatch.pl warning.
WARNING: space required after that close brace '}'
Signed-off-by: Alberto Pires de Oliveira Neto
---
drivers/staging/lustre/lustre/fld/lproc_fld.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/lustre/
This patch fixes checkpatch.pl warning.
WARNING: void function return statements are not generally useful
Signed-off-by: Alberto Pires de Oliveira Neto
---
drivers/staging/lustre/lustre/fld/fld_request.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/lustre/lustre/fld
Alberto Pires de Oliveira Neto (3):
staging: lustre: space prohibited between function name and open parenthesis
'('
staging: lustre: void function return statements are not generally useful.
staging: lustre: space required after that close brace '}'
drivers/stagi
This patch fixes checkpatch.pl warning.
WARNING: space prohibited between function name and open parenthesis '('
Signed-off-by: Alberto Pires de Oliveira Neto
---
drivers/staging/lustre/lustre/fld/fld_request.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drive
This patch fixes checkpatch.pl warning.
WARNING: else is not generally useful after a break or return
Signed-off-by: Alberto Pires de Oliveira Neto
---
drivers/staging/lustre/lustre/fld/fld_request.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers
that's true.
I think it's ok now.
On Sun, Mar 1, 2015 at 10:30 PM, Greg KH wrote:
> On Sat, Feb 28, 2015 at 01:37:23PM -0300, Alberto Pires de Oliveira Neto
> wrote:
>> This patch fixes checkpatch.pl warning.
>> WARNING: else is not generally useful after a
Ok, it's done.
Thanks again.
On Sat, Feb 28, 2015 at 6:29 AM, Sudip Mukherjee
wrote:
> On Fri, Feb 27, 2015 at 11:59:21AM -0300, Alberto Pires de Oliveira Neto
> wrote:
>> Sorry, still getting the hang of it. Should I resend the patch ?
> better. and then your subject lin
This patch fixes checkpatch.pl warning.
WARNING: else is not generally useful after a break or return
Signed-off-by: Alberto Pires de Oliveira Neto
---
drivers/staging/lustre/lustre/fld/fld_request.c | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/drivers
h again.
Thanks, for the patience.
On Thu, Feb 26, 2015 at 10:30 PM, Greg KH wrote:
> On Thu, Feb 26, 2015 at 10:20:06PM -0300, Alberto Pires de Oliveira Neto
> wrote:
>> Signed-off-by: Alberto Pires de Oliveira Neto
>>
>> drivers/staging/lustre/lustre/fld/fld_request.c
very sorry for the html patch. It was my first one.
Signed-off-by: Alberto Pires de Oliveira Neto
---
drivers/staging/lustre/lustre/fld/fld_request.c | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/lustre/lustre/fld/fld_request.c
b/drivers/staging
Hi Jeff,
I'm really sorry, but I forget to mention, that I had the same problems on a
RedHat RH4 (x86_64) too.
> On Thu, Aug 16, 2007 at 09:30:56AM -0700, Arjan van de Ven wrote:
> > this sounds like something really stupid and bad... why would the kernel
> > need to have a per-distro note se
19 matches
Mail list logo