<br><br>On Wednesday, January 19, 2022, Lucas De Marchi <<a href="mailto:lucas.demarchi@intel.com">lucas.demarchi@intel.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Follow the yes/no logic and add helpers for enabled/disabled and<br>
enable/disable - those are not so common throughout the kernel,<br>
but they give a nice way to reuse the strings to log things as<br>
enabled/disabled or enable/disable.<br>
<br>
Signed-off-by: Lucas De Marchi <<a href="mailto:lucas.demarchi@intel.com">lucas.demarchi@intel.com</a>><br>
---<br>
 drivers/gpu/drm/i915/i915_<wbr>utils.h | 10 ----------<br>
 include/linux/string_helpers.h    |  2 ++<br>
 2 files changed, 2 insertions(+), 10 deletions(-)<br>
<br>
diff --git a/drivers/gpu/drm/i915/i915_<wbr>utils.h b/drivers/gpu/drm/i915/i915_<wbr>utils.h<br>
index 2a8781cc648b..cbec79bae0d2 100644<br>
--- a/drivers/gpu/drm/i915/i915_<wbr>utils.h<br>
+++ b/drivers/gpu/drm/i915/i915_<wbr>utils.h<br>
@@ -419,16 +419,6 @@ static inline const char *onoff(bool v)<br>
        return v ? "on" : "off";<br>
 }<br>
<br>
-static inline const char *enabledisable(bool v)<br>
-{<br>
-       return v ? "enable" : "disable";<br>
-}<br>
-<br>
-static inline const char *enableddisabled(bool v)<br>
-{<br>
-       return v ? "enabled" : "disabled";<br>
-}<br>
-<br>
 void add_taint_for_CI(struct drm_i915_private *i915, unsigned int taint);<br>
 static inline void __add_taint_for_CI(unsigned int taint)<br>
 {<br>
diff --git a/include/linux/string_<wbr>helpers.h b/include/linux/string_<wbr>helpers.h<br>
index e980dec05d31..e4b82f364ee1 100644<br>
--- a/include/linux/string_<wbr>helpers.h<br>
+++ b/include/linux/string_<wbr>helpers.h<br>
@@ -103,5 +103,7 @@ char *kstrdup_quotable_file(struct file *file, gfp_t gfp);<br>
 void kfree_strarray(char **array, size_t n);<br>
<br>
 static inline const char *yesno(bool v) { return v ? "yes" : "no"; }<br>
+static inline const char *enabledisable(bool v) { return v ? "enable" : "disable"; }<br>
+static inline const char *enableddisabled(bool v) { return v ? "enabled" : "disabled"; }</blockquote><div><br></div><div>Looks not readable even if takes 80 characters. Please, keep original style.</div><div><br></div><div><br></div><div>I believe you wanted to have nice negative statistics from day 1, then you may add more patches in the series to cleanup more users.</div><div><br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
 #endif<br>
-- <br>
2.34.1<br>
<br>
</blockquote><br><br>-- <br>With Best Regards,<br>Andy Shevchenko<br><br><br>