All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeff Garzik <jeff@garzik.org>
To: Mikael Pettersson <mikpe@it.uu.se>
Cc: linux-ide@vger.kernel.org
Subject: Re: [PATCH 2.6.21-rc7] sata_promise: SATAII-150/300 TX4 port numbering fix
Date: Sat, 28 Apr 2007 14:49:22 -0400	[thread overview]
Message-ID: <46339732.8000205@garzik.org> (raw)
In-Reply-To: <200704222228.l3MMSadi026578@harpo.it.uu.se>

Mikael Pettersson wrote:
> There is a known problem with sata_promise on SATAII-150/300 TX4
> controller cards: it enumerates drives in an order that differs
> from the port numbers printed on the controller cards. However,
> Promise's BIOS and Linux driver both get the order right.
> 
> I investigated Promise's Linux driver (v1.01.0.23), and found
> that it explicitly changes the mapping from logical port number
> to ATA engine MMIO address on the SATAII TX4 cards. It does this
> on all SATAII TX4 cards, without inspecting revision etc. The
> SATAII TX2plus cards continue to use the same mapping that was
> used for the first-generation chips.
> 
> This patch updates sata_promise to use the new port number to
> ATA engine mapping on SATAII TX4 cards, which fixes the drive
> enumeration order problem on those cards. Tested on 300 TX4,
> 300 TX2plus, and SATAII-150 TX2plus chips.
> 
> Signed-off-by: Mikael Pettersson <mikpe@it.uu.se>
> ---
> This patch should apply to 2.6.21-rc7 and libata#upstream.
> It won't apply to libata#ALL because of the massive changes
> for the new init model. I will do a new and cleaner patch for
> #ALL once I can get it as a patch in -mm (I don't do git).

ACK; dropped, awaiting rediff and resend against new init model

FWIW:  The new init model was applied to #upstream, and subsequently 
merged into #ALL (which always is a superset of #upstream).  So from 
your description, it sounds like your #upstream may not have been updated.

FWIW2:  When I note that libata-dev.git has been rebased, this means 
that you will need to delete the #upstream branch on your side, and 
download it again.

	Jeff



      reply	other threads:[~2007-04-28 18:49 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-22 22:28 [PATCH 2.6.21-rc7] sata_promise: SATAII-150/300 TX4 port numbering fix Mikael Pettersson
2007-04-28 18:49 ` Jeff Garzik [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=46339732.8000205@garzik.org \
    --to=jeff@garzik.org \
    --cc=linux-ide@vger.kernel.org \
    --cc=mikpe@it.uu.se \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.