diff options
| author | Roland Dreier <rolandd@cisco.com> | 2008-11-04 11:18:56 -0800 |
|---|---|---|
| committer | Roland Dreier <rolandd@cisco.com> | 2008-11-04 11:18:56 -0800 |
| commit | 7b0f5df4c88bac46fe749d36d905fc7ad0296587 (patch) | |
| tree | f0e9d80ad6994c2e6b139ac9eccf9a6e2a249084 | |
| parent | 332edc2f7fa58b818dfed1cede60272eecc27c0a (diff) | |
| download | op-kernel-dev-7b0f5df4c88bac46fe749d36d905fc7ad0296587.zip op-kernel-dev-7b0f5df4c88bac46fe749d36d905fc7ad0296587.tar.gz | |
mlx4_core: Fix unused variable warning
Fix
drivers/net/mlx4/profile.c:55: warning: 'res_name' defined but not used
by making mlx4_dbg() always use all of its parameters, regardless of
whether CONFIG_MLX4_DEBUG is set or not.
Reported-by: Alexander Beregalov <a.beregalov@gmail.com>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
| -rw-r--r-- | drivers/net/mlx4/mlx4.h | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/drivers/net/mlx4/mlx4.h b/drivers/net/mlx4/mlx4.h index fa431fa..56a2e21 100644 --- a/drivers/net/mlx4/mlx4.h +++ b/drivers/net/mlx4/mlx4.h @@ -87,6 +87,9 @@ enum { #ifdef CONFIG_MLX4_DEBUG extern int mlx4_debug_level; +#else /* CONFIG_MLX4_DEBUG */ +#define mlx4_debug_level (0) +#endif /* CONFIG_MLX4_DEBUG */ #define mlx4_dbg(mdev, format, arg...) \ do { \ @@ -94,12 +97,6 @@ extern int mlx4_debug_level; dev_printk(KERN_DEBUG, &mdev->pdev->dev, format, ## arg); \ } while (0) -#else /* CONFIG_MLX4_DEBUG */ - -#define mlx4_dbg(mdev, format, arg...) do { (void) mdev; } while (0) - -#endif /* CONFIG_MLX4_DEBUG */ - #define mlx4_err(mdev, format, arg...) \ dev_err(&mdev->pdev->dev, format, ## arg) #define mlx4_info(mdev, format, arg...) \ |
