Messages in this thread | | | Date | Fri, 15 Sep 2023 14:56:38 +0700 | From | Bagas Sanjaya <> | Subject | Re: [PATCH] staging: fbtft: Removed unnecessary parenthesis around conditions to comply with the checkpatch coding style. |
| |
On Fri, Sep 15, 2023 at 11:58:47AM +1000, angus gardner wrote: > Thanks for the feedback Bagas, > > "Then checkpatch is wrong, " lol > > Ill double check my SoB. >
tl;dr:
> A: http://en.wikipedia.org/wiki/Top_post > Q: Were do I find info about this thing called top-posting? > A: Because it messes up the order in which people normally read text. > Q: Why is top-posting such a bad thing? > A: Top-posting. > Q: What is the most annoying thing in e-mail? > > A: No. > Q: Should I include quotations after my reply? > > http://daringfireball.net/2007/07/on_top
Also, don't send HTML emails as mailing lists reject them away.
Last but not least, this is the third time you do parentheses fixup, for which Greg's bot said:
> You sent a patch that has been sent multiple times in the past few > days, and is identical to ones that has been recently rejected. > Please always look at the mailing list traffic to determine if you are > duplicating other people's work.
Again, read all Documentation/process/*.rst docs so that you won't make mistakes again as kernel developer.
Bye!
-- An old man doll... just what I always wanted! - Clara [unhandled content-type:application/pgp-signature] | |