[linux-dvb] removal em28xx from linuxtv.org

Markus Rechberger mrechberger at gmail.com
Fri Jun 29 20:05:16 CEST 2007


On 6/29/07, Johannes Stezenbach <js at linuxtv.org> wrote:
> On Thu, Jun 28, 2007, Markus Rechberger wrote:
> > On 6/28/07, Mauro Carvalho Chehab <mchehab at infradead.org> wrote:
> > >
> > > In general, it is a good idea to have the manufacturer supporting their
> > > drivers. However, it won't make any sense to allow Empiatech to touch on
> > > low-level drivers for hardware not developed by them. The original
> > > patches were touching on every DVB driver, even on some Empiatech
> > > competitors drivers.
> >
> > And it added support for several cx88 and saa7134 based devices. The
> > base change was required to get the Empia drivers in, the rest was
> > optional.
> > There was a patch from another company which added full xc3028 support
> > for their saa7134 drivers to the repository on mcentral.de and by
> > doing what you did you now force me to rework what that company did.
> > The whole issue is a bigger than you imagine.
>
> I wondered about the reasons for your reply in
> http://marc.info/?l=linux-dvb&m=117940327206449&w=2
>
> This now outlines what the problem is: You had a repository
> which contained code which you knew wasn't going to be merged
> because others disagreed, and instead of addressing the problem
> you decided to work in isolation for a year, piling patch upon patch.
>
> But the original problem is still unsolved, and instead of asking
> for help you try to force us to accept your code -- which
> naturally won't work.
>
> Knowing how difficult the situation was I tried to resolve it
> by suggesting to merge your code as-is, and then fix it up later
> until everyone's happy with the outcome.
> http://marc.info/?l=linux-video&m=117572695505842 ff.
>
> Everybody grudgingly agreed, and I thought the issue
> was resolved, and I even sent you in private mail a reminder
> to push your code in time to make the 2.6.22-rc1 merge window.
> I was surprised that you never did.
>
> A few weeks later you start all over again with the same
> "merge all my code" request, but this time I was thinking
> that the reason why you didn't push your patches earlier
> was that you don't want us to change anything to your code
> after it has been merged. Which you confirmed in
> http://marc.info/?l=linux-dvb&m=117940327206449&w=2
>
> But your hybrid tuner dvb-core changes are too ugly to live,
> if we can't fix them up then your code can't be merged.
>
> A few people have offered to help you, but it seems you
> are unable to accept any help and rather pull the plug
> than to allow others to change your code.
>
> I have the impression that you didn't get the basic idea
> of what open source development is all about :-(
>

Johannes,

any discussion leads nowhere about it, there are so many mails out
there everyone can fish some out of them and put things into every
light someone wants to see that issue.

I could start that I initially asked how I could solve that problem
and there are also logs available.
That I don't really trust several v4l/dvb developers is because I've
been pointed to a wrong direction initially and that's why I'm also
very skeptic about listening to several people here.
Sure I also made some mistakes but I was the one who came up with the
code the whole time where others were telling me stories that I should
better look at some other examples which do not solve that problem.
Also during that time other devices came up which again had different
requirements and that's why I redid some parts again, if everyone
would have agreed during the first approach to get that work in it
might have turned out to a community work in the end but the current
way how many things work out are not good.

If a company comes up with a new hardware AND with a driver which
would cover the parts which are not supported by the API some people
would fight against it and write that they know everything better.

I also remember one of the first few emails I received and it was like
you're doing it completly wrong without a technical reason why. In the
end there was nothing wrong with it - several other drivers which are
in the linuxtv repository just do not fit all requirements yet and
would need some updates if someone keeps the changes local to his
drivers it's just fine.

Markus



More information about the linux-dvb mailing list