Messages in this thread Patch in this message | | | From | Andrew Jeffery <> | Subject | [PATCH] Documentation: checkpatch: Tweak BIT() macro include | Date | Thu, 20 May 2021 11:27:04 +0930 |
| |
While include/linux/bitops.h brings in the BIT() macro, it was moved to include/linux/bits.h in [1]. Since [1] BIT() has moved again into include/vdso/bits.h via [2].
I think the move to the vDSO header can be considered a implementation detail, so for now update the checkpatch documentation to recommend use of include/linux/bits.h.
[1] commit 8bd9cb51daac ("locking/atomics, asm-generic: Move some macros from <linux/bitops.h> to a new <linux/bits.h> file") [2] commit 3945ff37d2f4 ("linux/bits.h: Extract common header for vDSO")
Cc: Jiri Slaby <jirislaby@kernel.org> Signed-off-by: Andrew Jeffery <andrew@aj.id.au> --- Documentation/dev-tools/checkpatch.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/dev-tools/checkpatch.rst b/Documentation/dev-tools/checkpatch.rst index 51fed1bd72ec..59fcc9f627ea 100644 --- a/Documentation/dev-tools/checkpatch.rst +++ b/Documentation/dev-tools/checkpatch.rst @@ -472,7 +472,7 @@ Macros, Attributes and Symbols **BIT_MACRO** Defines like: 1 << <digit> could be BIT(digit). - The BIT() macro is defined in include/linux/bitops.h:: + The BIT() macro is defined via include/linux/bits.h:: #define BIT(nr) (1UL << (nr)) -- 2.30.2
| |