Re: [PATCH] Re: [PATCH] Multi protocol support (stage #1)

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

 



Johannes Stezenbach wrote:


I am CC' ing the others as well.


Anyway, one last try. Let's go back to the very beginning:

- we need to extend frontend API for DVB-S2, DVB-H etc.
- we cannot do so without breaking backward compatibility
  because of mistakes made when defining the current API
- so now that we define new API, we don't want to repeat
  the same mistake, but design in a way that make future
  extensions possible without too much pain for
  driver and application developers

- right from the beginning my position was to do the
  *minimal* work necessary to support the *current*
  requirements -- keep it simple and stupid
- and add other stuff later when the need arises
- and all along you totally ignored this and instead
  want to cram every last detail into the API that
  you think *might* be necessary
- wrt requirements: you need to think through how
  an application will work (how does dvbscan discover
  DVB-S2 transmission? where do the tuning parameters
  come from?)

currently the tuning parameters come from the s2_satellite_delivery_system_descriptor what i have in there is the sis-mis flag and the transport stream identifier to select the streams

Remember: You cannot remove anything from the API
once it is in there. But you can always add stuff later.

Please ask yourself who are doing this API design for, and
how people are going to benefit from your work.
IMHO:
- users just expect to be able to buy a DVB-S2 card and
  watch the services broadcast *today* via DVB-S2
- application developers want to support DVB-S2 with
  as little hassle as possible

With the new API "as little hassle as possible" for
apps already means:
- testing for support for the new API
- fallback to using the old API for backwards compatibility

:-(

IMHO it is of no use to further complicate the matter by
adding lots of unnecessary stuff to the API.


Johannes, you are just accusing me with blunt statements like that. I really don't know what to say on your accusations.

I too agree with not adding unwanted stuff into the API.

BTW, I don't claim to be right on my interpretation of the
DVB-S2 spec, however as long as we can't agree (i.e.
no one can give a good explanation why his version
of the DVB-S2 API is the correct one, and *everyone*
can agree on it), I think the while DVB-S2 API is useless.


Are there any other versions ?

I also say it again: I am deeply dissatisfied that none
of the people you mentioned at the bottom of
http://linuxtv.org/pipermail/linux-dvb/2006-May/010076.html
seem to care enough to participate in the discussion.

[15:47] <Dnumgis> |MA|: application devs does not comment because they have no time/will to read through such an enourmously long thread [15:48] <|MA|> You may present that on the ML, in regards to Johannes's comment
[15:48] <adq> yeah
[15:48] <|MA|> that also will be a very good help that you will be doing
[15:49] <adq> I wish js would just use IRC to discuss this stuff - its so much easier hammering this stuff out with an interactive discusssion




Manu


_______________________________________________

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