[v4] fix CONFIG_DRM_USE_DYNAMIC_DEBUG=y

[PATCH v4 07/58] dyndbg: tweak pr_fmt to avoid expansion conflicts

Posted by Jim Cromie 2 months ago

Disambiguate pr_fmt(fmt) arg, by changing it to _FMT_, to avoid naming
confusion with many later macros also using that argname.

no functional change

Signed-off-by: Jim Cromie <jim.cromie@gmail.com>
---
 lib/dynamic_debug.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c
index 55df35df093b0..2751056a5240d 100644
--- a/lib/dynamic_debug.c
+++ b/lib/dynamic_debug.c
@@ -11,7 +11,7 @@
  * Copyright (C) 2013 Du, Changbin <changbin.du@gmail.com>
  */
 
-#define pr_fmt(fmt) "dyndbg: " fmt
+#define pr_fmt(_FMT_) "dyndbg: " _FMT_
 
 #include <linux/kernel.h>
 #include <linux/module.h>
-- 
2.50.1

Re: [PATCH v4 07/58] dyndbg: tweak pr_fmt to avoid expansion conflicts

Posted by Louis Chauvet 4 weeks, 1 day ago


Le 03/08/2025 à 05:57, Jim Cromie a écrit :
> Disambiguate pr_fmt(fmt) arg, by changing it to _FMT_, to avoid naming
> confusion with many later macros also using that argname.
> 
> no functional change
> 
> Signed-off-by: Jim Cromie <jim.cromie@gmail.com>

Reviewed-by: Louis Chauvet <louis.chauvet@bootlin.com>

> ---
>   lib/dynamic_debug.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c
> index 55df35df093b0..2751056a5240d 100644
> --- a/lib/dynamic_debug.c
> +++ b/lib/dynamic_debug.c
> @@ -11,7 +11,7 @@
>    * Copyright (C) 2013 Du, Changbin <changbin.du@gmail.com>
>    */
>   
> -#define pr_fmt(fmt) "dyndbg: " fmt
> +#define pr_fmt(_FMT_) "dyndbg: " _FMT_
>   
>   #include <linux/kernel.h>
>   #include <linux/module.h>

-- 
Louis Chauvet, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com