Differences

This shows you the differences between two versions of the page.

--- en:developers:documentation:submittingpatches [2017/05/31 12:42]
Kalle Valo [Format issues] Add link to linux-wireless git guide
+++ en:developers:documentation:submittingpatches [2018/03/26 15:03]
Kalle Valo [Checking state of patches from patchwork] Do not top post!
@@ Line 35: / Line 35: @@
 ===== Checking state of patches from patchwork =====
-All wireless patches are tracked in [[https://patchwork.kernel.org/project/linux-wireless/list/|linux-wireless patchwork project]] (only exception being ath10k which has its own [[https://patchwork.kernel.org/project/ath10k/list/?state=*|ath10k patchwork project]]). From patchwork you can check the state of the patch and to whom it is assigned. Here's a quick link to see all the patches, no matter what's the state:
+All wireless patches are tracked in [[https://patchwork.kernel.org/project/linux-wireless/list/|linux-wireless patchwork project]]. From patchwork you can check the state of the patch and to whom it is assigned. Here's a quick link to see all the patches, no matter what's the state:
 [[https://patchwork.kernel.org/project/linux-wireless/list/?state=*]]
-Always avoid contacting maintainers directly, they get way too much email already. Instead use the link above to find your patch and see the status. Only in last resort contact the maintainers, and do that by replying to your own patch and ask for status.
+Always avoid contacting maintainers directly, they get way too much email already. Instead use the link above to find your patch and see the status. Only in last resort contact the maintainers, and do that by replying to your own patch and ask for status. Do not top post!
 Different patchwork states and their meanings:
@@ Line 232: / Line 232: @@
 ==== Changelog missing ====
-When sending a new version of a patch or patchset you should **always** add a changelog so that maintainer can easily what has changed.
+When sending a new version of a patch or patchset you should **always** add a changelog so that maintainer can easily see what has changed.
 If you have just one patch you can add the changelog after the ''%%---%%'' (three dashes) line.
-If you have multiples patches (called a patchset) add the changelog to the cover letter. You can create the cover letter with switch ''--cover-letter'':
+If you have multiples patches (called a patchset) add the changelog to the cover letter. You can create the cover letter with the switch ''--cover-letter'':
   git format-patch --subject-prefix="PATCH v2" --cover-letter
@@ Line 305: / Line 305: @@
 a5f390562a37 mwifiex: add 802.11AC support
 </code>
+==== Too many patches ====
+The recommend size is 10-12 patches per patchset. More than that it gets difficult for reviewers and maintainers. Of course there's no hard rule, for simple patches more than that might be ok but then again for more complex patches even 10 patches per patchset might be too much.
+==== Resubmit the whole patchset =====
+Even if just one patch has changed in a patch series resubmit the whole patchset (and remember to increase the version number), do not just resubmit that one changed patch. The reason is that it's difficult to apply patches in correct order when some of them are submitted separately.
+==== Commit log does not answer "Why?" ====
+The commit log should //always// answer the question "Why?" and describe the reason what
+motivated to implement the patch. This is the most important part of the commit log as this helps maintainers, backports, distros etc to make decisions if the patch is important for them or not and to what release it should go.
+The commit log needs to tell why you wrote the patch. If you fixed a bug give a short summary of the bug (can be a long one as well, of course) from user's point of view, and if there's a publically available bug report include a link to that. If you are fixing a warning from a compiler or a static checker add the warning from tool. Or if it's just code cleanup or fixing a theoretical issue, and does not have practical user visible changes, mention that also.
 ===== More patch work references =====

Linux Wireless

User Tools

Site Tools

Differences

Page Tools