cron job: media_tree daily build: ERRORS

2016-10-14 Thread Hans Verkuil
This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date: Sat Oct 15 05:00:23 CEST 2016 media-tree git hash:11a1e0ed7908f04c896e69d0eb65e478c12f8519 media_build

Re: V4L2_DEC_CMD_STOP and last_buffer_dequeued

2016-10-14 Thread 李務誠
On Sat, Oct 15, 2016 at 2:20 AM, Nicolas Dufresne wrote: > > Le mercredi 12 octobre 2016 à 23:33 +0800, Wu-Cheng Li (李務誠) a écrit : > > I'm trying to use V4L2_DEC_CMD_STOP to implement flush. First the > > userspace sent V4L2_DEC_CMD_STOP to initiate the flush. The

Re: [PATCH 29/57] [media] ti-vpe: don't break long lines

2016-10-14 Thread Benoit Parrot
Acked-by: Benoit Parrot Mauro Carvalho Chehab wrote on Fri [2016-Oct-14 17:20:17 -0300]: > 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

Re: [PATCH 54/57] [media] platform: don't break long lines

2016-10-14 Thread Robert Jarzmik
Mauro Carvalho Chehab writes: > 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.

[PATCH 19/57] [media] solo6x10: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 57/57] [media] tuners: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 56/57] [media] rc: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 24/57] [media] marvell-ccic: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 28/57] [media] c8sectpfe: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 45/57] [media] tm6000: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 49/57] [media] uvc: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 27/57] [media] s5p-mfc: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 50/57] [media] zr364xx: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 14/57] [media] ivtv: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 17/57] [media] saa7134: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 18/57] [media] saa7164: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 20/57] [media] ttpci: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 22/57] [media] davinci: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 23/57] [media] exynos4-is: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 13/57] [media] dm1105: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 29/57] [media] ti-vpe: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 21/57] [media] tw68: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 51/57] [media] v4l2-core: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 53/57] [media] i2c: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 26/57] [media] omap3isp: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 25/57] [media] omap: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 10/57] [media] cx23885: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 15/57] [media] meye: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 48/57] [media] usbvision: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 42/57] [media] pwc: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 43/57] [media] siano: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 54/57] [media] platform: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 30/57] [media] si470x: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 37/57] [media] dvb-usb: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 32/57] [media] wl128x: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 41/57] [media] pvrusb2: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 46/57] [media] ttusb-budget: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 35/57] [media] cpia2: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 52/57] [media] common: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 47/57] [media] ttusb-dec: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 55/57] [media] radio: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 40/57] [media] hdpvr: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 36/57] [media] cx231xx: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 07/57] [media] b2c2: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 11/57] [media] cx88: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 31/57] [media] si4713: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 02/57] [media] dvb-frontends: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 38/57] [media] dvb-usb-v2: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 39/57] [media] em28xx: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 34/57] [media] b2c2: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 09/57] [media] cx18: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 08/57] [media] bt8xx: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 03/57] [media] firewire: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 44/57] [media] stkwebcam: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 33/57] [media] au0828: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 04/57] [media] cx25840: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 00/57] don't break long lines on strings

2016-10-14 Thread Mauro Carvalho Chehab
There are lots of drivers on media that breaks long line strings in order to fit into 80 columns. This was an usual practice to make checkpatch happy. However, it makes harder to detect where the strings are with grep. As we're right now fixing other drivers due to KERN_CONT, we need to be able

[PATCH 12/57] [media] ddbridge: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 01/57] [media] b2c2: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 06/57] [media] soc_camera: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 05/57] [media] smiapp: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

Re: [PATCH 11/25] [media] dvb-core: use pr_foo() instead of printk()

2016-10-14 Thread SF Markus Elfring
> diff --git a/drivers/media/dvb-core/dmxdev.c b/drivers/media/dvb-core/dmxdev.c > index 7b67e1dd97fd..1e96a6f1b6f0 100644 > --- a/drivers/media/dvb-core/dmxdev.c > +++ b/drivers/media/dvb-core/dmxdev.c > @@ -20,6 +20,8 @@ > * > */ > > +#define pr_fmt(fmt) "dmxdev: " fmt > + > #include >

Re: V4L2_DEC_CMD_STOP and last_buffer_dequeued

2016-10-14 Thread Nicolas Dufresne
Le mercredi 12 octobre 2016 à 23:33 +0800, Wu-Cheng Li (李務誠) a écrit : > I'm trying to use V4L2_DEC_CMD_STOP to implement flush. First the > userspace sent V4L2_DEC_CMD_STOP to initiate the flush. The driver > set > V4L2_BUF_FLAG_LAST on the last CAPTURE buffer. I thought implementing >

