Re: [PATCH 2/2 v2] gspca: Add MR97310A driver

2009-01-16 Thread Jean-Francois Moine
On Thu, 15 Jan 2009 22:36:49 -0600
Kyle Guinn ely...@gmail.com wrote:

 This patch adds support for USB webcams based on the MR97310A chip.
 It was tested with an Aiptek PenCam VGA+ webcam.

Hi Kyle,

I added your driver to my repository.

I just found a little bug. At line 250/251

data[8] = 0x01;
err_code = reg_w(gspca_dev, 10);

you set 9 values and you output 10 values.

BTW, as many of these USB control messages are constant, you should
better use static data or a table format (byte count, values)*.

Regards.

-- 
Ken ar c'hentan | ** Breizh ha Linux atav! **
Jef |   http://moinejf.free.fr/
--
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  http://vger.kernel.org/majordomo-info.html


Re: KWorld ATSC 115 all static

2009-01-16 Thread Jean Delvare
Hi Mauro, Trent,

On Fri, 16 Jan 2009 00:02:52 -0200, Mauro Carvalho Chehab wrote:
 On Thu, 15 Jan 2009 10:33:15 -0800 (PST)
 Trent Piepho xy...@speakeasy.org wrote:
 
  On Thu, 15 Jan 2009, Mauro Carvalho Chehab wrote:
   It has nothing to do with the load order. It is related to i2c binding. 
   With
   the current approach (before Hans patch), the i2c core will try to bind 
   the
   tuner after having 2 conditions satisfied:
 1) I2C bus were registered;
 2) tuner code is available.
  
   So, if you load tuner before saa7134 (or have it compiled in-kernel), the 
   code
   will try to probe tuners at the moment you register i2c bus. Otherwise, 
   it will
   try to bind only when request_module is handled.
  
   Some devices with DVB has an internal i2c gate. For a subset of these, 
   the i2c
   gate is inside the tuner. So, you need to bind the tuner device before 
   probing
   for the frontends. On the other set of devices, the gate is inside the 
   demod.
   So, you need to turn the i2c gate before running the i2c address seek for 
   tuner.
  
  I wonder if a better way to fix these problems is to use virtual I2C busses
  for the gate?
 
 For now, we should finish the Hans approach, since it also helps to stop using
 the legacy i2c methods. After having all drivers using it, we can do some
 cleanup at the drivers.
 
 However, I like the idea of providing a better support for those buses that
 have an i2c switch inside (I don't like to call it virtual - it is a real 
 i2c
 bus, where part of the bus is controlled by a switch).

I second this, the term virtual to qualify these buses is improper.
Better speak in terms of I2C segments, multiplexers and gates. Gates
can be seen as a degenerated form of multiplexers, but this is not
necessarily the best approach.

   When a device has some kind of i2c gate, it creates a new
  I2C adapter for the devices behind the gate.  The code for this virtual i2c
  adapter can just open the gate, pass of the request to the main i2c
  adapter, then close the gate.  Creating a new i2c adapter should trigger
  the i2c drivers that scan to do so and find new devices behind the gate.
  
  It seems like this would solve the scan order problem, since the bus the
  tuner/demod/whatever is on won't exist until the gate it is behind can be
  properly controlled.

Well, the scanning order problem is IMHO better resolved by
instantiating the devices explicitly instead of letting the i2c
subsystem probe for them. Everything is already in place to do this,
and a number of drivers are already being converted in this direction.

  There are a number of additional benefits too.  There are many devices that
  can be behind many different kinds of gates.  So we have all these gate
  control functions that must be called manually from all over the place.
  This adds bloat and developers are always forgetting to call them, which
  doesn't cause any problem they notice because their card doesn't have a
  gate.
  
  With manual gate control, we must remember to close the gate when done with
  the device.  But this isn't always done and the gate is left open.  These
  gates are there for a reason, to shield RF devices from noise created by
  the I2C bus, and so leaving them opens impairs RF performance.
  
  And when the gate is only controlled by the driver in the kernel, it is
  hard to manually debug/test i2c devices from userspace with i2c-dev.
 
 I'm not sure if we should implement it inside v4l core, or at i2c-core.

This is an excellent question, I don't know for sure myself.

 Maybe the latter could be more appropriate, since maybe some other devices 
 could have
 similar issues, like the embedded processor/controllers, where you have an i2c
 bus there connected to several different devices, like those used on cellular
 phones. 
 
 I bet that some embedded devices uses the same i2c bus for more than
 one subsystem (like having a radio and/or a webcam and some 
 temperature/battery
 sensors). In this case, a virtual i2c support could be interesting.
 
 Maybe Jean could give us some suggestions about the better approach for such 
 cases.

Support for I2C segments and multiplexers is needed for other systems,
yes. Some embedded systems have such multiplexers, and even some PC
motherboards do. The goal can be to isolate some portions of the I2C
bus, but in most cases it is simply a way to have more than one I2C
device with the same I2C address on a given bus.

Support for these multiplexers still isn't in the kernel so we are free
to imagine how an implementation could look like. As far as the gate
issue is concerned, the key point is whether multiplexers would always
have at least one outer segment selected, or not. In terms of
performance, that would be preferable, so that consecutive accesses to
a given chip on any outer segment are fast (no mux change). But then we
can't treat gates as muxes, because for gates we want the outer segment
isolated when we don't use it. This 

Re: Fw: [PATCH] E506r-composite-input

2009-01-16 Thread hermann pitton
Hi,

Am Donnerstag, den 15.01.2009, 23:35 -0200 schrieb Mauro Carvalho
Chehab:
 Message sent to the wrong address... it is not *-owner ;)
 
 Forwarded message:
 
 Date: Thu, 15 Jan 2009 21:58:55 +0900
 From: Tim Farrington t...@iinet.net.au
 To: Mauro Carvalho Chehab mche...@infradead.org,
 linux-media-ow...@vger.kernel.org
 Subject: [PATCH] E506r-composite-input
 
 
 Make correction to composite input plus svideo input
 to Avermedia E506R
 
 Signed-off-by: Tim Farrington t...@iinet.net.au
 
 
 
 
 
 
 
 Cheers,
 Mauro
 
 
 
 
 
 
 
 Unterschied
 zwischen
 Dateien-Anlage
 (E506r_composite.patch)
 
 Only in .: E506r_composite.patch
 diff
 -upr ./linux/drivers/media/video/saa7134/saa7134-cards.c 
 ../a/v4l-dvb/linux/drivers/media/video/saa7134/saa7134-cards.c
 --- ./linux/drivers/media/video/saa7134/saa7134-cards.c 2009-01-15
 21:42:05.0 +0900
 +++ ../a/v4l-dvb/linux/drivers/media/video/saa7134/saa7134-cards.c  
 2009-01-15 21:45:29.0 +0900
 @@ -4362,13 +4362,13 @@ struct saa7134_board saa7134_boards[] = 
  .amux = TV,
  .tv   = 1,
  }, {
 -.name = name_comp,
 -.vmux = 0,
 +.name = name_comp1,
 +.vmux = 3,
  .amux = LINE1,
  }, {
  .name = name_svideo,
  .vmux = 8,
 -.amux = LINE1,
 +.amux = LINE2,
  } },
  .radio = {
  .name = name_radio,
 

Mauro, I was never sure about why that patch, which introduced name_comp
was accepted very, very late in the drivers history. It previously
always started the enumeration with name_comp1.

If it should have any sense at all, I thought it was to avoid ambiguity
on such devices which have only one Composite input.

Tim, are you sure that Composite amux is LINE1 and S-Video LINE2?

It would be the first and only card ever seen with different amux for
those inputs and should be noted as unusual case. I doubt it has two
different audio-in connectors.

Cheers,
Hermann


--
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  http://vger.kernel.org/majordomo-info.html


Twinhan DST stops working under latest v4l-dvb

2009-01-16 Thread Aaron Theodore

It seems the latest v4l-dvb causes issues with my Twinhan 1020


bttv: driver version 0.9.17 loaded
bttv: using 8 buffers with 2080k (520 pages) each for capture
bttv: Bt8xx card found (0).
ACPI: PCI Interrupt :05:08.0[A] - Link [APC3] - GSI 18 (level, 
low) - IRQ 18
bttv0: Bt878 (rev 17) at :05:08.0, irq: 18, latency: 32, mmio: 
0xcb00

bttv0: using: Twinhan DST + clones [card=113,insmod option]
bttv0: gpio: en=, out= in=00fefffe [init]
bttv0: tuner absent
bttv0: add subdevice dvb0
bt878: AUDIO driver version 0.0.0 loaded
dvb_bt8xx: unable to determine DMA core of card 0,
dvb_bt8xx: if you have the ALSA bt87x audio driver installed, try 
removing it.

dvb-bt8xx: probe of dvb0 failed with error -14

i tried unloading all the sound modules made no difference (even though 
i didnt have the bt87x module loaded)


This card works on earlier kernel modules.

Any ideas?
--
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  http://vger.kernel.org/majordomo-info.html


Re: The pvrusb2 stuff you just pulled

2009-01-16 Thread Mauro Carvalho Chehab
On Thu, 15 Jan 2009 23:57:01 -0600 (CST)
Mike Isely is...@isely.net wrote:

 
 Mauro:
 
 Do you not find it strange that you show up as the credited source for 
 my recent changesets on the summary page http://linuxtv.org/hg/v4l-dvb?  
 (See 10236 - 10240.)  I suspect it's because you cherry picked them, 
 but that doesn't make it right.
 
 I could have sworn in the past that I've been able to pull in changes / 
 contributions into hg from other pvrusb2 users and successfully 
 preserved the credit in the change list summary.  What's the problem 
 here?

Hi Mike,

This is due to a deficiency at mercurial. On git, there are two concepts on
their meta-tags: patch author and committer. Unfortunately, mercurial has only
one meta-tag: user.

When a patch is cherry-picked, mercurial will use the current user as his
user meta-tag. This indicates who applied the patch at the mercurial tree.

In order to preserve both information on our tree, we are using the user meta
tag for the committer, and the From: field at the patch description as the
patch author, as stated at README.patches.

The committer info is important to allow us to see from what tree a patch came.

The -git conversion scripts properly handle the From: info when giving the
credits for the patch.

Cheers,
Mauro
--
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  http://vger.kernel.org/majordomo-info.html


Re: Fw: [PATCH] E506r-composite-input

2009-01-16 Thread Tim Farrington

hermann pitton wrote:

Hi,

Am Donnerstag, den 15.01.2009, 23:35 -0200 schrieb Mauro Carvalho
Chehab:
  

Message sent to the wrong address... it is not *-owner ;)

