| Johannes Stezenbach | 2add87a | 2005-05-16 21:54:10 -0700 | [diff] [blame] | 1 | This README escorted the skystar2-driver rewriting procedure. It describes the | 
 | 2 | state of the new flexcop-driver set and some internals are written down here | 
 | 3 | too. | 
 | 4 |  | 
| Johannes Stezenbach | 2add87a | 2005-05-16 21:54:10 -0700 | [diff] [blame] | 5 | This document hopefully describes things about the flexcop and its | 
| Johannes Stezenbach | c6ee197 | 2005-05-16 21:54:17 -0700 | [diff] [blame] | 6 | device-offsprings. Goal was to write an easy-to-write and easy-to-read set of | 
| Johannes Stezenbach | 2add87a | 2005-05-16 21:54:10 -0700 | [diff] [blame] | 7 | drivers based on the skystar2.c and other information. | 
 | 8 |  | 
| Johannes Stezenbach | 2add87a | 2005-05-16 21:54:10 -0700 | [diff] [blame] | 9 | Remark: flexcop-pci.c was a copy of skystar2.c, but every line has been | 
 | 10 | touched and rewritten. | 
 | 11 |  | 
| Johannes Stezenbach | c6ee197 | 2005-05-16 21:54:17 -0700 | [diff] [blame] | 12 | History & News | 
 | 13 | ============== | 
 | 14 |   2005-04-01 - correct USB ISOC transfers (thanks to Vadim Catana) | 
 | 15 |  | 
 | 16 |  | 
 | 17 |  | 
 | 18 |  | 
| Johannes Stezenbach | 2add87a | 2005-05-16 21:54:10 -0700 | [diff] [blame] | 19 | General coding processing | 
 | 20 | ========================= | 
 | 21 |  | 
 | 22 | We should proceed as follows (as long as no one complains): | 
 | 23 |  | 
 | 24 | 0) Think before start writing code! | 
 | 25 |  | 
 | 26 | 1) rewriting the skystar2.c with the help of the flexcop register descriptions | 
 | 27 | and splitting up the files to a pci-bus-part and a flexcop-part. | 
 | 28 | The new driver will be called b2c2-flexcop-pci.ko/b2c2-flexcop-usb.ko for the | 
 | 29 | device-specific part and b2c2-flexcop.ko for the common flexcop-functions. | 
 | 30 |  | 
 | 31 | 2) Search for errors in the leftover of flexcop-pci.c (compare with pluto2.c | 
 | 32 | and other pci drivers) | 
 | 33 |  | 
 | 34 | 3) make some beautification (see 'Improvements when rewriting (refactoring) is | 
 | 35 | done') | 
 | 36 |  | 
 | 37 | 4) Testing the new driver and maybe substitute the skystar2.c with it, to reach | 
 | 38 | a wider tester audience. | 
 | 39 |  | 
 | 40 | 5) creating an usb-bus-part using the already written flexcop code for the pci | 
 | 41 | card. | 
 | 42 |  | 
 | 43 | Idea: create a kernel-object for the flexcop and export all important | 
 | 44 | functions. This option saves kernel-memory, but maybe a lot of functions have | 
 | 45 | to be exported to kernel namespace. | 
 | 46 |  | 
 | 47 |  | 
 | 48 | Current situation | 
 | 49 | ================= | 
 | 50 |  | 
 | 51 | 0) Done :) | 
 | 52 | 1) Done (some minor issues left) | 
 | 53 | 2) Done | 
 | 54 | 3) Not ready yet, more information is necessary | 
 | 55 | 4) next to be done (see the table below) | 
 | 56 | 5) USB driver is working (yes, there are some minor issues) | 
 | 57 |  | 
 | 58 | What seems to be ready? | 
 | 59 | ----------------------- | 
 | 60 |  | 
 | 61 | 1) Rewriting | 
 | 62 | 1a) i2c is cut off from the flexcop-pci.c and seems to work | 
 | 63 | 1b) moved tuner and demod stuff from flexcop-pci.c to flexcop-tuner-fe.c | 
 | 64 | 1c) moved lnb and diseqc stuff from flexcop-pci.c to flexcop-tuner-fe.c | 
 | 65 | 1e) eeprom (reading MAC address) | 
 | 66 | 1d) sram (no dynamic sll size detection (commented out) (using default as JJ told me)) | 
 | 67 | 1f) misc. register accesses for reading parameters (e.g. resetting, revision) | 
 | 68 | 1g) pid/mac filter (flexcop-hw-filter.c) | 
 | 69 | 1i) dvb-stuff initialization in flexcop.c (done) | 
 | 70 | 1h) dma stuff (now just using the size-irq, instead of all-together, to be done) | 
 | 71 | 1j) remove flexcop initialization from flexcop-pci.c completely (done) | 
 | 72 | 1l) use a well working dma IRQ method (done, see 'Known bugs and problems and TODO') | 
 | 73 | 1k) cleanup flexcop-files (remove unused EXPORT_SYMBOLs, make static from | 
 | 74 | non-static where possible, moved code to proper places) | 
 | 75 |  | 
 | 76 | 2) Search for errors in the leftover of flexcop-pci.c (partially done) | 
 | 77 | 5a) add MAC address reading | 
| Johannes Stezenbach | c6ee197 | 2005-05-16 21:54:17 -0700 | [diff] [blame] | 78 | 5c) feeding of ISOC data to the software demux (format of the isochronous data | 
 | 79 | and speed optimization, no real error) (thanks to Vadim Catana) | 
| Johannes Stezenbach | 2add87a | 2005-05-16 21:54:10 -0700 | [diff] [blame] | 80 |  | 
 | 81 | What to do in the near future? | 
 | 82 | -------------------------------------- | 
 | 83 | (no special order here) | 
 | 84 |  | 
| Johannes Stezenbach | 2add87a | 2005-05-16 21:54:10 -0700 | [diff] [blame] | 85 | 5) USB driver | 
 | 86 | 5b) optimize isoc-transfer (submitting/killing isoc URBs when transfer is starting) | 
| Johannes Stezenbach | 2add87a | 2005-05-16 21:54:10 -0700 | [diff] [blame] | 87 |  | 
 | 88 | Testing changes | 
 | 89 | --------------- | 
 | 90 |  | 
 | 91 | O             = item is working | 
 | 92 | P             = item is partially working | 
 | 93 | X             = item is not working | 
 | 94 | N             = item does not apply here | 
 | 95 | <empty field> = item need to be examined | 
 | 96 |  | 
 | 97 |        | PCI                               | USB | 
 | 98 | item   | mt352 | nxt2002 | stv0299 | mt312 | mt352 | nxt2002 | stv0299 | mt312 | 
 | 99 | -------+-------+---------+---------+-------+-------+---------+---------+------- | 
 | 100 | 1a)    | O     |         |         |       | N     | N       | N       | N | 
 | 101 | 1b)    | O     |         |         |       |       |         | O       | | 
 | 102 | 1c)    | N     | N       |         |       | N     | N       | O       | | 
 | 103 | 1d)    |                 O                 |                 O | 
 | 104 | 1e)    |                 O                 |                 O | 
 | 105 | 1f)    |                                   P | 
 | 106 | 1g)    |                                   O | 
 | 107 | 1h)    |                 P                 | | 
 | 108 | 1i)    |                 O                 |                 N | 
 | 109 | 1j)    |                 O                 |                 N | 
 | 110 | 1l)    |                 O                 |                 N | 
 | 111 | 2)     |                 O                 |                 N | 
 | 112 | 5a)    |                 N                 |                 O | 
 | 113 | 5b)*   |                 N                 | | 
| Johannes Stezenbach | c6ee197 | 2005-05-16 21:54:17 -0700 | [diff] [blame] | 114 | 5c)    |                 N                 |                 O | 
| Johannes Stezenbach | 2add87a | 2005-05-16 21:54:10 -0700 | [diff] [blame] | 115 |  | 
 | 116 | * - not done yet | 
 | 117 |  | 
 | 118 | Known bugs and problems and TODO | 
 | 119 | -------------------------------- | 
 | 120 |  | 
 | 121 | 1g/h/l) when pid filtering is enabled on the pci card | 
 | 122 |  | 
 | 123 | DMA usage currently: | 
 | 124 |   The DMA is splitted in 2 equal-sized subbuffers. The Flexcop writes to first | 
 | 125 |   address and triggers an IRQ when it's full and starts writing to the second | 
 | 126 |   address. When the second address is full, the IRQ is triggered again, and | 
 | 127 |   the flexcop writes to first address again, and so on. | 
 | 128 |   The buffersize of each address is currently 640*188 bytes. | 
 | 129 |  | 
 | 130 |   Problem is, when using hw-pid-filtering and doing some low-bandwidth | 
 | 131 |   operation (like scanning) the buffers won't be filled enough to trigger | 
 | 132 |   the IRQ. That's why: | 
 | 133 |  | 
 | 134 |   When PID filtering is activated, the timer IRQ is used. Every 1.97 ms the IRQ | 
 | 135 |   is triggered.  Is the current write address of DMA1 different to the one | 
 | 136 |   during the last IRQ, then the data is passed to the demuxer. | 
 | 137 |  | 
 | 138 |   There is an additional DMA-IRQ-method: packet count IRQ. This isn't | 
 | 139 |   implemented correctly yet. | 
 | 140 |  | 
 | 141 |   The solution is to disable HW PID filtering, but I don't know how the DVB | 
 | 142 |   API software demux behaves on slow systems with 45MBit/s TS. | 
 | 143 |  | 
 | 144 | Solved bugs :) | 
 | 145 | -------------- | 
 | 146 | 1g) pid-filtering (somehow pid index 4 and 5 (EMM_PID and ECM_PID) aren't | 
 | 147 | working) | 
 | 148 | SOLUTION: also index 0 was affected, because net_translation is done for | 
 | 149 | these indexes by default | 
 | 150 |  | 
