[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 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 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 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 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 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 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 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

[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 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 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 16/57] [media] pt1: 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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

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.

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

[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 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 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 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

Re: [RFC PATCH 00/11] Introduce writeback connectors

2016-10-14 Thread Archit Taneja
Hi Brian, On 10/11/2016 08:23 PM, Brian Starkey wrote: Hi, This RFC series introduces a new connector type: DRM_MODE_CONNECTOR_WRITEBACK It is a follow-on from a previous discussion: [1] Writeback connectors are used to expose the memory writeback engines found in some display controllers,

Re: [PATCH 02/18] [media] RedRat3: Move two assignments in redrat3_transmit_ir()

2016-10-14 Thread Dan Carpenter
I have asked you about six or seven times to only send bug fixes and stop sending clean up patches. You have refused. But now I'm asking you to stop randomly doing things without at least thinking about it for a bit. The original code was correct. regards, dan carpenter -- To unsubscribe from

[patch] [media] st-hva: fix some error handling in hva_hw_probe()

2016-10-14 Thread Dan Carpenter
The devm_ioremap_resource() returns error pointers, never NULL. The platform_get_resource() returns NULL on error, never error pointers. The error code needs to be set, as well. The current code returns PTR_ERR(NULL) which is success. Fixes: 57b2c0628b60 ("[media] st-hva: multi-format video

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

[PATCH 3/5] [media] winbond-cir: Move assignments for three variables in wbcir_shutdown()

2016-10-14 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 14 Oct 2016 10:40:12 +0200 Move the setting for the local variables "mask", "match" and "rc6_csl" behind the source code for a condition check by this function at the beginning. Signed-off-by: Markus Elfring

[PATCH 4/5] [media] winbond-cir: One variable and its check less in wbcir_shutdown() after error detection

2016-10-14 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 14 Oct 2016 12:48:41 +0200 The local variable "do_wake" was set to "false" after an invalid system setting was detected so that a bit of error handling was triggered. * Replace these assignments by direct jumps to the source code

Re: [PATCH v6 1/2] media: i2c/ov5645: add the device tree binding document

2016-10-14 Thread Todor Tomov
Hi Laurent, Thank you for the review. On 09/08/2016 03:22 PM, Laurent Pinchart wrote: > Hi Todor, > > Thank you for the patch. > > On Thursday 08 Sep 2016 12:13:54 Todor Tomov wrote: >> Add the document for ov5645 device tree binding. >> >> Signed-off-by: Todor Tomov

[PATCH 1/5] [media] winbond-cir: Use kmalloc_array() in wbcir_tx()

2016-10-14 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 14 Oct 2016 07:19:00 +0200 A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "kmalloc_array". This issue was detected

[PATCH 5/5] [media] winbond-cir: Move a variable assignment in two functions

2016-10-14 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 14 Oct 2016 13:13:11 +0200 Move the assignment for the local variable "data" behind the source code for condition checks by these functions. Signed-off-by: Markus Elfring ---

[PATCH 2/5] [media] winbond-cir: Move a variable assignment in wbcir_tx()

2016-10-14 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 14 Oct 2016 07:34:46 +0200 Move the assignment for the local variable "data" behind the source code for a memory allocation by this function. Signed-off-by: Markus Elfring ---

Re: [media] RedRat3: Move two assignments in redrat3_transmit_ir()?

2016-10-14 Thread SF Markus Elfring
> The original code was correct. Your view can be appropriate for this function implementation to some degree. I got the impression that it contains the specification of assignments which will happen a bit too early here. Is this a weakness for which software developers can care about? Regards,

[PATCH 2/2] [media] vb2: Add support for use_dma_bidirectional queue flag

2016-10-14 Thread Thierry Escande
From: Pawel Osciak When this flag is set for CAPTURE queues by the driver on calling vb2_queue_init(), it forces the buffers on the queue to be allocated/mapped with DMA_BIDIRECTIONAL direction flag, instead of DMA_FROM_DEVICE. This allows the device not only to write to

[PATCH 1/2] [media] vb2: Store dma_dir in vb2_queue

2016-10-14 Thread Thierry Escande
From: Pawel Osciak Store dma_dir in struct vb2_queue and reuse it, instead of recalculating it each time. Signed-off-by: Pawel Osciak Tested-by: Pawel Osciak Reviewed-by: Tomasz Figa Reviewed-by: Owen Lin

[PATCH 0/2] [media] DMA direction support in vb2_queue

2016-10-14 Thread Thierry Escande
Hi, This series adds a dma_dir field to the vb2_queue structure in order to store the DMA direction once for all in vb2_queue_init(); It also adds a new use_dma_bidirectional flag to the vb2_queue structure allowing the hardware to read from the CAPTURE buffer. This flag is ignored for OUTPUT

[PATCH 0/5] [media] winbond-cir: Fine-tuning for four function implementations

2016-10-14 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 14 Oct 2016 13:24:35 +0200 A few update suggestions were taken into account from static source code analysis. Markus Elfring (5): Use kmalloc_array() in wbcir_tx() Move a variable assignment in wbcir_tx() Move assignments for

Re: [RFC PATCH 00/11] Introduce writeback connectors

2016-10-14 Thread Daniel Vetter
On Fri, Oct 14, 2016 at 2:39 PM, Brian Starkey wrote: >> - Besides the above property, writeback hardware can have provisions >> for scaling, color space conversion and rotation. This would mean that >> we'd eventually add more writeback specific props/params in >>

Re: [PATCH 18/22] [media] imx-ipuv3-csi: support downsizing

2016-10-14 Thread Philipp Zabel
Am Freitag, den 07.10.2016, 21:01 +0200 schrieb Marek Vasut: > On 10/07/2016 06:01 PM, Philipp Zabel wrote: > > Add support for the CSI internal horizontal and vertical downsizing. > > > > Signed-off-by: Philipp Zabel > > --- > >

Re: [RFC PATCH 00/11] Introduce writeback connectors

2016-10-14 Thread Brian Starkey
Hi Archit, On Fri, Oct 14, 2016 at 04:20:14PM +0530, Archit Taneja wrote: Hi Brian, On 10/11/2016 08:23 PM, Brian Starkey wrote: Hi, This RFC series introduces a new connector type: DRM_MODE_CONNECTOR_WRITEBACK It is a follow-on from a previous discussion: [1] Writeback connectors are used

Re: [PATCH 1/2] devicetree/bindings: display: Add bindings for LVDS panels

2016-10-14 Thread Rob Herring
On Sun, Oct 9, 2016 at 11:33 AM, Laurent Pinchart wrote: > Hi Rob, > > On Saturday 08 Oct 2016 20:29:39 Rob Herring wrote: >> On Tue, Oct 04, 2016 at 07:23:29PM +0300, Laurent Pinchart wrote: >> > LVDS is a physical layer specification defined in

Re: [RFC PATCH 00/11] Introduce writeback connectors

2016-10-14 Thread Ville Syrjälä
On Fri, Oct 14, 2016 at 01:39:15PM +0100, Brian Starkey wrote: > Hi Archit, > > On Fri, Oct 14, 2016 at 04:20:14PM +0530, Archit Taneja wrote: > >Hi Brian, > > > >On 10/11/2016 08:23 PM, Brian Starkey wrote: > >>Hi, > >> > >>This RFC series introduces a new connector type: > >>

[PATCH] [media] videobuf2-dma-contig: Support cacheable MMAP

2016-10-14 Thread Thierry Escande
From: Heng-Ruey Hsu DMA allocations for MMAP type are uncached by default. But for some cases, CPU has to access the buffers. ie: memcpy for format converter. Supporting cacheable MMAP improves huge performance. Signed-off-by: Heng-Ruey Hsu

Re: [PATCH 02/22] [media] v4l2-async: allow subdevices to add further subdevices to the notifier waiting list

2016-10-14 Thread Philipp Zabel
Am Donnerstag, den 13.10.2016, 14:56 +0300 schrieb Sakari Ailus: > Hi Philipp, > > On Wed, Oct 12, 2016 at 03:26:48PM +0200, Philipp Zabel wrote: > > On Fri, Oct 7, 2016 at 11:52 PM, Sakari Ailus wrote: > > > Hi Philipp, > > > > > > On Fri, Oct 07, 2016 at 06:00:47PM +0200,

Re: [PATCH 02/22] [media] v4l2-async: allow subdevices to add further subdevices to the notifier waiting list

2016-10-14 Thread Philipp Zabel
Am Samstag, den 08.10.2016, 01:43 +0300 schrieb Sakari Ailus: [...] > > void v4l2_async_notifier_unregister(struct v4l2_async_notifier *notifier) > > { > > struct v4l2_subdev *sd, *tmp; > > - unsigned int notif_n_subdev = notifier->num_subdevs; > > - unsigned int n_subdev =

Re: [PATCH 04/22] [media] v4l2-subdev.h: add prepare_stream op

2016-10-14 Thread Philipp Zabel
Am Samstag, den 08.10.2016, 02:16 +0300 schrieb Sakari Ailus: > Hi Philipp, > > On Fri, Oct 07, 2016 at 06:00:49PM +0200, Philipp Zabel wrote: > > In some cases, for example MIPI CSI-2 input on i.MX6, the sending and > > receiving subdevice need to be prepared in lock-step before the actual > >

Re: [PATCH 12/22] [media] tc358743: put lanes in STOP state before starting streaming

2016-10-14 Thread Philipp Zabel
Am Freitag, den 07.10.2016, 21:02 +0200 schrieb Marek Vasut: > On 10/07/2016 06:00 PM, Philipp Zabel wrote: > > Without calling tc358743_set_csi from the new prepare_stream callback > > (or calling tc358743_s_dv_timings or tc358743_set_fmt from userspace > > after stopping the stream), the i.MX6

Re: [PATCH 03/22] [media] v4l: of: add v4l2_of_subdev_registered

2016-10-14 Thread Philipp Zabel
Am Samstag, den 08.10.2016, 01:50 +0300 schrieb Sakari Ailus: > Hi Philipp, > > On Fri, Oct 07, 2016 at 06:00:48PM +0200, Philipp Zabel wrote: > > Provide a default registered callback for device tree probed subdevices > > that use OF graph bindings to add still missing source subdevices to > >

Re: [PATCH 03/22] [media] v4l: of: add v4l2_of_subdev_registered

2016-10-14 Thread Philipp Zabel
Am Freitag, den 07.10.2016, 20:50 +0200 schrieb Marek Vasut: > On 10/07/2016 06:00 PM, Philipp Zabel wrote: > > Provide a default registered callback for device tree probed subdevices > > that use OF graph bindings to add still missing source subdevices to > > the async notifier waiting list. > >

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

[PATCH v2 03/21] [media] v4l: of: add v4l2_of_subdev_registered

2016-10-14 Thread Philipp Zabel
Provide a default registered callback for device tree probed subdevices that use OF graph bindings to add still missing source subdevices to the async notifier waiting list. This is only necessary for subdevices that have input ports to which other subdevices are connected that are not initially

[PATCH v2 07/21] [media] imx-ipu: Add i.MX IPUv3 CSI subdevice driver

2016-10-14 Thread Philipp Zabel
This adds a V4L2 subdevice driver for the two CMOS Sensor Interface (CSI) modules contained in each IPUv3. These sample video data from the parallel CSI0/1 pads or from the the MIPI CSI-2 bridge via IOMUXC video bus multiplexers and write to IPU internal FIFOs to deliver data to either the IDMAC

[PATCH v2 04/21] [media] v4l2-async: add new subdevices to the tail of subdev_list

2016-10-14 Thread Philipp Zabel
That way the asynchronous notifier will pick them up in the order they were registered. Signed-off-by: Philipp Zabel --- drivers/media/v4l2-core/v4l2-async.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/v4l2-core/v4l2-async.c

[PATCH v2 06/21] [media] imx: Add IPUv3 media common code

2016-10-14 Thread Philipp Zabel
From: Sascha Hauer Add video4linux API routines common to drivers for units that accept or provide video data via the i.MX IPU IDMAC channels, such as capture, mem2mem scaler or deinterlacer drivers. Signed-off-by: Sascha Hauer Signed-off-by:

[PATCH v2 11/21] [media] tc358743: put lanes in STOP state before starting streaming

2016-10-14 Thread Philipp Zabel
Without calling tc358743_set_csi after stopping streaming (or calling tc358743_s_dv_timings or tc358743_set_fmt from userspace after stopping the stream), the i.MX6 MIPI CSI2 input fails waiting for lanes to enter STOP state when streaming is started again. Signed-off-by: Philipp Zabel

[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

[PATCH v2 12/21] ARM: dts: imx6qdl: Add capture-subsystem node

2016-10-14 Thread Philipp Zabel
Signed-off-by: Philipp Zabel --- arch/arm/boot/dts/imx6dl.dtsi | 5 + arch/arm/boot/dts/imx6q.dtsi | 5 + 2 files changed, 10 insertions(+) diff --git a/arch/arm/boot/dts/imx6dl.dtsi b/arch/arm/boot/dts/imx6dl.dtsi index 9a4c22c..3c817de 100644 ---

[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 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

  1   2   >