[PATCH 3/3] [media] s5p-fimc: Stop media entity pipeline if fimc_pipeline_validate fails

2012-05-24 Thread Sachin Kamat
Stops the media entity pipeline which was started earlier if fimc_pipeline_validate fails. Signed-off-by: Sachin Kamat --- drivers/media/video/s5p-fimc/fimc-capture.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/drivers/media/video/s5p-fimc/fimc-capture.c b/driver

[PATCH 2/3] [media] s5p-fimc: Fix compiler warning in fimc-capture.c file

2012-05-24 Thread Sachin Kamat
drivers/media/video/s5p-fimc/fimc-capture.c: In function ‘fimc_cap_streamon’: drivers/media/video/s5p-fimc/fimc-capture.c:1053:29: warning: ignoring return value of ‘media_entity_pipeline_start’, declared with attribute warn_unused_result [-Wunused-result] Signed-off-by: Sachin Kamat --- driver

[PATCH 1/3] [media] s5p-fimc: Fix compiler warning in fimc-lite.c

2012-05-24 Thread Sachin Kamat
Fixes the followng warning: drivers/media/video/s5p-fimc/fimc-lite.c: In function ‘fimc_lite_streamon’: drivers/media/video/s5p-fimc/fimc-lite.c:765:29: warning: ignoring return value of ‘media_entity_pipeline_start’, declared with attribute warn_unused_result [-Wunused-result] Signed-off-by: Sac

Re: [GIT PULL for v3.5-rc1] media updates for v3.5

2012-05-24 Thread Linus Torvalds
On Thu, May 24, 2012 at 3:33 PM, Mauro Carvalho Chehab wrote: > > The Kconfig default for DVB_FE_CUSTOMISE is 'n'. So, if no DVB bridge is > selected, > nothing will be compiled. Sadly, it looks like the default for distro kernels is 'y'. Which means that if you start with a distro kernel confi

Re: [GIT PULL for v3.5-rc1] media updates for v3.5

2012-05-24 Thread Mauro Carvalho Chehab
Hi Linus, Em 24-05-2012 17:42, Linus Torvalds escreveu: > Btw, I only noticed now, because I normally don't build DVB on my main > machine with "oldconfig" - why the hell does DVB add tuners with > "default m"? > > Why would *anybody* with an old config ever want to get those new > drivers as mod

[PATCH 1/2] Tuning fixed

2012-05-24 Thread André Roth
Signed-off-by: André Roth --- lib/libdvbv5/dvb-fe.c | 23 +-- 1 files changed, 13 insertions(+), 10 deletions(-) diff --git a/lib/libdvbv5/dvb-fe.c b/lib/libdvbv5/dvb-fe.c index ba7bdf0..9d0866e 100644 --- a/lib/libdvbv5/dvb-fe.c +++ b/lib/libdvbv5/dvb-fe.c @@ -503,11 +503,

[PATCH 2/2] LNB calculation fixed

2012-05-24 Thread André Roth
Signed-off-by: André Roth --- lib/libdvbv5/dvb-sat.c | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/lib/libdvbv5/dvb-sat.c b/lib/libdvbv5/dvb-sat.c index 0cef091..c93b882 100644 --- a/lib/libdvbv5/dvb-sat.c +++ b/lib/libdvbv5/dvb-sat.c @@ -349,7 +349,7 @@ in

Re: [GIT PULL for v3.5-rc1] media updates for v3.5

2012-05-24 Thread Linus Torvalds
Btw, I only noticed now, because I normally don't build DVB on my main machine with "oldconfig" - why the hell does DVB add tuners with "default m"? Why would *anybody* with an old config ever want to get those new drivers as modules? Get rid of all the stupid default m if DVB_FE_CUSTOMISE be

Re: Discussion: How to deal with radio tuners which can tune to multiple bands

