net: Change netdev_<level> logging functions to return void
authorJoe Perches <joe@perches.com>
Mon, 22 Sep 2014 18:10:50 +0000 (11:10 -0700)
committerDavid S. Miller <davem@davemloft.net>
Fri, 26 Sep 2014 19:17:17 +0000 (15:17 -0400)
No caller or macro uses the return value so make all
the functions return void.

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/linux/netdevice.h
net/core/dev.c

index 9f5d293a028118007d1fbe9fc1dff40dc4b94f1f..9b7fbacb6296f28c225807e33b851bcaeec2a59d 100644 (file)
@@ -3612,22 +3612,22 @@ static inline const char *netdev_reg_state(const struct net_device *dev)
 }
 
 __printf(3, 4)
-int netdev_printk(const char *level, const struct net_device *dev,
-                 const char *format, ...);
+void netdev_printk(const char *level, const struct net_device *dev,
+                  const char *format, ...);
 __printf(2, 3)
-int netdev_emerg(const struct net_device *dev, const char *format, ...);
+void netdev_emerg(const struct net_device *dev, const char *format, ...);
 __printf(2, 3)
-int netdev_alert(const struct net_device *dev, const char *format, ...);
+void netdev_alert(const struct net_device *dev, const char *format, ...);
 __printf(2, 3)
-int netdev_crit(const struct net_device *dev, const char *format, ...);
+void netdev_crit(const struct net_device *dev, const char *format, ...);
 __printf(2, 3)
-int netdev_err(const struct net_device *dev, const char *format, ...);
+void netdev_err(const struct net_device *dev, const char *format, ...);
 __printf(2, 3)
-int netdev_warn(const struct net_device *dev, const char *format, ...);
+void netdev_warn(const struct net_device *dev, const char *format, ...);
 __printf(2, 3)
-int netdev_notice(const struct net_device *dev, const char *format, ...);
+void netdev_notice(const struct net_device *dev, const char *format, ...);
 __printf(2, 3)
-int netdev_info(const struct net_device *dev, const char *format, ...);
+void netdev_info(const struct net_device *dev, const char *format, ...);
 
 #define MODULE_ALIAS_NETDEV(device) \
        MODULE_ALIAS("netdev-" device)
@@ -3645,7 +3645,6 @@ do {                                                              \
 ({                                                             \
        if (0)                                                  \
                netdev_printk(KERN_DEBUG, __dev, format, ##args); \
-       0;                                                      \
 })
 #endif
 
index e2ced01c01e4d58f8c53c7e67f98ad6648a8ea59..e55c546717d4d6530fa50f83e463f39fdc717ecf 100644 (file)
@@ -7066,53 +7066,45 @@ const char *netdev_drivername(const struct net_device *dev)
        return empty;
 }
 
-static int __netdev_printk(const char *level, const struct net_device *dev,
-                          struct va_format *vaf)
+static void __netdev_printk(const char *level, const struct net_device *dev,
+                           struct va_format *vaf)
 {
-       int r;
-
        if (dev && dev->dev.parent) {
-               r = dev_printk_emit(level[1] - '0',
-                                   dev->dev.parent,
-                                   "%s %s %s%s: %pV",
-                                   dev_driver_string(dev->dev.parent),
-                                   dev_name(dev->dev.parent),
-                                   netdev_name(dev), netdev_reg_state(dev),
-                                   vaf);
+               dev_printk_emit(level[1] - '0',
+                               dev->dev.parent,
+                               "%s %s %s%s: %pV",
+                               dev_driver_string(dev->dev.parent),
+                               dev_name(dev->dev.parent),
+                               netdev_name(dev), netdev_reg_state(dev),
+                               vaf);
        } else if (dev) {
-               r = printk("%s%s%s: %pV", level, netdev_name(dev),
-                          netdev_reg_state(dev), vaf);
+               printk("%s%s%s: %pV",
+                      level, netdev_name(dev), netdev_reg_state(dev), vaf);
        } else {
-               r = printk("%s(NULL net_device): %pV", level, vaf);
+               printk("%s(NULL net_device): %pV", level, vaf);
        }
-
-       return r;
 }
 
-int netdev_printk(const char *level, const struct net_device *dev,
-                 const char *format, ...)
+void netdev_printk(const char *level, const struct net_device *dev,
+                  const char *format, ...)
 {
        struct va_format vaf;
        va_list args;
-       int r;
 
        va_start(args, format);
 
        vaf.fmt = format;
        vaf.va = &args;
 
-       r = __netdev_printk(level, dev, &vaf);
+       __netdev_printk(level, dev, &vaf);
 
        va_end(args);
-
-       return r;
 }
 EXPORT_SYMBOL(netdev_printk);
 
 #define define_netdev_printk_level(func, level)                        \
-int func(const struct net_device *dev, const char *fmt, ...)   \
+void func(const struct net_device *dev, const char *fmt, ...)  \
 {                                                              \
-       int r;                                                  \
        struct va_format vaf;                                   \
        va_list args;                                           \
                                                                \
@@ -7121,11 +7113,9 @@ int func(const struct net_device *dev, const char *fmt, ...)     \
        vaf.fmt = fmt;                                          \
        vaf.va = &args;                                         \
                                                                \
-       r = __netdev_printk(level, dev, &vaf);                  \
+       __netdev_printk(level, dev, &vaf);                      \
                                                                \
        va_end(args);                                           \
-                                                               \
-       return r;                                               \
 }                                                              \
 EXPORT_SYMBOL(func);