linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Albert Lee <albertcc@tw.ibm.com>
To: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Cc: Jeff Garzik <jgarzik@pobox.com>, Doug Maxey <dwm@maxeymade.com>,
	IDE Linux <linux-ide@vger.kernel.org>,
	Mark Lord <mlord@pobox.com>
Subject: Re: [PATCH libata-dev-2.6] Issue INITIALIZE DEVICE PARAMETERS for CHS only devices
Date: Thu, 24 Feb 2005 11:05:40 +0800	[thread overview]
Message-ID: <421D4484.5090205@tw.ibm.com> (raw)
In-Reply-To: <58cb370e050223085039cb0731@mail.gmail.com>


Hi Bart,

> 
> [ generic comment about PATA support in libata, yes wrong thread 8) ]
> 
> I think that at one point libata and IDE drivers should be merged.
> Yes, this is doable and I plan to prove it soon (this shouldn't be so
> hard nowadays :-).  

   Do you mean the IDE subsystem and libata sharing the same lowlevel drivers?

> I really don't want to have two PATA drivers in
> the tree with subtle differences (i.e. CHS handling) when one would
> work in some cases (i.e. 4 Promise controllers) and the second one
> in other cases (i.e. ATAPI support).  I'm sure you get the point.
> 

   Yes, it is a problem. When I was asked to look at pdc2027x host
adapter hotplug issue, I saw libata already have the HBA hotplug feature.
So, that's why the pdc202xx_new driver was ported to libata as pata_pdc202xx driver.
However, currently libata needs enhancements like PATA, ATAPI, etc.
And the IDE subsystem needs some enhancements like HBA hotplug, etc.
   It will be great if some kind of integration can be done.

Before the dream come true, could I work on the following items on pdc202xx_new driver?
- Add PLL init code
   (This can fix the more than 2 adapters not supported limitation)
- Fix the hardware auto-set timing registers problem
- Fix the pdc2027x ATAPI DMA problem with the workaround
   (Currently pdc202xx_new driver alway use PIO mode for ATAPI)

The pdc202xx_new is the current driver used in production environment.
Fixing the above problems is really needed.

Best regards,

Albert



      reply	other threads:[~2005-02-24  3:06 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-02-06  6:02 [PATCH libata-dev-2.6 1/3] Add CHS support Albert Lee
2005-02-06  6:26 ` Jeff Garzik
2005-02-06 14:41   ` Mark Lord
2005-02-06 15:00     ` Jeff Garzik
2005-02-06 22:02       ` Mark Lord
2005-02-14 11:09   ` Albert Lee
2005-02-14 11:17     ` Albert Lee
2005-02-14 11:23       ` Albert Lee
2005-02-18 23:40 ` Jeff Garzik
2005-02-22  3:35   ` Albert Lee
2005-02-23 11:25   ` [PATCH libata-dev-2.6] Issue INITIALIZE DEVICE PARAMETERS for CHS only devices Albert Lee
2005-02-23 14:48     ` Mark Lord
2005-02-24  2:36       ` Albert Lee
2005-02-24  2:53         ` Jeff Garzik
2005-02-24  5:29           ` Albert Lee
2005-02-24  5:35             ` Jeff Garzik
2005-02-24  6:41               ` Albert Lee
2005-02-24  6:48                 ` Jeff Garzik
2005-02-24  7:01                   ` Albert Lee
2005-02-24  7:48                     ` Jeff Garzik
2005-02-25  5:01                       ` Albert Lee
2005-02-25  5:22                         ` Albert Lee
2005-02-25  5:40                         ` Jeff Garzik
2005-02-24  3:10         ` Mark Lord
2005-02-24  3:15           ` Jeff Garzik
2005-02-24  3:35             ` Mark Lord
2005-02-23 16:50     ` Bartlomiej Zolnierkiewicz
2005-02-24  3:05       ` Albert Lee [this message]

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=421D4484.5090205@tw.ibm.com \
    --to=albertcc@tw.ibm.com \
    --cc=bzolnier@gmail.com \
    --cc=dwm@maxeymade.com \
    --cc=jgarzik@pobox.com \
    --cc=linux-ide@vger.kernel.org \
    --cc=mlord@pobox.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).