Re: Re: [RFC/PATCHES] xc3028 hybrid tuner, em28xx/em2880-dvb, saa7134, cx88

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

 



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

[Index of Archives]     [Linux Media]     [Video 4 Linux]     [Asterisk]     [Samba]     [Xorg]     [Xfree86]     [Linux USB]

  Powered by Linux