Mailing List archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[linux-dvb] Re: [PATCH] SS2 : Fix for OpenWholeBandWidth



Roberto Ragusa wrote:
On Sun, 23 Nov 2003 11:44:09 +0000
Vincenzo Di Massa <hawk78_it@yahoo.it> wrote:

...
On a general side, I think the current pid handling is not very smart:
1) we check 0x27 slots even if only two are occupied (easily fixable)
2) when we openwholebandwith we don't close it back; this consumes
CPU time and causes interrupts and DMA transfers

Other considerations: is there any specific reason
1) to have exactly 0x27 slots?
2) to advertize 32 pids max?
3) to have stream1/stream2/pcr/pmt/emm/ecm specific routines, given that
it looks like they have no particular specialization, they're just generic
filters?
4) to have different coding style than linux kernel? ("CheckPID" instead
of "check_pid")

Great that this got patched at last!
The 0x27 slots are indeed 7 special-purpose hardware PID filters (for stream1, stream2, PES etc.) and 32 all-purpose data filters; alone the data filters are also hardware filters but the driver does not really support them, they are simulated in software. Nevertheless there are 32 unsused hardware filters on the flexcop.
I asked B2C2 & BBTI to tell me where the according registers are located on the chip, but I have no answer yet.

Niklas



--
Info:
To unsubscribe send a mail to ecartis@linuxtv.org with "unsubscribe linux-dvb" as subject.



Home | Main Index | Thread Index