[media] ivtv: don't break long lines
authorMauro Carvalho Chehab <mchehab@s-opensource.com>
Tue, 18 Oct 2016 19:44:05 +0000 (17:44 -0200)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Fri, 21 Oct 2016 11:11:09 +0000 (09:11 -0200)
Due to the 80-cols restrictions, and latter due to 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.

As we're right now fixing other drivers due to KERN_CONT, we need
to be able to identify what printk strings don't end with a "\n".
It is a way easier to detect those if we don't break long lines.

So, join those continuation lines.

The patch was generated via the script below, and manually
adjusted if needed.

</script>
use Text::Tabs;
while (<>) {
if ($next ne "") {
$c=$_;
if ($c =~ /^\s+\"(.*)/) {
$c2=$1;
$next =~ s/\"\n$//;
$n = expand($next);
$funpos = index($n, '(');
$pos = index($c2, '",');
if ($funpos && $pos > 0) {
$s1 = substr $c2, 0, $pos + 2;
$s2 = ' ' x ($funpos + 1) . substr $c2, $pos + 2;
$s2 =~ s/^\s+//;

$s2 = ' ' x ($funpos + 1) . $s2 if ($s2 ne "");

print unexpand("$next$s1\n");
print unexpand("$s2\n") if ($s2 ne "");
} else {
print "$next$c2\n";
}
$next="";
next;
} else {
print $next;
}
$next="";
} else {
if (m/\"$/) {
if (!m/\\n\"$/) {
$next=$_;
next;
}
}
}
print $_;
}
</script>

Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/pci/ivtv/ivtv-alsa-main.c
drivers/media/pci/ivtv/ivtv-driver.c
drivers/media/pci/ivtv/ivtv-firmware.c
drivers/media/pci/ivtv/ivtv-yuv.c
drivers/media/pci/ivtv/ivtvfb.c

index 8a86b61..374f45f 100644 (file)
@@ -177,8 +177,8 @@ static int snd_ivtv_init(struct v4l2_device *v4l2_dev)
 #if 0
        ret = snd_ivtv_mixer_create(itvsc);
        if (ret) {
-               IVTV_ALSA_WARN("%s: snd_ivtv_mixer_create() failed with err %d:"
-                              " proceeding anyway\n", __func__, ret);
+               IVTV_ALSA_WARN("%s: snd_ivtv_mixer_create() failed with err %d: proceeding anyway\n",
+                              __func__, ret);
        }
 #endif
 
@@ -235,8 +235,8 @@ static int ivtv_alsa_load(struct ivtv *itv)
 
        s = &itv->streams[IVTV_ENC_STREAM_TYPE_PCM];
        if (s->vdev.v4l2_dev == NULL) {
-               IVTV_DEBUG_ALSA_INFO("%s: PCM stream for card is disabled - "
-                                    "skipping\n", __func__);
+               IVTV_DEBUG_ALSA_INFO("%s: PCM stream for card is disabled - skipping\n",
+                                    __func__);
                return 0;
        }
 
@@ -250,8 +250,8 @@ static int ivtv_alsa_load(struct ivtv *itv)
                IVTV_ALSA_ERR("%s: failed to create struct snd_ivtv_card\n",
                              __func__);
        } else {
-               IVTV_DEBUG_ALSA_INFO("%s: created ivtv ALSA interface instance "
-                                    "\n", __func__);
+               IVTV_DEBUG_ALSA_INFO("%s: created ivtv ALSA interface instance \n",
+                                    __func__);
        }
        return 0;
 }