2012-05-24 Thread halli manjunatha
On Thu, May 24, 2012 at 2:12 PM, Hans de Goede wrote: > Hi, > > > On 05/24/2012 05:00 PM, Hans Verkuil wrote: >> >> On Wed 23 May 2012 20:29:20 Hans de Goede wrote: > > > > > >>> ### >>> >>> So given all of the above I would like to propose the following: >>> >>> 1) Add a "band" field to struct v

Re: Discussion: How to deal with radio tuners which can tune to multiple bands

2012-05-24 Thread Hans de Goede
Hi, On 05/24/2012 05:00 PM, Hans Verkuil wrote: On Wed 23 May 2012 20:29:20 Hans de Goede wrote: ### So given all of the above I would like to propose the following: 1) Add a "band" field to struct v4l2_tuner, and a capability indicating if the driver understands / uses this field 2)

Re: [PATCH 1/3] stv090x: Fix typo in register macros

2012-05-24 Thread Andreas Regel
Am 15.05.2012 14:43, schrieb Manu Abraham: > Hi Andreas, > > Sorry about the late reply. > > Which datasheet revision are you using ? I looked at RevG and found that the > register ERRCNT22 @ 0xF59D, 0xF39D do have bitfields by name Px_ERR_CNT2 > on Page 227. > > Did you overlook that by some ch

cron job: media_tree daily build: WARNINGS

