cron job: media_tree daily build: WARNINGS

2014-02-12 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 Feb 13 04:00:26 CET 2014 git branch: test git hash: 37e59f876bc710d67a30b660826a5e83e07101ce gcc versio

Re: [PATCH] s2255drv: upgrade to videobuf2

2014-02-12 Thread Dean Anderson
Updated: Only error is readbuffers now. I'll fix it and submit a v2 patch with 0 errors. FYI, the fix for the readbuffers v4l2-compliance fail will be setting v4l2_streamparm.capture.readbuffers to the minimum buffer value. Thanks, On 2014-02-12 10:10, Hans Verkuil wrote: On 02/12/14 17:

Re: [PATCH v2] s2255drv: upgrade to videobuf2

2014-02-12 Thread Dean Anderson
"./v4l2-compliance -s" Driver Info: Driver name : s2255 Card type : s2255 Bus info : usb-:00:1a.7-3.6 Driver version: 3.13.0 Capabilities : 0x8501 Video Capture Read/Write Streaming

[PATCH v2] s2255drv: upgrade to videobuf2

2014-02-12 Thread Dean Anderson
Upgrade to videobuf2 libraries. No errors reported with "v4l2-compliance -s". Signed-off-by: Dean Anderson --- drivers/media/usb/s2255/s2255drv.c | 512 +++- 1 file changed, 152 insertions(+), 360 deletions(-) diff --git a/drivers/media/usb/s2255/s2255drv.c b

[PATCH v2] media: soc-camera: OF cameras

2014-02-12 Thread Bryan Wu
From: Guennadi Liakhovetski With OF we aren't getting platform data any more. To minimise changes we create all the missing data ourselves, including compulsory struct soc_camera_link objects. Host-client linking is now done, based on the OF data. Media bus numbers also have to be assigned dynami

[GIT PULL] AF9035 / IT9135

2014-02-12 Thread Antti Palosaari
The following changes since commit 37e59f876bc710d67a30b660826a5e83e07101ce: [media, edac] Change my email address (2014-02-07 08:03:07 -0200) are available in the git repository at: git://linuxtv.org/anttip/media_tree.git af9035 for you to fetch changes up to 57f53ffd5db1c45f7b5253c1f6fad

[REVIEW PATCH 2/4] af9035: add default 0x9135 slave I2C address

2014-02-12 Thread Antti Palosaari
From: Malcolm Priestley On some devices the vendor has not set EEPROM_2ND_DEMOD_ADDR. Checks tmp is not zero after call to get EEPROM_2ND_DEMOD_ADDR and sets the default slave address of 0x3a on 0x9135 devices. Signed-off-by: Malcolm Priestley Signed-off-by: Antti Palosaari --- drivers/media

[REVIEW PATCH 3/4] af9035: Add remaining it913x dual ids to af9035.

2014-02-12 Thread Antti Palosaari
From: Malcolm Priestley As follow on to patch af9035: Move it913x single devices to af9035 and patch 1. SNR is reported as db/10 values. All dual ids are added to af9035 and it913x driver disabled. it913x/it913x-fe removal patches to follow. Signed-off-by: Malcolm Priestley Signed-off-by: An

[REVIEW PATCH 4/4] af9035: use default i2c slave address for af9035 too

2014-02-12 Thread Antti Palosaari
Some device vendors has forgotten set correct slave demod I2C address to eeprom. Use default I2C address when eeprom has no address at all. Signed-off-by: Antti Palosaari --- drivers/media/usb/dvb-usb-v2/af9035.c | 15 +++ 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a

[REVIEW PATCH 1/4] af9035: Move it913x single devices to af9035

2014-02-12 Thread Antti Palosaari
From: Malcolm Priestley The generic v1 and v2 devices have been all tested. IDs tested USB_PID_ITETECH_IT9135 v1 & v2 USB_PID_ITETECH_IT9135_9005 v1 USB_PID_ITETECH_IT9135_9006 v2 Current Issues There is no signal on USB_PID_ITETECH_IT9135 v2 No SNR reported all devices. All single devices t

[GIT PULL] rtl28xxu: add ID [0ccd:00b4] TerraTec NOXON DAB Stick (rev 3)

2014-02-12 Thread Antti Palosaari
The following changes since commit 37e59f876bc710d67a30b660826a5e83e07101ce: [media, edac] Change my email address (2014-02-07 08:03:07 -0200) are available in the git repository at: git://linuxtv.org/anttip/media_tree.git rtl28xxu_id for you to fetch changes up to 46a1dac26913fae389e79b7f

