cron job: media_tree daily build: ABI WARNING

2015-02-17 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 Feb 18 04:00:23 CET 2015 git branch: test git hash: 135f9be9194cf7778eb73594aa55791b229cf27c gcc versio

Re: Opening firmware source code (vhdl)

2015-02-17 Thread Abylay Ospan
Hi Luis, Great ! Please send me your address and phone in private email. I will arrange shipment. 2015-02-17 19:43 GMT-05:00 Luis de Bethencourt : > On 17 Feb 2015 21:40, "Abylay Ospan" wrote: >> >> Hi Luis, >> >> welcome ! >> >> We have some amount (5+ pcs) of boards available for developers. I

Re: Opening firmware source code (vhdl)

2015-02-17 Thread Abylay Ospan
Hi Luis, welcome ! We have some amount (5+ pcs) of boards available for developers. If someone seriously interested (have any ideas to check with board ?) we can ship it for free :) 2015-02-16 11:35 GMT-05:00 Luis de Bethencourt : > On Mon, Feb 16, 2015 at 11:04:47AM -0500, Abylay Ospan wrote:

Re: [PATCHv2 4/6] uvc gadget: switch to unlocked_ioctl.

2015-02-17 Thread Laurent Pinchart
Hi Hans, Thank you for the patch. On Tuesday 17 February 2015 09:44:07 Hans Verkuil wrote: > From: Hans Verkuil > > Instead of .ioctl use unlocked_ioctl. This allows us to finally remove > the old .ioctl op. > > Signed-off-by: Hans Verkuil Acked-by: Laurent Pinchart > --- > drivers/usb/ga

Re: [PATCHv2 3/6] uvc gadget: switch to v4l2 core locking

2015-02-17 Thread Laurent Pinchart
Hi Hans, Thank you for the patch. On Tuesday 17 February 2015 09:44:06 Hans Verkuil wrote: > From: Hans Verkuil > > Switch this driver over to the V4L2 core locking mechanism in preparation > for switching to unlocked_ioctl. Suggested by Laurent Pinchart. > > This patch introduces a new mutex

Re: [RFC PATCH 0/1] [media] pci: Add support for DVB PCIe cards from Prospero Technologies Ltd.

2015-02-17 Thread Antti Palosaari
On 02/17/2015 08:22 PM, Philip Downer wrote: Hi Mauro, On Mon, Feb 16, 2015 at 11:47 PM, Mauro Carvalho Chehab wrote: Em Mon, 16 Feb 2015 22:01:07 +0200 Antti Palosaari escreveu: Moikka! On 02/16/2015 09:48 PM, Philip Downer wrote: The Vortex PCIe card by Prospero Technologies Ltd is a mo

Re: [RFC PATCH 0/1] [media] pci: Add support for DVB PCIe cards from Prospero Technologies Ltd.

2015-02-17 Thread Philip Downer
Hi Mauro, On Mon, Feb 16, 2015 at 11:47 PM, Mauro Carvalho Chehab wrote: > Em Mon, 16 Feb 2015 22:01:07 +0200 > Antti Palosaari escreveu: > >> Moikka! >> >> On 02/16/2015 09:48 PM, Philip Downer wrote: >> > The Vortex PCIe card by Prospero Technologies Ltd is a modular DVB card >> > with a hardw

Re: [RFC v01] Driver for Toshiba TC358743 CSI-2 to HDMI bridge

2015-02-17 Thread Mats Randgaard (matrandg)
Thank you again for testing this driver! I am sorry I haven't had time to prepare a second RFC for this driver, but I will try to do that as soon as possible. On 02/17/2015 09:50 AM, Philipp Zabel wrote: Hi Mats, Am Montag, den 15.12.2014, 19:21 +0100 schrieb matra...@cisco.com: [...] +stati

Re: [PATCH] media/v4l2-ctrls: Always run s_ctrl on volatile ctrls

2015-02-17 Thread Jacek Anaszewski
On 02/17/2015 03:35 PM, Sakari Ailus wrote: Jacek Anaszewski wrote: On 02/17/2015 03:06 PM, Sakari Ailus wrote: Hi Jacek, Jacek Anaszewski wrote: Hi Hans, Sakari, On 02/17/2015 12:32 PM, Sakari Ailus wrote: Hi Hans, Hans Verkuil wrote: ... Unfortunately, it only works one time, because th

[PATCH v4 2/2] media/Documentation: Volatile writable

