Bug 7524 - AVerTV Studio 507UA support
Summary: AVerTV Studio 507UA support
Status: RESOLVED CODE_FIX
Alias: None
Product: v4l-dvb
Classification: Unclassified
Component: saa7134 (show other bugs)
Hardware: i386 Linux
: P2 normal
Assignee: Alexey Dobriyan
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-11-14 09:21 UTC by Andy Shevchenko
Modified: 2009-11-16 13:58 UTC (History)
4 users (show)

See Also:
Kernel Version: 2.6.18-1.2200.fc5
Subsystem:
Regression: No
Bisected commit-id:


Attachments
AVerTV Studio 507UA support (3.86 KB, patch)
2006-11-14 09:22 UTC, Andy Shevchenko
Details | Diff

Description Andy Shevchenko 2006-11-14 09:21:38 UTC
Most recent kernel where this bug did *NOT* occur: nothing
Distribution: ASPLinux v11.2 (fc5 kernel based)
Hardware Environment: AVerTV Studio 507UA (saa7135 based)
Software Environment: xawtv
Problem Description:
No support at all.

Steps to reproduce:
Try to use this card

Please, review attached patch. I've checked all functions except s-video input 
with this patch.
Comment 1 Andy Shevchenko 2006-11-14 09:22:44 UTC
Created attachment 9508 [details]
AVerTV Studio 507UA support
Comment 2 Nickolay V. Shmyrev 2006-11-14 13:40:56 UTC
What UA means here?
Comment 3 Andy Shevchenko 2006-11-15 01:39:05 UTC
UA means 'for Ukraine edition'.

I assume you can read in Russian and following link is told you something more
http://forum.avertv.ru/lofiversion/index.php/t757.html

I've found two differences between 507 and 507UA: a) the differ subdevice 
number and b) only one (?) composite input. In last v4l-dvb the two composite 
inputs are described in the sources for 507 [#102].
Comment 4 Nickolay V. Shmyrev 2006-11-23 13:26:40 UTC
Ok, actually I believe that two inputs for composite is just a misunderstanding
from the author of 507 patch. Looking at the 507 pictures from

http://www.hardwareportal.ru/Multimedia/Beholder.507/index.html

one can found that there is only one composite. Two composites are very rare case.

So I believe that correct patch should just add another pci entry with
appropriate comment and remove composite input.

Also, to apply the patch we need your Sign-Off-By.
Comment 5 Nickolay V. Shmyrev 2006-11-23 13:29:27 UTC
And, remote on this card should be different from 307. Did you test it works?
Comment 6 Nickolay V. Shmyrev 2006-11-23 13:59:00 UTC
But if it's really differs in tuner module and has MK5 tuner instead of MK3,
that's it's certainly better to add new entry.
Comment 7 Andy Shevchenko 2006-11-24 01:49:43 UTC
1. Really, I think so about two composits in the 507 patch.
2. No, I didn't test. Just copy these strings from 507 patch (And here good 
idea to add several words for testers in the sources)
3. Yes. The last digit in the tuner model is 5 instead of 3
Full model string: FM1216ME IH5
Comment 8 Andy Shevchenko 2006-11-24 02:26:50 UTC
Some words about sign-off-by. What is this? 
The my patch for the ALSA (#5461) has been applied without any additional 
information from me. Yes, the sign-off-by string with my name is present in 
that changelog. Thanks for explanation.
Comment 9 Andrew Morton 2007-01-31 02:18:06 UTC
Can we close this bug now?
Comment 10 Nickolay V. Shmyrev 2007-01-31 02:29:27 UTC
mchehab should include this patch into tree. It's not included still I suppose
Comment 11 Natalie Protasevich 2007-06-21 21:31:22 UTC
The code has been merged, it is currently in 2.6.22-rc4. Closing the bug.
Thanks.
Comment 12 Andy Shevchenko 2007-09-20 05:48:01 UTC
linux 2.6.22.5 from FC7 still not contain the patch.
Comment 13 Natalie Protasevich 2008-04-05 02:27:07 UTC
Was the patch submitted/merged? Can someone update the status please.
Thanks.
Comment 15 Natalie Protasevich 2008-04-08 13:43:59 UTC
Thank you Mauro. Closing the bug.
Comment 16 Andy Shevchenko 2008-09-27 13:59:27 UTC
May be I'm wrong, but specified patch doesn't cover the PCI SSID 0x1461:0xa11b.
Comment 17 Alan 2009-03-24 08:40:18 UTC
Mauro - ping - confirmed this one has still somehow escaped merging
Comment 18 Mauro Carvalho Chehab 2009-04-08 17:11:13 UTC
Alan,

Yes, you're right. It escaped from my eyes. I've committed it right now at the development tree:
   http://linuxtv.org/hg/v4l-dvb/rev/77ebdc14cc24

I should add it at -git later, after merging some other pending stuff.
Comment 19 Alexey Dobriyan 2009-11-16 13:57:36 UTC
commit df0dbbe24053b7c669f63341d3d3f090560c3217
"V4L/DVB (11442): saa7134: BZ#7524: Add AVerTV Studio 507UA support"

Note You need to log in before you can comment on or make changes to this bug.