[GIT PULL] tda10071 changes

2014-02-12 Thread Antti Palosaari
The following changes since commit 37e59f876bc710d67a30b660826a5e83e07101ce: [media, edac] Change my email address (2014-02-07 08:03:07 -0200) are available in the git repository at: git://linuxtv.org/anttip/media_tree.git tda10071 for you to fetch changes up to a47dd5674fda7c48d693cd785f4

[REVIEW PATCH 1/3] tda10071: remove a duplicative test

2014-02-12 Thread Antti Palosaari
From: Dan Carpenter "ret" is an error code here, we already tested that. Signed-off-by: Dan Carpenter Acked-by: Antti Palosaari Reviewed-by: Antti Palosaari Signed-off-by: Antti Palosaari --- drivers/media/dvb-frontends/tda10071.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) dif

[REVIEW PATCH 2/3] tda10071: do not check tuner PLL lock on read_status()

2014-02-12 Thread Antti Palosaari
Tuner PLL lock flag was mapped to FE_HAS_SIGNAL, which is wrong. PLL lock has nothing to do with received signal. In real life that flag is always set. Signed-off-by: Antti Palosaari --- drivers/media/dvb-frontends/tda10071.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git

[REVIEW PATCH 3/3] tda10071: coding style issues

2014-02-12 Thread Antti Palosaari
Fix some coding style issues, mostly reported by checkpatch.pl. Signed-off-by: Antti Palosaari --- drivers/media/dvb-frontends/tda10071.c | 60 +++--- drivers/media/dvb-frontends/tda10071.h | 2 +- 2 files changed, 34 insertions(+), 28 deletions(-) diff --git a/driv

[REVIEW PATCH] rtl28xxu: add ID [0ccd:00b4] TerraTec NOXON DAB Stick (rev 3)

2014-02-12 Thread Antti Palosaari
From: Till Dörges I've got the following DAB USB stick that also works fine with the DVB_USB_RTL28XXU driver after I added its USB ID: --- snip --- user@box:~> lsusb -d 0ccd:00b4 Bus 001 Device 009: ID 0ccd:00b4 TerraTec Electronic GmbH --- snap --- [cr...@iki.fi: apply patch partly manually] S

[GIT PULL 3.14] em28xx-dvb m88ts2022 tuner binding fix

2014-02-12 Thread Antti Palosaari
The following changes since commit 37e59f876bc710d67a30b660826a5e83e07101ce: [media, edac] Change my email address (2014-02-07 08:03:07 -0200) are available in the git repository at: git://linuxtv.org/anttip/media_tree.git pctv_461e_fix for you to fetch changes up to 6d56b487e83787e7c532ba

[PATCH FOR 3.14] em28xx-dvb: fix PCTV 461e tuner I2C binding

2014-02-12 Thread Antti Palosaari
Add missing m88ts2022 module reference counts as removing that module is not allowed when it is used by em28xx-dvb module. That same module was not unregistered correctly, fix it too. Error cases validated by returning errors from m88ds3103, m88ts2022 and a8293 probe(). Signed-off-by: Antti Palos

Re: [PATCH v2] em28xx: Only deallocate struct em28xx after finishing all extensions

2014-02-12 Thread Frank Schäfer
Am 09.02.2014 21:09, schrieb Mauro Carvalho Chehab: >> Ping ! >> > Are you going to continue working on this patch ? >> > I've planned to come up with some follow-up changes... ;-) > Yes, but I'm currently without time to do your proposal changes and > test, as I'm in a 2-week business trip. I did

Re: [PATCH v2] [media] of: move graph helpers from drivers/media/v4l2-core to drivers/media

2014-02-12 Thread Philipp Zabel
Hi Mauro, Am Mittwoch, den 12.02.2014, 22:35 +0900 schrieb Mauro Carvalho Chehab: > Em Wed, 12 Feb 2014 10:11:54 +0100 > Philipp Zabel escreveu: > > > Hi Mauro, > > > > Am Mittwoch, den 12.02.2014, 06:53 +0900 schrieb Mauro Carvalho Chehab: > > [...] > > > > diff --git a/include/media/of_graph.

Re: [PATCH] s2255drv: upgrade to videobuf2

