On 20.05.2012 12:56, Thomas Mair wrote:
On 18.05.2012 22:47, Antti Palosaari wrote:
Good evening!
On 18.05.2012 21:47, Thomas Mair wrote:
Good Evening!
This is the corrected version of the patch series to support the
RTL2832 demodulator. There where no major changes. The majority of
the changes consist in fixing style issues and adhering to proper
naming conventions.
Review done and seems to be OK for my eyes.
Thanks Antti! You have been a big help for developing the driver.
What are the next steps? I think the fc0012 and fc0013 driver
need to be reviewed before the patch may be included in
staging. Is that the way it works?
Yes those should be reviewed. At least Mauro will review those when
merging drivers to master but it is always better if there is some other
reviewers before that as he has million patches to review.
The next question for me is how to proceed when including new
devices. Poma already sent an extensive list a little while
ago (http://patchwork.linuxtv.org/patch/10982/). Should they
all be included at once, or should I wait until somone confirms
they are working correctly and include them one by one?
It has been rule that device is added after known to work.
That sounds good to me. In the meantime I will try to set up a
page for the driver on the linuxtv.org wiki to keep information
about the driver and the devices in one place.
Unfortunately DVB USB do not support dynamic USB ID. In order to workaround that I have done some small hackish solution for the dvb_usb_rtl28xxu driver. Currently it works for RTL2831U based devices, but I see it could be easily extended for RTL2832U too by adding module parameter.
If I understand it right, the problem is that the tuner/demod
combination is also hard coded in the dvb_usb_rtl28xxu driver?
Device USB IDs are hard coded to (static struct
dvb_usb_device_properties) and that structure is passed to the DVB USB
framework by calling dvb_usb_device_init(). DVB USB framework just
refuses to register device if USB ID is not found. So I added that
hackish solution to replace one USB ID by USB ID got as a dynamic USB
ID. Dynamic ID is USB core features. It will load and call some USB
driver even given USB ID is not advertised by the driver.
regards
Antti
--
http://palosaari.fi/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
