Re: [PATCH][RESEND 5/8] uio: uio_pruss: use gen_pool_dma_alloc() to allocate sram memory

2013-12-09 Thread Greg KH
On Fri, Nov 01, 2013 at 07:48:18PM +0800, Nicolin Chen wrote: Since gen_pool_dma_alloc() is introduced, we implement it to simplify code. Signed-off-by: Nicolin Chen b42...@freescale.com --- Acked-by: Greg Kroah-Hartman gre...@linuxfoundation.org -- To unsubscribe from this list: send the

[GIT PULL FOR v3.13] vb2: regression fix: always set length field.

2013-12-09 Thread Hans Verkuil
Mauro, Please queue this regression fix for 3.13. Regards, Hans The following changes since commit 3f823e094b935c1882605f8720336ee23433a16d: [media] exynos4-is: Simplify fimc-is hardware polling helpers (2013-12-04 15:54:19 -0200) are available in the git repository at:

[GIT PULL FOR v3.14] New si4713-usb driver

2013-12-09 Thread Hans Verkuil
Hi Mauro, I'm trying again to get this driver merged. The comments you made have all been addressed, so hopefully this will now be OK. Regards, Hans The following changes since commit 3f823e094b935c1882605f8720336ee23433a16d: [media] exynos4-is: Simplify fimc-is hardware polling

[PATCH][RESEND] [media] davinci: vpfe: remove deprecated IRQF_DISABLED

2013-12-09 Thread Michael Opdenacker
This patch proposes to remove the use of the IRQF_DISABLED flag It's a NOOP since 2.6.35 and it will be removed one day. Signed-off-by: Michael Opdenacker michael.opdenac...@free-electrons.com Acked-by: Lad, Prabhakar prabhakar.cse...@gmail.com ---

Re: Can't open mfc v5 encode but decode can

2013-12-09 Thread randy
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 ? 2013?12?05? 17:36, Kamil Debski ??: Hi Randy, -Original Message- From: randy [mailto:lxr1...@hotmail.com] Sent: Thursday, December 05, 2013 1:27 AM To: linux-media@vger.kernel.org Cc: Kamil Debski; m.szyprow...@samsung.com;

[GIT PULL FOR v3.13] vb2: regression fixes

2013-12-09 Thread Hans Verkuil
Mauro, Please queue these regression fixes for 3.13. This supersedes my previous earlier 3.13 pull request of today, since I realized that the 'fix possible memory leak' patch should also go to 3.13. Regards, Hans The following changes since commit

[GIT PULL FOR v3.14] vb2 fixes

2013-12-09 Thread Hans Verkuil
Various vb2 fixes. I've asked Marek if I could take care of these since I have more vb2 patches pending. Regards, Hans The following changes since commit 3f823e094b935c1882605f8720336ee23433a16d: [media] exynos4-is: Simplify fimc-is hardware polling helpers (2013-12-04 15:54:19

Re: vb2: various cleanups and improvements

2013-12-09 Thread Laurent Pinchart
Hi Hans, Thank you for the patches. For patches 1, 2, 4, 7 and 10, Acked-by: Laurent Pinchart laurent.pinch...@ideasonboard.com On Thursday 05 December 2013 09:21:39 Hans Verkuil wrote: This patch series does some cleanups in the qbuf/prepare_buf handling (the first two patches). The third

Re: omap3isp device tree support

2013-12-09 Thread Laurent Pinchart
Hi Enrico, On Friday 06 December 2013 11:13:50 Enrico wrote: Hi, i know there is some work going on for omap3isp device tree support, but right now is it possible to enable it in some other way in a DT kernel? I've tried enabling it in board-generic.c (omap3_init_camera(...) with proper

Any development for Hauppauge HVR 5500-HD started?

2013-12-09 Thread Sven Müller
Hi! Is there anyone, who has started driver development for this card? I would like to help testing. Best wishes. -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at

[PATCH] CHROMIUM: s5p-mfc: add controls to set vp8 enc profile

2013-12-09 Thread Arun Kumar K
Add v4l2 controls to set desired profile for VP8 encoder. Acceptable levels for VP8 encoder are 0: Version 0 1: Version 1 2: Version 2 3: Version 3 Signed-off-by: Pawel Osciak posc...@chromium.org Signed-off-by: Kiran AVND avnd.ki...@samsung.com Signed-off-by: Arun Kumar K arun...@samsung.com ---

Re: [PATCH] CHROMIUM: s5p-mfc: add controls to set vp8 enc profile

2013-12-09 Thread Hans Verkuil
Hi Arun, Some comments below... On 12/09/2013 02:16 PM, Arun Kumar K wrote: Add v4l2 controls to set desired profile for VP8 encoder. Acceptable levels for VP8 encoder are 0: Version 0 1: Version 1 2: Version 2 3: Version 3 Signed-off-by: Pawel Osciak posc...@chromium.org

[RFCv4 PATCH 0/8] vb2: various cleanups and improvements

2013-12-09 Thread Hans Verkuil
This patch series does some cleanups in the qbuf/prepare_buf handling (the first three patches). The fourth patch removes the 'fileio = NULL' hack. That hack no longer works when dealing with asynchronous calls from a kernel thread so it had to be fixed. The next three patches implement retrying

[RFCv4 PATCH 8/8] vb2: Improve file I/O emulation to handle buffers in any order

2013-12-09 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com videobuf2 file I/O emulation assumed that buffers dequeued from the driver would return in the order they were enqueued in the driver. Improve the file I/O emulator's book-keeping to remove this assumption. Also set the buf-size properly if a write()

[RFCv4 PATCH 2/8] vb2: simplify qbuf/prepare_buf by removing callback.

2013-12-09 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com The callback used to merge the common code of the qbuf/prepare_buf code can be removed now that the mmap_sem handling is pushed down to __buf_prepare(). This makes the code more readable. Signed-off-by: Hans Verkuil hans.verk...@cisco.com Acked-by:

[RFCv4 PATCH 3/8] vb2: fix race condition between REQBUFS and QBUF/PREPARE_BUF.

2013-12-09 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com When preparing a buffer the queue lock is released for a short while if the memory mode is USERPTR (see __buf_prepare for the details), which would allow a race with a REQBUFS which can free the buffers. Removing the buffers from underneath __buf_prepare

[RFCv4 PATCH 4/8] vb2: remove the 'fileio = NULL' hack.

2013-12-09 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com The read/write implementation in vb2 reuses existing vb2 functions, but it sets q-fileio to NULL before calling them in order to skip the 'q-fileio != NULL' check. This works today due to the synchronous nature of read/write, but it 1) is ugly, and 2)

[RFCv4 PATCH 7/8] vb2: return ENODATA in start_streaming in case of too few buffers.

2013-12-09 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com This works together with the retry_start_streaming mechanism to allow userspace to start streaming even if not all required buffers have been queued. Signed-off-by: Hans Verkuil hans.verk...@cisco.com Cc: Lad, Prabhakar prabhakar.cse...@gmail.com Cc:

[RFCv4 PATCH 1/8] vb2: push the mmap semaphore down to __buf_prepare()

2013-12-09 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com Rather than taking the mmap semaphore at a relatively high-level function, push it down to the place where it is really needed. It was placed in vb2_queue_or_prepare_buf() to prevent racing with other vb2 calls. The only way I can see that a race can

[RFCv4 PATCH 6/8] vb2: don't set index, don't start streaming for write()

2013-12-09 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com Two fixes: - there is no need to set the index when calling dqbuf: dqbuf will overwrite it. - __vb2_init_fileio already starts streaming for write(), so there is no need to do it again in __vb2_perform_fileio. It can never have worked anyway:

[RFCv4 PATCH 5/8] vb2: retry start_streaming in case of insufficient buffers.

2013-12-09 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com If start_streaming returns -ENODATA, then it will be retried the next time a buffer is queued. This means applications no longer need to know how many buffers need to be queued before STREAMON can be called. This is particularly useful for output stream

Re: [GIT PULL] git://linuxtv.org/mkrufky/dvb cx24117

2013-12-09 Thread Mauro Carvalho Chehab
Em Sun, 8 Dec 2013 20:18:04 -0500 Michael Krufky mkru...@linuxtv.org escreveu: Mauro, What is the status of this pull request? Patchwork says changes requested but I have no record of any changes requested https://patchwork.linuxtv.org/patch/20728/ Thanks for pinging about that. I

Re: [PATCHv2 06/11] si4713: Added the USB driver for Si4713

2013-12-09 Thread Mauro Carvalho Chehab
Em Fri, 6 Dec 2013 11:17:09 +0100 Hans Verkuil hverk...@xs4all.nl escreveu: From: Dinesh Ram dinesh@cern.ch This is the USB driver for the Silicon Labs development board. It contains the Si4713 FM transmitter chip. Signed-off-by: Dinesh Ram dinesh@cern.ch Signed-off-by: Hans

Re: [PATCHv2 11/11] si4713: coding style cleanups

2013-12-09 Thread Mauro Carvalho Chehab
Em Fri, 6 Dec 2013 11:17:14 +0100 Hans Verkuil hverk...@xs4all.nl escreveu: From: Hans Verkuil hans.verk...@cisco.com Fix most checkpatch errors/warnings. It's mostly whitespace changes, except for replacing msleep with usleep_range and the jiffies comparison with

Re: [PATCHv2 02/11] si4713: Modified i2c driver to handle cases where interrupts are not used

2013-12-09 Thread Mauro Carvalho Chehab
Em Fri, 6 Dec 2013 11:17:05 +0100 Hans Verkuil hverk...@xs4all.nl escreveu: From: Dinesh Ram dinesh@cern.ch Checks have been introduced at several places in the code to test if an interrupt is set or not. For devices which do not use the interrupt, to get a valid response, within a

Re: [PATCHv2 00/11] si4713: add si4713 usb driver

2013-12-09 Thread Mauro Carvalho Chehab
Em Fri, 6 Dec 2013 11:17:03 +0100 Hans Verkuil hverk...@xs4all.nl escreveu: This is the second version of this patch series from Dinesh: http://www.spinics.net/lists/linux-media/msg68927.html It's identical to the first version except for two new patches: patch 10/11 prints the product

Re: [PATCHv2 06/11] si4713: Added the USB driver for Si4713

2013-12-09 Thread Hans Verkuil
On 12/09/2013 04:47 PM, Mauro Carvalho Chehab wrote: Em Fri, 6 Dec 2013 11:17:09 +0100 Hans Verkuil hverk...@xs4all.nl escreveu: From: Dinesh Ram dinesh@cern.ch This is the USB driver for the Silicon Labs development board. It contains the Si4713 FM transmitter chip. Signed-off-by:

Re: [PATCHv2 11/11] si4713: coding style cleanups

2013-12-09 Thread Hans Verkuil
On 12/09/2013 04:51 PM, Mauro Carvalho Chehab wrote: Em Fri, 6 Dec 2013 11:17:14 +0100 Hans Verkuil hverk...@xs4all.nl escreveu: From: Hans Verkuil hans.verk...@cisco.com Fix most checkpatch errors/warnings. It's mostly whitespace changes, except for replacing msleep with usleep_range

Re: [PATCHv2 11/11] si4713: coding style cleanups

2013-12-09 Thread Mauro Carvalho Chehab
Em Mon, 09 Dec 2013 17:17:41 +0100 Hans Verkuil hverk...@xs4all.nl escreveu: On 12/09/2013 04:51 PM, Mauro Carvalho Chehab wrote: Em Fri, 6 Dec 2013 11:17:14 +0100 Hans Verkuil hverk...@xs4all.nl escreveu: From: Hans Verkuil hans.verk...@cisco.com Fix most checkpatch

[PATCH] radio-bcm2048: fix signal of value

2013-12-09 Thread Mauro Carvalho Chehab
As value can be initialized with a value lower than zero, change it to int, to avoid those warnings: drivers/staging/media/bcm2048/radio-bcm2048.c: In function 'bcm2048_rds_pi_read': drivers/staging/media/bcm2048/radio-bcm2048.c:1989:9: warning: comparison of unsigned expression = 0 is always

Re: SDR sampling rate - control or IOCTL?

2013-12-09 Thread Antti Palosaari
On 21.11.2013 23:19, Antti Palosaari wrote: On 21.11.2013 22:54, Mauro Carvalho Chehab wrote: Em Thu, 21 Nov 2013 22:22:49 +0200 Antti Palosaari cr...@iki.fi escreveu: On 21.11.2013 21:12, Mauro Carvalho Chehab wrote: Em Thu, 21 Nov 2013 20:33:15 +0200 Antti Palosaari cr...@iki.fi escreveu:

Re: [PATCH] CHROMIUM: s5p-mfc: add controls to set vp8 enc profile

2013-12-09 Thread Pawel Osciak
Hi Arun, On Mon, Dec 9, 2013 at 10:16 PM, Arun Kumar K arun...@samsung.com wrote: Add v4l2 controls to set desired profile for VP8 encoder. Acceptable levels for VP8 encoder are 0: Version 0 1: Version 1 2: Version 2 3: Version 3 Signed-off-by: Pawel Osciak posc...@chromium.org Sorry,

ATTENTION:

2013-12-09 Thread Membership Authentification Form
ATTENTION: Monday 9th of December 2013 08:16:22 MYT from 41.206.151.183, your account was recently accessed with this details, please if you recognize this details,ignore this message, or use this web link to reconfirm your account details to prevent spammers and unauthorized users to gain

[PATCH] omap3isp: Fix buffer flags handling when querying buffer

2013-12-09 Thread Laurent Pinchart
A missing break resulted in all done buffers being flagged with V4L2_BUF_FLAG_QUEUED. Fix it. Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com --- drivers/media/platform/omap3isp/ispqueue.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

cron job: media_tree daily build: ERRORS

2013-12-09 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 Dec 10 04:00:29 CET 2013 git branch: test git hash: 356ba0213dca4dd68906bb315f8c7a46e81510ea gcc

Re: [PATCH] CHROMIUM: s5p-mfc: add controls to set vp8 enc profile

2013-12-09 Thread Arun Kumar K
Hi Hans, Thanks for the review. On Mon, Dec 9, 2013 at 6:52 PM, Hans Verkuil hverk...@xs4all.nl wrote: Hi Arun, Some comments below... On 12/09/2013 02:16 PM, Arun Kumar K wrote: Add v4l2 controls to set desired profile for VP8 encoder. Acceptable levels for VP8 encoder are 0: Version 0

Re: [PATCH] CHROMIUM: s5p-mfc: add controls to set vp8 enc profile

2013-12-09 Thread Arun Kumar K
Hi Pawel, On Tue, Dec 10, 2013 at 6:21 AM, Pawel Osciak posc...@chromium.org wrote: Hi Arun, On Mon, Dec 9, 2013 at 10:16 PM, Arun Kumar K arun...@samsung.com wrote: Add v4l2 controls to set desired profile for VP8 encoder. Acceptable levels for VP8 encoder are 0: Version 0 1: Version 1

Re: [PATCH] CHROMIUM: s5p-mfc: add controls to set vp8 enc profile

2013-12-09 Thread Sachin Kamat
Hi Arun, On 10 December 2013 10:15, Arun Kumar K arunkk.sams...@gmail.com wrote: Hi Pawel, On Tue, Dec 10, 2013 at 6:21 AM, Pawel Osciak posc...@chromium.org wrote: Hi Arun, Ok will make the change in next version. While at it also update the patch subject appropriately. -- With warm

Re: [PATCH] CHROMIUM: s5p-mfc: add controls to set vp8 enc profile

2013-12-09 Thread Arun Kumar K
Hi Sachin, On Tue, Dec 10, 2013 at 10:38 AM, Sachin Kamat sachin.ka...@linaro.org wrote: Hi Arun, On 10 December 2013 10:15, Arun Kumar K arunkk.sams...@gmail.com wrote: Hi Pawel, On Tue, Dec 10, 2013 at 6:21 AM, Pawel Osciak posc...@chromium.org wrote: Hi Arun, Ok will make the change

ATTENTION:

2013-12-09 Thread Membership Authentification Form
ATTENTION: Monday 9th of December 2013 08:16:22 MYT from 41.206.151.183, your account was recently accessed with this details, please if you recognize this details,ignore this message, or use this web link to reconfirm your account details to prevent spammers and unauthorized users to gain

Re: [RFCv4 PATCH 5/8] vb2: retry start_streaming in case of insufficient buffers.

2013-12-09 Thread Guennadi Liakhovetski
Hi Hans On Mon, 9 Dec 2013, Hans Verkuil wrote: From: Hans Verkuil hans.verk...@cisco.com If start_streaming returns -ENODATA, then it will be retried the next time a buffer is queued. Wouldn't ENOBUFS be a better error code? Thanks Guennadi This means applications no longer need to

Re: [RFCv4 PATCH 5/8] vb2: retry start_streaming in case of insufficient buffers.

2013-12-09 Thread Hans Verkuil
On 12/10/2013 07:38 AM, Guennadi Liakhovetski wrote: Hi Hans On Mon, 9 Dec 2013, Hans Verkuil wrote: From: Hans Verkuil hans.verk...@cisco.com If start_streaming returns -ENODATA, then it will be retried the next time a buffer is queued. Wouldn't ENOBUFS be a better error code?

Re: [RFCv4 PATCH 7/8] vb2: return ENODATA in start_streaming in case of too few buffers.

2013-12-09 Thread Guennadi Liakhovetski
On Mon, 9 Dec 2013, Hans Verkuil wrote: From: Hans Verkuil hans.verk...@cisco.com This works together with the retry_start_streaming mechanism to allow userspace to start streaming even if not all required buffers have been queued. Signed-off-by: Hans Verkuil hans.verk...@cisco.com Cc:

Re: [RFCv4 PATCH 7/8] vb2: return ENODATA in start_streaming in case of too few buffers.

2013-12-09 Thread Hans Verkuil
As Guennadi mentioned in his review, ENODATA will be replaced by ENOBUFS, which is more appropriate. Prabhakar, Kamil, Tomasz, are you OK with this patch provided s/ENODATA/ENOBUFS/ ? Regards, Hans On 12/09/2013 02:43 PM, Hans Verkuil wrote: From: Hans Verkuil hans.verk...@cisco.com