public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Alan Cox <alan@lxorguk.ukuu.org.uk>
To: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: Parallel ATA with libata status with the patches I'm working on
Date: Thu, 03 Nov 2005 15:49:34 +0000	[thread overview]
Message-ID: <1131032974.18848.60.camel@localhost.localdomain> (raw)
In-Reply-To: <58cb370e0511030658tb23cecds2ed8cc63570a68d5@mail.gmail.com>

On Iau, 2005-11-03 at 15:58 +0100, Bartlomiej Zolnierkiewicz wrote:
> On 11/3/05, Alan Cox <alan@lxorguk.ukuu.org.uk> wrote:
> > Core Features Fixed
> > - Per drive tuning
> > - Filter quirk lists
> > - Single channel support
> 
> Are these the same changes that have been recently pushed into
> Linus' tree without any previous public review or some new ones?

Some of the changes have gone to Jeff Garzik others are pending
improvments/driver fixes for existing drivers and testing before Jeff
accepts them. Beyond that its up to Jeff what is merged into -mm and on.

> > Drivers so far written for the libata parallel work I'm doing
> 
> Are the patches available somewhere?

Yes. I'll post updated sets at some point soon. The patch I posted
pointers to before is now well out of date.

> > AMD
> > Driver written, given basic testing and equivalent to current
> > drivers/ide
> 
> Functionality can't be the same as drivers/ide because libata
> lacks some core features.

Well duh, other than core differences. Functionality is indeed different
- CRC errors dont crash SMP boxes for example ;)

> > CS5520
> > Driver written, some debug work to do. Works unlike the drivers/ide one
> 
> Please fix drivers/ide also if not a big problem.

I started this work in part because it was impossible to work with you.
I've no interest in fixing drivers/ide and in fact I'm now running as
much as I can with CONFIG_IDE=n, because the only way to really test
code is to use it all the time.

The things I've fixed however if you want to go fix them in drivers/ide
are

CS5520 is totally broken. You want the fixes I sent you over a year ago
and maybe more. Its probably best left as I doubt anyone else has a 5520
any more 8)

Serverworks runs the wrong cable detect routine in some cases
(conditions ordered wrongly)

I think the HPT366 driver contains several bugs looking at it in
comparison with hpts driver and the data sheets I can get hold of. It's
hard to be sure however because at times all three disagree with each
other. In particular it doesn't know about 302N but think its a 37x and
the PLL tune code appears to be unrelated to either data sheet or hpt
code. Can't be sure yet on the PLL until I've got PLL modes working in
the libata driver.


Alan (drivers/ide elimination department)


  reply	other threads:[~2005-11-03 15:19 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-11-03 14:54 Parallel ATA with libata status with the patches I'm working on Alan Cox
2005-11-03 14:48 ` Russell King
2005-11-03 15:02   ` Bartlomiej Zolnierkiewicz
2005-11-03 15:28     ` David S. Miller
2005-11-03 15:48       ` Bartlomiej Zolnierkiewicz
2005-11-03 16:27         ` Alan Cox
2005-11-03 15:56     ` Alan Cox
2005-11-03 20:55     ` Krzysztof Halasa
2005-11-03 21:29       ` Bartlomiej Zolnierkiewicz
2005-11-03 22:29         ` Krzysztof Halasa
2005-11-03 22:48           ` Bartlomiej Zolnierkiewicz
2005-11-03 22:54         ` Alan Cox
2005-11-04  1:30           ` Rogério Brito
2005-11-04 13:38             ` Alan Cox
2005-11-04 13:41             ` Alan Cox
2005-11-04 23:10               ` Russell King
2005-11-04 23:19                 ` Benjamin Herrenschmidt
2005-11-03 15:58   ` Alan Cox
2005-11-03 15:30     ` Russell King
2005-11-03 16:23       ` Alan Cox
2005-11-04  3:35       ` Jeff Garzik
2005-11-04  3:35     ` Jeff Garzik
2005-11-04  8:43       ` Xavier Bestel
2005-11-03 14:58 ` Bartlomiej Zolnierkiewicz
2005-11-03 15:49   ` Alan Cox [this message]
2005-11-03 15:34     ` Bartlomiej Zolnierkiewicz
2005-11-04  0:14     ` Mark Lord
2005-11-04  0:17       ` Mark Lord
2005-11-04  1:18         ` Alan Cox
2005-11-04  3:49 ` Jeff Garzik
2005-11-05 22:31   ` Mark Lord
2005-11-04  6:43 ` Benjamin Herrenschmidt
2005-11-04  7:00   ` Jeff Garzik
2005-11-04 13:34   ` Alan Cox
2005-11-04 21:50     ` Benjamin Herrenschmidt
2005-11-06 20:41       ` 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=1131032974.18848.60.camel@localhost.localdomain \
    --to=alan@lxorguk.ukuu.org.uk \
    --cc=bzolnier@gmail.com \
    --cc=linux-kernel@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox