Re: Fwd: Re: [PATCH] staging:lustre:lnet: lib-md.c fix checkpath warnings and errors.

[Janet Liu]

On 08/28/2014 10:20 AM, Janet Liu wrote:
> -------- Original Message --------
> Subject: Re: [PATCH] staging:lustre:lnet: lib-md.c fix checkpath
> warnings and errors.
> Date: Tue, 26 Aug 2014 18:22:45 -0700
> From: Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx>
> To: Janet Liu <janetliu526@xxxxxx>
> CC: devel@xxxxxxxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
>
> On Mon, Aug 25, 2014 at 01:18:54AM +0800, Janet Liu wrote:
> > Sliences the following warning and error:
> >
> >   WARNING: line over 80 characters
> >   WARNING: space prohibited between function name and open parenthesis
> > '('
> >   ERROR: do not use C99 // comments
> >   ERROR: trailing statements should be on next line
>
> You changed 4 things, in one patch :(
>
> Also, you sent 4 different patches, which one is correct?
>
> Please only do one thing per patch and please resend with the one that
> you wish to have reviewed.
>
> thanks,
>
> greg k-h
Hi Greg,

I'm very sorry for having troubled you.

The latest one is correct. It is send by my gmail box.

The first two is sent by this mailbox(janetliu526@xxxxxx) and is 
rejected by linux-kernel@xxxxxxxxxxxxxxxx

The third is tested by dan.carpenter@xxxxxxxxxx, he told me it broke 
build. So I fixed and send the fourth.

Those are all codestyle issue.

regards,
janet
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/