Re: cron job: media_tree daily build: ERRORS

2016-05-02 Thread Hans Verkuil
On 05/03/2016 05:05 AM, Hans Verkuil wrote: > 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: Tue May 3 04:00:18 CEST 2016 > git branch: test > git hash:

cron job: media_tree daily build: ERRORS

2016-05-02 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: Tue May 3 04:00:18 CEST 2016 git branch: test git hash: 68af062b5f38510dc96635314461c6bbe1dbf2fe gcc versi

Re: [PATCH v2] s5p-mfc: Don't try to put pm->clock if lookup failed

2016-05-02 Thread Arnd Bergmann
> > This leads to the following warning in clk_put() due s5p_mfc_final_pm(): > > WARNING: CPU: 3 PID: 1023 at drivers/clk/clk.c:2814 > s5p_mfc_final_pm+0x48/0x74 [s5p_mfc] > CPU: 3 PID: 1023 Comm: rmmod Tainted: G W > 4.6.0-rc6-next-20160502-5-g5a15a49106bc #9 &

Re: [PATCH v7 16/24] i2c: allow adapter drivers to override the adapter locking

2016-05-02 Thread Antti Palosaari
On 04/29/2016 12:16 PM, Peter Rosin wrote: On 2016-04-29 09:16, Wolfram Sang wrote: Yes, obviously... I'll make that change locally and wait for the rest. Another nit: You could use '--strict' with checkpatch and see if you want to fix the issues reported. I am not keen on those (except for 'sp

[PATCH v2] s5p-mfc: Don't try to put pm->clock if lookup failed

2016-05-02 Thread Javier Martinez Canillas
: CPU: 3 PID: 1023 at drivers/clk/clk.c:2814 s5p_mfc_final_pm+0x48/0x74 [s5p_mfc] CPU: 3 PID: 1023 Comm: rmmod Tainted: GW 4.6.0-rc6-next-20160502-5-g5a15a49106bc #9 Hardware name: SAMSUNG EXYNOS (Flattened Device Tree) [] (unwind_backtrace) from [] (show_stack+0x10/0x14

Re: [PATCH] s5p-mfc: Don't try to put pm->clock if lookup failed

2016-05-02 Thread Javier Martinez Canillas
Hello Arnd, Thanks for your feedback. On 05/02/2016 02:41 PM, Arnd Bergmann wrote: > On Monday 02 May 2016 13:27:54 Javier Martinez Canillas wrote: >> Failing to get the struct s5p_mfc_pm .clock is a non-fatal error so the >> clock field can have a errno pointer value. But s5p_mfc_final_pm() only

Re: [PATCH] s5p-mfc: Don't try to put pm->clock if lookup failed

2016-05-02 Thread Arnd Bergmann
On Monday 02 May 2016 13:27:54 Javier Martinez Canillas wrote: > Failing to get the struct s5p_mfc_pm .clock is a non-fatal error so the > clock field can have a errno pointer value. But s5p_mfc_final_pm() only > checks if .clock is not NULL before attempting to unprepare and put it. > > This lead

[PATCH] s5p-mfc: Don't try to put pm->clock if lookup failed

2016-05-02 Thread Javier Martinez Canillas
: CPU: 3 PID: 1023 at drivers/clk/clk.c:2814 s5p_mfc_final_pm+0x48/0x74 [s5p_mfc] CPU: 3 PID: 1023 Comm: rmmod Tainted: GW 4.6.0-rc6-next-20160502-5-g5a15a49106bc #9 Hardware name: SAMSUNG EXYNOS (Flattened Device Tree) [] (unwind_backtrace) from [] (show_stack+0x10/0x14

Re: [PATCH] [media] em28xx_dvb: add support for PLEX PX-BCUD (ISDB-S usb dongle)

2016-05-02 Thread Satoshi Nagahama
Hi Akihiro, Thank you for your review and comments. I did two changes as you commented. > * The names of _REG_ROWS / reg_index might be a bit vague to others. > I would prefer _CHIP_IDS / chip_id or something like that. > > * reg_index should not be static as it is per device property. > In

Re: [PATCH 2/2] media: s3c-camif: fix deadlock on driver probe()

