2021-07-14 09:45:24

by Geert Uytterhoeven

[permalink] [raw]
Subject: [PATCH trivial/resend] asm-generic: ffs: Drop bogus reference to ffz location

The generic definition of ffz() is not defined in the same header files
as the generic definitions of ffs().

Signed-off-by: Geert Uytterhoeven <[email protected]>
---
include/asm-generic/bitops/builtin-ffs.h | 2 +-
include/asm-generic/bitops/ffs.h | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/asm-generic/bitops/builtin-ffs.h b/include/asm-generic/bitops/builtin-ffs.h
index 1dacfdb4247ee1b7..7b129329046b7337 100644
--- a/include/asm-generic/bitops/builtin-ffs.h
+++ b/include/asm-generic/bitops/builtin-ffs.h
@@ -8,7 +8,7 @@
*
* This is defined the same way as
* the libc and compiler builtin ffs routines, therefore
- * differs in spirit from the above ffz (man ffs).
+ * differs in spirit from ffz (man ffs).
*/
#define ffs(x) __builtin_ffs(x)

diff --git a/include/asm-generic/bitops/ffs.h b/include/asm-generic/bitops/ffs.h
index e81868b2c0f07b87..323fd5d6ae263ab3 100644
--- a/include/asm-generic/bitops/ffs.h
+++ b/include/asm-generic/bitops/ffs.h
@@ -8,7 +8,7 @@
*
* This is defined the same way as
* the libc and compiler builtin ffs routines, therefore
- * differs in spirit from the above ffz (man ffs).
+ * differs in spirit from ffz (man ffs).
*/
static inline int ffs(int x)
{
--
2.25.1


2021-08-12 22:37:15

by Arnd Bergmann

[permalink] [raw]
Subject: Re: [PATCH trivial/resend] asm-generic: ffs: Drop bogus reference to ffz location

From: Arnd Bergmann <[email protected]>

On Wed, 14 Jul 2021 11:44:22 +0200, Geert Uytterhoeven wrote:
> The generic definition of ffz() is not defined in the same header files
> as the generic definitions of ffs().

Applied to asm-generic-uaccess-3, thanks!

[1/1] asm-generic: ffs: Drop bogus reference to ffz location
commit: fc062ad8e406a08b624b3ab3427434800ff886f8

Arnd