| Johannes Stezenbach | c6ee197 | 2005-05-16 21:54:17 -0700 | [diff] [blame] | 151 | 5b) isochronous transfer does only work in the first attempt (for the Sky2PC | 
 | 152 | USB, Air2PC is working) SOLUTION: the flexcop was going asleep and never really | 
 | 153 | woke up again (don't know if this need fixes, see | 
 | 154 | flexcop-fe-tuner.c:flexcop_sleep) | 
 | 155 |  | 
 | 156 | NEWS: when the driver is loaded and unloaded and loaded again (w/o doing | 
 | 157 | anything in the while the driver is loaded the first time), no transfers take | 
 | 158 | place anymore. | 
| Johannes Stezenbach | 2add87a | 2005-05-16 21:54:10 -0700 | [diff] [blame] | 159 |  | 
 | 160 | Improvements when rewriting (refactoring) is done | 
 | 161 | ================================================= | 
 | 162 |  | 
 | 163 | - split sleeping of the flexcop (misc_204.ACPI3_sig = 1;) from lnb_control | 
 | 164 |   (enable sleeping for other demods than dvb-s) | 
| Johannes Stezenbach | c6ee197 | 2005-05-16 21:54:17 -0700 | [diff] [blame] | 165 | - add support for CableStar (stv0297 Microtune 203x/ALPS) (almost done, incompatibilities with the Nexus-CA) | 
| Johannes Stezenbach | 2add87a | 2005-05-16 21:54:10 -0700 | [diff] [blame] | 166 |  | 
 | 167 | Debugging | 
 | 168 | --------- | 
 | 169 | - add verbose debugging to skystar2.c (dump the reg_dw_data) and compare it | 
 | 170 |   with this flexcop, this is important, because i2c is now using the | 
 | 171 |   flexcop_ibi_value union from flexcop-reg.h (do you have a better idea for | 
 | 172 |   that, please tell us so). | 
 | 173 |  | 
 | 174 | Everything which is identical in the following table, can be put into a common | 
 | 175 | flexcop-module. | 
 | 176 |  | 
| Mauro Carvalho Chehab | 674434c | 2005-12-12 00:37:28 -0800 | [diff] [blame] | 177 | 		  PCI                  USB | 
| Johannes Stezenbach | 2add87a | 2005-05-16 21:54:10 -0700 | [diff] [blame] | 178 | ------------------------------------------------------------------------------- | 
 | 179 | Different: | 
 | 180 | Register access:  accessing IO memory  USB control message | 
 | 181 | I2C bus:          I2C bus of the FC    USB control message | 
 | 182 | Data transfer:    DMA                  isochronous transfer | 
 | 183 | EEPROM transfer:  through i2c bus      not clear yet | 
 | 184 |  | 
 | 185 | Identical: | 
 | 186 | Streaming:                 accessing registers | 
 | 187 | PID Filtering:             accessing registers | 
 | 188 | Sram destinations:         accessing registers | 
 | 189 | Tuner/Demod:                     I2C bus | 
 | 190 | DVB-stuff:            can be written for common use | 
 | 191 |  | 
| Johannes Stezenbach | 2add87a | 2005-05-16 21:54:10 -0700 | [diff] [blame] | 192 | Acknowledgements (just for the rewriting part) | 
 | 193 | ================ | 
 | 194 |  | 
 | 195 | Bjarne Steinsbo thought a lot in the first place of the pci part for this code | 
 | 196 | sharing idea. | 
 | 197 |  | 
| Johannes Stezenbach | 3ed8a31 | 2005-05-16 21:54:13 -0700 | [diff] [blame] | 198 | Andreas Oberritter for providing a recent PCI initialization template | 
 | 199 | (pluto2.c). | 
 | 200 |  | 
 | 201 | Boleslaw Ciesielski for pointing out a problem with firmware loader. | 
 | 202 |  | 
 | 203 | Vadim Catana for correcting the USB transfer. | 
| Johannes Stezenbach | 2add87a | 2005-05-16 21:54:10 -0700 | [diff] [blame] | 204 |  | 
| Johannes Stezenbach | c6ee197 | 2005-05-16 21:54:17 -0700 | [diff] [blame] | 205 | comments, critics and ideas to linux-dvb@linuxtv.org. |