Commit 5ccb197d authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab

[media] em28xx: don't break long lines

Due to the 80-cols checkpatch warnings, several strings
were broken into multiple lines. This is not considered
a good practice anymore, as it makes harder to grep for
strings at the source code. So, join those continuation
lines.
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
parent 2af3eb64
...@@ -3,7 +3,7 @@ ...@@ -3,7 +3,7 @@
* *
* Copyright (C) 2006 Markus Rechberger <mrechberger@gmail.com> * Copyright (C) 2006 Markus Rechberger <mrechberger@gmail.com>
* *
* Copyright (C) 2007-2014 Mauro Carvalho Chehab * Copyright (C) 2007-2016 Mauro Carvalho Chehab
* - Port to work with the in-kernel driver * - Port to work with the in-kernel driver
* - Cleanups, fixes, alsa-controls, etc. * - Cleanups, fixes, alsa-controls, etc.
* *
...@@ -254,8 +254,7 @@ static int snd_em28xx_capture_open(struct snd_pcm_substream *substream) ...@@ -254,8 +254,7 @@ static int snd_em28xx_capture_open(struct snd_pcm_substream *substream)
int nonblock, ret = 0; int nonblock, ret = 0;
if (!dev) { if (!dev) {
em28xx_err("BUG: em28xx can't find device struct." em28xx_err("BUG: em28xx can't find device struct. Can't proceed with open\n");
" Can't proceed with open\n");
return -ENODEV; return -ENODEV;
} }
...@@ -902,10 +901,9 @@ static int em28xx_audio_init(struct em28xx *dev) ...@@ -902,10 +901,9 @@ static int em28xx_audio_init(struct em28xx *dev)
kref_get(&dev->ref); kref_get(&dev->ref);
printk(KERN_INFO "em28xx-audio.c: Copyright (C) 2006 Markus " printk(KERN_INFO "em28xx-audio.c: Copyright (C) 2006 Markus Rechberger\n");
"Rechberger\n");
printk(KERN_INFO printk(KERN_INFO
"em28xx-audio.c: Copyright (C) 2007-2014 Mauro Carvalho Chehab\n"); "em28xx-audio.c: Copyright (C) 2007-2016 Mauro Carvalho Chehab\n");
err = snd_card_new(&dev->udev->dev, index[devnr], "Em28xx Audio", err = snd_card_new(&dev->udev->dev, index[devnr], "Em28xx Audio",
THIS_MODULE, 0, &card); THIS_MODULE, 0, &card);
......
...@@ -2832,13 +2832,10 @@ static int em28xx_hint_board(struct em28xx *dev) ...@@ -2832,13 +2832,10 @@ static int em28xx_hint_board(struct em28xx *dev)
dev->tuner_type = em28xx_eeprom_hash[i].tuner; dev->tuner_type = em28xx_eeprom_hash[i].tuner;
em28xx_errdev("Your board has no unique USB ID.\n"); em28xx_errdev("Your board has no unique USB ID.\n");
em28xx_errdev("A hint were successfully done, " em28xx_errdev("A hint were successfully done, based on eeprom hash.\n");
"based on eeprom hash.\n");
em28xx_errdev("This method is not 100%% failproof.\n"); em28xx_errdev("This method is not 100%% failproof.\n");
em28xx_errdev("If the board were missdetected, " em28xx_errdev("If the board were missdetected, please email this log to:\n");
"please email this log to:\n"); em28xx_errdev("\tV4L Mailing List <linux-media@vger.kernel.org>\n");
em28xx_errdev("\tV4L Mailing List "
" <linux-media@vger.kernel.org>\n");
em28xx_errdev("Board detected as %s\n", em28xx_errdev("Board detected as %s\n",
em28xx_boards[dev->model].name); em28xx_boards[dev->model].name);
...@@ -2864,13 +2861,10 @@ static int em28xx_hint_board(struct em28xx *dev) ...@@ -2864,13 +2861,10 @@ static int em28xx_hint_board(struct em28xx *dev)
dev->model = em28xx_i2c_hash[i].model; dev->model = em28xx_i2c_hash[i].model;
dev->tuner_type = em28xx_i2c_hash[i].tuner; dev->tuner_type = em28xx_i2c_hash[i].tuner;
em28xx_errdev("Your board has no unique USB ID.\n"); em28xx_errdev("Your board has no unique USB ID.\n");
em28xx_errdev("A hint were successfully done, " em28xx_errdev("A hint were successfully done, based on i2c devicelist hash.\n");
"based on i2c devicelist hash.\n");
em28xx_errdev("This method is not 100%% failproof.\n"); em28xx_errdev("This method is not 100%% failproof.\n");
em28xx_errdev("If the board were missdetected, " em28xx_errdev("If the board were missdetected, please email this log to:\n");
"please email this log to:\n"); em28xx_errdev("\tV4L Mailing List <linux-media@vger.kernel.org>\n");
em28xx_errdev("\tV4L Mailing List "
" <linux-media@vger.kernel.org>\n");
em28xx_errdev("Board detected as %s\n", em28xx_errdev("Board detected as %s\n",
em28xx_boards[dev->model].name); em28xx_boards[dev->model].name);
...@@ -2878,17 +2872,14 @@ static int em28xx_hint_board(struct em28xx *dev) ...@@ -2878,17 +2872,14 @@ static int em28xx_hint_board(struct em28xx *dev)
} }
} }
em28xx_errdev("Your board has no unique USB ID and thus need a " em28xx_errdev("Your board has no unique USB ID and thus need a hint to be detected.\n");
"hint to be detected.\n"); em28xx_errdev("You may try to use card=<n> insmod option to workaround that.\n");
em28xx_errdev("You may try to use card=<n> insmod option to "
"workaround that.\n");
em28xx_errdev("Please send an email with this log to:\n"); em28xx_errdev("Please send an email with this log to:\n");
em28xx_errdev("\tV4L Mailing List <linux-media@vger.kernel.org>\n"); em28xx_errdev("\tV4L Mailing List <linux-media@vger.kernel.org>\n");
em28xx_errdev("Board eeprom hash is 0x%08lx\n", dev->hash); em28xx_errdev("Board eeprom hash is 0x%08lx\n", dev->hash);
em28xx_errdev("Board i2c devicelist hash is 0x%08lx\n", dev->i2c_hash); em28xx_errdev("Board i2c devicelist hash is 0x%08lx\n", dev->i2c_hash);
em28xx_errdev("Here is a list of valid choices for the card=<n>" em28xx_errdev("Here is a list of valid choices for the card=<n> insmod option:\n");
" insmod option:\n");
for (i = 0; i < em28xx_bcount; i++) { for (i = 0; i < em28xx_bcount; i++) {
em28xx_errdev(" card=%d -> %s\n", em28xx_errdev(" card=%d -> %s\n",
i, em28xx_boards[i].name); i, em28xx_boards[i].name);
...@@ -3035,11 +3026,9 @@ static void em28xx_card_setup(struct em28xx *dev) ...@@ -3035,11 +3026,9 @@ static void em28xx_card_setup(struct em28xx *dev)
if (dev->board.valid == EM28XX_BOARD_NOT_VALIDATED) { if (dev->board.valid == EM28XX_BOARD_NOT_VALIDATED) {
em28xx_errdev("\n\n"); em28xx_errdev("\n\n");
em28xx_errdev("The support for this board weren't " em28xx_errdev("The support for this board weren't valid yet.\n");
"valid yet.\n");
em28xx_errdev("Please send a report of having this working\n"); em28xx_errdev("Please send a report of having this working\n");
em28xx_errdev("not to V4L mailing list (and/or to other " em28xx_errdev("not to V4L mailing list (and/or to other addresses)\n\n");
"addresses)\n\n");
} }
/* Free eeprom data memory */ /* Free eeprom data memory */
...@@ -3360,8 +3349,7 @@ static int em28xx_init_dev(struct em28xx *dev, struct usb_device *udev, ...@@ -3360,8 +3349,7 @@ static int em28xx_init_dev(struct em28xx *dev, struct usb_device *udev,
/* Resets I2C speed */ /* Resets I2C speed */
retval = em28xx_write_reg(dev, EM28XX_R06_I2C_CLK, dev->board.i2c_speed); retval = em28xx_write_reg(dev, EM28XX_R06_I2C_CLK, dev->board.i2c_speed);
if (retval < 0) { if (retval < 0) {
em28xx_errdev("%s: em28xx_write_reg failed!" em28xx_errdev("%s: em28xx_write_reg failed! retval [%d]\n",
" retval [%d]\n",
__func__, retval); __func__, retval);
return retval; return retval;
} }
...@@ -3429,7 +3417,8 @@ static int em28xx_usb_probe(struct usb_interface *interface, ...@@ -3429,7 +3417,8 @@ static int em28xx_usb_probe(struct usb_interface *interface,
nr = find_first_zero_bit(em28xx_devused, EM28XX_MAXBOARDS); nr = find_first_zero_bit(em28xx_devused, EM28XX_MAXBOARDS);
if (nr >= EM28XX_MAXBOARDS) { if (nr >= EM28XX_MAXBOARDS) {
/* No free device slots */ /* No free device slots */
printk(DRIVER_NAME ": Supports only %i em28xx boards.\n", printk(DRIVER_NAME
": Supports only %i em28xx boards.\n",
EM28XX_MAXBOARDS); EM28XX_MAXBOARDS);
retval = -ENOMEM; retval = -ENOMEM;
goto err_no_slot; goto err_no_slot;
...@@ -3438,8 +3427,8 @@ static int em28xx_usb_probe(struct usb_interface *interface, ...@@ -3438,8 +3427,8 @@ static int em28xx_usb_probe(struct usb_interface *interface,
/* Don't register audio interfaces */ /* Don't register audio interfaces */
if (interface->altsetting[0].desc.bInterfaceClass == USB_CLASS_AUDIO) { if (interface->altsetting[0].desc.bInterfaceClass == USB_CLASS_AUDIO) {
em28xx_err(DRIVER_NAME " audio device (%04x:%04x): " em28xx_err(DRIVER_NAME
"interface %i, class %i\n", " audio device (%04x:%04x): interface %i, class %i\n",
le16_to_cpu(udev->descriptor.idVendor), le16_to_cpu(udev->descriptor.idVendor),
le16_to_cpu(udev->descriptor.idProduct), le16_to_cpu(udev->descriptor.idProduct),
ifnum, ifnum,
...@@ -3576,8 +3565,7 @@ static int em28xx_usb_probe(struct usb_interface *interface, ...@@ -3576,8 +3565,7 @@ static int em28xx_usb_probe(struct usb_interface *interface,
} }
printk(KERN_INFO DRIVER_NAME printk(KERN_INFO DRIVER_NAME
": New device %s %s @ %s Mbps " ": New device %s %s @ %s Mbps (%04x:%04x, interface %d, class %d)\n",
"(%04x:%04x, interface %d, class %d)\n",
udev->manufacturer ? udev->manufacturer : "", udev->manufacturer ? udev->manufacturer : "",
udev->product ? udev->product : "", udev->product ? udev->product : "",
speed, speed,
...@@ -3593,8 +3581,7 @@ static int em28xx_usb_probe(struct usb_interface *interface, ...@@ -3593,8 +3581,7 @@ static int em28xx_usb_probe(struct usb_interface *interface,
*/ */
if (udev->speed != USB_SPEED_HIGH && disable_usb_speed_check == 0) { if (udev->speed != USB_SPEED_HIGH && disable_usb_speed_check == 0) {
printk(DRIVER_NAME ": Device initialization failed.\n"); printk(DRIVER_NAME ": Device initialization failed.\n");
printk(DRIVER_NAME ": Device must be connected to a high-speed" printk(DRIVER_NAME ": Device must be connected to a high-speed USB 2.0 port.\n");
" USB 2.0 port.\n");
retval = -ENODEV; retval = -ENODEV;
goto err_free; goto err_free;
} }
......
...@@ -87,8 +87,8 @@ int em28xx_read_reg_req_len(struct em28xx *dev, u8 req, u16 reg, ...@@ -87,8 +87,8 @@ int em28xx_read_reg_req_len(struct em28xx *dev, u8 req, u16 reg,
return -EINVAL; return -EINVAL;
if (reg_debug) { if (reg_debug) {
printk(KERN_DEBUG "(pipe 0x%08x): " printk(KERN_DEBUG
"IN: %02x %02x %02x %02x %02x %02x %02x %02x ", "(pipe 0x%08x): IN: %02x %02x %02x %02x %02x %02x %02x %02x ",
pipe, pipe,
USB_DIR_IN | USB_TYPE_VENDOR | USB_RECIP_DEVICE, USB_DIR_IN | USB_TYPE_VENDOR | USB_RECIP_DEVICE,
req, 0, 0, req, 0, 0,
...@@ -165,8 +165,8 @@ int em28xx_write_regs_req(struct em28xx *dev, u8 req, u16 reg, char *buf, ...@@ -165,8 +165,8 @@ int em28xx_write_regs_req(struct em28xx *dev, u8 req, u16 reg, char *buf,
if (reg_debug) { if (reg_debug) {
int byte; int byte;
printk(KERN_DEBUG "(pipe 0x%08x): " printk(KERN_DEBUG
"OUT: %02x %02x %02x %02x %02x %02x %02x %02x >>>", "(pipe 0x%08x): OUT: %02x %02x %02x %02x %02x %02x %02x %02x >>>",
pipe, pipe,
USB_DIR_OUT | USB_TYPE_VENDOR | USB_RECIP_DEVICE, USB_DIR_OUT | USB_TYPE_VENDOR | USB_RECIP_DEVICE,
req, 0, 0, req, 0, 0,
...@@ -942,8 +942,7 @@ int em28xx_alloc_urbs(struct em28xx *dev, enum em28xx_mode mode, int xfer_bulk, ...@@ -942,8 +942,7 @@ int em28xx_alloc_urbs(struct em28xx *dev, enum em28xx_mode mode, int xfer_bulk,
usb_bufs->transfer_buffer[i] = usb_alloc_coherent(dev->udev, usb_bufs->transfer_buffer[i] = usb_alloc_coherent(dev->udev,
sb_size, GFP_KERNEL, &urb->transfer_dma); sb_size, GFP_KERNEL, &urb->transfer_dma);
if (!usb_bufs->transfer_buffer[i]) { if (!usb_bufs->transfer_buffer[i]) {
em28xx_err("unable to allocate %i bytes for transfer" em28xx_err("unable to allocate %i bytes for transfer buffer %i%s\n",
" buffer %i%s\n",
sb_size, i, sb_size, i,
in_interrupt() ? " while in int" : ""); in_interrupt() ? " while in int" : "");
em28xx_uninit_usb_xfer(dev, mode); em28xx_uninit_usb_xfer(dev, mode);
......
...@@ -934,8 +934,7 @@ static int em28xx_attach_xc3028(u8 addr, struct em28xx *dev) ...@@ -934,8 +934,7 @@ static int em28xx_attach_xc3028(u8 addr, struct em28xx *dev)
cfg.ctrl = &ctl; cfg.ctrl = &ctl;
if (!dev->dvb->fe[0]) { if (!dev->dvb->fe[0]) {
em28xx_errdev("/2: dvb frontend not attached. " em28xx_errdev("/2: dvb frontend not attached. Can't attach xc3028\n");
"Can't attach xc3028\n");
return -EINVAL; return -EINVAL;
} }
...@@ -1937,8 +1936,7 @@ static int em28xx_dvb_init(struct em28xx *dev) ...@@ -1937,8 +1936,7 @@ static int em28xx_dvb_init(struct em28xx *dev)
} }
break; break;
default: default:
em28xx_errdev("/2: The frontend of your DVB/ATSC card" em28xx_errdev("/2: The frontend of your DVB/ATSC card isn't supported yet\n");
" isn't supported yet\n");
break; break;
} }
if (NULL == dvb->fe[0]) { if (NULL == dvb->fe[0]) {
......
...@@ -703,8 +703,7 @@ static int em28xx_ir_init(struct em28xx *dev) ...@@ -703,8 +703,7 @@ static int em28xx_ir_init(struct em28xx *dev)
if (dev->board.ir_codes == NULL && !dev->board.has_ir_i2c) { if (dev->board.ir_codes == NULL && !dev->board.has_ir_i2c) {
/* No remote control support */ /* No remote control support */
em28xx_warn("Remote control support is not available for " em28xx_warn("Remote control support is not available for this card.\n");
"this card.\n");
return 0; return 0;
} }
......
...@@ -505,8 +505,7 @@ static void em28xx_copy_video(struct em28xx *dev, ...@@ -505,8 +505,7 @@ static void em28xx_copy_video(struct em28xx *dev,
if ((char *)startwrite + lencopy > (char *)buf->vb_buf + if ((char *)startwrite + lencopy > (char *)buf->vb_buf +
buf->length) { buf->length) {
em28xx_isocdbg("Overflow of %zu bytes past buffer end" em28xx_isocdbg("Overflow of %zu bytes past buffer end(2)\n",
"(2)\n",
((char *)startwrite + lencopy) - ((char *)startwrite + lencopy) -
((char *)buf->vb_buf + buf->length)); ((char *)buf->vb_buf + buf->length));
lencopy = remain = (char *)buf->vb_buf + buf->length - lencopy = remain = (char *)buf->vb_buf + buf->length -
...@@ -2204,8 +2203,7 @@ static int em28xx_v4l2_close(struct file *filp) ...@@ -2204,8 +2203,7 @@ static int em28xx_v4l2_close(struct file *filp)
em28xx_videodbg("setting alternate 0\n"); em28xx_videodbg("setting alternate 0\n");
errCode = usb_set_interface(dev->udev, 0, 0); errCode = usb_set_interface(dev->udev, 0, 0);
if (errCode < 0) { if (errCode < 0) {
em28xx_errdev("cannot change alternate number to " em28xx_errdev("cannot change alternate number to 0 (error=%i)\n", errCode);
"0 (error=%i)\n", errCode);
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment