Re: How I must report that a driver has been broken?

2012-05-13 Thread Devin Heitmueller
On Sun, May 13, 2012 at 1:50 AM, Hans de Goede wrote: > Hi, > > > On 05/12/2012 09:26 PM, Alfredo Jesús Delaiti wrote: >> >> Hi >> >> Thanks for your response Hans and Patrick >> >> Maybe I doing wrong this, because it reports twice: >> >> http://www.mail-archive.com/linux-media@vger.kernel.org/ms

Re: [PATCH 1/5] rtl2832 ver 0.3: suport for RTL2832 demodulator revised version

2012-05-13 Thread Antti Palosaari
I forget to ran checkpatch.pl. Looks like you too. Mostly simple fixes... You should always ran checkpatch.pl when dending patches. On 14.05.2012 04:37, Antti Palosaari wrote: Puuuh, there was more findings what I supposed earlier. Most of those are easy to fix. But it seems like all the stati

Re: [PATCH 1/5] rtl2832 ver 0.3: suport for RTL2832 demodulator revised version

2012-05-13 Thread Antti Palosaari
On 12.05.2012 21:08, Thomas Mair wrote: Changes compared to version 0.2: - removed reading of signal strength for tuners FC0012,FC0013 (is now supported by fc0012,fc0013 driver) - moved definition of register names to rtl2832_priv.h - cleaned up demod private structure - replaced rtl2832_log2 fu

Re: [PATCH v3 07/10] arm: omap4430sdp: Add support for omap4iss camera

2012-05-13 Thread Sakari Ailus
Hi Sergio, On Thu, May 03, 2012 at 10:20:47PM -0500, Aguirre, Sergio wrote: > Hi Sakari, > > On Thu, May 3, 2012 at 7:03 AM, Aguirre, Sergio wrote: > > Hi Sakari, > > > > Thanks for reviewing. > > > > On Wed, May 2, 2012 at 2:47 PM, Sakari Ailus wrote: > >> > >> Hi Sergio, > >> > >> Thanks for

Re: [PATCH 03/23] V4L: Add an extended camera white balance control

2012-05-13 Thread Sakari Ailus
Hi Sylwaster, Thanks for the patch. I noticed your pull req; I hope you could take into account a few more comments. :) On Thu, May 10, 2012 at 12:30:38PM +0200, Sylwester Nawrocki wrote: > This patch adds V4L2_CID_AUTO_N_PRESET_WHITE_BALANCE control which is > an extended version of the V4L2_CID

Re: [PATCH 0/3] V4L2_CID_PIXEL_RATE support in sensor drivers

2012-05-13 Thread Sakari Ailus
Hi Laurent, Thanks for the patches! On Wed, May 09, 2012 at 02:55:56PM +0200, Laurent Pinchart wrote: > Hi everybody, > > This patch implements support for the V4L2_CID_PIXEL_RATE control in the > mt9t001, mt9p031 and mt9m032 sensor drivers. > > Recent changes to the OMAP3 ISP driver (see the m

[PATCH] rtl28xxu: removed newlines from info strings

2012-05-13 Thread Thomas Mair
Signed-off-by: Thomas Mair --- drivers/media/dvb/dvb-usb/rtl28xxu.c | 20 ++-- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/media/dvb/dvb-usb/rtl28xxu.c b/drivers/media/dvb/dvb-usb/rtl28xxu.c index ef02f0f..f10cac2 100644 --- a/drivers/media/dvb/dvb-

Re: [PATCH 1/5] rtl2832 ver 0.3: suport for RTL2832 demodulator revised version

2012-05-13 Thread Thomas Mair
On 13.05.2012 20:53, Antti Palosaari wrote: > On 12.05.2012 21:08, Thomas Mair wrote: >> Changes compared to version 0.2: >> - removed reading of signal strength for tuners FC0012,FC0013 (is now >> supported by fc0012,fc0013 driver) >> - moved definition of register names to rtl2832_priv.h >> - cl

Re: [PATCH] fc0012 ver. 0.6: introduction of get_rf_strength function

2012-05-13 Thread poma
On 05/13/2012 12:59 PM, Antti Palosaari wrote: > On 13.05.2012 11:16, poma wrote: >> On 05/12/2012 11:11 AM, Hans-Frieder Vogt wrote: >>> Changes compared to version 0.5 of driver (sent 6 May): >>> - Initial implementation of get_rf_strength function. >>> >>> Signed-off-by: Hans-Frieder Vogt >>> >>

Re: [PATCH 2/2] staging: media: go7007: Adlink MPG24 board

2012-05-13 Thread Dan Carpenter
On Sun, May 13, 2012 at 10:52:42PM +0400, Volokh Konstantin wrote: > Changes: > - wis-tw2804.ko module code was incompatible with 3.4 branch in > initialization v4l2_subdev parts. now i2c_get_clientdata(...) contains > v4l2_subdev struct instead non standard wis_tw2804 struct > - Use V4L2

Re: [PATCH 1/2] staging: media: go7007: Adlink MPG24 board

2012-05-13 Thread Dan Carpenter
On Sun, May 13, 2012 at 10:52:41PM +0400, Volokh Konstantin wrote: > This patch applies only for Adlink MPG24 board with go7007, all these changes > were tested for continuous loading & restarting modes > > This is minimal changes needed for start up go7007 to work correctly > in 3.4 branch >

[PATCH] radio-sf16fmr2: add PnP support for SF16-FMD2

2012-05-13 Thread Ondrej Zary
Add PnP support to radio-sf16fmr2 driver to support SF16-FMD2 card (SB16 + TEA5757). The driver can now handle two cards (FMR2 is hardwired to 0x384, FMD2 can be put at 0x384 or 0x284 by PnP). Tested with both SF16-FMR2 and SF16-FMD2 (the can work at the same time by using kernel parameter "pnp_res

[PATCH 2/2] staging: media: go7007: Adlink MPG24 board

2012-05-13 Thread Volokh Konstantin
Changes: - wis-tw2804.ko module code was incompatible with 3.4 branch in initialization v4l2_subdev parts. now i2c_get_clientdata(...) contains v4l2_subdev struct instead non standard wis_tw2804 struct - Use V4L2 control framework Adds: - Additional chipset tw2804 controls with: gain,a

[PATCH 1/2] staging: media: go7007: Adlink MPG24 board

2012-05-13 Thread Volokh Konstantin
This patch applies only for Adlink MPG24 board with go7007, all these changes were tested for continuous loading & restarting modes This is minimal changes needed for start up go7007 to work correctly in 3.4 branch Changes: - When go7007 reset device, i2c was not working (need rewrite GPIO5)

Re: [PATCH 1/5] rtl2832 ver 0.3: suport for RTL2832 demodulator revised version

2012-05-13 Thread Antti Palosaari
On 12.05.2012 21:08, Thomas Mair wrote: Changes compared to version 0.2: - removed reading of signal strength for tuners FC0012,FC0013 (is now supported by fc0012,fc0013 driver) - moved definition of register names to rtl2832_priv.h - cleaned up demod private structure - replaced rtl2832_log2 fu

cron job: media_tree daily build: WARNINGS

2012-05-13 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:Sun May 13 19:00:19 CEST 2012 git hash:121b3ddbe4ad17df77cb7284239be0a63d9a66bd gcc version: i686-linux-gcc (GC

Re: AF9035 experimental header changes

2012-05-13 Thread Antti Palosaari
On 13.05.2012 20:26, Oliver Schinagl wrote: On 13-05-12 17:51, Antti Palosaari wrote: On 13.05.2012 18:24, Oliver Schinagl wrote: Hi antti, I've just updated my local branch of your experimental branch and got some conflicts because you moved the header inclusions from the C file to the header

Re: AF9035 experimental header changes

2012-05-13 Thread Oliver Schinagl
On 13-05-12 17:51, Antti Palosaari wrote: On 13.05.2012 18:24, Oliver Schinagl wrote: Hi antti, I've just updated my local branch of your experimental branch and got some conflicts because you moved the header inclusions from the C file to the header file. Why is that? I thought it was really b

Re: [PATCH] Support for tuner FC0012, revised version 0.4

2012-05-13 Thread Antti Palosaari
On 01.04.2012 01:26, Hans-Frieder Vogt wrote: Support for tuner Fitipower FC0012, revised version 0.4. Changes compared to version 0.3: - write function for multiple bytes at once removed again (because the maximum allowable length of an i2c transfer may be different between various demodulators

Re: Fwd: Bug#669715: dvb-apps: Channel/frequency/etc. data needs updating for London transmitters

2012-05-13 Thread Andy Furniss
Andy Furniss wrote: Russel Winder wrote: On Sun, 2012-05-13 at 15:23 +0100, Andy Furniss wrote: [...] Should that be 545833000 instead of 54580, and 529833000 instead of 52980? Possibly - I think if you calculate by hand from channel number and add or take the offset if it it+ or - th

Re: Fwd: Bug#669715: dvb-apps: Channel/frequency/etc. data needs updating for London transmitters

2012-05-13 Thread Andy Furniss
Russel Winder wrote: On Sun, 2012-05-13 at 15:23 +0100, Andy Furniss wrote: [...] Should that be 545833000 instead of 54580, and 529833000 instead of 52980? Possibly - I think if you calculate by hand from channel number and add or take the offset if it it+ or - then you do get the ex

Re: AF9035 experimental header changes

2012-05-13 Thread Antti Palosaari
On 13.05.2012 18:24, Oliver Schinagl wrote: Hi antti, I've just updated my local branch of your experimental branch and got some conflicts because you moved the header inclusions from the C file to the header file. Why is that? I thought it was really bad practice to have includes in header file

[PATCH] staging/media/as102: remove version.h include at as102_fe.c

2012-05-13 Thread joseph daniel
There was a warning when ran "make versioncheck" drivers/staging/media/as102/as102_fe.c: 20 linux/version.h not needed. Signed-off-by: joseph daniel --- drivers/staging/media/as102/as102_fe.c |2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/media/as102/as102_fe.c b/drive

AF9035 experimental header changes

2012-05-13 Thread Oliver Schinagl
Hi antti, I've just updated my local branch of your experimental branch and got some conflicts because you moved the header inclusions from the C file to the header file. Why is that? I thought it was really bad practice to have includes in header files. http://git.linuxtv.org/anttip/media_t

Re: Fwd: Bug#669715: dvb-apps: Channel/frequency/etc. data needs updating for London transmitters

2012-05-13 Thread Russel Winder
On Sun, 2012-05-13 at 15:23 +0100, Andy Furniss wrote: [...] > > Should that be 545833000 instead of 54580, and 529833000 instead of > > 52980? > > > Possibly - I think if you calculate by hand from channel number and add > or take the offset if it it + or - then you do get the extra 33.

Re: Fwd: Bug#669715: dvb-apps: Channel/frequency/etc. data needs updating for London transmitters

2012-05-13 Thread Andy Furniss
Russel Winder wrote: On Sun, 2012-05-13 at 11:15 +0100, Andy Furniss wrote: [...] Transmission mode changed from 2k to 8k in the uk after analogue switch off. Of course. I just failed to make that change in my files. With that changed I got some response from gnome-dvb-setup but it only analys

Re: [PATCH 5/8] added m4 directory to gitignore

2012-05-13 Thread Gregor Jasny
On 5/13/12 2:17 PM, André Roth wrote: > configure > +m4 > aclocal.m4 > autom4te.cache > build-aux The m4 directory also contains files that *are* under version control. Changes to these files will be undetected if the directory is being ignored. Maybe one can convince autotools to put generat

Re: [PATCH 1/6] libdvbv5 shared lib

2012-05-13 Thread Gregor Jasny
Hello, I noticed the dvb library is partially licensed as GPL2 and partially LGPL2.1+. Do you consider a re-licensing to LGPL2.1? For better maintainability in distributions I'd also suggest to hide non-public symbols like it's done for libv4l*. Thanks, Gregor -- To unsubscribe from this list: s

[PATCH 1/6] libdvbv5 shared lib

2012-05-13 Thread André Roth
--- Makefile.am|3 +- configure.ac |3 +- {utils/dvb => lib/include}/dvb-demux.h |0 {utils/dvb => lib/include}/dvb-fe.h|0 {utils/dvb => lib/include}/dvb-file.h |

[PATCH 5/8] added m4 directory to gitignore

2012-05-13 Thread André Roth
--- .gitignore |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/.gitignore b/.gitignore index f33eb98..d6bb3a3 100644 --- a/.gitignore +++ b/.gitignore @@ -18,6 +18,7 @@ config.status Makefile Makefile.in configure +m4 aclocal.m4 autom4te.cache build-aux -- 1.7.2.5

[PATCH 7/8] renamed dvb_frontend.h to dvb-frontend.h

2012-05-13 Thread André Roth
--- lib/include/dvb-fe.h |2 +- lib/include/dvb-frontend.h | 436 lib/include/dvb_frontend.h | 436 lib/libdvbv5/Makefile.am |2 +- lib/libdvbv5/descriptors.c |2 +- lib/libdvbv5/dvb-v5-

[PATCH 6/8] fixed bw for DVB-S

2012-05-13 Thread André Roth
--- utils/dvb/dvbv5-scan.c |9 +++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/utils/dvb/dvbv5-scan.c b/utils/dvb/dvbv5-scan.c index d54aa8c..63a8a44 100644 --- a/utils/dvb/dvbv5-scan.c +++ b/utils/dvb/dvbv5-scan.c @@ -138,7 +138,7 @@ static int new_freq_is_needed(stru

[PATCH 8/8] split README for utils/dvb/ and lib/libdvbv5

2012-05-13 Thread André Roth
--- lib/libdvbv5/Makefile.am |2 +- lib/libdvbv5/README | 59 ++ utils/dvb/README | 47 3 files changed, 60 insertions(+), 48 deletions(-) create mode 100644 lib/libdvbv5/README diff --git a/lib

[PATCH 4/8] use extern C for c++

2012-05-13 Thread André Roth
--- lib/include/dvb-fe.h |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/lib/include/dvb-fe.h b/lib/include/dvb-fe.h index a453424..4207ffe 100644 --- a/lib/include/dvb-fe.h +++ b/lib/include/dvb-fe.h @@ -88,6 +88,10 @@ struct dvb_v5_fe_parms { /* Open/close meth

[PATCH 2/8] fixed off by one parameter check error

2012-05-13 Thread André Roth
--- utils/dvb/dvbv5-scan.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/utils/dvb/dvbv5-scan.c b/utils/dvb/dvbv5-scan.c index d785998..d54aa8c 100644 --- a/utils/dvb/dvbv5-scan.c +++ b/utils/dvb/dvbv5-scan.c @@ -548,9 +548,9 @@ int main(int argc, char **argv)

[PATCH 1/8] workaround fix for segfault with CHANNEL file

2012-05-13 Thread André Roth
--- utils/dvb/dvb-file.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/utils/dvb/dvb-file.c b/utils/dvb/dvb-file.c index eb269b4..e1f2195 100644 --- a/utils/dvb/dvb-file.c +++ b/utils/dvb/dvb-file.c @@ -141,7 +141,7 @@ struct dvb_file *parse_format_oneline(const char *f

Re: [PATCH] fc0012 ver. 0.6: introduction of get_rf_strength function

2012-05-13 Thread Antti Palosaari
On 13.05.2012 11:16, poma wrote: On 05/12/2012 11:11 AM, Hans-Frieder Vogt wrote: Changes compared to version 0.5 of driver (sent 6 May): - Initial implementation of get_rf_strength function. Signed-off-by: Hans-Frieder Vogt fc0012.c | 72 ++

Re: Fwd: Bug#669715: dvb-apps: Channel/frequency/etc. data needs updating for London transmitters

2012-05-13 Thread Russel Winder
On Sun, 2012-05-13 at 11:15 +0100, Andy Furniss wrote: [...] > Transmission mode changed from 2k to 8k in the uk after analogue switch off. Of course. I just failed to make that change in my files. With that changed I got some response from gnome-dvb-setup but it only analysed one multiplex. > Ch

Re: Fwd: Bug#669715: dvb-apps: Channel/frequency/etc. data needs updating for London transmitters

2012-05-13 Thread Andy Furniss
Mark Purcell wrote: Not sure about w_scan, I have scan. It doesn't generate a channel list. I calculated the following replacement for the uk-CrystalPalace file: # UK, Crystal Palace # # Manually calculated by RLW 2012-04-21T10:20+01:00 # # T freq

Fwd: Bug#606728: dvbscan: Infinite loop parsing arguments

2012-05-13 Thread Mark Purcell
-- Forwarded Message -- Subject: Bug#606728: dvbscan: Infinite loop parsing arguments Date: Sat, 11 Dec 2010, 21:13:08 From: Vincent Pelletier To: Debian Bug Tracking System Package: dvb-apps Version: 1.1.1+rev1355-1 Severity: normal Tags: patch How to reproduce: dvbscan -

Fwd: Bug#669563: dvb-apps conflicts nmh

2012-05-13 Thread Mark Purcell
-- Forwarded Message -- Subject: Bug#669563: dvb-apps conflicts nmh Date: Fri, 20 Apr 2012, 06:37:37 From: Anders Hammarquist To: Debian Bug Tracking System Package: dvb-apps Version: 1.1.1+rev1457-4 Severity: normal The most recent dvb-apps conflicts with nmh. While they co

Fwd: Bug#669715: dvb-apps: Channel/frequency/etc. data needs updating for London transmitters

2012-05-13 Thread Mark Purcell
-- Forwarded Message -- Subject: Bug#669715: dvb-apps: Channel/frequency/etc. data needs updating for London transmitters Date: Sun, 22 Apr 2012, 13:17:48 From: Russel Winder To: Darren Salt CC: 669...@bugs.debian.org The representative of the Amalgamated Union of Philosophe

Re: [PATCH] fc0012 ver. 0.6: introduction of get_rf_strength function

2012-05-13 Thread poma
On 05/12/2012 11:11 AM, Hans-Frieder Vogt wrote: > Changes compared to version 0.5 of driver (sent 6 May): > - Initial implementation of get_rf_strength function. > > Signed-off-by: Hans-Frieder Vogt > > fc0012.c | 72 > ++- > 1 fil