Re: [PATCH 1/6] Use LUT based implementation for (co)sine functions

2014-12-16 Thread Mauro Carvalho Chehab
Em Tue, 16 Dec 2014 06:41:18 + Prashant Laddha (prladdha) prlad...@cisco.com escreveu: Antti, Mauro, Thanks for your comments. On 15/12/14 7:00 pm, Antti Palosaari cr...@iki.fi wrote: On 12/15/2014 03:13 PM, Mauro Carvalho Chehab wrote: Em Mon, 15 Dec 2014 14:49:17 +0530

Re: [PATCH] [media] vb2-vmalloc: Protect DMA-specific code by #ifdef CONFIG_HAS_DMA

2014-12-16 Thread Marek Szyprowski
Hello, On 2014-12-15 14:40, Geert Uytterhoeven wrote: If NO_DMA=y: drivers/built-in.o: In function `vb2_vmalloc_dmabuf_ops_detach': videobuf2-vmalloc.c:(.text+0x6f11b0): undefined reference to `dma_unmap_sg' drivers/built-in.o: In function `vb2_vmalloc_dmabuf_ops_map':

[PATCH 2/2] s5p-mfc: use VB2_FILEIO_ALLOW_ZERO_BYTESUSED flag

2014-12-16 Thread Kamil Debski
The MFC driver interprets a buffer with bytesused equal to 0 as a special case indicating end-of-stream. After vb2: fix bytesused == 0 handling (8a75ffb) patch videobuf2 modified the value of bytesused if it was 0. The VB2_FILEIO_ALLOW_ZERO_BYTESUSED flag was added to videobuf2 to keep backward

[PATCH 1/2] vb2: Add VB2_FILEIO_ALLOW_ZERO_BYTESUSED flag to vb2_fileio_flags

2014-12-16 Thread Kamil Debski
The vb2: fix bytesused == 0 handling (8a75ffb) patch changed the behavior of __fill_vb2_buffer function, so that if bytesused is 0 it is set to the size of the buffer. However, bytesused set to 0 is used by older codec drivers as as indication used to mark the end of stream. To keep backward

Re: [PATCH 1/6] Use LUT based implementation for (co)sine functions

2014-12-16 Thread Mauro Carvalho Chehab
Em Tue, 16 Dec 2014 08:45:53 -0200 Mauro Carvalho Chehab mche...@osg.samsung.com escreveu: Em Tue, 16 Dec 2014 06:41:18 + Prashant Laddha (prladdha) prlad...@cisco.com escreveu: Antti, Mauro, Thanks for your comments. On 15/12/14 7:00 pm, Antti Palosaari cr...@iki.fi wrote:

Re: [RFC] video support for Samsung SUR40

2014-12-16 Thread Florian Echtler
On 15.12.2014 23:34, Hans Verkuil wrote: On 12/15/2014 11:15 PM, Florian Echtler wrote: On 15.12.2014 17:01, Hans Verkuil wrote: On 12/15/2014 04:47 PM, Florian Echtler wrote: Why on earth is sur40_poll doing anything with video buffers? That's all handled by vb2. As far as I can tell you can

Re: Re: Fwd: Re: Re: Problems with Linux drivers (Debian Jessie kernel3.16.0-4)

2014-12-16 Thread Christopher Scheuring
Hello Max, sorry for the delayed answer - I'm currently busy. About your question where I have the issues: It concerns to all transponders using dvb-s or dvb-s2. Currently I have no chance for testing. Around Christmas I'll have some more time for testing. I'll check out the current drivers

Re: [RFC] video support for Samsung SUR40

2014-12-16 Thread Hans Verkuil
Hi Florian, On 12/16/14 13:35, Florian Echtler wrote: On 15.12.2014 23:34, Hans Verkuil wrote: On 12/15/2014 11:15 PM, Florian Echtler wrote: On 15.12.2014 17:01, Hans Verkuil wrote: On 12/15/2014 04:47 PM, Florian Echtler wrote: Why on earth is sur40_poll doing anything with video buffers?

Re: [RFC PATCH 1/8] v4l2 subdevs: replace get/set_crop by get/set_selection

2014-12-16 Thread Prabhakar Lad
On Thu, Dec 4, 2014 at 3:24 PM, Hans Verkuil hverk...@xs4all.nl wrote: From: Hans Verkuil hans.verk...@cisco.com The crop and selection pad ops are duplicates. Replace all uses of get/set_crop by get/set_selection. This will make it possible to drop get/set_crop altogether. Signed-off-by:

Re: [RFC PATCH 2/8] v4l2-subdev: drop get/set_crop pad ops

2014-12-16 Thread Prabhakar Lad
On Thu, Dec 4, 2014 at 3:24 PM, Hans Verkuil hverk...@xs4all.nl wrote: From: Hans Verkuil hans.verk...@cisco.com Drop the duplicate get/set_crop pad ops and only use get/set_selection. It makes no sense to have two duplicate ops in the internal subdev API. Signed-off-by: Hans Verkuil

Re: [RFC PATCH 3/8] v4l2-subdev: drop unused op enum_mbus_fmt

2014-12-16 Thread Prabhakar Lad
On Thu, Dec 4, 2014 at 3:24 PM, Hans Verkuil hverk...@xs4all.nl wrote: From: Hans Verkuil hans.verk...@cisco.com Weird, this op isn't used at all. Seems to be orphaned code. Remove it. Signed-off-by: Hans Verkuil hans.verk...@cisco.com Acked-by: Lad, Prabhakar prabhakar.cse...@gmail.com

[RFC PATCH] fixp-arith: replace sin/cos table by a better precision one

2014-12-16 Thread Mauro Carvalho Chehab
The cos table used at fixp-arith.h has only 8 bits of precision. That causes problems if it is reused on other drivers. As some media drivers require a higher precision sin/cos implementation, replace the current implementation by one that will provide 32 bits precision. The values generated by

[PATCH 2/2] [media] adv7180: Remove the unneeded 'err' label

2014-12-16 Thread Fabio Estevam
There is no need to jump to the 'err' label as we can simply return the error code directly and make the code shorter. Signed-off-by: Fabio Estevam fabio.este...@freescale.com --- drivers/media/i2c/adv7180.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git

[PATCH 1/2] [media] adv7180: Simplify PM hooks

2014-12-16 Thread Fabio Estevam
The macro SIMPLE_DEV_PM_OPS already takes care of the CONFIG_PM_SLEEP=n case, so move it out of the CONFIG_PM_SLEEP 'if' block and remove the unneeded ADV7180_PM_OPS definition to make the code simpler. Signed-off-by: Fabio Estevam fabio.este...@freescale.com --- drivers/media/i2c/adv7180.c | 9

Re: Instalation issue on S960

2014-12-16 Thread Carlos Diogo
Hi , thanks for the inputs. Some more details on the issue. When i try to build the drivers the m88ds3103 driver is not built (no .o /.ko file generated) but also no error shows up. For whatever reason the make process skips this drivers. I enabled the i2c and re-started the build process which

Re: [PATCH 1/2] [media] adv7180: Simplify PM hooks

2014-12-16 Thread Lars-Peter Clausen
On 12/16/2014 05:49 PM, Fabio Estevam wrote: The macro SIMPLE_DEV_PM_OPS already takes care of the CONFIG_PM_SLEEP=n case. I guess that's kind of debatable. The purpose of ifdef-ing stuff out is to decrease the driver size if PM support is disabled. With this change you are adding a rather

[PATCH] media: au0828 VBI support comment cleanup

2014-12-16 Thread Shuah Khan
This driver supports VBI and the comment VBI support is not yet working is inaccurate. Remove it. Signed-off-by: Shuah Khan shua...@osg.samsung.com --- drivers/media/usb/au0828/au0828-video.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/usb/au0828/au0828-video.c

[PATCH v2] mn88472: implement lock for all delivery systems

2014-12-16 Thread Benjamin Larsson
The increase of the lock timeout is needed for dvb-t2. Signed-off-by: Benjamin Larsson benja...@southpole.se --- drivers/staging/media/mn88472/mn88472.c | 24 1 file changed, 20 insertions(+), 4 deletions(-) diff --git a/drivers/staging/media/mn88472/mn88472.c

Re: [PATCH] mn88472: implement lock for all delivery systems

2014-12-16 Thread Benjamin Larsson
Hello. On 12/16/2014 04:34 AM, Antti Palosaari wrote: Moikka! On 12/16/2014 01:40 AM, Benjamin Larsson wrote: Signed-off-by: Benjamin Larsson benja...@southpole.se --- drivers/staging/media/mn88472/mn88472.c | 23 --- 1 file changed, 20 insertions(+), 3 deletions(-)

[linuxtv-media:master 7740/7741] drivers/staging/media/tlg2300/pd-main.c:81 send_set_req() error: doing dma on the stack (data)

2014-12-16 Thread kbuild test robot
tree: git://linuxtv.org/media_tree.git master head: 427ae153c65ad7a08288d86baf99000569627d03 commit: ea2e813e8cc3492c951b9895724fd47187e04a6f [7740/7741] [media] tlg2300: move to staging in preparation for removal drivers/staging/media/tlg2300/pd-main.c:81 send_set_req() error: doing dma on

cron job: media_tree daily build: ERRORS

2014-12-16 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: Wed Dec 17 04:00:20 CET 2014 git branch: test git hash: 427ae153c65ad7a08288d86baf99000569627d03 gcc

Re: [PATCH v2] mn88472: implement lock for all delivery systems

2014-12-16 Thread Antti Palosaari
On 12/17/2014 02:16 AM, Benjamin Larsson wrote: The increase of the lock timeout is needed for dvb-t2. Signed-off-by: Benjamin Larsson benja...@southpole.se Reviewed-by: Antti Palosaari cr...@iki.fi --- drivers/staging/media/mn88472/mn88472.c | 24 1 file

[PATCH] [media] s5p-jpeg: Clear JPEG_CODEC_ON bits in sw reset function

2014-12-16 Thread Tony K Nadackal
Bits EXYNOS4_DEC_MODE and EXYNOS4_ENC_MODE do not get cleared on software reset. These bits need to be cleared explicitly. Signed-off-by: Tony K Nadackal tony...@samsung.com --- This patch is created and tested on top of linux-next-20141210. It can be cleanly applied on media-next and

[PATCH] [media] s5p-jpeg: Fix crash in jpeg isr due to multiple interrupts.

2014-12-16 Thread Tony K Nadackal
In case of corrupt images, multiple interrupts may occur due to different error scenarios. Since we are removing the src and dest buffers in the first interrupt itself, crash occurs in the second error interrupts. Disable the global interrupt before we start processing the interrupt avoid the

Re: [RFC PATCH 0/8] Removing duplicate video/pad ops

2014-12-16 Thread Prabhakar Lad
On Thu, Dec 4, 2014 at 3:24 PM, Hans Verkuil hverk...@xs4all.nl wrote: This patch series attempts to remove some of the duplicate video/pad ops. The first two patches have been posted before. The only thing changed is that the subdevs no longer add checks for pad values != 0 as suggested. The

[PATCH] [media] s5p-jpeg: Initialize cb and cr to zero.

2014-12-16 Thread Tony K Nadackal
To avoid garbage value written into image base address planes, initialize cb and cr of structure s5p_jpeg_addr to zero. Signed-off-by: Tony K Nadackal tony...@samsung.com --- This patch is created and tested on top of linux-next-20141210. It can be cleanly applied on media-next and

[PATCH] [media] s5p-jpeg: Adding Exynos7 Jpeg variant support

2014-12-16 Thread Tony K Nadackal
Fimp_jpeg used in Exynos7 is a revised version. Some register configurations are slightly different from Jpeg in Exynos4. Added one more variant SJPEG_EXYNOS7 to handle these differences. Signed-off-by: Tony K Nadackal tony...@samsung.com --- This patch is created and tested on top of