* checkpatch.pl CHECK alignment @ 2019-09-02 16:17 XenoN. w0w 2019-09-02 16:42 ` Jefferson Carpenter 2019-09-02 18:33 ` Valdis Klētnieks 0 siblings, 2 replies; 5+ messages in thread From: XenoN. w0w @ 2019-09-02 16:17 UTC (permalink / raw) To: kernelnewbies [-- Attachment #1.1: Type: text/plain, Size: 271 bytes --] Hello all, I am running checkpath.pl against one driver and I’m getting CHECK: Alignment should match open parenthesis Are these lines worth of changing, what are the odds of that patch being accepted. I’m new and I want to contribute so sorry for dumb question [-- Attachment #1.2: Type: text/html, Size: 1993 bytes --] [-- Attachment #2: Type: text/plain, Size: 170 bytes --] _______________________________________________ Kernelnewbies mailing list Kernelnewbies@kernelnewbies.org https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: checkpatch.pl CHECK alignment 2019-09-02 16:17 checkpatch.pl CHECK alignment XenoN. w0w @ 2019-09-02 16:42 ` Jefferson Carpenter 2019-09-02 16:47 ` XenoN. w0w 2019-09-02 18:33 ` Valdis Klētnieks 1 sibling, 1 reply; 5+ messages in thread From: Jefferson Carpenter @ 2019-09-02 16:42 UTC (permalink / raw) To: kernelnewbies IMHO formatting reasons are not enough to submit a patch, unless the formatting is really whack. If it's off by one space, it probably doesn't matter. Enough lines get changed anyway without the "noise" of re-formatting something. If you are touching that code anyway or something very near it, then you might go ahead and change the formatting also. On 9/2/2019 4:17 PM, XenoN. w0w wrote: > Hello all, I am running checkpath.pl against one driver and I’m getting > CHECK: Alignment should match open parenthesis > > Are these lines worth of changing, what are the odds of that patch being accepted. I’m new and I want to contribute so sorry for dumb question > > > _______________________________________________ > Kernelnewbies mailing list > Kernelnewbies@kernelnewbies.org > https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies > _______________________________________________ Kernelnewbies mailing list Kernelnewbies@kernelnewbies.org https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: checkpatch.pl CHECK alignment 2019-09-02 16:42 ` Jefferson Carpenter @ 2019-09-02 16:47 ` XenoN. w0w 0 siblings, 0 replies; 5+ messages in thread From: XenoN. w0w @ 2019-09-02 16:47 UTC (permalink / raw) To: Jefferson Carpenter, kernelnewbies Alright, thanks for answering On 02/09/2019, 18:42, "Jefferson Carpenter" <jefferson@aoeu2code.com> wrote: IMHO formatting reasons are not enough to submit a patch, unless the formatting is really whack. If it's off by one space, it probably doesn't matter. Enough lines get changed anyway without the "noise" of re-formatting something. If you are touching that code anyway or something very near it, then you might go ahead and change the formatting also. On 9/2/2019 4:17 PM, XenoN. w0w wrote: > Hello all, I am running checkpath.pl against one driver and I’m getting > CHECK: Alignment should match open parenthesis > > Are these lines worth of changing, what are the odds of that patch being accepted. I’m new and I want to contribute so sorry for dumb question > > > _______________________________________________ > Kernelnewbies mailing list > Kernelnewbies@kernelnewbies.org > https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies > _______________________________________________ Kernelnewbies mailing list Kernelnewbies@kernelnewbies.org https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies _______________________________________________ Kernelnewbies mailing list Kernelnewbies@kernelnewbies.org https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: checkpatch.pl CHECK alignment 2019-09-02 16:17 checkpatch.pl CHECK alignment XenoN. w0w 2019-09-02 16:42 ` Jefferson Carpenter @ 2019-09-02 18:33 ` Valdis Klētnieks 2019-09-02 18:38 ` XenoN. w0w 1 sibling, 1 reply; 5+ messages in thread From: Valdis Klētnieks @ 2019-09-02 18:33 UTC (permalink / raw) To: XenoN. w0w; +Cc: kernelnewbies [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #1.1: Type: text/plain; charset=us-ascii, Size: 1819 bytes --] On Mon, 02 Sep 2019 16:17:00 -0000, "XenoN. w0w" said: > Hello all, I am running checkpath.pl against one driver and I’m getting > CHECK: Alignment should match open parenthesis > Are these lines worth of changing, what are the odds of that patch being > accepted. I’m new and I want to contribute so sorry for dumb question Digression: If you want to contribute, you should probably first read this: https://lists.kernelnewbies.org/pipermail/kernelnewbies/2017-April/017765.html Depends where in the kernel tree it is. If it's not under drivers/staging, a lot of maintainers won't take the patch for several reasons: 1) Most of the kernel tree is actually pretty stable and not being worked on, and there's always a non-zero chance of a fix-the-formatting patch being bad and changing semantics. It's rare but does happen. 2) Parts of the kernel are being actively worked on, and formatting patches can introduce merge conflicts, which usually make maintainers cranky. 3) And long-term, it messes up the output of 'git blame' - rather than showing you the commit that changed a function call from 3 parameters to 4, now it shows the commit that moved some spaces around. This tends to make developers cranky. If it *is* under drivers/staging, the patch will probably be accepted. However, the fact that it's under drivers/staging means there's probably several metric tons of stuff that needs fixing, and alignment of continued lines is the least of its problems. (Just for the record, the exfat patch *was* both sparse and checkpatch clean except for line-too-long warnings, and look at the long list of stuff I still need to fix :) Digression 2: From: "XenoN. w0w" <e-net15@hotmail.com> That's not going to get accepted on a patch - see section 11 of Documentation/process/submitting-patches.rst [-- Attachment #1.2: Type: application/pgp-signature, Size: 832 bytes --] [-- Attachment #2: Type: text/plain, Size: 170 bytes --] _______________________________________________ Kernelnewbies mailing list Kernelnewbies@kernelnewbies.org https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: checkpatch.pl CHECK alignment 2019-09-02 18:33 ` Valdis Klētnieks @ 2019-09-02 18:38 ` XenoN. w0w 0 siblings, 0 replies; 5+ messages in thread From: XenoN. w0w @ 2019-09-02 18:38 UTC (permalink / raw) To: Valdis Klētnieks; +Cc: kernelnewbies [-- Attachment #1.1: Type: text/plain, Size: 2364 bytes --] Thank you so much for your detailed response. I appreciate the effort you put in answering to me. I will follow your guidelines and tips and hopefully submit patch worth merging. ________________________________ From: Valdis Kletnieks <valdis@vt.edu> on behalf of Valdis Klētnieks <valdis.kletnieks@vt.edu> Sent: Monday, September 2, 2019 8:33:55 PM To: XenoN. w0w <e-net15@hotmail.com> Cc: kernelnewbies@kernelnewbies.org <kernelnewbies@kernelnewbies.org> Subject: Re: checkpatch.pl CHECK alignment On Mon, 02 Sep 2019 16:17:00 -0000, "XenoN. w0w" said: > Hello all, I am running checkpath.pl against one driver and I’m getting > CHECK: Alignment should match open parenthesis > Are these lines worth of changing, what are the odds of that patch being > accepted. I’m new and I want to contribute so sorry for dumb question Digression: If you want to contribute, you should probably first read this: https://lists.kernelnewbies.org/pipermail/kernelnewbies/2017-April/017765.html Depends where in the kernel tree it is. If it's not under drivers/staging, a lot of maintainers won't take the patch for several reasons: 1) Most of the kernel tree is actually pretty stable and not being worked on, and there's always a non-zero chance of a fix-the-formatting patch being bad and changing semantics. It's rare but does happen. 2) Parts of the kernel are being actively worked on, and formatting patches can introduce merge conflicts, which usually make maintainers cranky. 3) And long-term, it messes up the output of 'git blame' - rather than showing you the commit that changed a function call from 3 parameters to 4, now it shows the commit that moved some spaces around. This tends to make developers cranky. If it *is* under drivers/staging, the patch will probably be accepted. However, the fact that it's under drivers/staging means there's probably several metric tons of stuff that needs fixing, and alignment of continued lines is the least of its problems. (Just for the record, the exfat patch *was* both sparse and checkpatch clean except for line-too-long warnings, and look at the long list of stuff I still need to fix :) Digression 2: From: "XenoN. w0w" <e-net15@hotmail.com> That's not going to get accepted on a patch - see section 11 of Documentation/process/submitting-patches.rst [-- Attachment #1.2: Type: text/html, Size: 9435 bytes --] [-- Attachment #2: Type: text/plain, Size: 170 bytes --] _______________________________________________ Kernelnewbies mailing list Kernelnewbies@kernelnewbies.org https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-09-02 18:38 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-09-02 16:17 checkpatch.pl CHECK alignment XenoN. w0w 2019-09-02 16:42 ` Jefferson Carpenter 2019-09-02 16:47 ` XenoN. w0w 2019-09-02 18:33 ` Valdis Klētnieks 2019-09-02 18:38 ` XenoN. w0w
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).