iwlwifi: dvm: use macros for format strings
authorJohannes Berg <johannes.berg@intel.com>
Wed, 17 May 2017 11:45:29 +0000 (13:45 +0200)
committerLuca Coelho <luciano.coelho@intel.com>
Thu, 29 Jun 2017 10:26:24 +0000 (13:26 +0300)
Some static checkers (e.g. smatch) complain if a non-constant
format string is used, even if that's a static const variable.
Since there's no impact on code generation, just change those
format strings to be macros.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
drivers/net/wireless/intel/iwlwifi/dvm/debugfs.c

index 376c79337a0e04f86796494565749b7b333ab1ff..937be04c6c71e93ae35515ec31cfe790529c467e 100644 (file)
@@ -681,11 +681,10 @@ DEBUGFS_READ_FILE_OPS(temperature);
 DEBUGFS_READ_WRITE_FILE_OPS(sleep_level_override);
 DEBUGFS_READ_FILE_OPS(current_sleep_command);
 
-static const char *fmt_value = "  %-30s %10u\n";
-static const char *fmt_hex   = "  %-30s       0x%02X\n";
-static const char *fmt_table = "  %-30s %10u  %10u  %10u  %10u\n";
-static const char *fmt_header =
-       "%-32s    current  cumulative       delta         max\n";
+#define fmt_value      "  %-30s %10u\n"
+#define fmt_hex                "  %-30s       0x%02X\n"
+#define fmt_table      "  %-30s %10u  %10u  %10u  %10u\n"
+#define fmt_header     "%-32s    current  cumulative       delta         max\n"
 
 static int iwl_statistics_flag(struct iwl_priv *priv, char *buf, int bufsz)
 {