Hi all, After merging the mm-nonmm-stable tree, today's linux-next build (powerpc ppc64_defconfig) failed like this: In file included from include/linux/string_helpers.h:7, from include/linux/seq_file.h:7, from include/linux/cgroup.h:18, from include/linux/memcontrol.h:13, from include/linux/swap.h:9, from include/linux/suspend.h:5, from arch/powerpc/kernel/asm-offsets.c:21: include/linux/string_choices.h:74:27: error: redefinition of 'str_true_false' 74 | static inline const char *str_true_false(bool v) | ^~~~~~~~~~~~~~ include/linux/string_choices.h:68:27: note: previous definition of 'str_true_false' with type 'const char *(bool)' {aka 'const char *(_Bool)'} 68 | static inline const char *str_true_false(bool v) | ^~~~~~~~~~~~~~ Caused by commit 32cebfe1cc21 ("lib/string_choices: add str_true_false()/str_false_true() helper") merging badly with commit 6ff4cd1160af ("lib/string_choices: Add str_true_false()/str_false_true() helper") from Linus' tree. I applied the following merge fix patch for today: From: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> Date: Thu, 19 Sep 2024 09:07:58 +1000 Subject: [PATCH] fix up for bad merge of "lib/string_choices: add str_true_false()/str_false_true() helper" which also exists in Linus' tree. Signed-off-by: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> --- include/linux/string_choices.h | 6 ------ 1 file changed, 6 deletions(-) diff --git a/include/linux/string_choices.h b/include/linux/string_choices.h index ef2bb6826919..120ca0f28e95 100644 --- a/include/linux/string_choices.h +++ b/include/linux/string_choices.h @@ -71,12 +71,6 @@ static inline const char *str_true_false(bool v) } #define str_false_true(v) str_true_false(!(v)) -static inline const char *str_true_false(bool v) -{ - return v ? "true" : "false"; -} -#define str_false_true(v) str_true_false(!(v)) - /** * str_plural - Return the simple pluralization based on English counts * @num: Number used for deciding pluralization -- 2.45.2 -- Cheers, Stephen Rothwell
Attachment:
pgpNxEmPY9nTU.pgp
Description: OpenPGP digital signature