linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@osdl.org>
To: Arjan van de Ven <arjan@infradead.org>
Cc: oliver@neukum.org, rlrevell@joe-job.com, dax@gurulabs.com,
	linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org
Subject: Re: ETA for Areca RAID driver (arcmsr) in mainline?
Date: Thu, 22 Dec 2005 05:24:43 -0800	[thread overview]
Message-ID: <20051222052443.57ffe6f9.akpm@osdl.org> (raw)
In-Reply-To: <1135239601.2940.5.camel@laptopd505.fenrus.org>

Arjan van de Ven <arjan@infradead.org> wrote:
>
> On Thu, 2005-12-22 at 09:17 +0100, Oliver Neukum wrote:
> > Am Donnerstag, 22. Dezember 2005 06:34 schrieb Lee Revell:
> > > On Wed, 2005-12-21 at 22:20 -0700, Dax Kelson wrote:
> > > > I just got a shiny new (for me at least, the card has been out for
> > > > months) Areca RAID card.
> > > > 
> > > > The driver (arcmsr) is in the -mm kernel, but hasn't yet made it to the
> > > > mainline kernel. I'm curious what remains to be done before this can
> > > > happen?
> > > 
> > > Well, often all that's needed are some user reports that the driver
> > > works for them.
> > 
> > Is that a reasonable strategy? Why is a _new_ driver present only in -mm?
> > It hardly can break anything. It is possible that Andrew is quicker merging
> > but still I can't see the advantage if this persists for any length of time.
> 
> afaik that's not the strategy here.
> It's more "we're waiting for the structural issues to be resolved" which
> sounds quite reasonable.
> 

Yes, there are lots of stylistic and API-usage issues with the driver and
it needs someone to fix them all up.  Unfortunately the original
developer's English is very poor and he's obviously quite unfamiliar with
how Linux development happens.

This is all resolveable - it just needs someone to get down and work with
Erich on knocking the driver into shape.  But as yet, nobody has stepped up
to do that work.

And yes, the driver does apparently work.

  reply	other threads:[~2005-12-22 13:25 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-12-22  5:20 ETA for Areca RAID driver (arcmsr) in mainline? Dax Kelson
2005-12-22  5:34 ` Lee Revell
2005-12-22  6:26   ` Dax Kelson
2005-12-22  7:04     ` Lee Revell
2005-12-22  7:24       ` Dax Kelson
2005-12-22  8:17   ` Oliver Neukum
2005-12-22  8:20     ` Arjan van de Ven
2005-12-22 13:24       ` Andrew Morton [this message]
2005-12-22 19:31         ` Dax Kelson
2005-12-30 19:32           ` Randy.Dunlap
2005-12-31  5:21             ` [PATCH 2/2] Areca RAID driver (arcmsr) cleanups Randy.Dunlap
2006-01-02 17:04               ` Matthew Wilcox
2006-01-04  9:18             ` ETA for Areca RAID driver (arcmsr) in mainline? Andrew Morton
2006-01-07  5:17               ` [PATCH] Areca RAID driver (arcmsr) cleanups Randy.Dunlap
2005-12-22  6:43 ` ETA for Areca RAID driver (arcmsr) in mainline? Adrian Bunk

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=20051222052443.57ffe6f9.akpm@osdl.org \
    --to=akpm@osdl.org \
    --cc=arjan@infradead.org \
    --cc=dax@gurulabs.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=oliver@neukum.org \
    --cc=rlrevell@joe-job.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;
as well as URLs for NNTP newsgroup(s).