public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
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.


  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