From 93fbc7142e4ec38eea5549392b49fe339abe94e9 Mon Sep 17 00:00:00 2001 From: Paul Beesley Date: Mon, 21 Jan 2019 12:06:24 +0000 Subject: doc: Link coding guidelines to user guide Adds a link from user-guide.rst to coding-guidelines.rst and merges the information about using checkpatch from both files into the user guide document. Change-Id: Iffbb4225836a042d20024faf28b8bdd6b2c4043e Signed-off-by: Paul Beesley --- docs/coding-guidelines.rst | 18 ++---------------- 1 file changed, 2 insertions(+), 16 deletions(-) (limited to 'docs/coding-guidelines.rst') diff --git a/docs/coding-guidelines.rst b/docs/coding-guidelines.rst index 3075c63d..bd8cd128 100644 --- a/docs/coding-guidelines.rst +++ b/docs/coding-guidelines.rst @@ -15,20 +15,8 @@ Some of the guidelines may also apply to other codebases. **Note:** the existing TF codebase does not necessarily comply with all the below guidelines but the intent is for it to do so eventually. -Coding style ------------- - -Trusted Firmware re-uses the `Linux Coding Style`_ . This style is enforced by -the *checkpatch* tool which can be found in the Linux source code (in `Linus's tree`_ -, for example). - -For convenience, the top-level TF makefile has a `checkpatch` target, which -defines a set of checkpatch options used in TF. - -Checkpatch errors will gate upstream merging of pull requests. - -Checkpatch warnings will not gate merging but should be reviewed and fixed if -possible. +Checkpatch overrides +-------------------- Some checkpatch warnings in the TF codebase are deliberately ignored. These include: @@ -503,8 +491,6 @@ writeable data section, which may result in a smaller and faster binary. Note that this may require dependent functions (``init()`` in the above example) to have ``const`` arguments, assuming they don't need to modify the data. -.. _`Linux Coding Style`: https://www.kernel.org/doc/html/latest/process/coding-style.html -.. _`Linus's tree`: https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/scripts/checkpatch.pl .. _`Why the “volatile” type class should not be used`: https://www.kernel.org/doc/html/latest/process/volatile-considered-harmful.html .. _`Procedure Call Standard for the Arm Architecture`: http://infocenter.arm.com/help/topic/com.arm.doc.ihi0042f/IHI0042F_aapcs.pdf .. _`Procedure Call Standard for the Arm 64-bit Architecture`: http://infocenter.arm.com/help/topic/com.arm.doc.ihi0055b/IHI0055B_aapcs64.pdf -- cgit v1.2.3