]> git.karo-electronics.de Git - karo-tx-linux.git/commitdiff
[media] dvb-usb: don't break long lines
authorMauro Carvalho Chehab <mchehab@s-opensource.com>
Tue, 18 Oct 2016 19:44:15 +0000 (17:44 -0200)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Fri, 21 Oct 2016 11:39:51 +0000 (09:39 -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/usb/dvb-usb/cinergyT2-core.c
drivers/media/usb/dvb-usb/dib0700_core.c
drivers/media/usb/dvb-usb/dib0700_devices.c
drivers/media/usb/dvb-usb/dvb-usb-dvb.c
drivers/media/usb/dvb-usb/dvb-usb-firmware.c
drivers/media/usb/dvb-usb/dw2102.c
drivers/media/usb/dvb-usb/friio.c
drivers/media/usb/dvb-usb/gp8psk.c
drivers/media/usb/dvb-usb/opera1.c
drivers/media/usb/dvb-usb/technisat-usb2.c

index 8ac825413d5ac09a17f33f9a6537e0cd69e4564c..1a198d07e30eda8a07eba8051f100e1a4cfb4610 100644 (file)
@@ -34,8 +34,7 @@
 int dvb_usb_cinergyt2_debug;
 
 module_param_named(debug, dvb_usb_cinergyt2_debug, int, 0644);
-MODULE_PARM_DESC(debug, "set debugging level (1=info, xfer=2, rc=4 "
-               "(or-able)).");
+MODULE_PARM_DESC(debug, "set debugging level (1=info, xfer=2, rc=4 (or-able)).");
 
 DVB_DEFINE_MOD_OPT_ADAPTER_NR(adapter_nr);
 
@@ -94,8 +93,7 @@ static int cinergyt2_frontend_attach(struct dvb_usb_adapter *adap)
 
        ret = dvb_usb_generic_rw(d, st->data, 1, st->data, 3, 0);
        if (ret < 0) {
-               deb_rc("cinergyt2_power_ctrl() Failed to retrieve sleep "
-                       "state info\n");
+               deb_rc("cinergyt2_power_ctrl() Failed to retrieve sleep state info\n");
        }
        mutex_unlock(&st->data_mutex);
 
index 92d5408684ac1cdc33f65f578b5ee2ce525968b0..47df7135e9601dfc4e194cf48933ad1cbacb6595 100644 (file)
@@ -16,10 +16,7 @@ MODULE_PARM_DESC(debug, "set debugging level (1=info,2=fw,4=fwdata,8=data (or-ab
 static int nb_packet_buffer_size = 21;
 module_param(nb_packet_buffer_size, int, 0644);
 MODULE_PARM_DESC(nb_packet_buffer_size,
-       "Set the dib0700 driver data buffer size. This parameter "
-       "corresponds to the number of TS packets. The actual size of "
-       "the data buffer corresponds to this parameter "
-       "multiplied by 188 (default: 21)");
+       "Set the dib0700 driver data buffer size. This parameter corresponds to the number of TS packets. The actual size of the data buffer corresponds to this parameter multiplied by 188 (default: 21)");
 
 DVB_DEFINE_MOD_OPT_ADAPTER_NR(adapter_nr);
 
index ef1b8ee75c577f00c61b07b80859ef063722d907..b29d4894c2f1ca914088880785e0157c2cdfcff5 100644 (file)
@@ -26,8 +26,7 @@
 
 static int force_lna_activation;
 module_param(force_lna_activation, int, 0644);
-MODULE_PARM_DESC(force_lna_activation, "force the activation of Low-Noise-Amplifyer(s) (LNA), "
-               "if applicable for the device (default: 0=automatic/off).");
+MODULE_PARM_DESC(force_lna_activation, "force the activation of Low-Noise-Amplifyer(s) (LNA), if applicable for the device (default: 0=automatic/off).");
 
 struct dib0700_adapter_state {
        int (*set_param_save) (struct dvb_frontend *);
index a04c0a25062579c310c837c0ba83ff3f60e9d94c..e5675da286cb2afca37edb62f5ba28d636ae6f44 100644 (file)
@@ -277,8 +277,7 @@ int dvb_usb_adapter_frontend_init(struct dvb_usb_adapter *adap)
        for (i = 0; i < adap->props.num_frontends; i++) {
 
                if (adap->props.fe[i].frontend_attach == NULL) {
-                       err("strange: '%s' #%d,%d "
-                           "doesn't want to attach a frontend.",
+                       err("strange: '%s' #%d,%d doesn't want to attach a frontend.",
                            adap->dev->desc->name, adap->id, i);
 
                        return 0;
index dd048a7c461c8ffe1884b52ca80ff2b35233adaf..f0023dbb72761641112ac7b0bfcba51cf154eda2 100644 (file)
@@ -49,8 +49,7 @@ int usb_cypress_load_firmware(struct usb_device *udev, const struct firmware *fw
                ret = usb_cypress_writemem(udev,hx.addr,hx.data,hx.len);
 
                if (ret != hx.len) {
-                       err("error while transferring firmware "
-                               "(transferred size: %d, block size: %d)",
+                       err("error while transferring firmware (transferred size: %d, block size: %d)",
                                ret,hx.len);
                        ret = -EINVAL;
                        break;
@@ -81,8 +80,7 @@ int dvb_usb_download_firmware(struct usb_device *udev, struct dvb_usb_device_pro
        const struct firmware *fw = NULL;
 
        if ((ret = request_firmware(&fw, props->firmware, &udev->dev)) != 0) {
-               err("did not find the firmware file. (%s) "
-                       "Please see linux/Documentation/dvb/ for more details on firmware-problems. (%d)",
+               err("did not find the firmware file. (%s) Please see linux/Documentation/dvb/ for more details on firmware-problems. (%d)",
                        props->firmware,ret);
                return ret;
        }
index 2c720cb2fb00f0ddecd0eb82c6daf0f3a6731094..38794ab8874e9624a72ccc3788190a49a01bc8b5 100644 (file)
@@ -86,8 +86,7 @@ MODULE_PARM_DESC(debug, "set debugging level (1=info 2=xfer 4=rc(or-able))."
 /* demod probe */
 static int demod_probe = 1;
 module_param_named(demod, demod_probe, int, 0644);
-MODULE_PARM_DESC(demod, "demod to probe (1=cx24116 2=stv0903+stv6110 "
-                       "4=stv0903+stb6100(or-able)).");
+MODULE_PARM_DESC(demod, "demod to probe (1=cx24116 2=stv0903+stv6110 4=stv0903+stb6100(or-able)).");
 
 DVB_DEFINE_MOD_OPT_ADAPTER_NR(adapter_nr);
 
@@ -2343,12 +2342,7 @@ static struct usb_driver dw2102_driver = {
 module_usb_driver(dw2102_driver);
 
 MODULE_AUTHOR("Igor M. Liplianin (c) liplianin@me.by");
-MODULE_DESCRIPTION("Driver for DVBWorld DVB-S 2101, 2102, DVB-S2 2104,"
-                       " DVB-C 3101 USB2.0,"
-                       " TeVii S421, S480, S482, S600, S630, S632, S650,"
-                       " TeVii S660, S662, Prof 1100, 7500 USB2.0,"
-                       " Geniatech SU3000, T220,"
-                       " TechnoTrend S2-4600, Terratec Cinergy S2 devices");
+MODULE_DESCRIPTION("Driver for DVBWorld DVB-S 2101, 2102, DVB-S2 2104, DVB-C 3101 USB2.0, TeVii S421, S480, S482, S600, S630, S632, S650, TeVii S660, S662, Prof 1100, 7500 USB2.0, Geniatech SU3000, T220, TechnoTrend S2-4600, Terratec Cinergy S2 devices");
 MODULE_VERSION("0.1");
 MODULE_LICENSE("GPL");
 MODULE_FIRMWARE(DW2101_FIRMWARE);
index 474a17e4db0cc35f20e7423189d98ec1eff385b0..62abe6c43a324588af7f269d36a55e601ae6c962 100644 (file)
@@ -320,8 +320,8 @@ restart:
  */
        if (rbuf[0] & 0x80) {   /* still in PowerOnReset state? */
                if (++retry > 3) {
-                       deb_info("failed to get the correct"
-                                " FE demod status:0x%02x\n", rbuf[0]);
+                       deb_info("failed to get the correct FE demod status:0x%02x\n",
+                                rbuf[0]);
                        goto error;
                }
                msleep(100);
index adfd764914517bdbbbff823ee31e5d08ebcac7fb..db9e9a9721a039952dc5c485f0ccc095ebe61801 100644 (file)
@@ -131,8 +131,7 @@ static int gp8psk_load_bcm4500fw(struct dvb_usb_device *d)
        u8 *buf;
        if ((ret = request_firmware(&fw, bcm4500_firmware,
                                        &d->udev->dev)) != 0) {
-               err("did not find the bcm4500 firmware file. (%s) "
-                       "Please see linux/Documentation/dvb/ for more details on firmware-problems. (%d)",
+               err("did not find the bcm4500 firmware file. (%s) Please see linux/Documentation/dvb/ for more details on firmware-problems. (%d)",
                        bcm4500_firmware,ret);
                return ret;
        }
index 2566d2f1c2ad0220467462524926eabcbb5c1ecc..946a5ccc8f1a97e8117292810b5d9146bec96d79 100644 (file)
@@ -453,8 +453,7 @@ static int opera1_xilinx_load_firmware(struct usb_device *dev,
        info("start downloading fpga firmware %s",filename);
 
        if ((ret = request_firmware(&fw, filename, &dev->dev)) != 0) {
-               err("did not find the firmware file. (%s) "
-                       "Please see linux/Documentation/dvb/ for more details on firmware-problems.",
+               err("did not find the firmware file. (%s) Please see linux/Documentation/dvb/ for more details on firmware-problems.",
                        filename);
                return ret;
        } else {
index 4706628a3ed5ea5345e30a9dc59cad7fb385286d..02c3bee6f83bdd789af978578254cc85cfe7c62d 100644 (file)
@@ -50,8 +50,7 @@ MODULE_PARM_DESC(debug,
 static int disable_led_control;
 module_param(disable_led_control, int, 0444);
 MODULE_PARM_DESC(disable_led_control,
-               "disable LED control of the device "
-               "(default: 0 - LED control is active).");
+               "disable LED control of the device (default: 0 - LED control is active).");
 
 /* device private data */
 struct technisat_usb2_state {