This shows you the differences between two versions of the page.
Both sides previous revision Previous revision Next revision | Previous revision Next revision Both sides next revision | ||
en:users:drivers:ath10k:submittingpatches [2018/04/13 04:22] Kalle Valo [Guidelines] Remove "the" from commit log |
en:users:drivers:ath10k:submittingpatches [2019/02/15 11:59] Kalle Valo Add links to submitting patches and how to write git commit messages |
||
---|---|---|---|
Line 16: | Line 16: | ||
More info about submitting patches: | More info about submitting patches: | ||
* [[en/developers/Documentation/SubmittingPatches|linux-wireless submitting patches instructions]] | * [[en/developers/Documentation/SubmittingPatches|linux-wireless submitting patches instructions]] | ||
- | * [[en/developers/Documentation/git-guide|linux-wireless git guide]] | + | * [[en/developers/Documentation/git-guide|linux-wireless git guide]] |
+ | * [[https://www.kernel.org/doc/html/latest/process/submitting-patches.html|Linux kernel submitting patches]] | ||
+ | * [[https://medium.com/@steveamaza/how-to-write-a-proper-git-commit-message-e028865e5791|How to write proper git commit messages]] | ||
==== Guidelines ==== | ==== Guidelines ==== | ||
Line 39: | Line 41: | ||
* QCA6174 hw3.2 PCI WLAN.RM.4.4.1-00102-QCARMSWP-1 | * QCA6174 hw3.2 PCI WLAN.RM.4.4.1-00102-QCARMSWP-1 | ||
* QCA9377 SDIO WLAN.TF.1.1.1-00061-QCATFSWPZ-1 | * QCA9377 SDIO WLAN.TF.1.1.1-00061-QCATFSWPZ-1 | ||
- | * SHOULD use Reported-by: and Tested-by: tags if others have reported the issue. | + | * SHOULD use [[https://www.kernel.org/doc/html/latest/process/submitting-patches.html#using-reported-by-tested-by-reviewed-by-suggested-by-and-fixes|Reported-by: and Tested-by: tags]] if others have reported the issue. |
+ | * SHOULD use [[https://www.kernel.org/doc/html/latest/process/submitting-patches.html#describe-changes|Fixes: tag]] if it fixes a regression caused by known commit. | ||
* SHOULD be //mostly// checkpatch clean (though not all patchworks warnings make sense), it's RECOMMENDED to test patches with [[en/users/Drivers/ath10k/CodingStyle#checking_code|ath10k-check]] as that will disable the useless warnings. | * SHOULD be //mostly// checkpatch clean (though not all patchworks warnings make sense), it's RECOMMENDED to test patches with [[en/users/Drivers/ath10k/CodingStyle#checking_code|ath10k-check]] as that will disable the useless warnings. | ||