Przeglądaj źródła

drm/amd/powerplay: add prefix for all powerplay pr_* prints

Powerplay will use them instead of raw printk, and we can dynamic
change the debug level with it.

The prefix is like below:

[  xxx.xxxxxx] amdgpu: [powerplay] ...

Suggested-by: Grazvydas Ignotas <notasas@gmail.com>
Signed-off-by: Huang Rui <ray.huang@amd.com>
Cc: Arindam Nath <Arindam.Nath@amd.com>
Reviewed-by: Edward O'Callaghan <funfunctor@folklore1984.net>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Huang Rui 8 lat temu
rodzic
commit
7739b1e7e1
1 zmienionych plików z 8 dodań i 2 usunięć
  1. 8 2
      drivers/gpu/drm/amd/powerplay/inc/pp_debug.h

+ 8 - 2
drivers/gpu/drm/amd/powerplay/inc/pp_debug.h

@@ -24,6 +24,12 @@
 #ifndef PP_DEBUG_H
 #ifndef PP_DEBUG_H
 #define PP_DEBUG_H
 #define PP_DEBUG_H
 
 
+#ifdef pr_fmt
+#undef pr_fmt
+#endif
+
+#define pr_fmt(fmt) "amdgpu: [powerplay] " fmt
+
 #include <linux/types.h>
 #include <linux/types.h>
 #include <linux/kernel.h>
 #include <linux/kernel.h>
 #include <linux/slab.h>
 #include <linux/slab.h>
@@ -31,7 +37,7 @@
 #define PP_ASSERT_WITH_CODE(cond, msg, code)	\
 #define PP_ASSERT_WITH_CODE(cond, msg, code)	\
 	do {					\
 	do {					\
 		if (!(cond)) {			\
 		if (!(cond)) {			\
-			printk("%s\n", msg);	\
+			pr_warning("%s\n", msg);	\
 			code;			\
 			code;			\
 		}				\
 		}				\
 	} while (0)
 	} while (0)
@@ -39,7 +45,7 @@
 
 
 #define PP_DBG_LOG(fmt, ...) \
 #define PP_DBG_LOG(fmt, ...) \
 	do { \
 	do { \
-		if(0)printk(KERN_INFO "[ pp_dbg ] " fmt, ##__VA_ARGS__); \
+		pr_debug(fmt, ##__VA_ARGS__); \
 	} while (0)
 	} while (0)