Lines Matching full:should

246     The first argument for kcalloc or kmalloc_array should be the
259     should be::
276     Architecture specific defines should be avoided wherever
286     BUG() or BUG_ON() should be avoided totally.
355     The function names should preferably follow the above pattern.
407     otherwise valid syscalls.  This should be avoided in new code.
412     ENOTSUPP is not a standard error code and should be avoided in new patches.
413     EOPNOTSUPP should be used instead.
418     EXPORT_SYMBOL should immediately follow the symbol to be exported.
433     It should not be used for any other purpose.
443     lockdep_assert_held() annotations should be preferred over
449     No #include statements in include/uapi should use a uapi/ path.
452     usleep_range() should be preferred over udelay(). The proper way of
478     C99 style single line comments (//) should not be used.
484     Applications of data_race() should have a comment so as to document the
504     signal to be delayed on the PCB; this unusual configuration should be
524     For adding version info, the following comment style should be used::
535     description of the changes made by the patch should be added.
560     line should be added according to Developer's certificate of
567     required that a simple sign off line should be present at the
621     should be written as::
638     side of the test should be avoided.
645     Code indent should use tabs instead of spaces.
661     switch should be at the same indent as case.
711     grepped, should not be split across multiple lines.
743     Trailing statements (for example after any conditional) should be
749     should be::
759     The ARRAY_SIZE(foo) macro should be preferred over
772     Local symbol names that are prefixed with `.L` should be avoided,
779     but `.L` prefixed local symbol names should not be used to denote
795     read-only.  The __read_mostly annotation thus should be removed.
814     should either use weak functions (appropriate for some cases), or
815     the symbol that protects them should be the same symbol we use.
820     do {} while(0) macros should not have a trailing semicolon.
823     Const init definitions should use __initconst instead of
830     The inline keyword should sit between storage class and type.
839     should be::
861     Macros with multiple statements should be enclosed in a
862     do - while block.  Same should also be the case for macros
878     Macro definition should not end with a semicolon. The macro
879     invocation style should be consistent with function calls.
967          This should be corrected to::
976          All arguments should have identifier names.
983     should be::
988     Global variables should not be initialized explicitly to
994     Static variables should not be initialized explicitly to zero.
1037     Permission bits should use 4 digit octal permissions (like 0700 or 0444).
1056     Assignment operators should not be written at the start of a
1057     line but should follow the operand at the previous line.
1096     Concatenated elements should have a space in between.
1101     should be::
1106     `else {` should follow the closing block `}` on the same line.
1117     The opening brace should be following the function definitions on the
1118     next line.  For any non-functional block it should be on the same line
1141     Trailing whitespace should always be removed.
1179     while should follow the closing bracket on the same line::
1192     Kconfig symbols should have a help text which fully describes
1201     So, CVS style keywords ($Id$, $Revision$, $Log$) should not be
1208     A "break;" should be added after empty default statement to avoid
1213     the line.  These should be removed.
1222     Devicetree bindings should be their own patch.  This is because
1249     Prefixing 0x with decimal output is defective and should be corrected.