index ee48c3e..0a3b80a 100644 (file)
@@ -885,8 +885,8 @@ static int ivtv_setup_pci(struct ivtv *itv, struct pci_dev *pdev,
        pci_read_config_byte(pdev, PCI_LATENCY_TIMER, &pci_latency);
 
        if (pci_latency < 64 && ivtv_pci_latency) {
-               IVTV_INFO("Unreasonably low latency timer, "
-                              "setting to 64 (was %d)\n", pci_latency);
+               IVTV_INFO("Unreasonably low latency timer, setting to 64 (was %d)\n",
+                         pci_latency);
                pci_write_config_byte(pdev, PCI_LATENCY_TIMER, 64);
                pci_read_config_byte(pdev, PCI_LATENCY_TIMER, &pci_latency);
        }
@@ -896,8 +896,7 @@ static int ivtv_setup_pci(struct ivtv *itv, struct pci_dev *pdev,
           these problems. */
        pci_write_config_dword(pdev, 0x40, 0xffff);
 
-       IVTV_DEBUG_INFO("%d (rev %d) at %02x:%02x.%x, "
-                  "irq: %d, latency: %d, memory: 0x%llx\n",
+       IVTV_DEBUG_INFO("%d (rev %d) at %02x:%02x.%x, irq: %d, latency: %d, memory: 0x%llx\n",
                   pdev->device, pdev->revision, pdev->bus->number,
                   PCI_SLOT(pdev->devfn), PCI_FUNC(pdev->devfn),
                   pdev->irq, pci_latency, (u64)itv->base_addr);
@@ -1047,13 +1046,10 @@ static int ivtv_probe(struct pci_dev *pdev, const struct pci_device_id *pci_id)
        itv->enc_mem = ioremap_nocache(itv->base_addr + IVTV_ENCODER_OFFSET,
                                       IVTV_ENCODER_SIZE);
        if (!itv->enc_mem) {
-               IVTV_ERR("ioremap failed. Can't get a window into CX23415/6 "
-                        "encoder memory\n");
-               IVTV_ERR("Each capture card with a CX23415/6 needs 8 MB of "
-                        "vmalloc address space for this window\n");
+               IVTV_ERR("ioremap failed. Can't get a window into CX23415/6 encoder memory\n");
+               IVTV_ERR("Each capture card with a CX23415/6 needs 8 MB of vmalloc address space for this window\n");
                IVTV_ERR("Check the output of 'grep Vmalloc /proc/meminfo'\n");
-               IVTV_ERR("Use the vmalloc= kernel command line option to set "
-                        "VmallocTotal to a larger value\n");
+               IVTV_ERR("Use the vmalloc= kernel command line option to set VmallocTotal to a larger value\n");
                retval = -ENOMEM;
                goto free_mem;
        }
@@ -1064,14 +1060,10 @@ static int ivtv_probe(struct pci_dev *pdev, const struct pci_device_id *pci_id)
                itv->dec_mem = ioremap_nocache(itv->base_addr + IVTV_DECODER_OFFSET,
                                IVTV_DECODER_SIZE);
                if (!itv->dec_mem) {
-                       IVTV_ERR("ioremap failed. Can't get a window into "
-                                "CX23415 decoder memory\n");
-                       IVTV_ERR("Each capture card with a CX23415 needs 8 MB "
-                                "of vmalloc address space for this window\n");
-                       IVTV_ERR("Check the output of 'grep Vmalloc "
-                                "/proc/meminfo'\n");
-                       IVTV_ERR("Use the vmalloc= kernel command line option "
-                                "to set VmallocTotal to a larger value\n");
+                       IVTV_ERR("ioremap failed. Can't get a window into CX23415 decoder memory\n");
+                       IVTV_ERR("Each capture card with a CX23415 needs 8 MB of vmalloc address space for this window\n");
+                       IVTV_ERR("Check the output of 'grep Vmalloc /proc/meminfo'\n");
+                       IVTV_ERR("Use the vmalloc= kernel command line option to set VmallocTotal to a larger value\n");
                        retval = -ENOMEM;
                        goto free_mem;
                }
@@ -1086,13 +1078,10 @@ static int ivtv_probe(struct pci_dev *pdev, const struct pci_device_id *pci_id)
        itv->reg_mem =
            ioremap_nocache(itv->base_addr + IVTV_REG_OFFSET, IVTV_REG_SIZE);
        if (!itv->reg_mem) {
-               IVTV_ERR("ioremap failed. Can't get a window into CX23415/6 "
-                        "register space\n");
-               IVTV_ERR("Each capture card with a CX23415/6 needs 64 kB of "
-                        "vmalloc address space for this window\n");
+               IVTV_ERR("ioremap failed. Can't get a window into CX23415/6 register space\n");
+               IVTV_ERR("Each capture card with a CX23415/6 needs 64 kB of vmalloc address space for this window\n");
                IVTV_ERR("Check the output of 'grep Vmalloc /proc/meminfo'\n");
-               IVTV_ERR("Use the vmalloc= kernel command line option to set "
-                        "VmallocTotal to a larger value\n");
+               IVTV_ERR("Use the vmalloc= kernel command line option to set VmallocTotal to a larger value\n");
                retval = -ENOMEM;
                goto free_io;
        }
index 5b3095f..ba279fd 100644 (file)
@@ -376,8 +376,8 @@ int ivtv_firmware_check(struct ivtv *itv, char *where)
        /* If something failed & currently idle, try to reload */
        if (res && !atomic_read(&itv->capturing) &&
                                                !atomic_read(&itv->decoding)) {
-               IVTV_INFO("Detected in %s that firmware had failed - "
-                         "Reloading\n", where);
+               IVTV_INFO("Detected in %s that firmware had failed - Reloading\n",
+                         where);
                res = ivtv_firmware_restart(itv);
                /*
                 * Even if restarted ok, still signal a problem had occurred.
index b094054..a61f632 100644 (file)
@@ -88,8 +88,8 @@ static int ivtv_yuv_prep_user_dma(struct ivtv *itv, struct ivtv_user_dma *dma,
 
                if (y_pages == y_dma.page_count) {
                        IVTV_DEBUG_WARN
-                               ("failed to map uv user pages, returned %d "
-                                "expecting %d\n", uv_pages, uv_dma.page_count);
+                               ("failed to map uv user pages, returned %d expecting %d\n",
+                                uv_pages, uv_dma.page_count);
 
                        if (uv_pages >= 0) {
                                for (i = 0; i < uv_pages; i++)
@@ -100,8 +100,8 @@ static int ivtv_yuv_prep_user_dma(struct ivtv *itv, struct ivtv_user_dma *dma,
                        }
                } else {
                        IVTV_DEBUG_WARN
-                               ("failed to map y user pages, returned %d "
-                                "expecting %d\n", y_pages, y_dma.page_count);
+                               ("failed to map y user pages, returned %d expecting %d\n",
+                                y_pages, y_dma.page_count);
                }
                if (y_pages >= 0) {
                        for (i = 0; i < y_pages; i++)
index 8b95eef..612a840 100644 (file)
@@ -293,8 +293,7 @@ static int ivtvfb_prep_dec_dma_to_device(struct ivtv *itv,
        /* Map User DMA */
        if (ivtv_udma_setup(itv, ivtv_dest_addr, userbuf, size_in_bytes) <= 0) {
                mutex_unlock(&itv->udma.lock);
-               IVTVFB_WARN("ivtvfb_prep_dec_dma_to_device, "
-                              "Error with get_user_pages: %d bytes, %d pages returned\n",
+               IVTVFB_WARN("ivtvfb_prep_dec_dma_to_device, Error with get_user_pages: %d bytes, %d pages returned\n",
                               size_in_bytes, itv->udma.page_count);
 
                /* get_user_pages must have failed completely */