Forwarded message:

Date: Thu, 15 Jan 2009 21:58:55 +0900
From: Tim Farrington t...@iinet.net.au
To: Mauro Carvalho Chehab mche...@infradead.org,
linux-media-ow...@vger.kernel.org
Subject: [PATCH] E506r-composite-input


Make correction to composite input plus svideo input
to Avermedia E506R

Signed-off-by: Tim Farrington t...@iinet.net.au







Cheers,
Mauro







Unterschied
zwischen
Dateien-Anlage
(E506r_composite.patch)

Only in .: E506r_composite.patch
diff
-upr ./linux/drivers/media/video/saa7134/saa7134-cards.c 
../a/v4l-dvb/linux/drivers/media/video/saa7134/saa7134-cards.c
--- ./linux/drivers/media/video/saa7134/saa7134-cards.c 2009-01-15
21:42:05.0 +0900
+++ ../a/v4l-dvb/linux/drivers/media/video/saa7134/saa7134-cards.c  
2009-01-15 21:45:29.0 +0900
@@ -4362,13 +4362,13 @@ struct saa7134_board saa7134_boards[] = 
 .amux = TV,

 .tv   = 1,
 }, {
-.name = name_comp,
-.vmux = 0,
+.name = name_comp1,
+.vmux = 3,
 .amux = LINE1,
 }, {
 .name = name_svideo,
 .vmux = 8,
-.amux = LINE1,
+.amux = LINE2,
 } },
 .radio = {
 .name = name_radio,




Mauro, I was never sure about why that patch, which introduced name_comp
was accepted very, very late in the drivers history. It previously
always started the enumeration with name_comp1.

If it should have any sense at all, I thought it was to avoid ambiguity
on such devices which have only one Composite input.

Tim, are you sure that Composite amux is LINE1 and S-Video LINE2?

It would be the first and only card ever seen with different amux for
those inputs and should be noted as unusual case. I doubt it has two
different audio-in connectors.

Cheers,
Hermann


--
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  http://vger.kernel.org/majordomo-info.html

  

Hermann,
For the record, Avermedia are well known for not respecting the GPL, and 
finding ways to
have as many different chip combinations on as many different cards as 
they can,

and as many different input configurations as they can invent.
(Try looking at all the different chip combinations they use on all 
their Volar sticks.)


Avermedia make the E506R, which is a PCMCIA card.

It has a FM Radio input, a DVB-T/Analog Television input and a Remote 
Control.


It also has a mini-SCSI input, to which is connected an Audio Left 
cable, an Audio Right cable, and a Composite Video cable;

also separately connected to the SCSI is a S-Video input cable.


Regards,
Timf
--
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  http://vger.kernel.org/majordomo-info.html


Re: KWorld ATSC 115 all static

2009-01-16 Thread Trent Piepho
On Fri, 16 Jan 2009, Jean Delvare wrote:
 Hi Mauro, Trent,
 On Fri, 16 Jan 2009 00:02:52 -0200, Mauro Carvalho Chehab wrote:
  On Thu, 15 Jan 2009 10:33:15 -0800 (PST)
  Trent Piepho xy...@speakeasy.org wrote:
 
   On Thu, 15 Jan 2009, Mauro Carvalho Chehab wrote:
  For now, we should finish the Hans approach, since it also helps to stop 
  using
  the legacy i2c methods. After having all drivers using it, we can do some
  cleanup at the drivers.

How will this work for drivers like bttv, where the i2c address of the
tuner chips isn't know for every supported card?

  However, I like the idea of providing a better support for those buses that
  have an i2c switch inside (I don't like to call it virtual - it is a real 
  i2c
  bus, where part of the bus is controlled by a switch).

 I second this, the term virtual to qualify these buses is improper.
 Better speak in terms of I2C segments, multiplexers and gates. Gates
 can be seen as a degenerated form of multiplexers, but this is not
 necessarily the best approach.

Maybe I should say virtual i2c adapter.  The driver registers another i2c
adapter that does not represent a new i2c master, but instead a new i2c bus
segment on an existing i2c master.

When a device has some kind of i2c gate, it creates a new
   I2C adapter for the devices behind the gate.  The code for this virtual 
   i2c
   adapter can just open the gate, pass of the request to the main i2c
   adapter, then close the gate.  Creating a new i2c adapter should trigger
   the i2c drivers that scan to do so and find new devices behind the gate.
  
   It seems like this would solve the scan order problem, since the bus the
   tuner/demod/whatever is on won't exist until the gate it is behind can be
   properly controlled.

 Well, the scanning order problem is IMHO better resolved by
 instantiating the devices explicitly instead of letting the i2c
 subsystem probe for them. Everything is already in place to do this,
 and a number of drivers are already being converted in this direction.

The problem is that then one needs to know where the i2c devices are, and
this is not known for all cards.  When the bttv driver was converted to use
the new i2c subsystem a decade ago and used probing as the only way to find
devices, I said it was a bad idea.  But now we have this database of
cards with no information about what chips and what address.

   There are a number of additional benefits too.  There are many devices 
   that
   can be behind many different kinds of gates.  So we have all these gate
   control functions that must be called manually from all over the place.
   This adds bloat and developers are always forgetting to call them, which
   doesn't cause any problem they notice because their card doesn't have a
   gate.
  
   With manual gate control, we must remember to close the gate when done 
   with
   the device.  But this isn't always done and the gate is left open.  These
   gates are there for a reason, to shield RF devices from noise created by
   the I2C bus, and so leaving them opens impairs RF performance.
  
   And when the gate is only controlled by the driver in the kernel, it is
   hard to manually debug/test i2c devices from userspace with i2c-dev.
 
  I'm not sure if we should implement it inside v4l core, or at i2c-core.

 This is an excellent question, I don't know for sure myself.

Why does either need support?

static int cx88_gate_i2c_xfer(struct i2c_adapter *i2c_adap,
  struct i2c_msg *msgs, int num)
{
struct cx88_gate_i2c *gate= i2c_get_adapdata(i2c_adap);
int ret;

cx88_dvb_gate_ctrl(gate-core, 1);
ret = i2c_transfer(gate-core-i2c_adap, msgs, num);
cx88_dvb_gate_ctrl(gate-core, 0);
return ret;
}

...
static struct i2c_algorithm cx88_gate = {
.master_xfer = cx88_gate_i2c_xfer,
...
};
static struct i2c_adapter adap = {
.algo = cx88_gate,
...
};
struct cx88_gate_i2c *gate = kzalloc(sizeof(typeof(*gate)), GFP_KERNEL);
gate-core = core;
i2c_set_adapdata(adap, gate);
i2c_add_adapter(adap);

What part needs to go somewhere else?

 of 2 other:

 * At I2C bus driver level. We can have a pre-transfer handler and a
 post-transfer handler, which does what needs to be done (like opening
 and closing a gate) based on the address of the device that is being
 accessed. I had discussed this approach with Michael Krufky long ago.

This won't work when muxes are used to put multiple i2c devices with the
same address behind a single master.

It still has the problem of probe order when one wants to scan for chips.
The i2c core does scanning when a new adapter is created.  But, suppose the
mux is an i2c device on the adapter?  In order for the scanner to find
anything behind the mux, it needs to be detected and working before the bus
is scanned.  But this may not happen.  Say one 

[PATCH 2/4] em28xx: Clock (XCLK) Cleanup

2009-01-16 Thread Robert Krakora
em28xx: Clock (XCLK) Cleanup

From: Robert Krakora rob.krak...@messagenetsystems.com

Clock (XCLK) Cleanupt

Priority: normal

Signed-off-by: Robert Krakora rob.krak...@messagenetsystems.com

diff -r 6896782d783d linux/drivers/media/video/em28xx/em28xx-core.c
--- a/linux/drivers/media/video/em28xx/em28xx-core.cWed Jan 14
10:06:12 2009 -0200
+++ b/linux/drivers/media/video/em28xx/em28xx-core.cWed Jan 14
12:47:00 2009 -0500
@@ -424,7 +424,7 @@

xclk = dev-board.xclk  0x7f;
if (!dev-mute)
-   xclk |= 0x80;
+   xclk |= EM28XX_XCLK_AUDIO_UNMUTE;

ret = em28xx_write_reg(dev, EM28XX_R0F_XCLK, xclk);
if (ret  0)
@@ -437,6 +437,10 @@
/* Sets volume */
if (dev-audio_mode.ac97 != EM28XX_NO_AC97) {
int vol;
+
+   em28xx_write_ac97(dev, AC97_POWER_DOWN_CTRL, 0x4200);
+   em28xx_write_ac97(dev, AC97_EXT_AUD_CTRL, 0x0031);
+   em28xx_write_ac97(dev, AC97_PCM_IN_SRATE, 0xbb80);

/* LSB: left channel - both channels with the same level */
vol = (0x1f - dev-volume) | ((0x1f - dev-volume)  8);
--
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  http://vger.kernel.org/majordomo-info.html


[PATCH 1/4] em28xx: Fix audio URB transfer buffer memory leak and race condition/corruption of capture pointer

2009-01-16 Thread Robert Krakora
em28xx: Fix audio URB transfer buffer memory leak and race
condition/corruption of capture pointer

From: Robert Krakora rob.krak...@messagenetsystems.com

Fix audio URB transfer buffer memory leak and race
condition/corruption of capture pointer

Leak fix kindly contributed by Pádraig Brady.

Priority: normal

Signed-off-by: Robert Krakora rob.krak...@messagenetsystems.com

diff -r 7981bdd4e25a linux/drivers/media/video/em28xx/em28xx-audio.c
--- a/linux/drivers/media/video/em28xx/em28xx-audio.c   Mon Jan 12
00:18:04 2009 +
+++ b/linux/drivers/media/video/em28xx/em28xx-audio.c   Thu Jan 15
17:27:27 2009 -0500
@@ -66,6 +66,9 @@
   usb_unlink_urb(dev-adev.urb[i]);
   usb_free_urb(dev-adev.urb[i]);
   dev-adev.urb[i] = NULL;
+
+   kfree(dev-adev.transfer_buffer[i]);
+   dev-adev.transfer_buffer[i] = NULL;
   }

   return 0;
@@ -458,11 +461,15 @@
   *substream)
 #endif
 {
+   unsigned long flags;
+
   struct em28xx *dev;
-
   snd_pcm_uframes_t hwptr_done;
+
   dev = snd_pcm_substream_chip(substream);
+   spin_lock_irqsave(dev-adev.slock, flags);
   hwptr_done = dev-adev.hwptr_done_capture;
+   spin_unlock_irqrestore(dev-adev.slock, flags);

   return hwptr_done;
 }
