dynamic_debug: change __dynamic_<foo>_dbg return types to void
authorJoe Perches <joe@perches.com>
Wed, 24 Sep 2014 18:17:56 +0000 (11:17 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 3 Oct 2014 21:55:48 +0000 (14:55 -0700)
The return value is not used by callers of these functions
so change the functions to return void.

Signed-off-by: Joe Perches <joe@perches.com>
Acked-by: Jason Baron <jbaron@akamai.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
include/linux/dynamic_debug.h
lib/dynamic_debug.c

index 2fe93b26b42fa71aa741689cf174cfbc86d1d781..4f1bbc68cd1bcc2312064bfcd2621b8c23edc012 100644 (file)
@@ -42,7 +42,7 @@ int ddebug_add_module(struct _ddebug *tab, unsigned int n,
 #if defined(CONFIG_DYNAMIC_DEBUG)
 extern int ddebug_remove_module(const char *mod_name);
 extern __printf(2, 3)
-int __dynamic_pr_debug(struct _ddebug *descriptor, const char *fmt, ...);
+void __dynamic_pr_debug(struct _ddebug *descriptor, const char *fmt, ...);
 
 extern int ddebug_dyndbg_module_param_cb(char *param, char *val,
                                        const char *modname);
@@ -50,15 +50,15 @@ extern int ddebug_dyndbg_module_param_cb(char *param, char *val,
 struct device;
 
 extern __printf(3, 4)
-int __dynamic_dev_dbg(struct _ddebug *descriptor, const struct device *dev,
-                     const char *fmt, ...);
+void __dynamic_dev_dbg(struct _ddebug *descriptor, const struct device *dev,
+                      const char *fmt, ...);
 
 struct net_device;
 
 extern __printf(3, 4)
-int __dynamic_netdev_dbg(struct _ddebug *descriptor,
-                        const struct net_device *dev,
-                        const char *fmt, ...);
+void __dynamic_netdev_dbg(struct _ddebug *descriptor,
+                         const struct net_device *dev,
+                         const char *fmt, ...);
 
 #define DEFINE_DYNAMIC_DEBUG_METADATA(name, fmt)               \
        static struct _ddebug  __aligned(8)                     \
index c9afbe2c445af155e2d2912ca362efae4ce02950..31fe79e31ab8d4940aa9d4e930c674e52ecee467 100644 (file)
@@ -537,10 +537,9 @@ static char *dynamic_emit_prefix(const struct _ddebug *desc, char *buf)
        return buf;
 }
 
-int __dynamic_pr_debug(struct _ddebug *descriptor, const char *fmt, ...)
+void __dynamic_pr_debug(struct _ddebug *descriptor, const char *fmt, ...)
 {
        va_list args;
-       int res;
        struct va_format vaf;
        char buf[PREFIX_SIZE];
 
@@ -552,21 +551,17 @@ int __dynamic_pr_debug(struct _ddebug *descriptor, const char *fmt, ...)
        vaf.fmt = fmt;
        vaf.va = &args;
 
-       res = printk(KERN_DEBUG "%s%pV",
-                    dynamic_emit_prefix(descriptor, buf), &vaf);
+       printk(KERN_DEBUG "%s%pV", dynamic_emit_prefix(descriptor, buf), &vaf);
 
        va_end(args);
-
-       return res;
 }
 EXPORT_SYMBOL(__dynamic_pr_debug);
 
-int __dynamic_dev_dbg(struct _ddebug *descriptor,
+void __dynamic_dev_dbg(struct _ddebug *descriptor,
                      const struct device *dev, const char *fmt, ...)
 {
        struct va_format vaf;
        va_list args;
-       int res;
 
        BUG_ON(!descriptor);
        BUG_ON(!fmt);
@@ -577,30 +572,27 @@ int __dynamic_dev_dbg(struct _ddebug *descriptor,
        vaf.va = &args;
 
        if (!dev) {
-               res = printk(KERN_DEBUG "(NULL device *): %pV", &vaf);
+               printk(KERN_DEBUG "(NULL device *): %pV", &vaf);
        } else {
                char buf[PREFIX_SIZE];
 
-               res = dev_printk_emit(7, dev, "%s%s %s: %pV",
-                                     dynamic_emit_prefix(descriptor, buf),
-                                     dev_driver_string(dev), dev_name(dev),
-                                     &vaf);
+               dev_printk_emit(7, dev, "%s%s %s: %pV",
+                               dynamic_emit_prefix(descriptor, buf),
+                               dev_driver_string(dev), dev_name(dev),
+                               &vaf);
        }
 
        va_end(args);
-
-       return res;
 }
 EXPORT_SYMBOL(__dynamic_dev_dbg);
 
 #ifdef CONFIG_NET
 
-int __dynamic_netdev_dbg(struct _ddebug *descriptor,
-                        const struct net_device *dev, const char *fmt, ...)
+void __dynamic_netdev_dbg(struct _ddebug *descriptor,
+                         const struct net_device *dev, const char *fmt, ...)
 {
        struct va_format vaf;
        va_list args;
-       int res;
 
        BUG_ON(!descriptor);
        BUG_ON(!fmt);
@@ -613,23 +605,21 @@ int __dynamic_netdev_dbg(struct _ddebug *descriptor,
        if (dev && dev->dev.parent) {
                char buf[PREFIX_SIZE];
 
-               res = dev_printk_emit(7, dev->dev.parent,
-                                     "%s%s %s %s%s: %pV",
-                                     dynamic_emit_prefix(descriptor, buf),
-                                     dev_driver_string(dev->dev.parent),
-                                     dev_name(dev->dev.parent),
-                                     netdev_name(dev), netdev_reg_state(dev),
-                                     &vaf);
+               dev_printk_emit(7, dev->dev.parent,
+                               "%s%s %s %s%s: %pV",
+                               dynamic_emit_prefix(descriptor, buf),
+                               dev_driver_string(dev->dev.parent),
+                               dev_name(dev->dev.parent),
+                               netdev_name(dev), netdev_reg_state(dev),
+                               &vaf);
        } else if (dev) {
-               res = printk(KERN_DEBUG "%s%s: %pV", netdev_name(dev),
-                            netdev_reg_state(dev), &vaf);
+               printk(KERN_DEBUG "%s%s: %pV", netdev_name(dev),
+                      netdev_reg_state(dev), &vaf);
        } else {
-               res = printk(KERN_DEBUG "(NULL net_device): %pV", &vaf);
+               printk(KERN_DEBUG "(NULL net_device): %pV", &vaf);
        }
 
        va_end(args);
-
-       return res;
 }
 EXPORT_SYMBOL(__dynamic_netdev_dbg);