diff options
author | Arnd Bergmann <arnd@arndb.de> | 2017-02-01 19:22:25 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-02-02 15:01:32 +0300 |
commit | b07525b89f95060dc2d8d954a7a77df4918d8bb8 (patch) | |
tree | 2c5d104bae12b3ca757eed15f20352f33f350a6a /drivers/staging/bcm2835-audio | |
parent | fe822dc6c12b1e3be3e820d088d431b55d57693b (diff) | |
download | linux-b07525b89f95060dc2d8d954a7a77df4918d8bb8.tar.xz |
staging: bcm2835-audio: fix empty-body warning
gcc-7 warns about debug statements being left out here:
drivers/staging/bcm2835-audio/bcm2835-vchiq.c: In function 'bcm2835_audio_set_ctls':
drivers/staging/bcm2835-audio/bcm2835-vchiq.c:572:54: error: suggest braces around empty body in an 'else' statement [-Werror=empty-body]
It's better to use no_printk(), which has the added advantage of checking
the format strings.
Fixes: 23b028c871e1 ("staging: bcm2835-audio: initial staging submission")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/bcm2835-audio')
-rw-r--r-- | drivers/staging/bcm2835-audio/bcm2835-vchiq.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/staging/bcm2835-audio/bcm2835-vchiq.c b/drivers/staging/bcm2835-audio/bcm2835-vchiq.c index 7c952bae4e52..4bdc8d8eacb9 100644 --- a/drivers/staging/bcm2835-audio/bcm2835-vchiq.c +++ b/drivers/staging/bcm2835-audio/bcm2835-vchiq.c @@ -50,9 +50,9 @@ #define LOG_DBG( fmt, arg... ) pr_info( "%s:%d " fmt, __func__, __LINE__, ##arg) #else #define LOG_ERR( fmt, arg... ) pr_err( "%s:%d " fmt, __func__, __LINE__, ##arg) -#define LOG_WARN( fmt, arg... ) -#define LOG_INFO( fmt, arg... ) -#define LOG_DBG( fmt, arg... ) +#define LOG_WARN( fmt, arg... ) no_printk(fmt, ##arg) +#define LOG_INFO( fmt, arg... ) no_printk(fmt, ##arg) +#define LOG_DBG( fmt, arg... ) no_printk(fmt, ##arg) #endif struct bcm2835_audio_instance { |