2014-02-12 Thread Dean Anderson
"./utils/v4l2-compliance/v4l2-compliance -s" Driver Info: Driver name : s2255 Card type : s2255 Bus info : usb-:00:1a.7-3.6 Driver version: 3.13.0 Capabilities : 0x8401 Video Capture Streaming

Re: [PATCH] s2255drv: upgrade to videobuf2

2014-02-12 Thread Hans Verkuil
On 02/12/14 17:01, Dean Anderson wrote: > "./utils/v4l2-compliance/v4l2-compliance -s" > > Driver Info: > Driver name : s2255 > Card type : s2255 > Bus info : usb-:00:1a.7-3.6 > Driver version: 3.13.0 > Capabilities : 0x8401 > Video Capture >

Re: OMAP3 ISP capabilities

2014-02-12 Thread Peter Meerwald
Hello Laurent, > > (3) it should be possible to use the ISP resizer input / output > > (memory-to-memory) independently; it there any example code doing this? > I haven't written any sample code as such for memory-to-memory operation. I > usually use the following media-ctl and yavta commands t

Re: OMAP3 ISP capabilities

2014-02-12 Thread Laurent Pinchart
Hi Peter, On Tuesday 11 February 2014 15:54:00 Peter Meerwald wrote: > Hello Laurent, > > some quick question about the OMAP3 ISP pipeline capabilities: > > (1) can the OMAP3 ISP CCDC output concurrently to memory AND the resizer > in YUV mode? I think the answer is no due to hardware limitation

OMAP3 ISP capabilities, resizer

2014-02-12 Thread Peter Meerwald
Hello, some more findings: * the driver bug seen below was observed with kernel 3.7 and is already fixed in more recent kernels, likely with commit 864a121274, [media] v4l: Don't warn during link validation when encountering a V4L2 devnode * there still is a a confusing warning: omap3isp oma

Re: [PATCH 36/47] adv7604: Make output format configurable through pad format operations

2014-02-12 Thread Hans Verkuil
On 02/05/14 17:42, Laurent Pinchart wrote: > Replace the dummy video format operations by pad format operations that > configure the output format. > > Signed-off-by: Laurent Pinchart > --- > drivers/media/i2c/adv7604.c | 243 > +++- > include/media/adv76

Re: [REVIEWv2 PATCH 24/34] v4l2-ctrls/videodev2.h: add u8 and u16 types.

2014-02-12 Thread Hans Verkuil
On 02/12/14 14:40, Ricardo Ribalda Delgado wrote: > Hello Hans > > > > On Wed, Feb 12, 2014 at 2:26 PM, Hans Verkuil wrote: >> On 02/12/14 14:13, Ricardo Ribalda Delgado wrote: >>> Hello Hans >>> >>> Thanks for you promptly response >>> >>> On Wed, Feb 12, 2014 at 1:40 PM, Hans Verkuil wrote

Re: [REVIEWv2 PATCH 24/34] v4l2-ctrls/videodev2.h: add u8 and u16 types.

2014-02-12 Thread Ricardo Ribalda Delgado
Hello Hans On Wed, Feb 12, 2014 at 2:26 PM, Hans Verkuil wrote: > On 02/12/14 14:13, Ricardo Ribalda Delgado wrote: >> Hello Hans >> >> Thanks for you promptly response >> >> On Wed, Feb 12, 2014 at 1:40 PM, Hans Verkuil wrote: >>> On 02/12/14 13:11, Ricardo Ribalda Delgado wrote: Hi Hans

Re: [PATCH v2] [media] of: move graph helpers from drivers/media/v4l2-core to drivers/media

2014-02-12 Thread Mauro Carvalho Chehab
Em Wed, 12 Feb 2014 10:11:54 +0100 Philipp Zabel escreveu: > Hi Mauro, > > Am Mittwoch, den 12.02.2014, 06:53 +0900 schrieb Mauro Carvalho Chehab: > [...] > > > diff --git a/include/media/of_graph.h b/include/media/of_graph.h > > > new file mode 100644 > > > index 000..3bbeb60 > > > --- /dev

Re: [REVIEWv2 PATCH 24/34] v4l2-ctrls/videodev2.h: add u8 and u16 types.

2014-02-12 Thread Hans Verkuil
On 02/12/14 14:13, Ricardo Ribalda Delgado wrote: > Hello Hans > > Thanks for you promptly response > > On Wed, Feb 12, 2014 at 1:40 PM, Hans Verkuil wrote: >> On 02/12/14 13:11, Ricardo Ribalda Delgado wrote: >>> Hi Hans >>> >>> Thanks for your reply >>> >>> On Wed, Feb 12, 2014 at 12:20 PM, Ha