--
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  http://vger.kernel.org/majordomo-info.html


[PATCH 3/4] em28xx: Fix for KWorld 330U Board

2009-01-16 Thread Robert Krakora
em28xx: Fix for KWorld 330U Board

From: Robert Krakora rob.krak...@messagenetsystems.com

Fix for KWorld 330U Board

Many thanks to Devin and Mauro!!!

Priority: normal

Signed-off-by: Robert Krakora rob.krak...@messagenetsystems.com

diff -r 6896782d783d linux/drivers/media/video/em28xx/em28xx-cards.c
--- a/linux/drivers/media/video/em28xx/em28xx-cards.c   Wed Jan 14
10:06:12 2009 -0200
+++ b/linux/drivers/media/video/em28xx/em28xx-cards.c   Wed Jan 14
12:46:43 2009 -0500
@@ -114,6 +114,18 @@
   {  -1,  -1, -1, -1},
 };
 #endif
+
+static struct em28xx_reg_seq kworld_330u_analog[] = {
+   {EM28XX_R08_GPIO,   0x6d,   ~EM_GPIO_4, 10},
+  {EM2880_R04_GPO,0x00,   0xff,   10},
+   {   -1, -1, -1, -1},
+};
+
+static struct em28xx_reg_seq kworld_330u_digital[] = {
+   {EM28XX_R08_GPIO,   0x6e,   ~EM_GPIO_4, 10},
+   {EM2880_R04_GPO,0x08,   0xff,   10},
+   {   -1, -1, -1, -1},
+};

 /* Callback for the most boards */
 static struct em28xx_reg_seq default_tuner_gpio[] = {
@@ -1242,29 +1254,33 @@
   .gpio = hauppauge_wintv_hvr_900_analog,
   } },
   },
-   [EM2883_BOARD_KWORLD_HYBRID_A316] = {
+   [EM2883_BOARD_KWORLD_HYBRID_330U] = {
   .name = Kworld PlusTV HD Hybrid 330,
   .tuner_type   = TUNER_XC2028,
   .tuner_gpio   = default_tuner_gpio,
   .decoder  = EM28XX_TVP5150,
   .mts_firmware = 1,
   .has_dvb  = 1,
-   .dvb_gpio = default_digital,
+   .dvb_gpio = kworld_330u_digital,
+   .xclk = EM28XX_XCLK_FREQUENCY_12MHZ,
+   .i2c_speed= EM28XX_I2C_CLK_WAIT_ENABLE |
EM28XX_I2C_EEPROM_ON_BOARD | EM28XX_I2C_EEPROM_KEY_VALID,
   .input= { {
   .type = EM28XX_VMUX_TELEVISION,
   .vmux = TVP5150_COMPOSITE0,
   .amux = EM28XX_AMUX_VIDEO,
-   .gpio = default_analog,
+   .gpio = kworld_330u_analog,
+   .aout = EM28XX_AOUT_PCM_IN | EM28XX_AOUT_PCM_STEREO,
   }, {
   .type = EM28XX_VMUX_COMPOSITE1,
   .vmux = TVP5150_COMPOSITE1,
   .amux = EM28XX_AMUX_LINE_IN,
-   .gpio = hauppauge_wintv_hvr_900_analog,
+   .gpio = kworld_330u_analog,
+   .aout = EM28XX_AOUT_PCM_IN | EM28XX_AOUT_PCM_STEREO,
   }, {
   .type = EM28XX_VMUX_SVIDEO,
   .vmux = TVP5150_SVIDEO,
   .amux = EM28XX_AMUX_LINE_IN,
-   .gpio = hauppauge_wintv_hvr_900_analog,
+   .gpio = kworld_330u_analog,
   } },
   },
   [EM2820_BOARD_COMPRO_VIDEOMATE_FORYOU] = {
@@ -1318,7 +1334,7 @@
   .driver_info = EM2880_BOARD_KWORLD_DVB_310U },
 #endif
   { USB_DEVICE(0xeb1a, 0xa316),
-   .driver_info = EM2883_BOARD_KWORLD_HYBRID_A316 },
+   .driver_info = EM2883_BOARD_KWORLD_HYBRID_330U },
   { USB_DEVICE(0xeb1a, 0xe320),
   .driver_info = EM2880_BOARD_MSI_DIGIVOX_AD_II },
   { USB_DEVICE(0xeb1a, 0xe323),
@@ -1594,6 +1610,10 @@
   case EM2880_BOARD_PINNACLE_PCTV_HD_PRO:
   /* FIXME: Better to specify the needed IF */
   ctl-demod = XC3028_FE_DEFAULT;
+   break;
+   case EM2883_BOARD_KWORLD_HYBRID_330U:
+   ctl-demod = XC3028_FE_CHINA;
+   ctl-fname = XC2028_DEFAULT_FIRMWARE;
   break;
   default:
   ctl-demod = XC3028_FE_OREN538;
diff -r 6896782d783d linux/drivers/media/video/em28xx/em28xx-dvb.c
--- a/linux/drivers/media/video/em28xx/em28xx-dvb.c Wed Jan 14
10:06:12 2009 -0200
+++ b/linux/drivers/media/video/em28xx/em28xx-dvb.c Wed Jan 14
12:47:00 2009 -0500
@@ -29,6 +29,7 @@

 #include lgdt330x.h
 #include zl10353.h
+#include s5h1409.h
 #ifdef EM28XX_DRX397XD_SUPPORT
 #include drx397xD.h
 #endif
@@ -231,6 +232,15 @@
  .no_tuner = 1,
  .parallel_ts = 1,
  .if2 = 45600,
+};
+
+static struct s5h1409_config em28xx_s5h1409_with_xc3028 = {
+   .demod_address = 0x32  1,
+   .output_mode   = S5H1409_PARALLEL_OUTPUT,
+   .gpio  = S5H1409_GPIO_OFF,
+   .inversion = S5H1409_INVERSION_OFF,
+   .status_mode   = S5H1409_DEMODLOCKING,
+   .mpeg_timing   = S5H1409_MPEGTIMING_CONTINOUS_NONINVERTING_CLOCK
 };

 #ifdef EM28XX_DRX397XD_SUPPORT
@@ -413,7 +423,6 @@
  case EM2883_BOARD_HAUPPAUGE_WINTV_HVR_850:
  case EM2883_BOARD_HAUPPAUGE_WINTV_HVR_950:
  case EM2880_BOARD_PINNACLE_PCTV_HD_PRO:
-  

[PATCH 4/4] em28xx: Fix for KWorld 330U AC97

2009-01-16 Thread Robert Krakora
em28xx: Fix for KWorld 330U AC97

From: Robert Krakora rob.krak...@messagenetsystems.com

Fix for KWorld 330U AC97

Many thanks to Devin and Mauro again!!!

Priority: normal

Signed-off-by: Robert Krakora rob.krak...@messagenetsystems.com

diff -r 6896782d783d linux/drivers/media/video/em28xx/em28xx-core.c
--- a/linux/drivers/media/video/em28xx/em28xx-core.cWed Jan 14
10:06:12 2009 -0200
+++ b/linux/drivers/media/video/em28xx/em28xx-core.cWed Jan 14
12:47:00 2009 -0500
@@ -424,7 +424,7 @@

   xclk = dev-board.xclk  0x7f;
   if (!dev-mute)
-   xclk |= 0x80;
+   xclk |= EM28XX_XCLK_AUDIO_UNMUTE;

   ret = em28xx_write_reg(dev, EM28XX_R0F_XCLK, xclk);
   if (ret  0)
@@ -437,6 +437,10 @@
   /* Sets volume */
   if (dev-audio_mode.ac97 != EM28XX_NO_AC97) {
   int vol;
+
+   em28xx_write_ac97(dev, AC97_POWER_DOWN_CTRL, 0x4200);
+   em28xx_write_ac97(dev, AC97_EXT_AUD_CTRL, 0x0031);
+   em28xx_write_ac97(dev, AC97_PCM_IN_SRATE, 0xbb80);

   /* LSB: left channel - both channels with the same level */
   vol = (0x1f - dev-volume) | ((0x1f - dev-volume)  8);
--
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  http://vger.kernel.org/majordomo-info.html


Re: KWorld ATSC 115 all static

2009-01-16 Thread Jean Delvare
Hi Trent,

On Fri, 16 Jan 2009 05:34:59 -0800 (PST), Trent Piepho wrote:
 On Fri, 16 Jan 2009, Jean Delvare wrote:
  Hi Mauro, Trent,
  On Fri, 16 Jan 2009 00:02:52 -0200, Mauro Carvalho Chehab wrote:
   For now, we should finish the Hans approach, since it also helps to stop 
   using
   the legacy i2c methods. After having all drivers using it, we can do some
   cleanup at the drivers.
 
 How will this work for drivers like bttv, where the i2c address of the
 tuner chips isn't know for every supported card?

Is this a problem in practice? My understanding was that I2C gates were
relatively recent in the history of V4L devices, so I assumed that for
devices with an I2C gate we would know where the devices are.

  (...)
  Well, the scanning order problem is IMHO better resolved by
  instantiating the devices explicitly instead of letting the i2c
  subsystem probe for them. Everything is already in place to do this,
  and a number of drivers are already being converted in this direction.
 
 The problem is that then one needs to know where the i2c devices are, and
 this is not known for all cards.  When the bttv driver was converted to use
 the new i2c subsystem a decade ago and used probing as the only way to find
 devices, I said it was a bad idea.  But now we have this database of
 cards with no information about what chips and what address.

I understand the problem. This is why even the new-style (standard) i2c
device driver binding still offers two flavors of device detection, to
handle the older devices.

   I'm not sure if we should implement it inside v4l core, or at i2c-core.
 
  This is an excellent question, I don't know for sure myself.
 
 Why does either need support?
 
 static int cx88_gate_i2c_xfer(struct i2c_adapter *i2c_adap,
 struct i2c_msg *msgs, int num)
 {
   struct cx88_gate_i2c *gate= i2c_get_adapdata(i2c_adap);
   int ret;
 
   cx88_dvb_gate_ctrl(gate-core, 1);
   ret = i2c_transfer(gate-core-i2c_adap, msgs, num);
   cx88_dvb_gate_ctrl(gate-core, 0);
   return ret;
 }
 
 ...
   static struct i2c_algorithm cx88_gate = {
   .master_xfer = cx88_gate_i2c_xfer,
   ...
   };
   static struct i2c_adapter adap = {
   .algo = cx88_gate,
   ...
   };
   struct cx88_gate_i2c *gate = kzalloc(sizeof(typeof(*gate)), GFP_KERNEL);
   gate-core = core;
   i2c_set_adapdata(adap, gate);
   i2c_add_adapter(adap);
 
 What part needs to go somewhere else?

This is indeed a possible implementation. One could argue though that
it's a bit overkill to instantiate a separate i2c_adapter just for a
gate. There are also caveats you must pay attention to. Two things in
particular that come to my mind:

* You must set the adapter depth properly, otherwise lockdep will
complain.

* Your proposal above, in its simple form, is incompatible with I2C
device detection, because devices which are before the gate would be
double-detected (once on each segment.)

The first point is very easy to handle, the second is a little more
complicated. Basically you should add an address check at the beginning
of cx88_gate_i2c_xfer() to reject all transfers except to the device
you know is behind the gate.

At which point it is no longer clear why you want to have 2
i2c_adapters. You can have just one which opens (and closes) the gate
automatically for the right I2C address and not for the other addresses.

  of 2 other:
 
  * At I2C bus driver level. We can have a pre-transfer handler and a
  post-transfer handler, which does what needs to be done (like opening
  and closing a gate) based on the address of the device that is being
  accessed. I had discussed this approach with Michael Krufky long ago.
 
 This won't work when muxes are used to put multiple i2c devices with the
 same address behind a single master.

Sorry for not being clear, I was only trying to address the gate issue
here, not the (more complex) multiplexing issue. I am not aware of V4L
devices having real I2C muxes?

 It still has the problem of probe order when one wants to scan for chips.
 The i2c core does scanning when a new adapter is created.  But, suppose the
 mux is an i2c device on the adapter?  In order for the scanner to find
 anything behind the mux, it needs to be detected and working before the bus
 is scanned.  But this may not happen.  Say one calls:
 i2c_add_adapter(adap); i2c_new_device(adap, the_mux);
 
 If the client driver for the device behind the mux uses probing, and is
 already loaded when i2c_add_adapter() is called, then it will be probed for
 before the i2c_new_device() call and won't be found because the mux isn't
 working.

You are right, this is a problem.

 If instead the mux driver created a new i2c adapter for the segment(s)
 behind it, which would happen when i2c_new_device() is called, then those
 segments would be probed at the correct time.

This doesn't fully solve the problem, because you have 

Re: [PULL] http://linuxtv.org/hg/~mcisely/pvrusb2

2009-01-16 Thread Mike Isely
On Thu, 15 Jan 2009, Mauro Carvalho Chehab wrote:

 
 OK. Well, the usage he wants is something that is better fitted by using sysfs
 info. There, you should have all information to uniquely identify a device:
 driver, bus location (on PCI this can be relevant), MAC (for dvb devices), 
 etc.
 IMO, the proper way is to add the serial number at sysfs, and let the bus_info
 point to the proper bus location. Having the bus location, an userspace app 
 can
 seek the sysfs and look for the udev info.
 
 For example, on an em28xx analog device I have here, bus_info returns
 1-3. Ok, this is a very bad way to specify the bus. IMO, we should use
 usb_make_path() to generate the canonical name for USB buses.

I will review what the pvrusb2 driver is doing and change it to use 
usb_make_path() if needed.

However given all the other information about the device that querycap 
returns, a serial number in that batch would be right at home there.  
Requiring an app to go through everything you described is a pretty 
complex process for what should be very simple datum.

In any case, right now the serial number in the pvrusb2 is not available 
through that means because I haven't done anything to make it available 
to udev.  I'd like to do something, but so far I have found no 
information on how to make that happen.

  -Mike

-- 

Mike Isely
isely @ pobox (dot) com
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F5 C1 E8
--
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  http://vger.kernel.org/majordomo-info.html


Cross-posting linux-media, linux-dvb etc

2009-01-16 Thread Patrick Boettcher

Hi Mauro,

Since the creation of linux-media@vger.kernel.org I'm seeing lots of 
cross-postings between linux-dvb, linux-media and video4linux. This is a 
little bit annoying if you are subscribed to all of those lists.


Worse is, that some people only send requests to linux-media. Like that 
linux-dvb-only subscribers can't help...


Why not closing linux-dvb (and video4linux) and transferring the currently 
subscribed users to linux-media automatically?


regards,
Patrick.
--
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  http://vger.kernel.org/majordomo-info.html


Re: [linux-dvb] Cross-posting linux-media, linux-dvb etc

2009-01-16 Thread Hans Verkuil
On Friday 16 January 2009 15:48:45 Patrick Boettcher wrote:
 Hi Mauro,

 Since the creation of linux-media@vger.kernel.org I'm seeing lots of
 cross-postings between linux-dvb, linux-media and video4linux. This
 is a little bit annoying if you are subscribed to all of those lists.

 Worse is, that some people only send requests to linux-media. Like
 that linux-dvb-only subscribers can't help...

 Why not closing linux-dvb (and video4linux) and transferring the
 currently subscribed users to linux-media automatically?

I agree with Patrick. I suggest a daily automatic posting to linux-dvb 
and video4linux telling people that on February 1st these lists 
disappear and that they should use linux-media instead. Then they can 
be closed down at the end of the month. I definitely wouldn't wait any 
longer since it is rather messy right now. One month transition period 
seems reasonable to me.

Regards,

Hans

-- 
Hans Verkuil - video4linux developer - sponsored by TANDBERG
--
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  http://vger.kernel.org/majordomo-info.html


RFC - Flexcop Streaming watchdog (VDSB)

2009-01-16 Thread Patrick Boettcher

Hi lists,

For years there has been the Video Data Stream Borken-error with VDR and
Technisat cards: The error occured randomly and unfrequently. A 
work-around for that problem was to restart VDR and let the driver reset 
the pid-filtering and streaming interface.


In fact it turned out, that the problem is worse with setups not based on 
VDR and the VDSB-error could be really easily reproduced (I'm not sure 
if this applies to all generations on SkyStar2-card). I'm skipping the 
description of the problem here...


Attached you'll find a patch which works around the hardware bug which is 
causing VDSB-error without needing to reload the driver.


There a struct-work-watchdog looking at the number of irq-received while 
having PIDs active in the PID-filter. If no IRQs are received, the 
pid-filter-system is reset.


It seems to fix the problem and so far I've not seen any false positives 
(like resetting the pid-filter even though streaming is working fine).


Before asking to pull the patch I'd like to discuss an issue: my 
work-around is iterating over the pid-filter-list in the dvb_demux. I'm 
doing this in the struct-work-callback. In dvb_demux.c I see that this 
list is protected with a spinlock. When I now try to take the spinlock in 
the work-function I'll get a nice message saying, that I cannot do take a 
spinlock in a work-function.


What can I do? What is the proper way to protect access to this list? Is 
it needed at all?


thanks for you input in advance,
Patrick.

--
  Mail: patrick.boettc...@desy.de
  WWW:  http://www.wi-bw.tfh-wildau.de/~pboettch/diff -r 7981bdd4e25a linux/drivers/media/dvb/b2c2/flexcop-hw-filter.c
--- a/linux/drivers/media/dvb/b2c2/flexcop-hw-filter.c  Mon Jan 12 00:18:04 
2009 +
+++ b/linux/drivers/media/dvb/b2c2/flexcop-hw-filter.c  Fri Jan 16 15:46:32 
2009 +0100
@@ -179,7 +179,7 @@
 
/* if it was the first or last feed request change the stream-status */
if (fc-feedcount == onoff) {
-   flexcop_rcv_data_ctrl(fc,onoff);
+   flexcop_rcv_data_ctrl(fc, onoff);
if (fc-stream_control) /* device specific stream control */
fc-stream_control(fc,onoff);
 
@@ -192,6 +192,7 @@
 
return 0;
 }
+EXPORT_SYMBOL(flexcop_pid_feed_control);
 
 void flexcop_hw_filter_init(struct flexcop_device *fc)
 {
diff -r 7981bdd4e25a linux/drivers/media/dvb/b2c2/flexcop-pci.c
--- a/linux/drivers/media/dvb/b2c2/flexcop-pci.cMon Jan 12 00:18:04 
2009 +
+++ b/linux/drivers/media/dvb/b2c2/flexcop-pci.cFri Jan 16 15:46:32 
2009 +0100
@@ -13,9 +13,9 @@
 module_param(enable_pid_filtering, int, 0444);
 MODULE_PARM_DESC(enable_pid_filtering, enable hardware pid filtering: 
supported values: 0 (fullts), 1);
 
-static int irq_chk_intv;
+static int irq_chk_intv = 100;
 module_param(irq_chk_intv, int, 0644);
-MODULE_PARM_DESC(irq_chk_intv, set the interval for IRQ watchdog (currently 
just debugging).);
+MODULE_PARM_DESC(irq_chk_intv, set the interval for IRQ streaming watchdog.);
 
 #ifdef CONFIG_DVB_B2C2_FLEXCOP_DEBUG
 #define dprintk(level,args...) \
@@ -34,7 +34,7 @@
 
 static int debug;
 module_param(debug, int, 0644);
-MODULE_PARM_DESC(debug, set debug level (1=info,2=regs,4=TS,8=irqdma 
(|-able)). DEBSTATUS);
+MODULE_PARM_DESC(debug, set debug level (1=info,2=regs,4=TS,8=irqdma,16=check 
(|-able)). DEBSTATUS);
 
 #define DRIVER_VERSION 0.1
 #define DRIVER_NAME Technisat/B2C2 FlexCop II/IIb/III Digital TV PCI Driver
@@ -58,6 +58,8 @@
int active_dma1_addr; /* 0 = addr0 of dma1; 1 = addr1 of dma1 */
u32 last_dma1_cur_pos; /* position of the pointer last time the 
timer/packet irq occured */
int count;
+   int count_prev;
+   int stream_problem;
 
spinlock_t irq_lock;
 
@@ -115,18 +117,47 @@
 #endif
struct flexcop_device *fc = fc_pci-fc_dev;
 
-   flexcop_ibi_value v = fc-read_ibi_reg(fc,sram_dest_reg_714);
+   if (fc-feedcount) {
+   flexcop_ibi_value v = fc-read_ibi_reg(fc,sram_dest_reg_714);
 
-   flexcop_dump_reg(fc_pci-fc_dev,dma1_000,4);
+   //  flexcop_dump_reg(fc_pci-fc_dev,dma1_000,4);
 
-   if (v.sram_dest_reg_714.net_ovflow_error)
-   deb_chk(sram net_ovflow_error\n);
-   if (v.sram_dest_reg_714.media_ovflow_error)
-   deb_chk(sram media_ovflow_error\n);
-   if (v.sram_dest_reg_714.cai_ovflow_error)
-   deb_chk(sram cai_ovflow_error\n);
-   if (v.sram_dest_reg_714.cai_ovflow_error)
-   deb_chk(sram cai_ovflow_error\n);
+#if 0
+   deb_chk(net_ovflow_error: %d, media_ovflow_error: %d, 
cai_ovflow_error: %d, cao_ovflow_error: %d, sram_dma: %d, maximumfill: %d\n,
+   v.sram_dest_reg_714.net_ovflow_error,
+   v.sram_dest_reg_714.media_ovflow_error,
+   v.sram_dest_reg_714.cai_ovflow_error,
+   

Re: [linux-dvb] Cross-posting linux-media, linux-dvb etc

2009-01-16 Thread Mike Isely
On Fri, 16 Jan 2009, Hans Verkuil wrote:

 On Friday 16 January 2009 15:48:45 Patrick Boettcher wrote:
  Hi Mauro,
 
  Since the creation of linux-media@vger.kernel.org I'm seeing lots of
  cross-postings between linux-dvb, linux-media and video4linux. This
  is a little bit annoying if you are subscribed to all of those lists.
 
  Worse is, that some people only send requests to linux-media. Like
  that linux-dvb-only subscribers can't help...
 
  Why not closing linux-dvb (and video4linux) and transferring the
  currently subscribed users to linux-media automatically?
 
 I agree with Patrick. I suggest a daily automatic posting to linux-dvb 
 and video4linux telling people that on February 1st these lists 
 disappear and that they should use linux-media instead. Then they can 
 be closed down at the end of the month. I definitely wouldn't wait any 
 longer since it is rather messy right now. One month transition period 
 seems reasonable to me.
 

Amen to that.  I've been telling people to go over to linux-media, but 
old habits are hard to break.  It's time to actually make a clean break 
from the old lists.

  -Mike


-- 

Mike Isely
isely @ pobox (dot) com
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F5 C1 E8
--
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  http://vger.kernel.org/majordomo-info.html


Re: [PULL] http://linuxtv.org/hg/~mcisely/pvrusb2

2009-01-16 Thread Janne Grunau
On Friday 16 January 2009 15:39:33 Mike Isely wrote:
 In any case, right now the serial number in the pvrusb2 is not available
 through that means because I haven't done anything to make it available
 to udev.  I'd like to do something, but so far I have found no
 information on how to make that happen.

You shouldn't need to do anything special. The serial number is available 
through the parent USB device. It can be used for udev rules through 
ATTRS{serial} and in sysfs entry of the video device through device/serial.

Janne
--
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  http://vger.kernel.org/majordomo-info.html


Re: [linux-dvb] Cross-posting linux-media, linux-dvb etc

2009-01-16 Thread Luca Tettamanti
On Fri, Jan 16, 2009 at 4:52 PM, Benny Amorsen benny+use...@amorsen.dk wrote:
 Mike Isely is...@isely.net writes:

 Amen to that.  I've been telling people to go over to linux-media, but
 old habits are hard to break.  It's time to actually make a clean break
 from the old lists.

 Is linux-media available on gmane?

Yup, here it is:
http://dir.gmane.org/gmane.linux.drivers.video-input-infrastructure

and also:
http://www.mail-archive.com/linux-media@vger.kernel.org/

Luca
--
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  http://vger.kernel.org/majordomo-info.html


Re: [linux-dvb] mxl5005s tuner analog support

2009-01-16 Thread Devin Heitmueller
On Fri, Jan 16, 2009 at 11:32 AM, Andreas linuxdr...@dslextreme.com wrote:
 Devin

 I seem to be affected by that 3db loss between the old mxl500x and the
 mxl5005s driver. Did you ever get the datasheets and is anyone perhaps
 even working on squeezing a little more out of the driver?

 --
 Gruß
 Andreas

Hello Andreas,

The guys at Pinnacle very kindly introduced me to someone over at
Maxlinear, and I am trying to get the datasheet.  I will be looking at
the device's performance more closely over the next couple of weeks as
I get it working with the Pinnacle 880e.

Devin

-- 
Devin J. Heitmueller
http://www.devinheitmueller.com
AIM: devinheitmueller
--
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  http://vger.kernel.org/majordomo-info.html


Re: [linux-dvb] Cross-posting linux-media, linux-dvb etc

2009-01-16 Thread Benny Amorsen
Mike Isely is...@isely.net writes:

 Amen to that.  I've been telling people to go over to linux-media, but 
 old habits are hard to break.  It's time to actually make a clean break 
 from the old lists.

Is linux-media available on gmane?


/Benny

--
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  http://vger.kernel.org/majordomo-info.html


Re: [linux-dvb] Cross-posting linux-media, linux-dvb etc

2009-01-16 Thread Lars Hanisch

Mike Isely wrote:

On Fri, 16 Jan 2009, Hans Verkuil wrote:


On Friday 16 January 2009 15:48:45 Patrick Boettcher wrote:

Hi Mauro,

Since the creation of linux-media@vger.kernel.org I'm seeing lots of
cross-postings between linux-dvb, linux-media and video4linux. This
is a little bit annoying if you are subscribed to all of those lists.

Worse is, that some people only send requests to linux-media. Like
that linux-dvb-only subscribers can't help...

Why not closing linux-dvb (and video4linux) and transferring the
currently subscribed users to linux-media automatically?
I agree with Patrick. I suggest a daily automatic posting to linux-dvb 
and video4linux telling people that on February 1st these lists 
disappear and that they should use linux-media instead. Then they can 
be closed down at the end of the month. I definitely wouldn't wait any 
longer since it is rather messy right now. One month transition period 
seems reasonable to me.




Amen to that.  I've been telling people to go over to linux-media, but 
old habits are hard to break.  It's time to actually make a clean break 
from the old lists.


 +1 from me

 Although I'm not an active developer (I'm just an interested user), 
reading the lists at the moment is hard...


Lars.
--
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  http://vger.kernel.org/majordomo-info.html


Re: [linux-dvb] Cross-posting linux-media, linux-dvb etc

2009-01-16 Thread Mauro Carvalho Chehab
On Fri, 16 Jan 2009 18:13:26 +0100
Lars Hanisch d...@cinnamon-sage.de wrote:

 Mike Isely wrote:
  On Fri, 16 Jan 2009, Hans Verkuil wrote:
  
  On Friday 16 January 2009 15:48:45 Patrick Boettcher wrote:
  Hi Mauro,
 
  Since the creation of linux-media@vger.kernel.org I'm seeing lots of
  cross-postings between linux-dvb, linux-media and video4linux. This
  is a little bit annoying if you are subscribed to all of those lists.
 
  Worse is, that some people only send requests to linux-media. Like
  that linux-dvb-only subscribers can't help...
 
  Why not closing linux-dvb (and video4linux) and transferring the
  currently subscribed users to linux-media automatically?
  I agree with Patrick. I suggest a daily automatic posting to linux-dvb 
  and video4linux telling people that on February 1st these lists 
  disappear and that they should use linux-media instead. Then they can 
  be closed down at the end of the month. I definitely wouldn't wait any 
  longer since it is rather messy right now. One month transition period 
  seems reasonable to me.
 
  
  Amen to that.  I've been telling people to go over to linux-media, but 
  old habits are hard to break.  It's time to actually make a clean break 
  from the old lists.
 
   +1 from me
 
   Although I'm not an active developer (I'm just an interested user), 
 reading the lists at the moment is hard...

Instead of just removing the ML, maybe the better is to change the reply to to
linux-media and send an autoreply message to the sender.
 
 Lars.
 --
 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  http://vger.kernel.org/majordomo-info.html




Cheers,
Mauro
--
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  http://vger.kernel.org/majordomo-info.html


Re: [linux-dvb] Cross-posting linux-media, linux-dvb etc

2009-01-16 Thread Mauro Carvalho Chehab
On Fri, 16 Jan 2009 15:59:12 -0200
Mauro Carvalho Chehab mche...@infradead.org wrote:

 Instead of just removing the ML, maybe the better is to change the reply to to
 linux-media and send an autoreply message to the sender.

Done. Any posts to linux-dvb will receive this message:

On Fri, 16 Jan 2009 18:59:48 +0100
linux-dvb-boun...@linuxtv.org wrote:

 This ML is deprecated. Please use linux-media@vger.kernel.org instead.
 For more info about linux-media@vger.kernel.org, please read:
 http://vger.kernel.org/vger-lists.html#linux-media




Cheers,
Mauro



Cheers,
Mauro
--
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  http://vger.kernel.org/majordomo-info.html


Re: [linux-dvb] mxl5005s tuner analog support

2009-01-16 Thread Andreas
Am Freitag, 16. Januar 2009 08:35:49 schrieb Devin Heitmueller:
 Hello Andreas,

 The guys at Pinnacle very kindly introduced me to someone over at
 Maxlinear, and I am trying to get the datasheet.  I will be looking
 at the device's performance more closely over the next couple of
 weeks as I get it working with the Pinnacle 880e.

 Devin

Devin,
Thanks for the heads up and the good news!

-- 
Gruß
Andreas
--
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  http://vger.kernel.org/majordomo-info.html


[cron job] ERRORS: armv5 armv5-ixp armv5-omap2 i686 m32r mips powerpc64 x86_64 v4l-dvb build

2009-01-16 Thread Hans Verkuil
(This message is generated daily by a cron job that builds v4l-dvb for
the kernels and architectures in the list below.)

Results of the daily build of v4l-dvb:

date:Fri Jan 16 19:00:02 CET 2009
path:http://www.linuxtv.org/hg/v4l-dvb
changeset:   10241:7981bdd4e25a
gcc version: gcc (GCC) 4.3.1
hardware:x86_64
host os: 2.6.26

linux-2.6.16.61-armv5: OK
linux-2.6.17.14-armv5: OK
linux-2.6.18.8-armv5: OK
linux-2.6.19.5-armv5: OK
linux-2.6.20.21-armv5: OK
linux-2.6.21.7-armv5: OK
linux-2.6.22.19-armv5: OK
linux-2.6.23.12-armv5: OK
linux-2.6.24.7-armv5: OK
linux-2.6.25.11-armv5: OK
linux-2.6.26-armv5: OK
linux-2.6.27-armv5: WARNINGS
linux-2.6.28-armv5: WARNINGS
linux-2.6.29-rc1-armv5: ERRORS
linux-2.6.27-armv5-ixp: WARNINGS
linux-2.6.28-armv5-ixp: WARNINGS
linux-2.6.29-rc1-armv5-ixp: ERRORS
linux-2.6.27-armv5-omap2: WARNINGS
linux-2.6.28-armv5-omap2: WARNINGS
linux-2.6.29-rc1-armv5-omap2: ERRORS
linux-2.6.16.61-i686: OK
linux-2.6.17.14-i686: OK
linux-2.6.18.8-i686: WARNINGS
linux-2.6.19.5-i686: WARNINGS
linux-2.6.20.21-i686: WARNINGS
linux-2.6.21.7-i686: WARNINGS
linux-2.6.22.19-i686: WARNINGS
linux-2.6.23.12-i686: WARNINGS
linux-2.6.24.7-i686: WARNINGS
linux-2.6.25.11-i686: WARNINGS
linux-2.6.26-i686: WARNINGS
linux-2.6.27-i686: WARNINGS
linux-2.6.28-i686: WARNINGS
linux-2.6.29-rc1-i686: ERRORS
linux-2.6.16.61-m32r: OK
linux-2.6.17.14-m32r: OK
linux-2.6.18.8-m32r: OK
linux-2.6.19.5-m32r: OK
linux-2.6.20.21-m32r: OK
linux-2.6.21.7-m32r: OK
linux-2.6.23.12-m32r: OK
linux-2.6.24.7-m32r: OK
linux-2.6.25.11-m32r: OK
linux-2.6.26-m32r: OK
linux-2.6.27-m32r: OK
linux-2.6.28-m32r: OK
linux-2.6.29-rc1-m32r: ERRORS
linux-2.6.16.61-mips: OK
linux-2.6.26-mips: WARNINGS
linux-2.6.27-mips: WARNINGS
linux-2.6.28-mips: WARNINGS
linux-2.6.29-rc1-mips: ERRORS
linux-2.6.27-powerpc64: WARNINGS
linux-2.6.28-powerpc64: WARNINGS
linux-2.6.29-rc1-powerpc64: ERRORS
linux-2.6.16.61-x86_64: OK
linux-2.6.17.14-x86_64: OK
linux-2.6.18.8-x86_64: WARNINGS
linux-2.6.19.5-x86_64: WARNINGS
linux-2.6.20.21-x86_64: WARNINGS
linux-2.6.21.7-x86_64: WARNINGS
linux-2.6.22.19-x86_64: WARNINGS
linux-2.6.23.12-x86_64: WARNINGS
linux-2.6.24.7-x86_64: WARNINGS
linux-2.6.25.11-x86_64: WARNINGS
linux-2.6.26-x86_64: WARNINGS
linux-2.6.27-x86_64: WARNINGS
linux-2.6.28-x86_64: WARNINGS
linux-2.6.29-rc1-x86_64: ERRORS
fw/apps: OK
sparse (linux-2.6.28): ERRORS
sparse (linux-2.6.29-rc1): ERRORS

Detailed results are available here:

http://www.xs4all.nl/~hverkuil/logs/Friday.log

Full logs are available here:

http://www.xs4all.nl/~hverkuil/logs/Friday.tar.bz2
--
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  http://vger.kernel.org/majordomo-info.html


Re: RFC: Where to store camera properties (upside down, needs sw whitebalance, etc). ?

2009-01-16 Thread Adam Baker
On Friday 16 January 2009, Olivier Lorin wrote:
snip discussion of cameras needing image flipping capability in libv4l
 The use of the buffer flags makes the life easier as this flag
 is read for every image. So we can solve for the flip/rotation
 issues with the help of two new buffer flags: V4L2_BUF_FLAG_NEEDS_HFLIP
 and V4L2_BUF_FLAG_NEEDS_VFLIP.

 A driver has to set them properly. Does the rotation or flip(s)
 apply to every image (e.g. sensor upside down) or change with frames
 (e.g. Genesys webcams), that no more matters .
 I can do the patch these days.
 I do not remember if there is h/v flip functions in the libv4l,
 maybe they have to be added.

 Regards,
 Nol

That sounds like a sensible approach. The flip code is already in libv4l  but 
is currently controlled by a static table (v4lconvert_flags) of cameras that 
need it.

It doesn't address the issue of whether libv4l should also provide gamma 
adjustment, auto white balance and auto gain for cameras that could benefit 
from it. Again that could be done with a static table but keeping the 
knowledge in libv4l in step with the list of supported cameras in the kernel 
won't be easy.

Adam
--
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  http://vger.kernel.org/majordomo-info.html


Re: Fw: [PATCH] E506r-composite-input

2009-01-16 Thread hermann pitton
Hi,

Am Freitag, den 16.01.2009, 20:26 +0900 schrieb Tim Farrington:
 hermann pitton wrote:
  Hi,
 
  Am Donnerstag, den 15.01.2009, 23:35 -0200 schrieb Mauro Carvalho
  Chehab:

  Message sent to the wrong address... it is not *-owner ;)
 
  Forwarded message:
 
  Date: Thu, 15 Jan 2009 21:58:55 +0900
  From: Tim Farrington t...@iinet.net.au
  To: Mauro Carvalho Chehab mche...@infradead.org,
  linux-media-ow...@vger.kernel.org
  Subject: [PATCH] E506r-composite-input
 
 
  Make correction to composite input plus svideo input
  to Avermedia E506R
 
  Signed-off-by: Tim Farrington t...@iinet.net.au
 
 
 
 
 
 
 
  Cheers,
  Mauro
 
 
 
 
 
 
 
  Unterschied
  zwischen
  Dateien-Anlage
  (E506r_composite.patch)
 
  Only in .: E506r_composite.patch
  diff
  -upr ./linux/drivers/media/video/saa7134/saa7134-cards.c 
  ../a/v4l-dvb/linux/drivers/media/video/saa7134/saa7134-cards.c
  --- ./linux/drivers/media/video/saa7134/saa7134-cards.c 2009-01-15
  21:42:05.0 +0900
  +++ ../a/v4l-dvb/linux/drivers/media/video/saa7134/saa7134-cards.c  
  2009-01-15 21:45:29.0 +0900
  @@ -4362,13 +4362,13 @@ struct saa7134_board saa7134_boards[] = 
   .amux = TV,
   .tv   = 1,
   }, {
  -.name = name_comp,
  -.vmux = 0,
  +.name = name_comp1,
  +.vmux = 3,
   .amux = LINE1,
   }, {
   .name = name_svideo,
   .vmux = 8,
  -.amux = LINE1,
  +.amux = LINE2,
   } },
   .radio = {
   .name = name_radio,
 
  
 
  Mauro, I was never sure about why that patch, which introduced name_comp
  was accepted very, very late in the drivers history. It previously
  always started the enumeration with name_comp1.
 
  If it should have any sense at all, I thought it was to avoid ambiguity
  on such devices which have only one Composite input.
 
  Tim, are you sure that Composite amux is LINE1 and S-Video LINE2?
 
  It would be the first and only card ever seen with different amux for
  those inputs and should be noted as unusual case. I doubt it has two
  different audio-in connectors.
 
  Cheers,
  Hermann
 
 
  --
  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  http://vger.kernel.org/majordomo-info.html
 

 Hermann,
 For the record, Avermedia are well known for not respecting the GPL, and 
 finding ways to
 have as many different chip combinations on as many different cards as 
 they can,
 and as many different input configurations as they can invent.
 (Try looking at all the different chip combinations they use on all 
 their Volar sticks.)

At least they use unique PCI subsystem IDs these days.

 Avermedia make the E506R, which is a PCMCIA card.

Can remember when it for the first time appeared on the list ;)

 It has a FM Radio input, a DVB-T/Analog Television input and a Remote 
 Control.
 
 It also has a mini-SCSI input, to which is connected an Audio Left 
 cable, an Audio Right cable, and a Composite Video cable;
 also separately connected to the SCSI is a S-Video input cable.