[PATCH 13/25] [media] dvb_demux: uncomment a packet loss check code

2016-10-14 Thread Mauro Carvalho Chehab
There is a commented code that also detects packet loss. Uncomment it and put into the DVB_DEMUX_SECTION_LOSS_LOG debug Kconfig option. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/dvb-core/dvb_demux.c | 10 ++ 1 file changed, 6 insertions(+), 4

[PATCH 06/25] [media] em28xx: convert the remaining printks to pr_foo

2016-10-14 Thread Mauro Carvalho Chehab
There are still several places with printk's called directly. Convert them to pr_foo() macros, except for the debug printk's, as those are enabled via modprobe vars. While here, realign the pr_foo() arguments to match the recommended CodingStyle. Signed-off-by: Mauro Carvalho Chehab

[PATCH 16/25] [media] dvb_filter: get rid of dead code

2016-10-14 Thread Mauro Carvalho Chehab
There are lots of stuff here commented out for a really long time. Get rid of them. If one wants it again, it could always use git log. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/pci/ttpci/dvb_filter.c | 485 --- 1 file

[PATCH 19/25] [media] imon: use %*ph to do small hexa dumps

2016-10-14 Thread Mauro Carvalho Chehab
Since commit 563873318d32 ("Merge branch 'printk-cleanups"), continuation lines require KERN_CONT. Instead, let's just use %*ph to print the buffer. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/rc/imon.c | 7 ++- 1 file changed, 2 insertions(+), 5

[PATCH 11/25] [media] dvb-core: use pr_foo() instead of printk()

2016-10-14 Thread Mauro Carvalho Chehab
The dvb-core directly calls printk() without using the modern printk macros, or using the proper printk levels. Change it to use pr_foo(). Signed-off-by: Mauro Carvalho Chehab --- drivers/media/dvb-core/dmxdev.c | 24 ++

[PATCH 10/25] [media] radio-bcm2048: don't ignore errors

2016-10-14 Thread Mauro Carvalho Chehab
Remove this warning: drivers/staging/media/bcm2048/radio-bcm2048.c: In function 'bcm2048_set_rds_no_lock': drivers/staging/media/bcm2048/radio-bcm2048.c:467:6: warning: variable 'err' set but not used [-Wunused-but-set-variable] int err; ^~~ By returning the error code. Signed-off-by:

[PATCH 24/25] [media] dvb-pll: use pr_foo() macros instead of printk()

2016-10-14 Thread Mauro Carvalho Chehab
Replace printk() macros by their pr_foo() counterparts. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/dvb-frontends/dvb-pll.c | 19 --- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/drivers/media/dvb-frontends/dvb-pll.c

[PATCH 20/25] [media] mt20xx: use %*ph to do small hexa dumps

2016-10-14 Thread Mauro Carvalho Chehab
Since commit 563873318d32 ("Merge branch 'printk-cleanups"), continuation lines require KERN_CONT. Instead, let's just use %*ph to print the buffer. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/tuners/mt20xx.c | 21 + 1 file changed, 5

[PATCH 21/25] [media] tvaudio: mark printk continuation lines as such

2016-10-14 Thread Mauro Carvalho Chehab
This driver has printk continuation lines for debugging purposes. Since commit 563873318d32 ("Merge branch 'printk-cleanups'"), this won't work as expected anymore. So, let's add KERN_CONT to those lines. Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 12/25] [media] dvb_demux: convert an internal ifdef into a Kconfig option

2016-10-14 Thread Mauro Carvalho Chehab
There are some ifdefs inside the dvb_demux that are meant to enable advanced debug capabilities, at the cost of being very verbose. Keeping those as internal ifdefs is a very bad idea, as it doesn't make easy to check if the code there was broken by some patch. So, let's add an explicit Kconfig

[PATCH 03/25] [media] em28xx: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 01/25] [media] tuner-xc2028: mark printk continuation lines as such

2016-10-14 Thread Mauro Carvalho Chehab
This driver has a lot of printk continuation lines for debugging purposes. Since commit 563873318d32 ("Merge branch 'printk-cleanups"), this won't work as expected anymore. So, let's add KERN_CONT to those lines. Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 04/25] [media] em28xx: mark printk continuation lines as such

2016-10-14 Thread Mauro Carvalho Chehab
This driver has a lot of printk continuation lines for debugging purposes. Since commit 563873318d32 ("Merge branch 'printk-cleanups"), this won't work as expected anymore. So, let's add KERN_CONT to those lines. Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 09/25] [media] tuner-core: use % for small buffer dumps

2016-10-14 Thread Mauro Carvalho Chehab
This driver has a printk with a continuation lines for debugging purposes. Since commit 563873318d32 ("Merge branch 'printk-cleanups'"), this won't work anymore. We might be using KERNEL_CONT, but it is better to just use a single printk line using %*ph for buffer dump. Signed-off-by: Mauro

[PATCH 02/25] [media] tuner-xc2028: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 00/25] Do some printk cleanups to avoid troubles with continuation lines

2016-10-14 Thread Mauro Carvalho Chehab
This commit: commit 563873318d328d9bbab4b00dfd835ac7c7e28697 Merge: 24532f768121 bfd8d3f23b51 Author: Linus Torvalds Date:���Mon Oct 10 09:29:50 2016 -0700 Merge branch 'printk-cleanups' Affects dmesg prints like:

[PATCH 18/25] [media] uvc_driver: use KERN_CONT where needed

2016-10-14 Thread Mauro Carvalho Chehab
Some continuation messages are not using KERN_CONT. Since commit 563873318d32 ("Merge branch 'printk-cleanups"), this won't work as expected anymore. So, let's add KERN_CONT to those lines. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/usb/uvc/uvc_driver.c |

[PATCH 22/25] [media] flexcop-i2c: mark printk continuation lines as such

2016-10-14 Thread Mauro Carvalho Chehab
This driver has printk continuation lines for debugging purposes. Since commit 563873318d32 ("Merge branch 'printk-cleanups'"), this won't work as expected anymore. So, let's add KERN_CONT to those lines. Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 15/25] [media] dvb-core: move dvb_filter out of the DVB core

2016-10-14 Thread Mauro Carvalho Chehab
The dvb_filter.c can hardly be considered as part of the DVB core. More than half of the code there is commented out by av7110 and ttusb_dec. On the latter, just two small helper functions and a struct definition is used. Being part of the core means that it would require an amount of work to

[PATCH 05/25] [media] em28xx: use pr_foo instead of em28xx-specific printk macros

2016-10-14 Thread Mauro Carvalho Chehab
There's no reason to keep using em28xx-specific printk macros here. Just use pr_foo(). Signed-off-by: Mauro Carvalho Chehab --- drivers/media/usb/em28xx/em28xx-audio.c | 31 - drivers/media/usb/em28xx/em28xx-camera.c | 34 +-

[PATCH 23/25] [media] cx2341x: mark printk continuation lines as such

2016-10-14 Thread Mauro Carvalho Chehab
This driver has printk continuation lines for debugging purposes. Since commit 563873318d32 ("Merge branch 'printk-cleanups'"), this won't work as expected anymore. So, let's add KERN_CONT to those lines. Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 07/25] [media] dvb-core: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH 17/25] [media] dvb_filter: use KERN_CONT where needed

2016-10-14 Thread Mauro Carvalho Chehab
Some continuation messages are not using KERN_CONT. Since commit 563873318d32 ("Merge branch 'printk-cleanups"), this won't work as expected anymore. So, let's add KERN_CONT to those lines. While here, add missing log level annotations. Signed-off-by: Mauro Carvalho Chehab

[PATCH 14/25] [media] dvb-core: get rid of demux optional circular buffer

2016-10-14 Thread Mauro Carvalho Chehab
There is a provision at the dvb_demux.c to use a vmalloc'ed circular buffer, enabled via an extra #ifdef option that it is not at Kconfig. Enabling it will only make the Kernel to allocate/deallocate such buffer, but no code would actually use it. So, no practical effect, except for sparing some

[PATCH 25/25] [media] nxt6000: use pr_foo() macros instead of printk()

2016-10-14 Thread Mauro Carvalho Chehab
Replace printk() macros by their pr_foo() counterparts and use pr_cont() for the continuation lines. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/dvb-frontends/nxt6000.c | 136 +++--- 1 file changed, 76 insertions(+), 60

[PATCH 08/25] [media] tuner-core: don't break long lines

2016-10-14 Thread Mauro Carvalho Chehab
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: Mauro Carvalho Chehab

[PATCH v2 15/21] ARM: dts: nitrogen6x: Add dtsi for BD_HDMI_MIPI HDMI to MIPI CSI-2 receiver board