2012-05-24 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 May 24 19:00:36 CEST 2012 git hash:5472d3f17845c4398c6a510b46855820920c2181 gcc version: i686-linux-gcc (GC

Help requested, frame grabber stops working on Debian upgrade (Lenny to Squeeze)

2012-05-24 Thread Rob Murgatroyd
Hello, Please could anyone try to assist with this problem with video capture after a system upgrade? I have a frame grabber that has 8 BT878 (or very similar) chips on one card, also a V4L2 application that reads single frames from attached video cameras. I was using this set-up successfully

Re: [GIT PULL for v3.5-rc1] media updates for v3.5

2012-05-24 Thread Mauro Carvalho Chehab
Hi Ezequiel, Em 24-05-2012 13:33, Ezequiel Garcia escreveu: > Hi Mauro, > > On Thu, May 24, 2012 at 12:34 PM, Mauro Carvalho Chehab > wrote: >> Ezequiel Garcia (2): >> [media] em28xx: Make card_setup() and pre_card_setup() static >> [media] em28xx: Remove unused list_head struct for qu

Re: [GIT PULL for v3.5-rc1] media updates for v3.5

2012-05-24 Thread Ezequiel Garcia
Hi Mauro, On Thu, May 24, 2012 at 12:34 PM, Mauro Carvalho Chehab wrote: > Ezequiel Garcia (2): >      [media] em28xx: Make card_setup() and pre_card_setup() static >      [media] em28xx: Remove unused list_head struct for queued buffers > > Ezequiel García (19): >      [media] em28xx: Remove red

[patch] [media] videobuf-dma-contig: use gfp_t for GFP flags

2012-05-24 Thread Dan Carpenter
Sparse complains if about using unsigned long. videobuf-dma-contig.c:47:67: warning: restricted gfp_t degrades to integer videobuf-dma-contig.c:47:65: warning: incorrect type in argument 2 (different base types) videobuf-dma-contig.c:47:65:expected restricted gfp_t [usertype] gfp_mask videobuf

Re: [GIT PULL for v3.5-rc1] media updates for v3.5

2012-05-24 Thread Antti Palosaari
On 24.05.2012 18:34, Mauro Carvalho Chehab wrote: Hi Linus, Please pull from: git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media v4l_for_linus For a series of patches for 3.5, including: - some V4L2 API updates needed by embedded devices; - DVB API extensions

Re: [PATCH 0/7] s5p-fimc driver fixes

2012-05-24 Thread Sylwester Nawrocki
On 05/24/2012 05:15 PM, Sylwester Nawrocki wrote: > The following is a couple of s5p-fimc driver bug fixes for v3.4-rcX. ^^^ Oops, of course v3.5-rc, not v3.4-rc. -- To unsubscribe from this list: send the line "unsubscribe linux-medi

[PATCH 6/7] s5p-fimc: Fix fimc-lite system wide suspend procedure

2012-05-24 Thread Sylwester Nawrocki
Only suspend the video pipeline devices if they were active before the pm.suspend() helper is called. This patch prevents following error: /# echo mem > /sys/power/state [ 34.965000] PM: Syncing filesystems ... done. [ 35.035000] Freezing user space processes ... (elapsed 0.01 seconds) done. .

[PATCH 3/7] s5p-fimc: Honour sizeimage in VIDIOC_S_FMT

2012-05-24 Thread Sylwester Nawrocki
Allow memory buffer size to be increased by means of struct v4l2_pix_plane_format::sizeimage at VIDIOC_S_FMT ioctl. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- drivers/media/video/s5p-fimc/fimc-capture.c |7 --- drivers/media/video/s5p-fimc/fimc-core.c|9 +

[PATCH 4/7] s5p-fimc: Remove superfluous checks for buffer type

2012-05-24 Thread Sylwester Nawrocki
The checks are already done at the v4l2 framework. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- drivers/media/video/s5p-fimc/fimc-capture.c |8 1 file changed, 8 deletions(-) diff --git a/drivers/media/video/s5p-fimc/fimc-capture.c b/drivers/media/video/s5p-

[PATCH 7/7] s5p-fimc: Shorten pixel formats description

2012-05-24 Thread Sylwester Nawrocki
Shorten pixel format descriptions that exceed 32 characters so they're not being truncated when queried from user space. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- drivers/media/video/s5p-fimc/fimc-core.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) di

[PATCH 5/7] s5p-fimc: Prevent lock-up in multiple sensor systems

2012-05-24 Thread Sylwester Nawrocki
The camera clocks managed by the driver were improperly reference counted and remained disabled when multiple video nodes were opened simultaneously. It manifested itself with following warning: [12.92] WARNING: at drivers/media/video/s5p-fimc/fimc-mdevice.c:787 __fimc_md_set_camclk+0x1c0/0x

[PATCH 0/7] s5p-fimc driver fixes

2012-05-24 Thread Sylwester Nawrocki
The following is a couple of s5p-fimc driver bug fixes for v3.4-rcX. Sylwester Nawrocki (7): s5p-fimc: Fix bug in capture node open() s5p-fimc: Don't create multiple active links to same sink entity s5p-fimc: Honour sizeimage in VIDIOC_S_FMT s5p-fimc: Remove superfluous checks for buffer

[PATCH 2/7] s5p-fimc: Don't create multiple active links to same sink entity

2012-05-24 Thread Sylwester Nawrocki
The driver is supposed to create active media link from sensor N (or its corresponding s5p-mipi-csis entity) to FIMC.N by default. Instead s5p-mipi-csis.N entity gets always connected by a default active link to FIMC.N, regardless of there are parallel bus sensor entities already connected to FIMC

[PATCH 1/7] s5p-fimc: Fix bug in capture node open()

2012-05-24 Thread Sylwester Nawrocki
When video pipeline initialization fails, the ST_CAPT_BUSY flag needs to be cleared before pm_runtime_put_sync is called. Otherwise the runtime suspend routine tries to suspend device, rather than just turning it off. Also fix potential null pointer dereference in fimc_pipeline_shutdown(). Signed-

Re: Discussion: How to deal with radio tuners which can tune to multiple bands

2012-05-24 Thread Hans Verkuil
On Wed 23 May 2012 20:29:20 Hans de Goede wrote: > Hi, > > As discussed before 2 different use-cases have come up where we want some > knowledge of there being different radio bands added to the v4l2 API. > > In Manjunatha Halli's case, if I understand things correctly, he wants > to limit hw_fre

Re: RFC: V4L2 API and radio devices with multiple tuners

2012-05-24 Thread Antti Palosaari
On 22.05.2012 19:26, Rémi Denis-Courmont wrote: Le samedi 19 mai 2012 21:36:23 Antti Palosaari, vous avez écrit : On 19.05.2012 21:20, Hans de Goede wrote: Currently the V4L2 API does not allow for radio devices with more then 1 tuner, which is a bit of a historical oversight, since many radio

Re: [PATCH 3/3] fc001x: tuner driver for FC0013

2012-05-24 Thread Antti Palosaari
On 22.05.2012 00:19, Hans-Frieder Vogt wrote: Am Sonntag, 20. Mai 2012 schrieb Antti Palosaari: On 06.05.2012 23:57, Hans-Frieder Vogt wrote: Support for tuner Fitipower FC0013 +EXPORT_SYMBOL(fc0013_rc_cal_add); Could you explain what is use of that exported function? I did not see any usa

Re: gspca maintenance handover

2012-05-24 Thread Hans de Goede
Hi, On 05/24/2012 12:21 PM, Mauro Carvalho Chehab wrote: Hi Jean-François, Em 24-05-2012 05:34, Jean-Francois Moine escreveu: Hi Mauro, I was glad and proud to work for 4 years in the Linux media team, but, now, I am a bit weary of webcams! I proposed to Hans de Goede to take the maintenance

RE: [Q] vb2 userptr: struct vb2_ops::buf_cleanup() is called without buf_init()

2012-05-24 Thread Marek Szyprowski
Hi Laurent, Thanks for CC. On Tuesday, May 22, 2012 6:22 PM Laurent Pinchart wrote: > Hi Guennadi, > > (CC'ing Pawel and Marek) > > On Monday 21 May 2012 10:30:19 Guennadi Liakhovetski wrote: > > Hi > > > > A recent report > > > > http://thread.gmane.org/gmane.linux.drivers.video-input-infrast

Re: [PATCH 01/10] string: introduce memweight

2012-05-24 Thread Akinobu Mita
2012/5/23 Matthew Wilcox : > On Wed, May 23, 2012 at 09:12:18PM +0900, Akinobu Mita wrote: >> size_t memweight(const void *ptr, size_t bytes) > > Why should this return size_t instead of unsigned long? I just use the same type as the bytes argument without mature consideration. If unsigned long i

[PATCH] gspca: Maintainer change

2012-05-24 Thread Jean-Francois Moine
Hans de Goede accepted to be the new gspca maintainer. Signed-off-by: Jean-François Moine --- MAINTAINERS |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index f175f44..aaa63da 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -3061,8 +3061,7 @@ S:

Re: gspca maintenance handover

2012-05-24 Thread Mauro Carvalho Chehab
Hi Jean-François, Em 24-05-2012 05:34, Jean-Francois Moine escreveu: > Hi Mauro, > > I was glad and proud to work for 4 years in the Linux media team, but, > now, I am a bit weary of webcams! > > I proposed to Hans de Goede to take the maintenance of gspca. He > perfectly knows the code and does

Re: [PATCH] bttv: Use btv->has_radio rather then the card info when registering the tuner

2012-05-24 Thread Andrew Benham
On 20/05/12 12:28, Hans de Goede wrote: > After bttv_init_card2(), bttv_init_tuner(), and it should clearly use > the updated, dynamic has_radio value from btv->has_radio, rather then > the const value in the tvcards array. > > This fixes the radio not working on my Hauppauge WinTV. > > Signed-o

gspca maintenance handover

2012-05-24 Thread Jean-Francois Moine
Hi Mauro, I was glad and proud to work for 4 years in the Linux media team, but, now, I am a bit weary of webcams! I proposed to Hans de Goede to take the maintenance of gspca. He perfectly knows the code and does a good work. The driver will be in safe hands. Anyway, I will not fully give up. I