diff options
author | Hans Verkuil <hverkuil@xs4all.nl> | 2010-03-24 19:09:55 -0300 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2010-05-06 19:20:50 -0300 |
commit | 03b1930efd3c2320b1dcba76c8af15f7e454919d (patch) | |
tree | 436dfae66ed4444578bc7d14cc43600a4f5119e1 /drivers/media/video/hexium_orion.c | |
parent | d2f2d6d0a11e892263ea511e46af449113fd2081 (diff) | |
download | linux-stable-03b1930efd3c2320b1dcba76c8af15f7e454919d.tar.gz linux-stable-03b1930efd3c2320b1dcba76c8af15f7e454919d.tar.bz2 linux-stable-03b1930efd3c2320b1dcba76c8af15f7e454919d.zip |
V4L/DVB: saa7146: fix regression of the av7110/budget-av driver
An earlier regression fix for the mxb driver (V4L/DVB: saa7146_vv: fix
regression where v4l2_device was registered too late) caused a new
regression in the av7110 driver.
Reverted the old fix and fixed the problem in the mxb driver instead.
Tested on mxb and budget-av cards.
The real problem is that the saa7146 framework has separate probe()
and attach() driver callbacks which should be rolled into one. This
is now done for the mxb driver, but others should do the same. Lack
of hardware makes this hard to do, though. I hope to get hold of some
hexium cards and then I can try to improve the framework to prevent
this from happening again.
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/video/hexium_orion.c')
-rw-r--r-- | drivers/media/video/hexium_orion.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/media/video/hexium_orion.c b/drivers/media/video/hexium_orion.c index fe596a1c12a8..938a1f8f880a 100644 --- a/drivers/media/video/hexium_orion.c +++ b/drivers/media/video/hexium_orion.c @@ -216,10 +216,6 @@ static int hexium_probe(struct saa7146_dev *dev) return -EFAULT; } - err = saa7146_vv_devinit(dev); - if (err) - return err; - hexium = kzalloc(sizeof(struct hexium), GFP_KERNEL); if (NULL == hexium) { printk("hexium_orion: hexium_probe: not enough kernel memory.\n"); |