2015-02-17 Thread Ricardo Ribalda Delgado
Volatile variables are also writable now. Update the documentation accordingly. Acked-by: Hans Verkuil Signed-off-by: Ricardo Ribalda Delgado --- v4: Hans Verkuil: Fix typos and add ack-by Hans Documentation/DocBook/media/v4l/vidioc-dqevent.xml | 7 --- Documentation/DocBook/media/v4l/vi

[PATCH v4 1/2] media/v4l2-ctrls: Always run s_ctrl on volatile ctrls

2015-02-17 Thread Ricardo Ribalda Delgado
Volatile controls can change their value outside the v4l-ctrl framework. We should ignore the cached written value of the ctrl when evaluating if we should run s_ctrl. Signed-off-by: Ricardo Ribalda Delgado --- v4: Hans Verkuil: explicity set has_changed to false. and add comment drivers/media

Re: [PATCH 2/2] media/Documentation: Volatile writable

2015-02-17 Thread Hans Verkuil
On 02/17/2015 03:41 PM, Ricardo Ribalda Delgado wrote: > Volatile variables are also writable now. Update the documentation > accordingly. > > Signed-off-by: Ricardo Ribalda Delgado > --- > Documentation/DocBook/media/v4l/vidioc-dqevent.xml | 7 --- > Documentation/DocBook/media/v4l/vidioc

Re: [PATCH 1/2] media/v4l2-ctrls: Always run s_ctrl on volatile ctrls

2015-02-17 Thread Hans Verkuil
On 02/17/2015 03:41 PM, Ricardo Ribalda Delgado wrote: > Volatile controls can change their value outside the v4l-ctrl framework. > We should ignore the cached written value of the ctrl when evaluating if > we should run s_ctrl. > > Signed-off-by: Ricardo Ribalda Delgado > --- > drivers/media/v4

Re: [PATCH 1/2] media/v4l2-ctrls: Always run s_ctrl on volatile ctrls

2015-02-17 Thread Ricardo Ribalda Delgado
This is v3 of the patch. I forgot to add it to the subject. I have marked v1 and v2 as Superseded on patchwork Thanks On Tue, Feb 17, 2015 at 3:41 PM, Ricardo Ribalda Delgado wrote: > Volatile controls can change their value outside the v4l-ctrl framework. > We should ignore the cached written

Re: [PATCH v2] media/v4l2-ctrls: Always run s_ctrl on volatile ctrls

2015-02-17 Thread Ricardo Ribalda Delgado
Hello Sakari On Tue, Feb 17, 2015 at 1:47 PM, Sakari Ailus wrote: > Hi Ricardo, > > Thanks for the patch! Thank you > > > Acked-by: Sakari Ailus Could you take a look to v3 of the patch Thanks! > > -- > Sakari Ailus > sakari.ai...@linux.intel.com -- Ricardo Ribalda -- To unsubscribe from

[PATCH 2/2] media/Documentation: Volatile writable

2015-02-17 Thread Ricardo Ribalda Delgado
Volatile variables are also writable now. Update the documentation accordingly. Signed-off-by: Ricardo Ribalda Delgado --- Documentation/DocBook/media/v4l/vidioc-dqevent.xml | 7 --- Documentation/DocBook/media/v4l/vidioc-queryctrl.xml | 6 -- Documentation/video4linux/v4l2-controls.tx

[PATCH 1/2] media/v4l2-ctrls: Always run s_ctrl on volatile ctrls

2015-02-17 Thread Ricardo Ribalda Delgado
Volatile controls can change their value outside the v4l-ctrl framework. We should ignore the cached written value of the ctrl when evaluating if we should run s_ctrl. Signed-off-by: Ricardo Ribalda Delgado --- drivers/media/v4l2-core/v4l2-ctrls.c | 6 ++ 1 file changed, 6 insertions(+) dif

Re: [PATCH] media/v4l2-ctrls: Always run s_ctrl on volatile ctrls

2015-02-17 Thread Sakari Ailus
Jacek Anaszewski wrote: > On 02/17/2015 03:06 PM, Sakari Ailus wrote: >> Hi Jacek, >> >> Jacek Anaszewski wrote: >>> Hi Hans, Sakari, >>> >>> On 02/17/2015 12:32 PM, Sakari Ailus wrote: Hi Hans, Hans Verkuil wrote: ... >> Unfortunately, it only works one time, because the ne

[PATCH] cxusb: Use enum to represent table offsets rather than hard-coding numbers [ver #3]

2015-02-17 Thread David Howells
Use enum to represent table offsets rather than hard-coding numbers to avoid problems with the numbers becoming out of sync with the table. Signed-off-by: David Howells --- drivers/media/usb/dvb-usb/cxusb.c | 155 ++--- 1 file changed, 111 insertions(+), 44 dele

Re: [PATCH] media/v4l2-ctrls: Always run s_ctrl on volatile ctrls

2015-02-17 Thread Jacek Anaszewski
On 02/17/2015 03:06 PM, Sakari Ailus wrote: Hi Jacek, Jacek Anaszewski wrote: Hi Hans, Sakari, On 02/17/2015 12:32 PM, Sakari Ailus wrote: Hi Hans, Hans Verkuil wrote: ... Unfortunately, it only works one time, because the next time the user writes a zero to the control cluster_changed retu

Re: [PATCH] media/v4l2-ctrls: Always run s_ctrl on volatile ctrls

2015-02-17 Thread Sakari Ailus
Hi Jacek, Jacek Anaszewski wrote: > Hi Hans, Sakari, > > On 02/17/2015 12:32 PM, Sakari Ailus wrote: >> Hi Hans, >> >> Hans Verkuil wrote: >> ... Unfortunately, it only works one time, because the next time the user writes a zero to the control cluster_changed returns false. >

Re: [PATCH] cxusb: Use enum to represent table offsets rather than hard-coding numbers

2015-02-17 Thread Mauro Carvalho Chehab
Em Tue, 17 Feb 2015 13:45:50 + David Howells escreveu: > Mauro Carvalho Chehab wrote: > > > I would do a s/ix_USB_PID_// in the above, in order to simplify the > > namespace and to avoid giving the false impression that those are vendor > > IDs. > > Okay. > > > If you look below on your p

[PATCH] cxusb: Use enum to represent table offsets rather than hard-coding numbers [ver #2]

2015-02-17 Thread David Howells
Use enum to represent table offsets rather than hard-coding numbers to avoid problems with the numbers becoming out of sync with the table. Signed-off-by: David Howells --- drivers/media/usb/dvb-usb/cxusb.c | 113 +++-- 1 file changed, 69 insertions(+), 44 delet

Re: [PATCH] cxusb: Use enum to represent table offsets rather than hard-coding numbers

2015-02-17 Thread David Howells
David Howells wrote: > That should really be: > > [VID_MEDION_MD95700] = {USB_VID_MEDION, USB_PID_MEDION_MD95700}, > > since the index number is the model, not the vendor, which brings me to: > > [DVICO_BLUEBIRD_DVB_T_NANO_2_NFW_WARM] = {USB_VID_DVICO, > USB_PID_DVICO_BLUEBIRD_DVB

Re: [PATCH] media/v4l2-ctrls: Always run s_ctrl on volatile ctrls

2015-02-17 Thread Jacek Anaszewski
Hi Hans, Sakari, On 02/17/2015 12:32 PM, Sakari Ailus wrote: Hi Hans, Hans Verkuil wrote: ... Unfortunately, it only works one time, because the next time the user writes a zero to the control cluster_changed returns false. I think on volatile controls it is safer to run s_ctrl twice than mis

Re: [PATCH] cxusb: Use enum to represent table offsets rather than hard-coding numbers

2015-02-17 Thread David Howells
Mauro Carvalho Chehab wrote: > I would do a s/ix_USB_PID_// in the above, in order to simplify the > namespace and to avoid giving the false impression that those are vendor > IDs. Okay. > If you look below on your patch, even you forgot to add a "ix_" prefix into > one of the entires ;) Bah.

Re: [PATCH v2] media/v4l2-ctrls: Always run s_ctrl on volatile ctrls

2015-02-17 Thread Sakari Ailus
Hi Ricardo, Thanks for the patch! Ricardo Ribalda Delgado wrote: > Volatile controls can change their value outside the v4l-ctrl framework. > We should ignore the cached written value of the ctrl when evaluating if > we should run s_ctrl. > > Signed-off-by: Ricardo Ribalda Delgado Acked-by: Sa

RE: [BUG, workaround] HVR-2200/saa7164 problem with C7 power state

2015-02-17 Thread dCrypt
Hi, Forcing C-state to C6 doesn't help either. It's just the errors show up in the log later. Last reboot was the 12th of February, and I started getting errors this morning and VDR shutdown (five days working without issues). Find the log attached. I am testing again with the next C-state bio

Re: [PATCH] media/v4l2-ctrls: Always run s_ctrl on volatile ctrls

2015-02-17 Thread Hans Verkuil
On 02/17/15 13:21, Ricardo Ribalda Delgado wrote: > Hello Hans > > On Tue, Feb 17, 2015 at 1:03 PM, Hans Verkuil wrote: >> Should be done after the 'ctrl == NULL' check. > > Good catch. Fixed on v2 > >> >>> >>> if (ctrl == NULL) >>> continue; >>> >> >> There

[PATCH v2] media/v4l2-ctrls: Always run s_ctrl on volatile ctrls

2015-02-17 Thread Ricardo Ribalda Delgado
Volatile controls can change their value outside the v4l-ctrl framework. We should ignore the cached written value of the ctrl when evaluating if we should run s_ctrl. Signed-off-by: Ricardo Ribalda Delgado --- v2: Do volatile test, once you know ctrl is not NULL drivers/media/v4l2-core/v4l2-c

Re: [PATCH] media/v4l2-ctrls: Always run s_ctrl on volatile ctrls

2015-02-17 Thread Ricardo Ribalda Delgado
Hello Hans On Tue, Feb 17, 2015 at 1:03 PM, Hans Verkuil wrote: > Should be done after the 'ctrl == NULL' check. Good catch. Fixed on v2 > >> >> if (ctrl == NULL) >> continue; >> > > There is one more change that has to be made: setting a volatile control > s

Re: [PATCH] media/v4l2-ctrls: Always run s_ctrl on volatile ctrls

2015-02-17 Thread Hans Verkuil
Hi Ricardo, I've thought about this some more and I agree that this should be allowed. But I have some comments, see below. On 02/17/15 12:02, Ricardo Ribalda Delgado wrote: > Volatile controls can change their value outside the v4l-ctrl framework. > > We should ignore the cached written value

Re: [PATCH] cxusb: Use enum to represent table offsets rather than hard-coding numbers

2015-02-17 Thread Mauro Carvalho Chehab
Em Mon, 16 Feb 2015 15:33:07 + David Howells escreveu: > Use enum to represent table offsets rather than hard-coding numbers to avoid > problems with the numbers becoming out of sync with the table. > > Signed-off-by: David Howells > --- > > drivers/media/usb/dvb-usb/cxusb.c | 115 >

Re: [PATCH] media/v4l2-ctrls: Always run s_ctrl on volatile ctrls

2015-02-17 Thread Sakari Ailus
Hi Hans, Hans Verkuil wrote: ... >> Unfortunately, it only works one time, because the next time the user writes >> a zero to the control cluster_changed returns false. >> >> I think on volatile controls it is safer to run s_ctrl twice than missing a >> valid s_ctrl. >> >> I know I am abusing a bi

Re: [PATCH] media/v4l2-ctrls: Always run s_ctrl on volatile ctrls

2015-02-17 Thread Ricardo Ribalda Delgado
Hello Hans I need to figure out how can you reply that fast. Thanks a lot! On Tue, Feb 17, 2015 at 12:17 PM, Hans Verkuil wrote: >> I have a control that tells the user when there has been a external trigger >> overrun. (Trigger while processing old image). This is a volatile control. > > Does

Re: [PATCH] media/v4l2-ctrls: Always run s_ctrl on volatile ctrls

2015-02-17 Thread Hans Verkuil
Hi Ricardo, On 02/17/15 12:02, Ricardo Ribalda Delgado wrote: > Volatile controls can change their value outside the v4l-ctrl framework. > > We should ignore the cached written value of the ctrl when evaluating if > we should run s_ctrl. > > Signed-off-by: Ricardo Ribalda Delgado > --- > > I h

[PATCH] media/v4l2-ctrls: Always run s_ctrl on volatile ctrls

2015-02-17 Thread Ricardo Ribalda Delgado
Volatile controls can change their value outside the v4l-ctrl framework. We should ignore the cached written value of the ctrl when evaluating if we should run s_ctrl. Signed-off-by: Ricardo Ribalda Delgado --- I have a control that tells the user when there has been a external trigger overrun.

Re: [PATCH] r820t: add DVBC profile in sysfreq_sel

2015-02-17 Thread Benjamin Larsson
Forgot to say that with this patch the Astrometa DVB-T/T2/C usb stick now has working DVB-C reception at all the frequencies I have at home. Before it was not working at around 290-314 MHz. The current status of the Astrometa stick is that DVB-T and DVB-C (8MHz tested, 6MHz testers wanted) sho

Re: [PATCH 1/2] [media] mantis: Move jump label to activate dead code

2015-02-17 Thread Silvan Jegen
Thanks for the review, Dan! On Mon, Feb 16, 2015 at 10:04 AM, Dan Carpenter wrote: > On Sun, Feb 15, 2015 at 01:11:04PM +0100, Silvan Jegen wrote: >> diff --git a/drivers/media/pci/mantis/mantis_cards.c >> b/drivers/media/pci/mantis/mantis_cards.c >> index 801fc55..e566061 100644 >> --- a/driver

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

2015-02-17 Thread Hans Verkuil
Hi Kamil, On 12/16/14 12:36, Kamil Debski wrote: > 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 indicati

Re: [RFC v01] Driver for Toshiba TC358743 CSI-2 to HDMI bridge

2015-02-17 Thread Philipp Zabel
Hi Mats, Am Montag, den 15.12.2014, 19:21 +0100 schrieb matra...@cisco.com: [...] > +static void tc358743_set_pll(struct v4l2_subdev *sd) > +{ > + struct tc358743_state *state = to_state(sd); > + struct tc358743_platform_data *pdata = &state->pdata; > + u16 pllctl0 = i2c_rd16(sd, PLLCT

[PATCHv2 4/6] uvc gadget: switch to unlocked_ioctl.

2015-02-17 Thread Hans Verkuil
From: Hans Verkuil Instead of .ioctl use unlocked_ioctl. This allows us to finally remove the old .ioctl op. Signed-off-by: Hans Verkuil --- drivers/usb/gadget/function/uvc_v4l2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/gadget/function/uvc_v4l2.c b/driv

[PATCHv2 3/6] uvc gadget: switch to v4l2 core locking

2015-02-17 Thread Hans Verkuil
From: Hans Verkuil Switch this driver over to the V4L2 core locking mechanism in preparation for switching to unlocked_ioctl. Suggested by Laurent Pinchart. This patch introduces a new mutex at the struct uvc_video level and drops the old mutex at the queue level. The new lock is now used for al

[PATCHv2 5/6] uvc gadget: set device_caps in querycap.

2015-02-17 Thread Hans Verkuil
From: Hans Verkuil The V4L2 core will warn if this is not done. Unfortunately this driver wasn't updated. Signed-off-by: Hans Verkuil Acked-by: Laurent Pinchart --- drivers/usb/gadget/function/uvc_v4l2.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/usb/gadget/

[PATCHv2 6/6] v4l2-core: remove the old .ioctl BKL replacement

2015-02-17 Thread Hans Verkuil
From: Hans Verkuil To keep V4L2 drivers that did not yet convert to unlocked_ioctl happy, the v4l2 core had a .ioctl file operation that took a V4L2 lock. The last drivers are now converted to unlocked_ioctl, so all this old code can now be removed. Signed-off-by: Hans Verkuil --- drivers/med

[PATCHv2 1/6] pvrusb2: replace .ioctl by .unlocked_ioctl.

2015-02-17 Thread Hans Verkuil
From: Hans Verkuil As far as I can tell pvrusb2 does its own locking, so there is no need to use .ioctl. Signed-off-by: Hans Verkuil --- drivers/media/usb/pvrusb2/pvrusb2-v4l2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/usb/pvrusb2/pvrusb2-v4l2.c b/driv

[PATCHv2 2/6] radio-bcm2048: use unlocked_ioctl instead of ioctl

2015-02-17 Thread Hans Verkuil
From: Hans Verkuil This driver does its own locking, so there is no need to use ioctl instead of unlocked_ioctl. Signed-off-by: Hans Verkuil Acked-by: Pali Rohár --- drivers/staging/media/bcm2048/radio-bcm2048.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/stagi

[PATCHv2 0/6] Remove .ioctl from v4l2_file_operations

2015-02-17 Thread Hans Verkuil
Changes in v1: - use core locking in the uvc gadget driver All V4L2 drivers should use .unlocked_ioctl instead of .ioctl. There are only three drivers left that do not do that: pvrusb2, radio-bcm2048 and the uvc gadget driver. The pvrusb2 driver does its own locking as far as I can tell, so it ca

Updated DVB-T scan tables for Adelaide region

2015-02-17 Thread Daniel Merritt
Greetings, I have attached a couple of updated scan tables for the two Adelaide transmission areas: * au-Adelaide (Australia / Adelaide / Mt Lofty) * au-AdelaideFoothills (Australia / Adelaide / Grenfell Street) A number of channels in the area underwent a final "retune" in late 2013 as per http