From: "Hans Werner" <HWerner4@gmx.de>
To: Mauro Carvalho Chehab <mchehab@infradead.org>
Cc: linux-media@vger.kernel.org
Subject: Re: Results of the 'dropping support for kernels <2.6.22' poll
Date: Fri, 20 Mar 2009 21:47:07 +0100 [thread overview]
Message-ID: <20090320204707.227110@gmx.net> (raw)
In-Reply-To: <20090304141715.0a1af14d@pedra.chehab.org>
> Hans,
>
> On Mon, 2 Mar 2009 22:18:24 +0100
> Hans Verkuil <hverkuil@xs4all.nl> wrote:
>
> > In the final analysis I'm the boss of my own time. And I've decided that
> > once the conversion of all the i2c modules is finished I'll stop
> spending
> > time on the compatibility code for kernels <2.6.22 as it is simply no
> > longer an effective use of my time. If someone else wants to spend time
> on
> > that, then that's great and I will of course answer questions or help in
> > whatever way is needed.
> >
> > I know that Mauro thinks he can keep the backwards compat code in by
> doing
> > nifty code transformations. It would be nice if he succeeds (and I have
> no
> > doubt that it is possible given enough time and effort), but personally
> I
> > think it is time better spent elsewhere.
>
> It required just a couple of hours today, in order to add the I2C
> conversion
> rules on the backport tree:
>
> http://linuxtv.org/hg/~mchehab/backport/
>
> There, I used, as example, the tea6415c.c file that you sent me, meant to
> be an
> example of a driver converted to use just the new I2C API. I've removed
> from it
> all the other #ifdefs for 2.6.26, so, the module doesn't have any compat
> bits
> (except for "compat.h" that can also be handled by the script). I didn't
> compile
> the entire tree, since several drivers will break, as they aren't ported
> yet
> to the new I2C style.
>
> Maybe a few adjustments on the backport.pl may be needed, after having all
> drivers converted to 2.6.22, since the final version may need some other
> patching rules.
>
> That's said, the backport tree is still an experimental work. The scripts
> require more time to be tested, and the Makefiles need some cleanups.
Mauro,
neat, but I still think time spent on backporting work would be better
spent on cutting-edge development. Two hours here ... two hours there
... it all adds up to a significant investment of time.
> Beside the fact that we don't need to strip support for legacy kernels,
> the
> advantage of using this method is that we can evolute to a new development
> model. As several developers already required, we should really use the
> standard -git tree as everybody's else. This will simplify a lot the way
> we
> work, and give us more agility to send patches upstream.
Great! Do you have a plan for how soon a move to the standard git tree
will happen? The sooner the better IMO.
> With this backport script, plus the current v4l-dvb building systems, and
> after
> having all backport rules properly mapped, we can generate a "test tree"
> based on -git drivers/media, for the users to test the drivers against
> their
> kernels, and still use a clean tree for development.
>
> Cheers,
> Mauro
If I understand correctly you are suggesting that a backporting
system would continue to exist?
Why? Surely this would be a heavy ball-and-chain to drag around for
eternity. Why not lose the backporting and go for the simplicity and
agility you mentioned above?
Regards,
Hans W.
--
Release early, release often.
Psssst! Schon vom neuen GMX MultiMessenger gehört? Der kann`s mit allen: http://www.gmx.net/de/go/multimessenger01
next prev parent reply other threads:[~2009-03-20 20:47 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-03-02 21:18 Results of the 'dropping support for kernels <2.6.22' poll Hans Verkuil
2009-03-02 22:46 ` kilgota
2009-03-02 23:28 ` Simon Kenyon
2009-03-02 22:47 ` Trent Piepho
2009-03-03 7:19 ` Hans Verkuil
2009-03-03 8:06 ` Trent Piepho
2009-03-04 17:17 ` Mauro Carvalho Chehab
2009-03-05 18:56 ` Guennadi Liakhovetski
2009-03-05 20:19 ` Trent Piepho
2009-03-05 20:36 ` Guennadi Liakhovetski
2009-03-05 20:57 ` Trent Piepho
2009-03-05 21:29 ` Hans Verkuil
2009-03-05 22:20 ` Guennadi Liakhovetski
2009-03-07 0:01 ` Trent Piepho
2009-03-07 0:46 ` Guennadi Liakhovetski
2009-03-07 1:46 ` hermann pitton
2009-03-15 16:39 ` Trent Piepho
2009-03-15 16:48 ` Hans Verkuil
2009-03-19 9:10 ` Trent Piepho
2009-03-15 19:00 ` Devin Heitmueller
2009-03-07 21:12 ` Adam Baker
2009-03-20 20:47 ` Hans Werner [this message]
2009-03-20 22:20 ` Mauro Carvalho Chehab
2009-03-21 2:03 ` Devin Heitmueller
2009-03-21 3:04 ` Mauro Carvalho Chehab
2009-03-21 5:21 ` Trent Piepho
2009-03-21 12:05 ` Devin Heitmueller
2009-03-21 17:35 ` Mauro Carvalho Chehab
2009-03-24 12:04 ` Trent Piepho
2009-03-21 5:16 ` Trent Piepho
-- strict thread matches above, loose matches on Subject: below --
2009-03-24 12:25 Hans Verkuil
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20090320204707.227110@gmx.net \
--to=hwerner4@gmx.de \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.