Re: [REVIEWv2 PATCH 24/34] v4l2-ctrls/videodev2.h: add u8 and u16 types.

2014-02-12 Thread Ricardo Ribalda Delgado
Hello Hans Thanks for you promptly response On Wed, Feb 12, 2014 at 1:40 PM, Hans Verkuil wrote: > On 02/12/14 13:11, Ricardo Ribalda Delgado wrote: >> Hi Hans >> >> Thanks for your reply >> >> On Wed, Feb 12, 2014 at 12:20 PM, Hans Verkuil wrote: >>> Hi Ricardo, >>> >>> On 02/12/14 11:44, Rica

Re: [REVIEWv2 PATCH 08/34] v4l2-ctrls: create type_ops.

2014-02-12 Thread Ricardo Ribalda Delgado
Agree I wont complain until I have some performance measurement :) Thanks! On Wed, Feb 12, 2014 at 1:31 PM, Hans Verkuil wrote: > On 02/12/14 13:03, Ricardo Ribalda Delgado wrote: >> Hello Hans >> >> My usercase is different gain and offset per pixel (non uniformity >> correction). My sensor

Re: [PATCH 25/47] v4l: subdev: Remove deprecated video-level DV timings operations

2014-02-12 Thread Hans Verkuil
On 02/05/14 17:42, Laurent Pinchart wrote: > The video enum_dv_timings and dv_timings_cap operations are deprecated > and unused. Remove them. FYI: after this the adv7604 fails to compile since it is still using the video ops. You should move the patches that convert the adv7604 before this patch.

Re: [REVIEWv2 PATCH 24/34] v4l2-ctrls/videodev2.h: add u8 and u16 types.

2014-02-12 Thread Hans Verkuil
On 02/12/14 13:11, Ricardo Ribalda Delgado wrote: > Hi Hans > > Thanks for your reply > > On Wed, Feb 12, 2014 at 12:20 PM, Hans Verkuil wrote: >> Hi Ricardo, >> >> On 02/12/14 11:44, Ricardo Ribalda Delgado wrote: >>> Hello Hans >>> >>> In the case of U8 and U16 data types. Why dont you fill th

Re: [REVIEWv2 PATCH 08/34] v4l2-ctrls: create type_ops.

2014-02-12 Thread Hans Verkuil
On 02/12/14 13:03, Ricardo Ribalda Delgado wrote: > Hello Hans > > My usercase is different gain and offset per pixel (non uniformity > correction). My sensor is now 4 Mpx, and probably 12 Mpx soon :) The hardware actually supports per pixel gain and offset?! Wow. Anyway, I am not opposed to mak

Re: [REVIEWv2 PATCH 24/34] v4l2-ctrls/videodev2.h: add u8 and u16 types.

2014-02-12 Thread Ricardo Ribalda Delgado
Hi Hans Thanks for your reply On Wed, Feb 12, 2014 at 12:20 PM, Hans Verkuil wrote: > Hi Ricardo, > > On 02/12/14 11:44, Ricardo Ribalda Delgado wrote: >> Hello Hans >> >> In the case of U8 and U16 data types. Why dont you fill the elem_size >> automatically in v4l2_ctrl and request the driver t

Re: [REVIEWv2 PATCH 08/34] v4l2-ctrls: create type_ops.

2014-02-12 Thread Ricardo Ribalda Delgado
Hello Hans My usercase is different gain and offset per pixel (non uniformity correction). My sensor is now 4 Mpx, and probably 12 Mpx soon :) Right now it is implemented as a video output... but I think that I am exploiting an API :) Regards! On Wed, Feb 12, 2014 at 12:36 PM, Hans Verkuil wro

[REVIEWv2 PATCH 36/34] v4l2-ctrls: break off loop on first changed element

2014-02-12 Thread Hans Verkuil
There is no need to loop over all elements of a matrix checking if there are changes. Just stop at the first changed element. Signed-off-by: Hans Verkuil Reported-by: Ricardo Ribalda Delgado --- drivers/media/v4l2-core/v4l2-ctrls.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) dif

Re: [REVIEWv2 PATCH 08/34] v4l2-ctrls: create type_ops.

2014-02-12 Thread Hans Verkuil
On 02/12/14 11:55, Ricardo Ribalda Delgado wrote: > Hello Hans > > If we have a matrix control with a huge number of elements (ie, the > number of pixels), a control validation will imply over ahuge number > of function calls. > > In the case of the matrix you are always validating/init all the >

Re: [REVIEWv2 PATCH 24/34] v4l2-ctrls/videodev2.h: add u8 and u16 types.

2014-02-12 Thread Hans Verkuil
Hi Ricardo, On 02/12/14 11:44, Ricardo Ribalda Delgado wrote: > Hello Hans > > In the case of U8 and U16 data types. Why dont you fill the elem_size > automatically in v4l2_ctrl and request the driver to fill the field? When you create the control the control framework has to know the element si

Re: [REVIEWv2 PATCH 08/34] v4l2-ctrls: create type_ops.

2014-02-12 Thread Ricardo Ribalda Delgado
Hello Hans If we have a matrix control with a huge number of elements (ie, the number of pixels), a control validation will imply over ahuge number of function calls. In the case of the matrix you are always validating/init all the values of of the array. Why dont we modify the type_ops so they o

Re: [REVIEWv2 PATCH 24/34] v4l2-ctrls/videodev2.h: add u8 and u16 types.

2014-02-12 Thread Ricardo Ribalda Delgado
Hello Hans In the case of U8 and U16 data types. Why dont you fill the elem_size automatically in v4l2_ctrl and request the driver to fill the field? Other option would be not declaring the basic data types (U8, U16, U32...) and use elem_size. Ie. If type==V4L2_CTRL_COMPLEX_TYPES, then the type i

[PATCH] [media] s5p-fimc: Remove reference to outdated macro

2014-02-12 Thread Paul Bolle
The Kconfig symbol S5P_SETUP_MIPIPHY was removed in v3.13. Remove a reference to its macro from a list of Kconfig options. Signed-off-by: Paul Bolle --- See commit e66f233dc7f7 ("ARM: Samsung: Remove the MIPI PHY setup code"). Should one or more options be added to replace S5P_SETUP_MIPIPHY? I co

Cash Awaiting Pick Up..

2014-02-12 Thread 2014 Heritage Foundation Board
This is to re-notify you that you have $500,000.00 waiting for pick-up at Money Gram, Contact Mrs Hillary Florence via email : heritd...@xtra.co.nz for claims. -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More ma

Re: [PATCH v2] [media] of: move graph helpers from drivers/media/v4l2-core to drivers/media

2014-02-12 Thread Philipp Zabel
Hi Tomi, Am Mittwoch, den 12.02.2014, 09:15 +0200 schrieb Tomi Valkeinen: > Hi, > > On 11/02/14 23:41, Philipp Zabel wrote: > > From: Philipp Zabel > > > > This patch moves the parsing helpers used to parse connected graphs > > in the device tree, like the video interface bindings documented in

Re: [PATCH v2] [media] of: move graph helpers from drivers/media/v4l2-core to drivers/media

2014-02-12 Thread Philipp Zabel
Hi Mauro, Am Mittwoch, den 12.02.2014, 06:53 +0900 schrieb Mauro Carvalho Chehab: [...] > > diff --git a/include/media/of_graph.h b/include/media/of_graph.h > > new file mode 100644 > > index 000..3bbeb60 > > --- /dev/null > > +++ b/include/media/of_graph.h > > @@ -0,0 +1,46 @@ > > +/* > > + *

[PATCH 1/3] lm3560: remove FSF address from the license

2014-02-12 Thread Andy Shevchenko
There is no need to keep the FSF address inside each file. Moreover, it might change in future which will make this one obsolete. There is no functional change. Signed-off-by: Andy Shevchenko --- drivers/media/i2c/lm3560.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/media/i

[PATCH 2/3] lm3560: keep style for the comments

2014-02-12 Thread Andy Shevchenko
Let's keep the style for all comments in the code, namely using small letters whenever it's possible. There is no functional change. Signed-off-by: Andy Shevchenko --- drivers/media/i2c/lm3560.c | 13 - 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/drivers/media/i2c/

[PATCH 3/3] lm3560: prevent memory leak in case of pdata absence

2014-02-12 Thread Andy Shevchenko
If we have no pdata defined and driver fails to register we leak memory. Converting to devm_kzalloc prevents this to happen. Signed-off-by: Andy Shevchenko --- drivers/media/i2c/lm3560.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/media/i2c/lm3560.c b/drivers/me