2016-10-14 Thread Philipp Zabel
Add device tree nodes for the BD_HDMI_MIPI HDMI to MIPI CSI-2 receiver board with a TC358743 connected to the Nitrogen6X MIPI CSI-2 input connector. Signed-off-by: Philipp Zabel --- .../boot/dts/imx6qdl-nitrogen6x-bd-hdmi-mipi.dtsi | 73 ++ 1 file

[PATCH v2 20/21] [media] imx: Set i.MX MIPI CSI-2 entity function to bridge

2016-10-14 Thread Philipp Zabel
The i.MX6 MIPI CSI2 bridge converts the external MIPI CSI2 input into a SoC internal parallel bus connected to the IPU CSIs via the CSI2IPU gasket. Signed-off-by: Philipp Zabel --- drivers/media/platform/imx/imx-mipi-csi2.c | 1 + 1 file changed, 1 insertion(+) diff

[PATCH v2 19/21] [media] video-multiplexer: set entity function to mux

2016-10-14 Thread Philipp Zabel
Signed-off-by: Philipp Zabel --- drivers/media/platform/video-multiplexer.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/platform/video-multiplexer.c b/drivers/media/platform/video-multiplexer.c index f79b90e..466d9d0 100644 ---

[PATCH v2 21/21] [media] tc358743: set entity function to video interface bridge

2016-10-14 Thread Philipp Zabel
The TC358743 is an HDMI to MIPI CSI2-2 bridge. Signed-off-by: Philipp Zabel --- drivers/media/i2c/tc358743.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/i2c/tc358743.c b/drivers/media/i2c/tc358743.c index 7f82932..259ccf6 100644 ---

[PATCH v2 18/21] [media] add mux and video interface bridge entity functions

2016-10-14 Thread Philipp Zabel
Signed-off-by: Philipp Zabel --- Documentation/media/uapi/mediactl/media-types.rst | 22 ++ include/uapi/linux/media.h| 6 ++ 2 files changed, 28 insertions(+) diff --git a/Documentation/media/uapi/mediactl/media-types.rst

[PATCH v2 16/21] gpu: ipuv3: add ipu_csi_set_downsize

2016-10-14 Thread Philipp Zabel
Support downsizing to 1/2 width and/or height in the CSI. Signed-off-by: Philipp Zabel --- drivers/gpu/ipu-v3/ipu-csi.c | 16 include/video/imx-ipu-v3.h | 1 + 2 files changed, 17 insertions(+) diff --git a/drivers/gpu/ipu-v3/ipu-csi.c

[PATCH v2 14/21] ARM: dts: imx6qdl: Add MIPI CSI-2 D-PHY compatible and clocks

2016-10-14 Thread Philipp Zabel
>From the data sheets it is not quite clear what the clock inputs should be named, but freescale code calls them "dphy_clk" (would that be per?) and "pixel_clk" and connects them to the mipi_core_cfg and emi_podf clocks, respectively. The mipi_core_cfg control is called hsi_tx currently, but it

[PATCH v2 00/21] Basic i.MX IPUv3 capture support

2016-10-14 Thread Philipp Zabel
Hi, the second round removes the prepare_stream callback and instead lets the intermediate subdevices propagate s_stream calls to their sources rather than individually calling s_stream on each subdevice from the bridge driver. This is similar to how drm bridges recursively call into their next

[PATCH v2 05/21] [media] imx: Add i.MX SoC wide media device driver

2016-10-14 Thread Philipp Zabel
This driver registers a single, SoC wide media device, which all entities in the media graph can be registered to. It is probed from device tree using a capture-subsystem node, listing the IPUv3 input ports, similarly to the already existing display-subsystem node that lists the IPUv3 output

[PATCH v2 10/21] [media] imx: Add i.MX MIPI CSI-2 subdevice driver

2016-10-14 Thread Philipp Zabel
Add a v4l2 subdevice driver for the Synopsys DesignWare MIPI CSI-2 host controller on i.MX6. Here its output is connected to the CSI2IPU gasket, which distributes the time division multiplexed virtual channels to the four IPU CSI inputs, directly or through video bus multiplexers. Signed-off-by:

[PATCH v2 08/21] [media] imx: Add i.MX IPUv3 capture driver

2016-10-14 Thread Philipp Zabel
This driver uses the IDMAC module's double buffering feature to do the processing of finished frames in the new frame acknowledge (NFACK) interrupt handler while the next frame is already being captured. This avoids a race condition between the end of frame interrupt and NFACK for very short

  1   2   >