If you have a single left/right pair as analog audio in, how can it be
for Composite one the LINE1 input pair and for S-Video on the LINE2
input pair of the chip.

There is no gpio switching on an external audio mux visible for these
inputs in the card's entry.

The error is probably taken from the E500 cardbus or I simply don't
understand what should happen on that card with the external audio in.

Unless you confirm you have tested audio on both inputs you modified,
it looks for me like a bug is left.

I thought I did help to fix this on Markus tree already and there was
sound reported for s-video working on LINE1!

I know it have been hard times with this one, but yourself introduced
vmux 0 for Composite over S-Video and now you take it away again ...

Cheers,
Hermann

 Regards,
 Timf

--
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  http://vger.kernel.org/majordomo-info.html


Re: [linux-dvb] Cross-posting linux-media, linux-dvb etc

2009-01-16 Thread BOUWSMA Barry
On Fri, 16 Jan 2009, Patrick Boettcher wrote:

 Why not closing linux-dvb (and video4linux) and transferring the currently 
 subscribed users to linux-media automatically?

Can I offer my opinions to differ?

First, I'm only subscribed to -dvb in order to post, yet still
I haven't posted what I originally planned to post before
unsubscribing until another device fails to work.  Luckily
the video4linux list was impossible to access (even the
archives needed subsciption, furrfu).

