Re: [patch 2/2] [media] ds3000: off by one in ds3000_read_snr()

2012-01-19 Thread Dan Carpenter
On Wed, Jan 18, 2012 at 06:06:46PM +0100, walter harms wrote: Am 17.01.2012 08:30, schrieb Dan Carpenter: This is a static checker patch and I don't have the hardware to test this, so please review it carefully. The dvbs2_snr_tab[] array has 80 elements so when we cap it at 80, that's

Re: [PATCH 2/2] [media] dvb_frontend: Require FE_HAS_PARAMETERS for get_frontend()

2012-01-19 Thread Patrick Boettcher
Hi Mauro, On Wednesday 18 January 2012 18:51:25 Mauro Carvalho Chehab wrote: Calling get_frontend() before having either the frontend locked or the network signaling carriers locked won't work. So, block it at the DVB core. I like the idea and also the implementation. But before merging this

Re: [patch 2/2] [media] ds3000: off by one in ds3000_read_snr()

2012-01-19 Thread walter harms
Am 19.01.2012 10:33, schrieb Dan Carpenter: On Wed, Jan 18, 2012 at 06:06:46PM +0100, walter harms wrote: Am 17.01.2012 08:30, schrieb Dan Carpenter: This is a static checker patch and I don't have the hardware to test this, so please review it carefully. The dvbs2_snr_tab[] array has 80

Re: [PATCH 2/2] [media] dvb_frontend: Require FE_HAS_PARAMETERS for get_frontend()

2012-01-19 Thread Mauro Carvalho Chehab
Em 19-01-2012 08:07, Patrick Boettcher escreveu: Hi Mauro, On Wednesday 18 January 2012 18:51:25 Mauro Carvalho Chehab wrote: Calling get_frontend() before having either the frontend locked or the network signaling carriers locked won't work. So, block it at the DVB core. I like the idea

Re: DVBv5 test report

2012-01-19 Thread Mauro Carvalho Chehab
Em 18-01-2012 20:05, Antti Palosaari escreveu: I tested almost all DVB-T/T2/C devices I have and all seems to be working, excluding Anysee models when using legacy zap. Anysee anysee_streaming_ctrl() will fail because mutex_lock_interruptible() returns -EINTR in anysee_ctrl_msg() function

Re: DVBv5 test report

2012-01-19 Thread Antti Palosaari
On 01/19/2012 01:33 PM, Mauro Carvalho Chehab wrote: Em 18-01-2012 20:05, Antti Palosaari escreveu: I tested almost all DVB-T/T2/C devices I have and all seems to be working, excluding Anysee models when using legacy zap. Anysee anysee_streaming_ctrl() will fail because

Re: [patch 2/2] [media] ds3000: off by one in ds3000_read_snr()

2012-01-19 Thread Dan Carpenter
On Thu, Jan 19, 2012 at 11:26:41AM +0100, walter harms wrote: perhaps it is more useful to do it in the check above ? It looks like the check is correct but we need to shift all the values by one. Again, I don't have this hardware, I'm just going by the context. I do not have the

Re: [PATCH] Adding YUV input support for OMAP3ISP driver

2012-01-19 Thread Gary Thomas
On 2012-01-17 08:33, Laurent Pinchart wrote: snip I already had a couple of YUV support patches in my OMAP3 ISP tree at git.kernel.org. I've rebased them on top of the lastest V4L/DVB tree and pushed them to http://git.linuxtv.org/pinchartl/media.git/shortlog/refs/heads/omap3isp-

Re: [PATCH] Adding YUV input support for OMAP3ISP driver

2012-01-19 Thread Laurent Pinchart
Hi Gary, On Thursday 19 January 2012 13:41:57 Gary Thomas wrote: On 2012-01-17 08:33, Laurent Pinchart wrote: snip I already had a couple of YUV support patches in my OMAP3 ISP tree at git.kernel.org. I've rebased them on top of the lastest V4L/DVB tree and pushed them to

Re: [PATCH] Adding YUV input support for OMAP3ISP driver

2012-01-19 Thread Gary Thomas
On 2012-01-19 05:50, Laurent Pinchart wrote: Hi Gary, On Thursday 19 January 2012 13:41:57 Gary Thomas wrote: On 2012-01-17 08:33, Laurent Pinchart wrote: snip I already had a couple of YUV support patches in my OMAP3 ISP tree at git.kernel.org. I've rebased them on top of the lastest

Re: [PATCH] Adding YUV input support for OMAP3ISP driver

2012-01-19 Thread Laurent Pinchart
Hi Gary, On Thursday 19 January 2012 14:13:53 Gary Thomas wrote: On 2012-01-19 05:50, Laurent Pinchart wrote: On Thursday 19 January 2012 13:41:57 Gary Thomas wrote: On 2012-01-17 08:33, Laurent Pinchart wrote: snip I already had a couple of YUV support patches in my OMAP3 ISP

Re: DVBv5 test report

2012-01-19 Thread Mauro Carvalho Chehab
Em 19-01-2012 09:57, Antti Palosaari escreveu: On 01/19/2012 01:33 PM, Mauro Carvalho Chehab wrote: Em 18-01-2012 20:05, Antti Palosaari escreveu: I tested almost all DVB-T/T2/C devices I have and all seems to be working, excluding Anysee models when using legacy zap. Anysee

Siano DVB USB device called Smart Plus

2012-01-19 Thread Denilson Figueiredo de Sá
I bought a USB DVB device in Brazil, but it doesn't work yet on my Linux system. I've already documented it at: http://linuxtv.org/wiki/index.php/Smart_Plus The device works if I try to use it inside a VirtualBox virtual machine running Windows. I believe the kernel driver that claims this

Re: [PATCH] Adding YUV input support for OMAP3ISP driver

2012-01-19 Thread Gary Thomas
On 2012-01-19 06:28, Laurent Pinchart wrote: Hi Gary, On Thursday 19 January 2012 14:13:53 Gary Thomas wrote: On 2012-01-19 05:50, Laurent Pinchart wrote: On Thursday 19 January 2012 13:41:57 Gary Thomas wrote: On 2012-01-17 08:33, Laurent Pinchart wrote: snip I already had a couple

[PATCH] keytable: Fix copy and paste error for SANYO IR protocol

2012-01-19 Thread Gregor Jasny
Hi Mauro, There seems to be a copy and paste error in the Sanyo protocol handling. Please commit or ack this patch. Thanks, Gregor -- 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

Re: DVB-S2 multistream support

2012-01-19 Thread Marek Ochaba
Hello Christian, we interest to your patch for BBFrame demux (we want to read some BBF header flags and read GS decapsulated data). Could you please publish latest version of it ? And send link to it ? Is there also need adaptation in device driver ? We want to us it by DVB-S2 card TBS-6925,

[PATCH] keytable: Fix copy and paste error for SANYO IR protocol

2012-01-19 Thread Gregor Jasny
Signed-off-by: Gregor Jasny gja...@googlemail.com --- utils/keytable/keytable.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/utils/keytable/keytable.c b/utils/keytable/keytable.c index 93609d2..f03de26 100644 --- a/utils/keytable/keytable.c +++

Re: [PATCH] Adding YUV input support for OMAP3ISP driver

2012-01-19 Thread Gary Thomas
On 2012-01-19 06:35, Gary Thomas wrote: On 2012-01-19 06:28, Laurent Pinchart wrote: Hi Gary, On Thursday 19 January 2012 14:13:53 Gary Thomas wrote: On 2012-01-19 05:50, Laurent Pinchart wrote: On Thursday 19 January 2012 13:41:57 Gary Thomas wrote: On 2012-01-17 08:33, Laurent Pinchart

Re: [PATCH] keytable: Fix copy and paste error for SANYO IR protocol

2012-01-19 Thread Mauro Carvalho Chehab
Em 19-01-2012 11:42, Gregor Jasny escreveu: Signed-off-by: Gregor Jasny gja...@googlemail.com Acked-by: Mauro Carvalho Chehab mche...@redhat.com --- utils/keytable/keytable.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/utils/keytable/keytable.c

Re: [PATCH] Adding YUV input support for OMAP3ISP driver

2012-01-19 Thread Javier Martinez Canillas
On Thu, Jan 19, 2012 at 2:35 PM, Gary Thomas g...@mlbassoc.com wrote: On 2012-01-19 06:28, Laurent Pinchart wrote: Hi Gary, On Thursday 19 January 2012 14:13:53 Gary Thomas wrote: On 2012-01-19 05:50, Laurent Pinchart wrote: On Thursday 19 January 2012 13:41:57 Gary Thomas wrote: On

Re: [PATCH] Adding YUV input support for OMAP3ISP driver

2012-01-19 Thread Enrico
On Thu, Jan 19, 2012 at 2:52 PM, Gary Thomas g...@mlbassoc.com wrote: On 2012-01-19 06:35, Gary Thomas wrote: My camera init code is attached. In the previous kernel, the I2C bus was probed implicitly when I initialized the OMAP3ISP. I thought I remembered some discussion about how that worked

Re: [PATCH] Adding YUV input support for OMAP3ISP driver

2012-01-19 Thread Gary Thomas
On 2012-01-19 07:11, Javier Martinez Canillas wrote: On Thu, Jan 19, 2012 at 2:35 PM, Gary Thomasg...@mlbassoc.com wrote: On 2012-01-19 06:28, Laurent Pinchart wrote: Hi Gary, On Thursday 19 January 2012 14:13:53 Gary Thomas wrote: On 2012-01-19 05:50, Laurent Pinchart wrote: On

Re: Siano DVB USB device called Smart Plus

2012-01-19 Thread Mauro Carvalho Chehab
Hi Denilson, Em 19-01-2012 11:31, Denilson Figueiredo de Sá escreveu: I bought a USB DVB device in Brazil, but it doesn't work yet on my Linux system. I've already documented it at: http://linuxtv.org/wiki/index.php/Smart_Plus The device works if I try to use it inside a VirtualBox

Re: DVBv5 test report

2012-01-19 Thread Antti Palosaari
On 01/19/2012 03:31 PM, Mauro Carvalho Chehab wrote: Em 19-01-2012 09:57, Antti Palosaari escreveu: On 01/19/2012 01:33 PM, Mauro Carvalho Chehab wrote: Em 18-01-2012 20:05, Antti Palosaari escreveu: I tested almost all DVB-T/T2/C devices I have and all seems to be working, excluding Anysee

Re: DVBv5 test report

2012-01-19 Thread Mauro Carvalho Chehab
Em 19-01-2012 13:53, Antti Palosaari escreveu: On 01/19/2012 03:31 PM, Mauro Carvalho Chehab wrote: Em 19-01-2012 09:57, Antti Palosaari escreveu: On 01/19/2012 01:33 PM, Mauro Carvalho Chehab wrote: Em 18-01-2012 20:05, Antti Palosaari escreveu: I tested almost all DVB-T/T2/C devices I have

Re: [PATCH] Adding YUV input support for OMAP3ISP driver

2012-01-19 Thread Javier Martinez Canillas
On Thu, Jan 19, 2012 at 3:36 PM, Gary Thomas g...@mlbassoc.com wrote: On 2012-01-19 07:11, Javier Martinez Canillas wrote: On Thu, Jan 19, 2012 at 2:35 PM, Gary Thomasg...@mlbassoc.com  wrote: On 2012-01-19 06:28, Laurent Pinchart wrote: Hi Gary, On Thursday 19 January 2012 14:13:53 Gary

Re: [PATCH 14/23] omap3isp: Configure CSI-2 phy based on platform data

2012-01-19 Thread Laurent Pinchart
Hi Sakari, On Tuesday 17 January 2012 20:45:18 Sakari Ailus wrote: On Mon, Jan 16, 2012 at 03:22:42PM +0100, Laurent Pinchart wrote: On Wednesday 11 January 2012 22:26:51 Sakari Ailus wrote: Configure CSI-2 phy based on platform data in the ISP driver. For that, the new

Re: [PATCH 16/23] media: Add link_validate op to check links to the sink pad

2012-01-19 Thread Laurent Pinchart
Hi Sakari, On Tuesday 17 January 2012 21:09:58 Sakari Ailus wrote: On Mon, Jan 16, 2012 at 03:35:07PM +0100, Laurent Pinchart wrote: On Wednesday 11 January 2012 22:26:53 Sakari Ailus wrote: Signed-off-by: Sakari Ailus sakari.ai...@iki.fi --- drivers/media/media-entity.c | 73

Re: [PATCH 17/23] v4l: Implement v4l2_subdev_link_validate()

2012-01-19 Thread Laurent Pinchart
Hi Sakari, On Tuesday 17 January 2012 21:21:39 Sakari Ailus wrote: On Mon, Jan 16, 2012 at 03:44:08PM +0100, Laurent Pinchart wrote: On Wednesday 11 January 2012 22:26:54 Sakari Ailus wrote: v4l2_subdev_link_validate() is the default op for validating a link. In V4L2 subdev context, it

Re: DVBv5 test report

2012-01-19 Thread Antti Palosaari
On 01/19/2012 06:08 PM, Mauro Carvalho Chehab wrote: Em 19-01-2012 13:53, Antti Palosaari escreveu: On 01/19/2012 03:31 PM, Mauro Carvalho Chehab wrote: Em 19-01-2012 09:57, Antti Palosaari escreveu: On 01/19/2012 01:33 PM, Mauro Carvalho Chehab wrote: Em 18-01-2012 20:05, Antti Palosaari

Re: [PATCH] Adding YUV input support for OMAP3ISP driver

2012-01-19 Thread Gary Thomas
On 2012-01-19 10:08, Gary Thomas wrote: On 2012-01-19 07:36, Gary Thomas wrote: On 2012-01-19 07:11, Javier Martinez Canillas wrote: On Thu, Jan 19, 2012 at 2:35 PM, Gary Thomasg...@mlbassoc.com wrote: On 2012-01-19 06:28, Laurent Pinchart wrote: Hi Gary, On Thursday 19 January 2012

Re: [PATCH] Adding YUV input support for OMAP3ISP driver

2012-01-19 Thread Gary Thomas
On 2012-01-19 10:22, Gary Thomas wrote: On 2012-01-19 10:08, Gary Thomas wrote: On 2012-01-19 07:36, Gary Thomas wrote: On 2012-01-19 07:11, Javier Martinez Canillas wrote: On Thu, Jan 19, 2012 at 2:35 PM, Gary Thomasg...@mlbassoc.com wrote: On 2012-01-19 06:28, Laurent Pinchart wrote: Hi

[PATCH] stb0899: fix the limits for signal strength values

2012-01-19 Thread Klaus Schmidinger
stb0899_read_signal_strength() adds an offset to the result of the table lookup. That offset must correspond to the lowest value in the lookup table, to make sure the result doesn't get below 0, which would mean a very high value since the parameter is unsigned. 'strength' and 'snr' need to be

[PATCH] cxd2820r: remove unused parameter from cxd2820r_attach

2012-01-19 Thread Antti Palosaari
Fix bug introduced by multi-frontend to single-frontend change. This parameter is no longer used after multi-frontend to single-frontend change. Signed-off-by: Antti Palosaari cr...@iki.fi --- drivers/media/dvb/dvb-usb/anysee.c |3 +-- drivers/media/dvb/frontends/cxd2820r.h |

cron job: media_tree daily build: WARNINGS

2012-01-19 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:Thu Jan 19 19:00:20 CET 2012 git hash:36be126cb0ebe3000a65c1049f339a3e882a9a47 gcc version: i686-linux-gcc

Re: [RFCv1 2/4] v4l:vb2: add support for shared buffer (dma_buf)

2012-01-19 Thread Pawel Osciak
Hi Sumit, Thank you for your work. Please find my comments below. On Thu, Jan 5, 2012 at 2:41 AM, Sumit Semwal sumit.sem...@ti.com wrote: This patch adds support for DMABUF memory type in videobuf2. It calls relevant APIs of dma_buf for v4l reqbuf / qbuf / dqbuf operations. For this version,

Re: [PATCH 14/23] omap3isp: Configure CSI-2 phy based on platform data

2012-01-19 Thread Sakari Ailus
Hi Laurent, On Thu, Jan 19, 2012 at 05:16:50PM +0100, Laurent Pinchart wrote: On Tuesday 17 January 2012 20:45:18 Sakari Ailus wrote: On Mon, Jan 16, 2012 at 03:22:42PM +0100, Laurent Pinchart wrote: On Wednesday 11 January 2012 22:26:51 Sakari Ailus wrote: Configure CSI-2 phy based on

Re: [PATCH 16/23] media: Add link_validate op to check links to the sink pad

2012-01-19 Thread Sakari Ailus
Hi Laurent, On Thu, Jan 19, 2012 at 05:20:55PM +0100, Laurent Pinchart wrote: ... Do we really need a generic link walking code for a single user ? Merging this in the function below would result in much simpler code. It's a single funcition but it's being used from two locations in

Build change in media_build to support Debian

2012-01-19 Thread Benjamin Limmer
commit 2949a7393f3e2598d4de49b408587462b11f819f Author: Ben Limmer benjamin.lim...@readytalk.com Date: Thu Jan 19 16:01:15 2012 -0700 Update to build script to give Debian users the Ubunutu package hints. The aptitude package names are the same. diff --git a/build b/build index

Re: Siano DVB USB device called Smart Plus

2012-01-19 Thread Denilson Figueiredo de Sá
On Thu, Jan 19, 2012 at 12:42, Mauro Carvalho Chehab mche...@redhat.com wrote: From the product page, it is a 1-seg device. So, it likely uses a sms1xxx chip. Correct, Siano SMS1140. You'll likely need to add a new board entry there for it, and discover the GPIO pins linked to the leds and

Delivery Failure

2012-01-19 Thread Postmaster
- The message you sent to mptourism.com/thk was rejected because it would exceed the quota for the mailbox. The subject of the message follows: Subject: Returned mail: see transcript for details