diff options
| author | Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2016-10-12 13:21:43 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2016-10-17 16:13:36 +0200 |
| commit | 1aeb5b615cd10db7324d4e4d167c4916dfeca311 (patch) | |
| tree | 522542eb5e0633ab165f27797caa9f5ca8feef09 /drivers | |
| parent | [media] pctv452e: fix semicolon.cocci warnings (diff) | |
| download | linux-1aeb5b615cd10db7324d4e4d167c4916dfeca311.tar.xz linux-1aeb5b615cd10db7324d4e4d167c4916dfeca311.zip | |
[media] radio-bcm2048: don't ignore errors
Remove this warning:
drivers/staging/media/bcm2048/radio-bcm2048.c: In function 'bcm2048_set_rds_no_lock':
drivers/staging/media/bcm2048/radio-bcm2048.c:467:6: warning: variable 'err' set but not used [-Wunused-but-set-variable]
int err;
^~~
By returning the error code.
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/staging/media/bcm2048/radio-bcm2048.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers/staging/media/bcm2048/radio-bcm2048.c index 8dade197f053..0f088405e186 100644 --- a/drivers/staging/media/bcm2048/radio-bcm2048.c +++ b/drivers/staging/media/bcm2048/radio-bcm2048.c @@ -482,6 +482,8 @@ static int bcm2048_set_rds_no_lock(struct bcm2048_device *bdev, u8 rds_on) flags); memset(&bdev->rds_info, 0, sizeof(bdev->rds_info)); } + if (err) + return err; err = bcm2048_send_command(bdev, BCM2048_I2C_FM_RDS_SYSTEM, bdev->cache_fm_rds_system); |