Anyway, soon after the creation of -media, I saw that the
crossposts from v4linux were of no interest to me (I'm only
interested in delivery of already-digital payloads, and am
not concerned with webcams, analogue radio or TV, remote
controls, and so on) -- since then I've skipped something
like 2/3 of the posts on -media, and today, I wouldn't want
it to appear in my mailbox.  But that's just my interest.

Also, I seem to recall that one intent of -media was to
focus on developer interest, as the initial posts revealed,
which also frees developers with better things to do than
to explain how to, for example, get a list of channels, or
stream a particular channel and be bothered by beginner or
simple questions that could be answered by those without
developer abilities.  Like me.

Anyway, it's no big deal to me.  I'm used to how the one
FreeBSD -multimedia list covers everything including sound,
yet typically gets fewer posts in a week than -dvb could
see in a day, and I can't see myself investing in another
DVB-type receiver soon until DVB-S2 support gets properly
rounded out and 100% reliable for all `experimental'-tagged
devices, so I'm quite content to browse the list just as I
skim the kernel list, or peer in on a few dozen other BSD-
type lists whenever I feel like it.


yerz,
barry bouwsma
off to answer a newbie question next
--
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  http://vger.kernel.org/majordomo-info.html


Re: [linux-dvb] RFC - Flexcop Streaming watchdog (VDSB)

2009-01-16 Thread BOUWSMA Barry
On Fri, 16 Jan 2009, AlexW wrote:

 Patrick Boettcher wrote:

  For years there has been the Video Data Stream Borken-error with VDR and
  Technisat cards: The error occured randomly and unfrequently. A 

  In fact it turned out, that the problem is worse with setups not based 
  on VDR and the VDSB-error could be really easily reproduced (I'm not 
  sure if this applies to all generations on SkyStar2-card). I'm 

 Which generation of cards have this problem? I did not see any VDSB with 
 my two Skystar 2.6D.

Same here -- never experienced this ever in some four-ish years
with one SkyStar2 of model long forgotten, with that card being
the primary one used whenever possible...

(in use typically several times per day, sometimes half a day
uninterrupted, but on a production machine at 2.6.14-ish)


barry bouwsma
--
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  http://vger.kernel.org/majordomo-info.html


Re: [linux-dvb] Problem with video4linux

2009-01-16 Thread OM Ugarcina

Hello Guys,

I have installed the patch , and everything works ok now .

Best Regards

Milorad
--
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  http://vger.kernel.org/majordomo-info.html


Re: [linux-dvb] dvb-t config for Ukraine_Kiev (ua)

2009-01-16 Thread hermann pitton
Hi Barry,

Am Samstag, den 17.01.2009, 01:41 +0100 schrieb BOUWSMA Barry:
 Hi Dmitry, thank you for your mail!
 
 I am posting part of it to the linux-dvb list, in case someone
 there can give more or better information than I do...
 
 On Wed, 14 Jan 2009, vdp wrote:
 
  BB some random 8-bit chars to make sure this gets tagged as utf-8...
  sory, it's really Cyrillic - I can read it with
  code_table_windows_1251, not like UTF-8, but strange and interesting ;-)
 
 Off-topic here, but to explain --
 In the more-than-ten years since I last used the mail program
 I am using now, the language and multi-lingual support has
 greatly improved -- back then, it would make no effort to try
 and display your Cyrillic characters, be they in KOI8-U, or
 ISO-8859-5, or whatever - if I had selected to display, say
 8859-1, or 8859-2 for Czech.
 
 But today, even with my use of the text console and no windowing
 system, I can display Cyrillic, Polish, Slovak, French, Hebrew,
 Greek -- all at the same time.  Yay!
 
 However, when I sent out a message with Greek characters, I saw
 in my local copy of it, that it was sent as 8859-7.  But I do
 not know if many mailers are able to understand how to convert
 from that and display properly with a Unicode font.
 
 The same way, when I sent the Ukranian text, it could be that
 some people in western europe, or outside europe entirely, might
 not see the characters correctly, because my mailer was set up
 to use the smallest possible unique character set tagging,
 rather than UTF-8 which has become far more common now (yes,
 I should fix my mailer configuration).
 
 So, in order to give the message a UTF-8 character set tagging,
 so that it could be displayed simply by any utf-8-aware
 xterm with a -10646 font, or on a text console with Unicode
 enabled and a font that uses as many possible characters in
 the 512 that are available, with a mailer that does not know
 how to convert from 8859-x into Unicode, I needed to insert a
 few German and Greek and Hebrew characters that are not common
 to 8859-5.
 
 And that is why I did not send only the Cyrillic characters,
 in case some mailer fails, and displays them as western-european
 or something else, the way I used to see things...
 
 This is easy to set up with X as there are plenty of -10646
 fonts available in the years since I contributed an 8859-2
 font; for the very nice large font of a 25x80 text console on
 a nice large monitor that does not strain my eyes, I like
 SCREEN_FONT=/usr/local/src/fonty-rg-0.5/LatCyrGr-16.psf.gz
 that makes many useful g00gle results readable to me.
 
 Again, sorry for going off-topic for so long...
 
 
 Now, to your question, which maybe some linux-dvb reader can
 offer more help...
 
 
  now work next scheme:
  I tzap to some channel and read stream with emcast from 
  /dev/dvb/adapter0/dvr0
  but it is only one channel - I would like stream all transponder
  
  Could you help, with advice - is it possible ? (receive all
  transponder with several channels simultaneous at the same time)
 
 Yes, this is very possible.
 
 The one thing to be certain of, is that you are not using a
 USB-1 device, as the bandwidth of USB 1 is less than most DVB-T
 multiplexes today -- usually you can fit at least two services
 without problems, though, over USB 1.
 
 
 The special ``PID'' of 8192 is used by, for example, `dvbstream'
 meaning to send the entire datastream with all PIDs to its
 output.  It will also do all the tuning for you.
 
 Or, if you know all the PIDs that are used on a particular
 frequency, you can usually list all of them of interest, up
 to the limit of the hardware PID filter, if there is one.
 
 This can save some space, as PID 8191 usually takes up some
 bandwidth for null packets to fill the available bandwidth,
 and there may be unneeded services, such as data, teletext,
 or whatever, that you do not care about.
 
 For example, here is what I would use to record three of
 the RTVi services which are sometimes FTA on Hotbirds:
 /home/beer/bin/dvbstream  ${OPERA1}  -T  \
 -s 27500   -p h   -f 12322   -I 2   -D 2  \
 -o:${RECROOT:-/opt}/Partial_Transport_Streams/detskii_mir-fs-${DATE}.ts  \
 0 44  -v 45  -a 46   40 41 42  47 48 49   $*
 
 (I am actually guessing that the last 6 PIDs are correct,
 as I only recorded the one service...)
 
 Then you only need to select which programme you wish during
 playback with your media player (you may need to record some
 additional PIDs to see the service name).  Or you can split
 the three services into three separate files.
 
 
 I use the `8192' PID when I want the entire stream, but if
 you want to use `tzap' or similar, then whatever program you
 use after that needs to set all the PIDs -- for example,
 `dvbtraffic' after I've tuned to a DVB-H multiplex...
 -PID--FREQ-BANDWIDTH-BANDWIDTH-
 20 p/s 3 kb/s31 kbit0
 0011 3 p/s 0 kb/s 5 kbit   17
 001223 p/s 4 kb/s35 kbit   18
 

Re: [PATCH 2/2 v2] gspca: Add MR97310A driver

2009-01-16 Thread Kyle Guinn
On Friday 16 January 2009 02:47:31 Jean-Francois Moine wrote:
 On Thu, 15 Jan 2009 22:36:49 -0600
 Kyle Guinn ely...@gmail.com wrote:
  This patch adds support for USB webcams based on the MR97310A chip.
  It was tested with an Aiptek PenCam VGA+ webcam.

 Hi Kyle,

 I added your driver to my repository.

 I just found a little bug. At line 250/251

   data[8] = 0x01;
   err_code = reg_w(gspca_dev, 10);

 you set 9 values and you output 10 values.


Good catch.  Yes, that should be 9.

Also that data[8] should be set to 0x00.  I must have been asleep at the 
keyboard :)  I'll send a patch.

 BTW, as many of these USB control messages are constant, you should
 better use static data or a table format (byte count, values)*.


I'll look into this.  Those may not all be costants, I believe the camera 
supports gamma adjustment and autoexposure and maybe some other controls.  I 
plan on gathering some traces to figure that out next week.

Regards,
-Kyle
--
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  http://vger.kernel.org/majordomo-info.html


Re: [PULL] http://linuxtv.org/hg/~mcisely/pvrusb2

2009-01-16 Thread Mike Isely
On Fri, 16 Jan 2009, Janne Grunau wrote:

 On Friday 16 January 2009 15:39:33 Mike Isely wrote:
  In any case, right now the serial number in the pvrusb2 is not available
  through that means because I haven't done anything to make it available
  to udev.  I'd like to do something, but so far I have found no
  information on how to make that happen.
 
 You shouldn't need to do anything special. The serial number is available 
 through the parent USB device. It can be used for udev rules through 
 ATTRS{serial} and in sysfs entry of the video device through device/serial.
 

Ah yes!

What I said before was right in its own context, but now I see something 
else.

The serial number that the pvrusb2 driver itself knows about is parsed 
out of Hauppauge private data by the tveeprom module from the device's 
internal I2C ROM.  This data is formatted in a packetized way that is 
specific to Hauppauge.  What I was refering to was *that* serial number, 
and since it's in the private ROM I saw no means for udev to know about 
it.

However I just tested with two 24xxx devices using usbview, and the same 
serial number is in fact visible in the USB configuration data.  
There's simply no way for the USB core in Linux to be able to directly 
know about, get at, or even understand that internal ROM.  Yet there it 
is.  I'm thinking now that perhaps the FX2 microcontroller is either 
parsing out the data itself during initialization and then writing out 
the USB configuration accordingly, or the serial number is in fact 
written in two places within the device!  Up until now, I had not seen 
any evidence to suggest that the FX2 firmware ever actually read the 
nearby ROM on its own.  But that could be what is happening here.

Thanks for pointing that out.  These devices still surprise me.

For anyone looking at this, the serial number in the USB configuration 
data for the device is just a hex-formatted version of the same value 
that you can see via the pvrusb2 driver's sysfs interface.

  -Mike


-- 

Mike Isely
isely @ pobox (dot) com
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F5 C1 E8

Re: [PULL] http://linuxtv.org/hg/~mcisely/pvrusb2

2009-01-16 Thread Carsten Meier
Am Fri, 16 Jan 2009 12:36:43 +0100
schrieb Carsten Meier c...@trexity.de:

 To make this mechanism work reliably with USB, the meaning of the
 field could be adapted to something like a binary identifier string.

I really meant opaque identifier string.
--
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  http://vger.kernel.org/majordomo-info.html