Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1634501 > unrolled thread

Re: LKML: "Fixes as per checkpatch.pl" patches

Started byJoe Perches <joe@perches.com>
First post2017-05-02 18:20 +0200
Last post2017-05-02 18:20 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: LKML: "Fixes as per checkpatch.pl" patches Joe Perches <joe@perches.com> - 2017-05-02 18:20 +0200

#1634501 — Re: LKML: "Fixes as per checkpatch.pl" patches

FromJoe Perches <joe@perches.com>
Date2017-05-02 18:20 +0200
SubjectRe: LKML: "Fixes as per checkpatch.pl" patches
Message-ID<tCIGR-AC-11@gated-at.bofh.it>
On Tue, 2017-05-02 at 18:08 +0300, Ali Kaasinen wrote:
> Hellooo, presumably contacted the right people.
> 
> I often browse LKML, and patches saying "Fix style issues as reported by 
> checkpatch.pl", and Greg responding "That's really vague, you need to be 
> specific, and only fix one type of thing per patch" seem fairly common 
> these days, e.g.:
> 
> - http://lkml.iu.edu/hypermail/linux/kernel/1704.3/01867.html
> 
> - http://lkml.iu.edu/hypermail/linux/kernel/1705.0/00105.html
> 
> Just wondering, shouldn't checkpatch mention that?

This question should be asked while cc'ing lkml.

What Greg does with patches is up to Greg.

A checkpatch user guide in Documentation could be useful.

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web