2016-05-02 Thread Hans Verkuil
On 04/28/16 12:25, Marek Szyprowski wrote: > Commit 0c426c472b5585ed6e59160359c979506d45ae49 ("[media] media: Always > keep a graph walk large enough around") changed > media_device_register_entity() function to take mdev->graph_mutex. This > causes deadlock in driver probe, which calls (indirectly

Re: [PATCH v3] media: vb2-dma-contig: configure DMA max segment size properly

2016-05-02 Thread Hans Verkuil
On 05/02/16 12:59, Marek Szyprowski wrote: > This patch lets vb2-dma-contig memory allocator to configure DMA max > segment size properly for the client device. Setting it is needed to let > DMA-mapping subsystem to create a single, contiguous mapping in DMA > address space. This is essential for a

Re: [PATCH v3] media: vb2-dma-contig: configure DMA max segment size properly

2016-05-02 Thread Sakari Ailus
On Mon, May 02, 2016 at 12:59:13PM +0200, Marek Szyprowski wrote: > This patch lets vb2-dma-contig memory allocator to configure DMA max > segment size properly for the client device. Setting it is needed to let > DMA-mapping subsystem to create a single, contiguous mapping in DMA > address space.

[PATCH 2/2] uvc: correct speed testing

2016-05-02 Thread Oliver Neukum
Allow for SS+ USB devices Signed-off-by: Oliver Neukum --- drivers/media/usb/uvc/uvc_video.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/usb/uvc/uvc_video.c b/drivers/media/usb/uvc/uvc_video.c index 075a0fe..b5589d5 100644 --- a/drivers/media/usb/uvc/uvc_video.c +++ b/driv

[PATCH 1/2] gspca: correct speed testing

2016-05-02 Thread Oliver Neukum
Allow for SS+ devices Signed-off-by: Oliver Neukum --- drivers/media/usb/gspca/gspca.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/usb/gspca/gspca.c b/drivers/media/usb/gspca/gspca.c index af5cd82..69d56f3 100644 --- a/drivers/media/usb/gspca/gspca.c +++ b/d

Re: [PATCH 1/2] media: exynos4-is: fix deadlock on driver probe

2016-05-02 Thread Hans Verkuil
On 05/02/16 12:52, Mauro Carvalho Chehab wrote: > Em Mon, 2 May 2016 00:59:56 +0300 > Sakari Ailus escreveu: > >> Hi Marek, >> >> Marek Szyprowski wrote: >>> Commit 0c426c472b5585ed6e59160359c979506d45ae49 ("[media] media: Always >>> keep a graph walk large enough around") changed >>> media_devic

[PATCH v3] media: vb2-dma-contig: configure DMA max segment size properly

2016-05-02 Thread Marek Szyprowski
This patch lets vb2-dma-contig memory allocator to configure DMA max segment size properly for the client device. Setting it is needed to let DMA-mapping subsystem to create a single, contiguous mapping in DMA address space. This is essential for all devices, which use dma-contig videobuf2 memory a

Re: [PATCH 1/2] media: exynos4-is: fix deadlock on driver probe

2016-05-02 Thread Mauro Carvalho Chehab
Em Mon, 2 May 2016 00:59:56 +0300 Sakari Ailus escreveu: > Hi Marek, > > Marek Szyprowski wrote: > > Commit 0c426c472b5585ed6e59160359c979506d45ae49 ("[media] media: Always > > keep a graph walk large enough around") changed > > media_device_register_entity() function to take mdev->graph_mutex.

Re: [PATCH] media: fix use-after-free in cdev_put() when app exits after driver unbind

2016-05-02 Thread Lars-Peter Clausen
On 04/30/2016 12:37 AM, Shuah Khan wrote: [...] > diff --git a/include/media/media-devnode.h b/include/media/media-devnode.h > index 5bb3b0e..ce9b051 100644 > --- a/include/media/media-devnode.h > +++ b/include/media/media-devnode.h > @@ -72,6 +72,7 @@ struct media_file_operations { > * @fops:

Re: [PATCH v2] media: vb2-dma-contig: configure DMA max segment size properly

2016-05-02 Thread Hans Verkuil
On 04/29/16 15:56, Sakari Ailus wrote: > Hi Marek, > > On Fri, Apr 29, 2016 at 01:39:43PM +0200, Marek Szyprowski wrote: >> Hi Sakari, >> >> >> On 2016-04-29 13:21, Sakari Ailus wrote: >>> Hi Marek, >>> >>> Thanks for the patch! >>> >>> On Thu, Apr 28, 2016 at 03:20:03PM +0200, Marek Szyprowski wr

[PATCH] v4l2-dv-timings.h: fix polarity for 4k formats

2016-05-02 Thread Hans Verkuil
Backport to 3.16-4.0 of mainline commit 3020ca711871fdaf0c15c8bab677a6bc302e28fe The VSync polarity was negative instead of positive for the 4k CEA formats. I probably copy-and-pasted these from the DMT 4k format, which does have a negative VSync polarity. Signed-off-by: Hans Verkuil Reported-by

Re: [RFC PATCH 00/24] Make Nokia N900 cameras working

2016-05-02 Thread Pavel Machek
Hi! > On Wed, Apr 27, 2016 at 06:45:29PM +0200, Pavel Machek wrote: > > > I don't have pre-production N900s. The phone I use for development > > > is HW revision 2101 with Finish keyboard layout. Apart from that > > > I have my productive phone, which is rev 2204 with German layout. > > > > How d