On Wed, Apr 18, 2012 at 08:38:10PM +0200, Ninja wrote: > sorry for the late response. Unfortunately my Skystar HD2 died on me > and it looks like i get an other card in exchange. > Anyway, as far I could test, the patches were ok to me. I can still > put my marks, but I'm not sure which ones I should use or I'm > allowed to use. All except 01 (IRQ0 handling) and 11 (enable CA) are written by me. 01 should probably be safe (it's already pending review here); 11 is more unclear. Anyway, I'm sure 11 is pretty trivial once it's cleaned up, so someone should just rewrite it without the part that moves a function around, and I'm sure it can go in. 02, 04 and 06 are fixes for debugging output only. They should be safe. 03 (dvbdev mutex) should probably be fixed in a more general way, by someone more knowledgeable of the DVB subsystem. 06 and 08 are general cleanups. They should be pretty safe. 07 (atomic accesses) contains important SMP fixes. Should definitely go in. 09 and 10 contain fixes for IRQ0 timeouts on SMP. They should go in, but only as a pair; if you do 09 without 10, you will unmask a lot of the timeouts that are there. (09 doesn't introduce any new timeouts, it just makes userspace see them properly, and as an end result, things get worse.) So, as a start, pull 01, 02, and 04 through 10? We'll still be without CA support, but as a whole the Mantis driver will be better. /* Steinar */ -- Homepage: http://www.sesse.net/ -- 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
