From: Christoph Hellwig <hch@infradead.org>
To: Michael Hunold <hunold@convergence.de>
Cc: linux-kernel@vger.kernel.org, torvalds@transmeta.com
Subject: Re: [PATCH[[2.5][3-11] update dvb subsystem core
Date: Wed, 7 May 2003 10:22:56 +0100 [thread overview]
Message-ID: <20030507102256.B14040@infradead.org> (raw)
In-Reply-To: <3EB8C67A.4020500@convergence.de>; from hunold@convergence.de on Wed, May 07, 2003 at 10:40:26AM +0200
On Wed, May 07, 2003 at 10:40:26AM +0200, Michael Hunold wrote:
> Hello Christoph,
>
> > What the problem with 2.5, dvb and devfs?
>
> The main problem is that our development "dvb-kernel" CVS tree *should*
> compile under 2.4 aswell, because most of the dvb-users don't want to
> participate in kernel development in general, but only on the
> development of the dvb subsystem. So work is done on the "dvb-kernel"
> tree, which should be synced with the 2.5 kernel frequently.
That okay in principle, but I'd like to ask you nicely to not touch any
devfs-related stuff currently. I'ts in flux and any external change
makes my life in cleaning up the mess a lot harder.
> > But first I have to fix the devfs API on 2.5 and randomly bringing
> > back old crap and lots of ifdefs in those changing areas won't help.
>
> I understand. But delaying the dvb updates just because a few calls to
> the devfs subsystem (which are now separated by #ifdefs and can easily
> be found) is not a good option either, or is it?
I think it is :) Esepcially as you don't just add ifdefs (which give
me lots of rejects and you much uglier code than just using the
compat header I'll send to lkml once I'm done with the API changes) but
you also change the code that's ifdefed for 2.5 to reverse change I
did. There is a reason why I removed every occurance of devfs_handle_t
from all drivers and the particular reason is that it will go away in
the next series of patches.
next prev parent reply other threads:[~2003-05-07 9:10 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-05-06 16:04 [PATCH[[2.5][3-11] update dvb subsystem core Michael Hunold
2003-05-06 20:49 ` Christoph Hellwig
2003-05-07 9:19 ` Michael Hunold
2003-05-07 9:28 ` Christoph Hellwig
2003-05-07 15:56 ` Michael Hunold
2003-05-07 16:04 ` Christoph Hellwig
2003-05-06 21:08 ` Christoph Hellwig
2003-05-07 8:40 ` Michael Hunold
2003-05-07 9:22 ` Christoph Hellwig [this message]
2003-05-07 15:56 ` Michael Hunold
2003-05-07 15:59 ` Christoph Hellwig
2003-05-07 16:14 ` Michael Hunold
2003-05-07 16:15 ` Christoph Hellwig
2003-05-07 18:30 ` Alan Cox
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=20030507102256.B14040@infradead.org \
--to=hch@infradead.org \
--cc=hunold@convergence.de \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@transmeta.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox