Hi Mauro,
Mauro Carvalho Chehab wrote:
> Subject: [media] dvb-bt8xx: handle errors from dvb_net_init
[...]
> [mchehab.redhat.com: codingstyle fix: printk() should include KERN_ facility level]
[...]
> --- a/drivers/media/dvb/bt8xx/dvb-bt8xx.c
> +++ b/drivers/media/dvb/bt8xx/dvb-bt8xx.c
> @@ -782,7 +782,12 @@ static int __devinit dvb_bt8xx_load_card(struct dvb_bt8xx_card *card, u32 type)
> goto err_remove_mem_frontend;
> }
>
> - dvb_net_init(&card->dvb_adapter, &card->dvbnet, &card->demux.dmx);
> + result = dvb_net_init(&card->dvb_adapter, &card->dvbnet, &card->demux.dmx);
> + if (result < 0) {
> + printk(KERN_ERR,
> + "dvb_bt8xx: dvb_net_init failed (errno = %d)\n", result);
I think there is an extra comma here:
$ make drivers/media/dvb/bt8xx/dvb-bt8xx.o
CHK include/linux/version.h
CHK include/generated/utsrelease.h
CALL scripts/checksyscalls.sh
CC [M] drivers/media/dvb/bt8xx/dvb-bt8xx.o
drivers/media/dvb/bt8xx/dvb-bt8xx.c: In function ‘dvb_bt8xx_load_card’:
drivers/media/dvb/bt8xx/dvb-bt8xx.c:788:10: warning: too many arguments for format [-Wformat-extra-args]
Perhaps it would be better to add the KERN_ levels throughout the file
with a separate patch. Like this:
Jonathan Nieder (2):
[media] dvb-bt8xx: use dprintk for debug statements
[media] dvb-bt8xx: convert printks to pr_err()
drivers/media/dvb/bt8xx/dvb-bt8xx.c | 41 +++++++++++++++++------------------
1 files changed, 20 insertions(+), 21 deletions(-)
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
