Commit 68616504 authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab

[media] tm6000: don't break long lines

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: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
parent b4b3f99c
...@@ -58,9 +58,7 @@ MODULE_PARM_DESC(index, "Index value for tm6000x capture interface(s)."); ...@@ -58,9 +58,7 @@ MODULE_PARM_DESC(index, "Index value for tm6000x capture interface(s).");
MODULE_DESCRIPTION("ALSA driver module for tm5600/tm6000/tm6010 based TV cards"); MODULE_DESCRIPTION("ALSA driver module for tm5600/tm6000/tm6010 based TV cards");
MODULE_AUTHOR("Mauro Carvalho Chehab"); MODULE_AUTHOR("Mauro Carvalho Chehab");
MODULE_LICENSE("GPL"); MODULE_LICENSE("GPL");
MODULE_SUPPORTED_DEVICE("{{Trident,tm5600}," MODULE_SUPPORTED_DEVICE("{{Trident,tm5600},{{Trident,tm6000},{{Trident,tm6010}");
"{{Trident,tm6000},"
"{{Trident,tm6010}");
static unsigned int debug; static unsigned int debug;
module_param(debug, int, 0644); module_param(debug, int, 0644);
MODULE_PARM_DESC(debug, "enable debug messages"); MODULE_PARM_DESC(debug, "enable debug messages");
......
...@@ -602,8 +602,8 @@ int tm6000_init(struct tm6000_core *dev) ...@@ -602,8 +602,8 @@ int tm6000_init(struct tm6000_core *dev)
for (i = 0; i < size; i++) { for (i = 0; i < size; i++) {
rc = tm6000_set_reg(dev, tab[i].req, tab[i].reg, tab[i].val); rc = tm6000_set_reg(dev, tab[i].req, tab[i].reg, tab[i].val);
if (rc < 0) { if (rc < 0) {
printk(KERN_ERR "Error %i while setting req %d, " printk(KERN_ERR "Error %i while setting req %d, reg %d to value %d\n",
"reg %d to value %d\n", rc, rc,
tab[i].req, tab[i].reg, tab[i].val); tab[i].req, tab[i].reg, tab[i].val);
return rc; return rc;
} }
...@@ -761,9 +761,8 @@ int tm6000_tvaudio_set_mute(struct tm6000_core *dev, u8 mute) ...@@ -761,9 +761,8 @@ int tm6000_tvaudio_set_mute(struct tm6000_core *dev, u8 mute)
if (dev->dev_type == TM6010) if (dev->dev_type == TM6010)
tm6010_set_mute_sif(dev, mute); tm6010_set_mute_sif(dev, mute);
else { else {
printk(KERN_INFO "ERROR: TM5600 and TM6000 don't has" printk(KERN_INFO "ERROR: TM5600 and TM6000 don't has SIF audio inputs. Please check the %s configuration.\n",
" SIF audio inputs. Please check the %s" dev->name);
" configuration.\n", dev->name);
return -EINVAL; return -EINVAL;
} }
break; break;
...@@ -822,9 +821,8 @@ void tm6000_set_volume(struct tm6000_core *dev, int vol) ...@@ -822,9 +821,8 @@ void tm6000_set_volume(struct tm6000_core *dev, int vol)
if (dev->dev_type == TM6010) if (dev->dev_type == TM6010)
tm6010_set_volume_sif(dev, vol); tm6010_set_volume_sif(dev, vol);
else else
printk(KERN_INFO "ERROR: TM5600 and TM6000 don't has" printk(KERN_INFO "ERROR: TM5600 and TM6000 don't has SIF audio inputs. Please check the %s configuration.\n",
" SIF audio inputs. Please check the %s" dev->name);
" configuration.\n", dev->name);
break; break;
case TM6000_AMUX_ADC1: case TM6000_AMUX_ADC1:
case TM6000_AMUX_ADC2: case TM6000_AMUX_ADC2:
......
...@@ -35,9 +35,7 @@ MODULE_DESCRIPTION("DVB driver extension module for tm5600/6000/6010 based TV ca ...@@ -35,9 +35,7 @@ MODULE_DESCRIPTION("DVB driver extension module for tm5600/6000/6010 based TV ca
MODULE_AUTHOR("Mauro Carvalho Chehab"); MODULE_AUTHOR("Mauro Carvalho Chehab");
MODULE_LICENSE("GPL"); MODULE_LICENSE("GPL");
MODULE_SUPPORTED_DEVICE("{{Trident, tm5600}," MODULE_SUPPORTED_DEVICE("{{Trident, tm5600},{{Trident, tm6000},{{Trident, tm6010}");
"{{Trident, tm6000},"
"{{Trident, tm6010}");
static int debug; static int debug;
...@@ -292,13 +290,11 @@ static int register_dvb(struct tm6000_core *dev) ...@@ -292,13 +290,11 @@ static int register_dvb(struct tm6000_core *dev)
} }
if (!dvb_attach(xc2028_attach, dvb->frontend, &cfg)) { if (!dvb_attach(xc2028_attach, dvb->frontend, &cfg)) {
printk(KERN_ERR "tm6000: couldn't register " printk(KERN_ERR "tm6000: couldn't register frontend (xc3028)\n");
"frontend (xc3028)\n");
ret = -EINVAL; ret = -EINVAL;
goto frontend_err; goto frontend_err;
} }
printk(KERN_INFO "tm6000: XC2028/3028 asked to be " printk(KERN_INFO "tm6000: XC2028/3028 asked to be attached to frontend!\n");
"attached to frontend!\n");
break; break;
} }
case TUNER_XC5000: { case TUNER_XC5000: {
...@@ -315,13 +311,11 @@ static int register_dvb(struct tm6000_core *dev) ...@@ -315,13 +311,11 @@ static int register_dvb(struct tm6000_core *dev)
} }
if (!dvb_attach(xc5000_attach, dvb->frontend, &dev->i2c_adap, &cfg)) { if (!dvb_attach(xc5000_attach, dvb->frontend, &dev->i2c_adap, &cfg)) {
printk(KERN_ERR "tm6000: couldn't register " printk(KERN_ERR "tm6000: couldn't register frontend (xc5000)\n");
"frontend (xc5000)\n");
ret = -EINVAL; ret = -EINVAL;
goto frontend_err; goto frontend_err;
} }
printk(KERN_INFO "tm6000: XC5000 asked to be " printk(KERN_INFO "tm6000: XC5000 asked to be attached to frontend!\n");
"attached to frontend!\n");
break; break;
} }
} }
......
...@@ -173,8 +173,7 @@ static int tm6000_i2c_xfer(struct i2c_adapter *i2c_adap, ...@@ -173,8 +173,7 @@ static int tm6000_i2c_xfer(struct i2c_adapter *i2c_adap,
* immediately after a 1 or 2 byte write to select * immediately after a 1 or 2 byte write to select
* a register. We cannot fulfil this request. * a register. We cannot fulfil this request.
*/ */
i2c_dprintk(2, " read without preceding write not" i2c_dprintk(2, " read without preceding write not supported");
" supported");
rc = -EOPNOTSUPP; rc = -EOPNOTSUPP;
goto err; goto err;
} else if (i + 1 < num && msgs[i].len <= 2 && } else if (i + 1 < num && msgs[i].len <= 2 &&
......
...@@ -464,8 +464,7 @@ static int tm6000_load_std(struct tm6000_core *dev, struct tm6000_reg_settings * ...@@ -464,8 +464,7 @@ static int tm6000_load_std(struct tm6000_core *dev, struct tm6000_reg_settings *
for (i = 0; set[i].req; i++) { for (i = 0; set[i].req; i++) {
rc = tm6000_set_reg(dev, set[i].req, set[i].reg, set[i].value); rc = tm6000_set_reg(dev, set[i].req, set[i].reg, set[i].value);
if (rc < 0) { if (rc < 0) {
printk(KERN_ERR "Error %i while setting " printk(KERN_ERR "Error %i while setting req %d, reg %d to value %d\n",
"req %d, reg %d to value %d\n",
rc, set[i].req, set[i].reg, set[i].value); rc, set[i].req, set[i].reg, set[i].value);
return rc; return rc;
} }
......
...@@ -615,8 +615,7 @@ static int tm6000_prepare_isoc(struct tm6000_core *dev) ...@@ -615,8 +615,7 @@ static int tm6000_prepare_isoc(struct tm6000_core *dev)
return -ENOMEM; return -ENOMEM;
} }
dprintk(dev, V4L2_DEBUG_QUEUE, "Allocating %d x %d packets" dprintk(dev, V4L2_DEBUG_QUEUE, "Allocating %d x %d packets (%d bytes) of %d bytes each to handle %u size\n",
" (%d bytes) of %d bytes each to handle %u size\n",
max_packets, num_bufs, sb_size, max_packets, num_bufs, sb_size,
dev->isoc_in.maxsize, size); dev->isoc_in.maxsize, size);
...@@ -939,8 +938,8 @@ static int vidioc_try_fmt_vid_cap(struct file *file, void *priv, ...@@ -939,8 +938,8 @@ static int vidioc_try_fmt_vid_cap(struct file *file, void *priv,
fmt = format_by_fourcc(f->fmt.pix.pixelformat); fmt = format_by_fourcc(f->fmt.pix.pixelformat);
if (NULL == fmt) { if (NULL == fmt) {
dprintk(dev, 2, "Fourcc format (0x%08x)" dprintk(dev, 2, "Fourcc format (0x%08x) invalid.\n",
" invalid.\n", f->fmt.pix.pixelformat); f->fmt.pix.pixelformat);
return -EINVAL; return -EINVAL;
} }
...@@ -1366,14 +1365,13 @@ static int __tm6000_open(struct file *file) ...@@ -1366,14 +1365,13 @@ static int __tm6000_open(struct file *file)
fh->width = dev->width; fh->width = dev->width;
fh->height = dev->height; fh->height = dev->height;
dprintk(dev, V4L2_DEBUG_OPEN, "Open: fh=0x%08lx, dev=0x%08lx, " dprintk(dev, V4L2_DEBUG_OPEN, "Open: fh=0x%08lx, dev=0x%08lx, dev->vidq=0x%08lx\n",
"dev->vidq=0x%08lx\n",
(unsigned long)fh, (unsigned long)dev, (unsigned long)fh, (unsigned long)dev,
(unsigned long)&dev->vidq); (unsigned long)&dev->vidq);
dprintk(dev, V4L2_DEBUG_OPEN, "Open: list_empty " dprintk(dev, V4L2_DEBUG_OPEN, "Open: list_empty queued=%d\n",
"queued=%d\n", list_empty(&dev->vidq.queued)); list_empty(&dev->vidq.queued));
dprintk(dev, V4L2_DEBUG_OPEN, "Open: list_empty " dprintk(dev, V4L2_DEBUG_OPEN, "Open: list_empty active=%d\n",
"active=%d\n", list_empty(&dev->vidq.active)); list_empty(&dev->vidq.active));
/* initialize hardware on analog mode */ /* initialize hardware on analog mode */
rc = tm6000_init_analog_mode(dev); rc = tm6000_init_analog_mode(dev);
......
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