Some users may only need the string choice APIs. Split the respective header, i.e. string_choices.h. Include it in the string_helpers.h. Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> --- MAINTAINERS | 1 + include/linux/string_choices.h | 37 ++++++++++++++++++++++++++++++++++ include/linux/string_helpers.h | 31 +--------------------------- 3 files changed, 39 insertions(+), 30 deletions(-) create mode 100644 include/linux/string_choices.h diff --git a/MAINTAINERS b/MAINTAINERS index c1f0310dae18..b16c2488f05a 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -8722,6 +8722,7 @@ GENERIC STRING LIBRARY R: Andy Shevchenko <andy@xxxxxxxxxx> S: Maintained F: include/linux/string.h +F: include/linux/string_choices.h F: include/linux/string_helpers.h F: lib/string.c F: lib/string_helpers.c diff --git a/include/linux/string_choices.h b/include/linux/string_choices.h new file mode 100644 index 000000000000..ff7e202808a9 --- /dev/null +++ b/include/linux/string_choices.h @@ -0,0 +1,37 @@ +/* SPDX-License-Identifier: GPL-2.0 */ +#ifndef _LINUX_STRING_CHOICES_H_ +#define _LINUX_STRING_CHOICES_H_ + +#include <linux/types.h> + +static inline const char *str_enable_disable(bool v) +{ + return v ? "enable" : "disable"; +} + +static inline const char *str_enabled_disabled(bool v) +{ + return v ? "enabled" : "disabled"; +} + +static inline const char *str_high_low(bool v) +{ + return v ? "high" : "low"; +} + +static inline const char *str_read_write(bool v) +{ + return v ? "read" : "write"; +} + +static inline const char *str_on_off(bool v) +{ + return v ? "on" : "off"; +} + +static inline const char *str_yes_no(bool v) +{ + return v ? "yes" : "no"; +} + +#endif diff --git a/include/linux/string_helpers.h b/include/linux/string_helpers.h index fd72393e7975..88fb8e1d0421 100644 --- a/include/linux/string_helpers.h +++ b/include/linux/string_helpers.h @@ -5,6 +5,7 @@ #include <linux/bits.h> #include <linux/ctype.h> #include <linux/string.h> +#include <linux/string_choices.h> #include <linux/types.h> struct device; @@ -108,34 +109,4 @@ void kfree_strarray(char **array, size_t n); char **devm_kasprintf_strarray(struct device *dev, const char *prefix, size_t n); -static inline const char *str_yes_no(bool v) -{ - return v ? "yes" : "no"; -} - -static inline const char *str_on_off(bool v) -{ - return v ? "on" : "off"; -} - -static inline const char *str_enable_disable(bool v) -{ - return v ? "enable" : "disable"; -} - -static inline const char *str_enabled_disabled(bool v) -{ - return v ? "enabled" : "disabled"; -} - -static inline const char *str_high_low(bool v) -{ - return v ? "high" : "low"; -} - -static inline const char *str_read_write(bool v) -{ - return v ? "read" : "write"; -} - #endif -- 2.35.1