mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@osdl.org>
To: Brent Casavant <bcasavan@sgi.com>
Cc: pavel@suse.cz, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/2] external interrupts
Date: Mon, 22 Aug 2005 14:43:30 -0700	[thread overview]
Message-ID: <20050822144330.791ba7b3.akpm@osdl.org> (raw)
In-Reply-To: <20050822155852.N325@chenjesu.americas.sgi.com>

Brent Casavant <bcasavan@sgi.com> wrote:
>
> On Sun, 21 Aug 2005, Pavel Machek wrote:
> 
> > > Here is a set of patches that implements an external interrupt capability
> > > in Linux, along with a device driver for a specific hardware device.  I
> > > submitted the patches several weeks ago, and they drew no comments, which
> > > I take to be a good sign.  Anyway, I'm 
> > It was not good sign in this particular case. My reaction was "this is _so_
> > overengineered tjat he's probably joking".
> 
> Laughter was not wholly unexpected, though I wasn't joking.  I'm trying
> to be realistic about the lifetime of any given hardware, and IOC4 is
> several years old at this point.  Couple that with a sincere desire to
> preserve application source compatability when (not if) new hardware
> appears, and an abstraction layer seemed to be a logical choice.  I'm
> more than happy to discuss problems in the abstraction layer's interface
> and make appropriate changes -- I'm nothing if not obliging.

Having an abstraction layer for a single client driver does seem a bit
pointless.  It would become more pointful if other client drivers were to
pop up.

Hence an option would be to merge an IOC4-specific driver which just does
what you need, no abstraction layer.  If someone later comes up with a
requirement for a driver for similar-looking hardware then we can resurrect
the abstraction layer at that stage.

  reply	other threads:[~2005-08-22 21:41 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-08-19 21:55 Brent Casavant
2005-08-20 22:21 ` Pavel Machek
2005-08-22 21:27   ` Brent Casavant
2005-08-22 21:43     ` Andrew Morton [this message]
2005-08-22 22:17       ` Brent Casavant
2005-08-23  9:57       ` Christoph Hellwig
  -- strict thread matches above, loose matches on Subject: below --
2005-07-28 21:13 Brent Casavant

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=20050822144330.791ba7b3.akpm@osdl.org \
    --to=akpm@osdl.org \
    --cc=bcasavan@sgi.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pavel@suse.cz \
    /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

Powered by JetHome