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


Groups > linux.kernel > #1634501

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

From Joe Perches <joe@perches.com>
Newsgroups linux.kernel
Subject Re: LKML: "Fixes as per checkpatch.pl" patches
Date 2017-05-02 18:20 +0200
Message-ID <tCIGR-AC-11@gated-at.bofh.it> (permalink)
References <tCIGR-AC-13@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


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.

Back to linux.kernel | Previous | Next | Find similar | Unroll thread


Thread

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

csiph-web