Markus Rechberger wrote: > to be more accurate where all the changes happened: > b/linux/drivers/media/tuners/Kconfig | 14 > b/linux/drivers/media/tuners/Makefile | 7 > b/linux/drivers/media/tuners/xc3028-tuner.c | 601 ++++ > b/linux/drivers/media/tuners/xc3028-tuner.h | 20 > b/linux/drivers/media/video/em28xx/em2880-dvb.c | 748 ++++ > b/linux/drivers/media/video/em28xx/em28xx-audio.c | 439 ++ > b/linux/drivers/media/video/em28xx/em28xx-webcam.c | 365 ++ > b/linux/include/media/v4l_dvb_tuner.h | 131 > linux/Documentation/video4linux/CARDLIST.cx88 | 3 > linux/Documentation/video4linux/CARDLIST.em28xx | 69 > linux/Documentation/video4linux/CARDLIST.tuner | 1 > linux/drivers/media/Kconfig | 2 > linux/drivers/media/Makefile | 1 > linux/drivers/media/common/ir-keymaps.c | 221 + > linux/drivers/media/dvb/b2c2/flexcop-fe-tuner.c | 21 > linux/drivers/media/dvb/bt8xx/dvb-bt8xx.c | 34 > linux/drivers/media/dvb/dvb-core/dmxdev.c | 6 > linux/drivers/media/dvb/dvb-core/dmxdev.h | 1 > linux/drivers/media/dvb/dvb-core/dvb_demux.c | 3 > linux/drivers/media/dvb/dvb-core/dvb_demux.h | 3 > linux/drivers/media/dvb/dvb-core/dvb_frontend.c | 29 > linux/drivers/media/dvb/dvb-core/dvb_frontend.h | 64 > linux/drivers/media/dvb/dvb-core/dvb_net.c | 29 > linux/drivers/media/dvb/dvb-core/dvb_net.h | 1 > linux/drivers/media/dvb/dvb-usb/af9005-fe.c | 12 > linux/drivers/media/dvb/dvb-usb/au6610.c | 3 > linux/drivers/media/dvb/dvb-usb/cxusb.c | 150 > linux/drivers/media/dvb/dvb-usb/cxusb.h | 2 > linux/drivers/media/dvb/dvb-usb/dib0700_devices.c | 7 > linux/drivers/media/dvb/dvb-usb/dibusb-common.c | 6 > linux/drivers/media/dvb/dvb-usb/dibusb-mb.c | 17 > linux/drivers/media/dvb/dvb-usb/digitv.c | 7 > linux/drivers/media/dvb/dvb-usb/dvb-usb-i2c.c | 11 > linux/drivers/media/dvb/dvb-usb/dvb-usb-ids.h | 2 > linux/drivers/media/dvb/dvb-usb/dvb-usb.h | 8 > linux/drivers/media/dvb/dvb-usb/gl861.c | 3 > linux/drivers/media/dvb/dvb-usb/m920x.c | 14 > linux/drivers/media/dvb/dvb-usb/opera1.c | 3 > linux/drivers/media/dvb/dvb-usb/ttusb2.c | 3 > linux/drivers/media/dvb/dvb-usb/umt-010.c | 3 > linux/drivers/media/dvb/frontends/at76c651.c | 2 > linux/drivers/media/dvb/frontends/bsbe1.h | 3 > linux/drivers/media/dvb/frontends/bsru6.h | 3 > linux/drivers/media/dvb/frontends/cx22700.c | 2 > linux/drivers/media/dvb/frontends/cx22702.c | 2 > linux/drivers/media/dvb/frontends/cx24110.c | 2 > linux/drivers/media/dvb/frontends/dib3000mb.c | 2 > linux/drivers/media/dvb/frontends/dib3000mc.c | 2 > linux/drivers/media/dvb/frontends/dib7000m.c | 2 > linux/drivers/media/dvb/frontends/dib7000p.c | 2 > linux/drivers/media/dvb/frontends/dvb-pll.c | 65 > linux/drivers/media/dvb/frontends/dvb-pll.h | 19 > linux/drivers/media/dvb/frontends/dvb_dummy_fe.c | 2 > linux/drivers/media/dvb/frontends/l64781.c | 2 > linux/drivers/media/dvb/frontends/lgdt330x.c | 6 > linux/drivers/media/dvb/frontends/mt2060.c | 43 > linux/drivers/media/dvb/frontends/mt2060.h | 6 > linux/drivers/media/dvb/frontends/mt312.c | 2 > linux/drivers/media/dvb/frontends/mt352.c | 15 > linux/drivers/media/dvb/frontends/mt352.h | 3 > linux/drivers/media/dvb/frontends/nxt200x.c | 2 > linux/drivers/media/dvb/frontends/nxt6000.c | 2 > linux/drivers/media/dvb/frontends/or51132.c | 2 > linux/drivers/media/dvb/frontends/qt1010.c | 59 > linux/drivers/media/dvb/frontends/qt1010.h | 4 > linux/drivers/media/dvb/frontends/s5h1420.c | 6 > linux/drivers/media/dvb/frontends/sp8870.c | 2 > linux/drivers/media/dvb/frontends/sp887x.c | 4 > linux/drivers/media/dvb/frontends/stv0297.c | 2 > linux/drivers/media/dvb/frontends/stv0299.c | 4 > linux/drivers/media/dvb/frontends/tda10021.c | 2 > linux/drivers/media/dvb/frontends/tda10023.c | 2 > linux/drivers/media/dvb/frontends/tda1004x.c | 2 > linux/drivers/media/dvb/frontends/tda10086.c | 4 > linux/drivers/media/dvb/frontends/tda8083.c | 2 > linux/drivers/media/dvb/frontends/tda80xx.c | 2 > linux/drivers/media/dvb/frontends/tda826x.c | 21 > linux/drivers/media/dvb/frontends/tda826x.h | 4 > linux/drivers/media/dvb/frontends/tda827x.c | 88 > linux/drivers/media/dvb/frontends/tda827x.h | 4 > linux/drivers/media/dvb/frontends/tua6100.c | 33 > linux/drivers/media/dvb/frontends/tua6100.h | 4 > linux/drivers/media/dvb/frontends/ves1820.c | 2 > linux/drivers/media/dvb/frontends/ves1x93.c | 2 > linux/drivers/media/dvb/frontends/zl10353.c | 24 > linux/drivers/media/dvb/frontends/zl10353.h | 7 > linux/drivers/media/dvb/frontends/zl10353_priv.h | 2 > linux/drivers/media/dvb/pluto2/pluto2.c | 7 > linux/drivers/media/dvb/ttpci/av7110.c | 50 > linux/drivers/media/dvb/ttpci/budget-av.c | 32 > linux/drivers/media/dvb/ttpci/budget-ci.c | 22 > linux/drivers/media/dvb/ttpci/budget-patch.c | 10 > linux/drivers/media/dvb/ttpci/budget.c | 28 > linux/drivers/media/dvb/ttusb-budget/dvb-ttusb-budget.c | 35 > linux/drivers/media/video/cx88/cx88-cards.c | 105 > linux/drivers/media/video/cx88/cx88-dvb.c | 51 > linux/drivers/media/video/cx88/cx88-i2c.c | 72 > linux/drivers/media/video/cx88/cx88.h | 5 > linux/drivers/media/video/em28xx/Kconfig | 29 > linux/drivers/media/video/em28xx/Makefile | 6 > linux/drivers/media/video/em28xx/em28xx-cards.c | 1898 +++++++++++- > linux/drivers/media/video/em28xx/em28xx-core.c | 773 +++-- > linux/drivers/media/video/em28xx/em28xx-i2c.c | 344 ++ > linux/drivers/media/video/em28xx/em28xx-input.c | 158 + > linux/drivers/media/video/em28xx/em28xx-video.c | 2402 ++++++++++++---- > linux/drivers/media/video/em28xx/em28xx.h | 552 ++- > linux/drivers/media/video/saa7134/saa7134-cards.c | 85 > linux/drivers/media/video/saa7134/saa7134-dvb.c | 67 > linux/drivers/media/video/saa7134/saa7134-i2c.c | 44 > linux/drivers/media/video/saa7134/saa7134.h | 4 > linux/drivers/media/video/tuner-core.c | 171 + > linux/drivers/media/video/tveeprom.c | 2 > linux/include/media/ir-common.h | 6 > linux/include/media/tuner.h | 7 > linux/include/media/v4l2-common.h | 2 > v4l/Makefile | 4 > v4l/scripts/cardlist | 1 > v4l/scripts/em28xx.pl | 97 > v4l/scripts/gentree.pl | 1 > v4l/scripts/tuner.pl | 3 > 120 files changed, 9118 insertions(+), 1496 deletions(-) Sorry Markus, imho it is an offence to post a patchbomb like this. It does not help if you add support for 50+ devices while possibly breaking 25 other ones. Please remember that the typical driver maintainer is not able to test all devices supported by the drivers which he is responsible for. (For example, I can only test 4 out of 52 devices of the dvb/ttpci.) > As from my side I don't have the time to do a major rewrite or doing > some reordering again, please try to make the best out of what's > available now. You don't have the time but you expect that all other maintainers take the time to review your patch monster. Sorry, my time is limited, too. If this style of development is the way to go in v4l/dvb, it is time for me to quit. Patches which affect core code must be discussed _before_ being applied. At least this is what we did it in the past... Imho core changes need to be done this way: 1. Propose changes. 2. Discuss. (Not very pleasant sometimes. ;-) 3a. Implement the changes. 3b. Modify existing drivers (if required, should be avoided if possible). 4. Add new drivers. I'm missing steps 1 and 2 here. Just my 2 cents. CU Oliver -- -------------------------------------------------------- VDR Remote Plugin 0.3.9 available at http://www.escape-edv.de/endriss/vdr/ -------------------------------------------------------- _______________________________________________ linux-dvb mailing list linux-dvb@xxxxxxxxxxx http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb